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회차 과제-최성임 #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

immms
Copy link

@immms immms commented May 3, 2024

1. 내가 개발한 기능

  • map() 이용하여 상품요소 생성하고 화면 상에 출력
  • 상품 이름 alert창 생성 함수인 alertName()생성
  • 해당 요소 클릭 시, alertName() 실행

2. 내가 개발할 때 유의깊게 개발한 부분

  • onClick이벤트 처리 시 초반에 헤맸지만 해결함.

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

기능 별로 commit하는 것을 잊고 있다가 한꺼번에 올렸습니다.. 양해 부탁드립니다.

Copy link

@juiuj juiuj left a comment

Choose a reason for hiding this comment

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

성임님께서 구현하신 코드 리뷰를 통해 가독성 있고 효율적으로 코드 짜는 방법을 많이 배워가는 것 같습니다. 감사합니다:)

<h1>상품 목록</h1>
<ul>
{/* map() 이용하여 요소 생성 반복*/}
{products.map((product) => (
Copy link

@juiuj juiuj May 4, 2024

Choose a reason for hiding this comment

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

불필요하게 낭비되는 부분없이 잘 짜여진 거 같아서 가독성이 좋은 거 같습니다!

{products.map((product) => (
// key속성은 고유한 값인 id값으로 설정
// 상품 클릭 시 해당 품목을 alertName()로 전달
<li key={product.id} onClick={() => {alertName(product)}}>
Copy link

@juiuj juiuj May 4, 2024

Choose a reason for hiding this comment

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

alert함수 구현하는 과정에서 어려움을 겪고 있었는데 성임님 코드 덕분에 해결점을 모색할 수 있었습니다!

Copy link
Author

Choose a reason for hiding this comment

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

저도 alert함수 구현 시 헤맸었는데 도움이 됐다니 다행입니다~~!!

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