🛫 체크인

오늘 컨디션 상태를 아래 테이블에 작성해주세요 😃 점수는 1~10점 사이로 오늘의 컨디션을 작성해주시면 됩니다.

💬 팀별 토크(~20:30)

• 지난 시간 이후에 이번 스텝2를 하며 느꼈던 점들, 신경쓴 점들 이야기 나눠보기
• 그리고 나의 PR 본문에 오늘 중점적으로 리뷰 받고 싶은 부분, 어려웠어서 피드백 받고 싶은 2~3가지 정도를 적어보기
  • ex_ 모듈분리에 신경을 많이 썼는데 관련된 부분에 피드백 남겨주시면 좋을 것 같아요~!
  • ex_ 작명소 차릴뻔 했어요... 변수명 짓느라 시간 엄청 썼는데 네이밍에 대한 피드백 기다립니다 😆

| 디스코드 닉네임 | 어떤 이야기를 나누셨나요? 그리고 오늘 내 코드에서 어떤 점을 중점적으로 리뷰 받고 싶으신가요~? | | --- | --- | | BO(강보현) | - 작업에 대해서: 이번주에 갑자기 확 바빠서 바빠서 작업을 못하였습니다.ㅠㅠ Step 3 PR을 날리면서 함께 같이 올리도록 하겠습니다 🥲

• 팀원분들이랑 근황 토크: 이력서 쓰느라 정신 없었던 나와.. 기획팀과 협업이 힘드셨던 용석님과.. 깃허브 레포지토리 클론 문제가 있어서 힘드셨던 Jay님까지... 다들 쫌쫌따리로 힘들었던 일들이.... 그래도 파이팅! 👊 (결론은 K-엔딩으로...) | | Ysw | 1. 코드 분리 : 함수에서 반복적으로 나오는 this.current~~를 깔끔하게 정리하고 싶습니다 ⇒ index.js에 정의된 상태들 (this.menu)을 따로 분류하면 하위에 있는 함수들을 좀더 깔끔이 처리할수 있을거 같은데, 어떤식으로 해야할지 고민입니다.. ⇒ 강의를 보며 따라친 코드 인데, 리팩토링의 방향성을 주시면 좋을것 같습니다

2. 나눈 이야기 : 상태 관리가 어려웠다..!🔥 로컬스토리지 신기하다..!👍 | | 용석 | - 한줄요약 : 각자 힘든일이 있지만... 화이팅!

• 컴포넌트 별로 쪼개보았는데 더 잘 쪼갤 수 있는 방법이 있을까요? | | 초롱 | - 나눈 이야기: 스텝2 미션 하면서 상태관리가 쉽지 않았다는 분들이 꽤 있었습니다.
• 중점적으로 리뷰 받고싶은 부분: 강의를 보면서 따라 친 코드라서.. 이렇게 하면 더 좋은 코드가 될 거 같다하는 개선사항...? 을 피드백으로 받으면 좋을거 같아요. | | 치이즈 | 강의를 보면서 코드를 작성을 해서 어려운부분이 뭔지 고민하는 시간이 적었고, 지식을 얻었다고 생각하며 진행을 했어서 큰 질문사항이 없던 것 같아요. 함수가 선언된 순서가 가독성이 있는지도 확인 받고 싶습니다. | | 김정환 | - 코드 작성하는 점은 별로 큰 문제가 없는데 아무래도 바닐라 스크립트를 전혀 모르다 보니 파일 분리는 어떻게 하는 게 좋을지 잘 모르겠네요. 보통 사용하는 폴더 명이라던가 ...
• 프로그램이 작동하는 데는 문제가 없는 부분인데 바로 사용되지는 않는 상태를 초기화 할 때 저장을 해야할까 궁금하네요.
• 함수를 얼마나 잘게 나누면 좋을지, 편의를 위해서 유사한 기능을 하는 함수를 모아서 어떤 조건에서는 작동하지 않는 함수가 들어가있어도 될지 모르겠아용. | | 승희 | 함수들을 작은단위로 작성했는데 가독성은 괜찮은지?, 함수명은 어떤지?, 함수의 역할이 너무 크지는 않은지 적절한지? 리뷰를 받고 싶습니다 | | 트루워터 | 동작은 하는데 인스턴스를 만드는 부분이 정확하게 이해가 가지 않아요 ㅠㅠ | | 최원준 | 상태관리나 코드를 짤 때 어려운 점 어떻게 하면 좀 더 나눌 수 있고 깨끗한 코드가 될 수 있는지 이점에 대해 리뷰 받고 싶습니다. | | 주영 | 상태 관리, 함수 모듈화 과정에서 많이 헤맸던 것 같습니다.

1. 함수 모듈화를 해봤는데, 다른 분들이 리뷰하기에도 적절한 만큼 가독성이 있는지 궁금합니다.
2. 상태 관리에 관한 부분은 나중에 적도록 하겠습니다! | | Jay | - Step 1 진행 중 | 직접 돔 조작을 하던 방식에서 배열 관리를 통해 화면을 렌더링하는 방식으로 작업을 하는 것에 초점을 두었습니다.

• 클론 관련된 문제를 해결해야하는 숙제가 남아있고 이 문제가 해결되면 step1 & 2작업을 추가해서 피드백을 받기로 했습니다 😀 | | 쌤(상민) | 1. 모듈화가 잘 된 것 같은지?

2. 변수명이 직관적인지?
3. 함수가 단일 책임인지? | | 창오 | - 고민: 적지도 않고 많지도 않은 적정 수준의 모듈화는 어떻게 해야할지 고민입니다.

• 코드리뷰: 네이밍 가독성 괜찮은지, 분리가 필요하거나 병합이 필요한 함수가 있을까 고민입니다. | | HAN(한아름) | 1. 배운 것, 신경 쓴 것
• localStorage를 세팅하고 불러오는 방법(세팅할 때 값이 문자열로 들어가고 불러올 때 다시 객체로 변환시켜야 하는 것)
• 상태관리 처음 해봐서 너무 어려웠음
• 조건을 변수화 해서 사용하는 것도 재미있고 신선했음

2. 코드 리뷰 받고 싶은 부분

• 명확한 변수 이름
• 중복되는 코드
• 모듈분리를 처음 해봤는데 앞으로 이런 부분도 모듈화 했으면 좋겠다 하는 부분 에 대해서 피드백 부탁드립니다~ | | nelldklee(이동규) | 상태값과 rendering에 대해서 고민을 많이 했던 것 같습니다. | | Rita | 코드의 흐름상 이상한 게 있거나 코드 중에 혹시 어색한 부분이 없는지, 개선하면 더 좋을 부분 등... 어떤 리뷰든 환영입니다! | | Yummy ( 염상권 ) | 1. 각자 과제를 하면서 느꼈던 점 그리고 크게 localhost로 상태관리에 대한 이야기가 주를 이뤘던 것 같습니다.

2.코드의 사이즈가 조금 커진 감이 없지 않아 있는데, 이를 어떻게 하면 간단하게 줄일 수 있을까요?? | | 은선 | 타입스크립트를 적용해보았는데 부적절한 사용이나 또는 더 적극적으로 활용할 방법이 있을지 | | | | | | |

💻 코드 리뷰 이터레이션1 (~21:00)

• 팀 내 이름 순서를 한 번 적어주시고, 오른쪽 사람의 코드를 리뷰하도록 하겠습니다.
  • ex_ 효진 ⇒ 정수 ⇒ 리타
  • 이 때 PR을 올리시지 못한 경우, 이름 순서에는 넣지 않고 깍두기 역할을 합니다.
  • 깍두기는 원하는 다른 사람의 PR을 자유롭게 보면서 리뷰를 남길 수 있습니다. (절대권능 깍두기!)
• 팀원 간의 순서를 정해서 다른 한 사람의 코드를 20분간 리뷰해주세요

📝 중간회고 (~21:10)

| 디스코드 닉네임 | 1. 첫번째 이터레이션에서 코드리뷰 하며 새롭게 알게 된 것 (최대한 구체적으로 작성해주세요 😄) 2. 두번째 이터레이션에서 코드리뷰에서 좀 더 신경 쓸 점 | | --- | --- | | BO(강보현) | - 우선 저번 주에 팀원 분 PR을 못 봐드렸는데, 그걸 마저 보았고(깍두기)

• 바닐라 JS를 컴포넌트화해서 작업하셨는데, 말로만 들어봤지 실제로 구현된 코드를 본 적은 없었어서 되게 신선했고, 보면서 즐거웠습니다. 참고 자료도 잘 첨부해주셔서 저도 한번 보고 도전해봐야겠습니다.
• 컴포넌트 쪼개기에 대해서도 고민을 많이 해봐야겠다고 생각이 들었습니다. 매번 이게 정답인 거 같다가도 또 저게 정답인 거 같고 하는.... 😥 | | Ysw | 1. 비슷한 코드여도 세심함이 다르다는걸 알았습니다..

2. 최대한 나의 생각과 좋은 의견이 반영 될 수 있도록 신경써야겠다 | | 용석 | - className.toggle로 제어할 수 있다는 것을 새롭게 알았습니다!

• 코드보는 눈을 조금 더 키워야될것 같아요...!
• 주석도 너무 잘 남겨주셨어요! 주석을 남기는거에 대해 고민해 볼 필요가 있을 것 같습니다!
• 함수를 어디서 정의했고, 어디서 사용하고 있는지에 대해 조금 더 신경써서 봐야할 것 같습니다! | | 초롱 | 1. 새롭게 알게 된 것은 없는것 같네요.

2. 강의를 안 듣고 짜신 코드라면 리뷰하기 위해서 세세하게 봐야 할 거 같아요. | | 치이즈 | 1.문자열대신 상수를 이용하면 조금 더 안정적일것 같다는 리뷰를 받았습니다. 함수 분리를 하는 스타일이 다 달라서 신기합니다.. 2.지식이 부족해서 그런지 수긍만 하게 되고 빠른 시간안에 코드를 보고 리뷰를 하고 질문을 해야하는 상황에서 의견이 많이 떠오르지가 않네요 ㅠㅠ | | 김정환 | querySelector() 가 연산 비용이 높은 메소드라는 것은 알겠는데, 가독성이나 편의성을 위해 어느 정도 선에서 바인딩을 하거나 호출해도 될지 잘 감이 잡히지 않네요. 함수가 한번 실행될 때 한 번 호출하는 정도에 대해서도 고려를 해야할까요?? | | 승희 | if~else if문 대신 가독성 높은 switch문으로 이벤트위임를 하는 코드를 작성해보면서 리팩토링했습니다. utils 함수들의 역할과 어떤것들을 두어야할지 고민해봐야겠습니다 | | 트루워터 | 돔조작을 인스턴스에 바인딩 한다는 생각을 배웠습니다. 메뉴가 많아 지고 복잡해지면 인스턴스를 나워서 관리할 수도 있는데 인스턴스에서만 사용하는 변수를 만들어 사용하는 방법을 배웠습니다. 상수를 사용하여 안전성을 높이는 법도 배웠습니다. 의미가 명확한 함수에 대해서 자세히 보겠습니다. | | 최원준 | 중복되는 코드가 있는 줄은 몰랐습니다. 이벤트 객체를 넘겨주는 부분이 같았는데 그럴 필요 없이 Id를 넘겨주는 방식으로 코드를 조금 줄일 수 있었네요. 다른 분들 코드리뷰를 할 때도 중복되는 코드가 있는지 없는지를 최우선 순위로 두고 리뷰해야 할 것 같습니다. | | 주영 | 모듈화를 하려고 노력하신 흔적이 엿보인 좋은 코드를 리뷰하게 되었습니다. 로직이 예상되는 함수는 기능별로 따로 빠져있어서 제 코드에도 적용해봐야겠다고 생각했습니다. | | Jay | 이벤트 핸들러를 잘 분리해 두신걸 보고 제 코드에 반영해봐야겠다고 생각했습니다. 코드를 읽어보는 것으로 이해가 잘 되진 않는데 코드리뷰를 늘려가면서 익숙해지면 좋을 것 같아요. | | 쌤(상민) | 바닐라 자바스크립트로 프레임워크를 구현한 코드를 경험해볼 수 있어서 좋았습니다. 제 실력이 부족해 모든 코드를 이해할 수는 없었지만 새로운 경험이었습니다. 나중에 기회가 된다면 직접 해보고 싶습니다. | | 창오 | - 초기 이벤트리스너 등록 부분을 한꺼번에 빼두는 것이 깔끔하고 보기 좋았습니다.

• 모듈화를 세세하게 해주신 덕분에 어떤 방식으로 하면 좋을지 고민해보는 시간을 가졌습니다. | | HAN(한아름) | 1. 새롭게 알게 된 점
• 쪼갤려면 엄청 쪼갤 수 있구나(기능별로, 함수별로 모듈화 시킨 부분이 넘나리 인상👀적)
• 나는 이벤트를 하나하나 리스너 달아줬는데 팀원은 메뉴를 관리하는 모듈을 만들고 액션을 정의하고(add, update, delete ...) 그 모듈이 실행될 때 넘겨 받은 액션 타입에 따라 기능을 분리한 점이 너무 인상적이었다...

2. 좀 더 신경쓸 점

• 정확한 변수명을 짓기란 너무 어렵구나, 처음 코드를 보는 사람이 봐도 어떤 것을 실행하는 지 알수있도록 더 정확한 변수명을 쓰는 것에 유의해서 피드백을 해드려야겠다.
• 중복되는 부분도 없는지 더욱 신경써야겠다. | | nelldklee(이동규) | 좀 더 상태관리 관점에서 반복을 어떻게 줄일지 집중해서 보게 된거같고, 코드를 보니 비슷한 고민을 했을 것 같아 즐거운 마음으로 다른 분들의 코드를 본 것 같습니다. | | Rita | 새롭게 알게된 점 : MVC패턴을 적용하셔서 좋은 예시를 볼 수 있었다. 타입스크립트 적용하신 코드를 보면서 궁금한 점이 많았다. | | Yummy ( 염상권 ) | 1. 새롭게 알게된 점 - 모듈 분리를 팀원들이 너무 잘한 것 같습니다.

2. 아직 변수명이나 가독성 있는 코드를 짜는것이 부족하다는 것을 알게 되었습니다. | | 은선 | 1. 새롭게 알게된 점: pubsub 패턴(용어?ㅎㅎ), 각 부분별로 나누어 렌더링하고 있어서 더 효율적인 렌더링이 가능한 구조였다. 나 역시도 효율성을 좀 더 고민해 보았으면 좋았을 것 같다. 또한 개인적으로는 처음부터 기존에 있는 패턴구조를 사용해서 작성을 했는데, 다른 분들이 작성한 코드를 보니 좀 더 개인적으로 고민해보고 구현 했어도 공부가 되었을 듯! | | | | | | |

💻 코드 리뷰 이터레이션2 (~21:30)

• 팀 내 이름 순서를 한 번 적어주시고, 오른쪽 사람의 코드를 리뷰하도록 하겠습니다.
  • ex_ 이터레이션1: 효진 ⇒ 정수 ⇒ 리타 / 이터레이션2: 리타 ⇒ 정수 ⇒ 효진
  • 이 때 PR을 올리시지 못한 경우, 이름 순서에는 넣지 않고 깍두기 역할을 합니다.
  • 깍두기는 원하는 다른 사람의 PR을 자유롭게 보면서 리뷰를 남길 수 있습니다. (절대권능 깍두기!)
• 팀원 간의 순서를 정해서 다른 한 사람의 코드를 20분간 리뷰해주세요

📝 의미 있는 주석 달기

1. 주석에 담아야하는 대상