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

[1팀 전명익] [Chapter 1-2] 프레임워크 없이 SPA 만들기 #46

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

Conversation

nopinokio
Copy link

과제 체크포인트

기본과제

가상돔을 기반으로 렌더링하기

  • createVNode 함수를 이용하여 vNode를 만든다.
  • normalizeVNode 함수를 이용하여 vNode를 정규화한다.
  • createElement 함수를 이용하여 vNode를 실제 DOM으로 만든다.
  • 결과적으로, JSX를 실제 DOM으로 변환할 수 있도록 만들었다.

이벤트 위임

  • 노드를 생성할 때 이벤트를 직접 등록하는게 아니라 이벤트 위임 방식으로 등록해야 한다
  • 동적으로 추가된 요소에도 이벤트가 정상적으로 작동해야 한다
  • 이벤트 핸들러가 제거되면 더 이상 호출되지 않아야 한다

심화 과제

1) Diff 알고리즘 구현

  • 초기 렌더링이 올바르게 수행되어야 한다
  • diff 알고리즘을 통해 변경된 부분만 업데이트해야 한다
  • 새로운 요소를 추가하고 불필요한 요소를 제거해야 한다
  • 요소의 속성만 변경되었을 때 요소를 재사용해야 한다
  • 요소의 타입이 변경되었을 때 새로운 요소를 생성해야 한다

2) 포스트 추가/좋아요 기능 구현

  • 비사용자는 포스트 작성 폼이 보이지 않는다
  • 비사용자는 포스트에 좋아요를 클릭할 경우, 경고 메세지가 발생한다.
  • 사용자는 포스트 작성 폼이 보인다.
  • 사용자는 포스트를 추가할 수 있다.
  • 사용자는 포스트에 좋아요를 클릭할 경우, 좋아요가 토글된다.

과제 셀프회고

기술적 성장

코드 품질

학습 효과 분석

과제 피드백

리뷰 받고 싶은 내용

@BongjoonKim
Copy link

renderElement.js에서 normalizeVNode함수를 if랑 else문에 공통으로쓰고 있어서 조건문 밖으로 빼도 괜찮을 것 같습니다!
updateElement에서 텍스트 노드 변경에서 조건문이
if (typeof newNode === "string" || typeof newNode === "number")
이고 document.createTextNode(newNode.toString())으로 값을 생성하면 될 것 같은 느낌입니다

주석을 잘 달아주셔서 코드를 좀 더 빠르게 이해할 수 있었던 것 같습니다! 이번주도 너무 수고하셨습니다.

@CRITICBANGGU
Copy link

코드에 주석 잘 달아주셔서 흐름이 잘 읽혔어요 ㅎㅎ
이번주도 고생 많으셨습니다:)

@devsuzy
Copy link

devsuzy commented Dec 28, 2024

VaildCheck 함수를 따로 만들어서 데이터 처리하신게 인상 깊네요!!
과제하시느라 넘 수고 많으셨습니당 👍

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.

4 participants