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

5회차 과제 - 구준혁 #10

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

Conversation

gugitgugit
Copy link

1. 개발한 기능

  • 상품 목록 진열
  • 제품명-제품가격 나열
  • 클릭시 해당 품목 이름 알림창 생성

2. 유의깊게 개발한 부분

  • 각 JSX의 key 속성을 id(고유한 값)으로 설정
  • 따로 변수를 선언하지 않고 map 함수 내에서 onClick까지 구현했습니다.

3. 개발하면서 들었던 의문 사항

  • 위에서 말했듯이 따로 변수를 선언하지 않고 map 함수 내에서 구현을 했는데, 오히려 가독성이 떨어지지 않을까 싶습니다.

return (
<div>
<h1>상품 목록</h1>
<ul>
Copy link

Choose a reason for hiding this comment

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

<ul> 태그의 내용들을 컴포넌트로 만들어서 적용시키는 방법도 좋을 것 같습니다 !

Copy link
Author

Choose a reason for hiding this comment

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

그러면 확실히 깔끔해지겠네요 감사합니다!

{products.map((product) => (
<li
onClick={() => {
alert(product.info.name);
Copy link

Choose a reason for hiding this comment

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

alert() 함수를 코드 내에 직접 삽입 해 코드의 길이를 줄인 것이 좋은 것 같습니다.
현재는 상품의 이름을 alert하는 간단한 함수이지만, 이후 복잡한 내용을 실행해야하는 함수라면 해당 함수를 따로 만들어 적용시키는 것이 좋은 방법 같습니다!

Copy link
Author

@gugitgugit gugitgugit May 5, 2024

Choose a reason for hiding this comment

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

저 또한 그렇게 생각합니다. 복잡한 내용이 아니라서 한 번 시도해 보았는데 실무적으로 생각해봤을 때는 함수를 따로 만드는 습관을 들이는 것도 괜찮은 것 같습니다. 좋은 리뷰 감사합니다!

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