Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tablemin03] 프론트엔드 1주차 미션 제출합니다. #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tablemin03
Copy link

No description provided.

Copy link
Collaborator

@hhhkdev hhhkdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

프론트엔드 스터디를 들었다는게 티가 나는 부분이 몇몇 있네요

React에서 코드짜는건 확실히 JS를 그냥 시키는 것과 느낌이 달라서 지금처럼 컴포넌트를 어떻게 만들고 관리할지에 치중하여 공부하면 좋을거 같아여

Comment on lines +19 to +41
.hoby{
width : 50px;
}
.food{
width : 160px;
}
.self{
background-color: gray;
color:white;
border: 5px solid navy;
width:50%;
}
.burger{
background-color: gray;
color:white;
border: 5px solid navy;
width:50%;
}
.movie{
background-color: gray;
color:white;
border: 5px solid navy;
width:50%;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

className이 많은걸보니 자기소개에 많은 내용을 담으셨나보네요 좋습니다 + 추가적으로 여러 CSS 속성을 이용해보면 재밋을거 같아요

Comment on lines +16 to +22
<h2 class ="hoby">취미</h2>
<div class="movie">
<p>취미는 영화관에서 영화보는 걸 좋아합니다. 그리고 그냥 집에서 쉬는 것도 좋아합니다.</p>
</div>
<h2 class ="food">좋아하는 음식</h2>
<div class= "burger">
<p>좋아하는 음식은 햄버거입니다. <del>햄버거 먹고 키 컸습니다.</del></p>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HTML 태그도 CSS와 마찬가지로 여러 종류를 이용하면 더욱 풍성한 페이지가 되지 않을까 싶습니다!

Comment on lines +1 to +12
function plus(n1, n2){
return n1 + n2;
}
function minus(n1, n2){
return n1 - n2;
}
function mul(n1, n2){
return n1 * n2;
}
function div(n1, n2){
return n1 / n2;
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

딱 제가 의도하신 대로 함수 코드를 작성하셧네요 React로 코드를 짜게된다면 이처럼 함수를 이용하는일이 많을거예요 특히 다른 언어와 헷갈리지 않게 조심해야 합니당

Comment on lines +16 to +18
var a = +inp.substring(0, inp.indexOf(' '));
var oper = inp.substring(inp.indexOf(' ') + 1, inp.indexOf(' ', inp.indexOf(' ') + 1));
var b = +inp.substring(inp.lastIndexOf(' ') + 1);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

자바스크립트에서 문자열을 다루는 메서드가 여러가지 있는데, 이중에서 split에 대해서 알아보면 좋을거 같아여

그리고 +를 붙인건 문자로 표현된 숫자를 숫자형으로 바꾸기 위한 작업으로 보이는데 이 역시 parseInt를 이용해보면 좋을듯합니당

var oper = inp.substring(inp.indexOf(' ') + 1, inp.indexOf(' ', inp.indexOf(' ') + 1));
var b = +inp.substring(inp.lastIndexOf(' ') + 1);

var ans;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

필요없는 변수가 선언돼서 짚고 갑니다..!

function App() {
return (
<div className="App">
<Button />
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

딱 깔끔하게 Button 컴포넌트 이용하셧네요 이렇게 React에서는 컴포넌트를 불러올 수 있답니다

Comment on lines +6 to +11
const buttonStyle = {
fontsize: '100px',
padding: '25%',
background: 'blue',
color: 'white'
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

따로 css 안빼내고 스타일 적용시킨게 인상적이네요

이번 2주차 미션 서브 미션을 보시면 이렇게 객체 형식으로 빼낸 스타일처럼 관리할 수 있는 방법을 알 수 있으니 styled component에 대한 내용을 공부해보면 좋을거 같아요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants