체크인
오늘 컨디션 상태를 아래 테이블에 작성해주세요
점수는 1~10점 사이로 오늘의 컨디션을 작성해주시면 됩니다.
점수는 1~10점 사이로 오늘의 컨디션을 작성해주시면 됩니다.디스코드 닉네임 | 체크인 점수(1~10) | 이유 |
메이커준 | 8 | 비가 와서 그런지 몸이 무겁네용! 스터디 끝나면 간단히 운동하고 와서 텐션을 올릴려고 합니다 ㅎㅎ 조만간 원티드에서 이력서 워크샵을 진행해볼 것 같은데 열리게 되면 블랙커피에 가장 먼저 공유드릴게요! |
BO(강보현) | 7 | 비가 와서 기분이 울적합니다ㅠ(날씨에 영향받는 1인..) 이력서 작업을 하는 중인데, 뭔가 잔뜩 벌려 놓은 일들을 문서화하는 작업이 왜이렇게 힘든지 모르겠네요...! 파이팅..! |
Ysw | 8 | 비 오는 날에 게을러 지는 제가 싫습니다.. |
용석 | 5 | 회사 프로젝트 reset 되어서 다시하는데.. 우울하네요..허허;; |
초롱 | 8 | 그럭 저럭 평소와 같은 하루네요 ^~^ |
치이즈 | 7 | 하루종일 정신이 없었는데, 금방 8시가 되었네요 |
김정환 | 9 | 학교 프로젝트 코드가 너무 엉망이라 싹 다 고치고 있는데 오늘은 비가 와서 아무것도 안 했어요 ㅎㅎ!! 집에서 한 발자국도 안 움직였네용 |
승희 | 7 | 이력서 쓰는중인데 잘 쓰고있는지 모르겠네요ㅠㅠ |
트루워터 | 9 | 블프에서 패딩하나 진하나 건졌어요 ㅋㅋ |
최원준 | 7 | 회사일이 잘 안풀려서 아쉽네요. |
주영 | 10 | 이번주에 새롭게 배우고 알게 된 부분이 많아 기분이 좋네요! |
Jay | 7 | 미션 완료를 못했는데 그마저 풀리케 보내다가 문제가 발견되어서 씁... 찬찬히 다시 보려고 합니다. 오늘 스터디에서 배우는 내용들은 잘 얻어가고싶어요~ |
쌤(상민) | 9 | 하루종일 정신이 없지만 컨디션은 좋네요 |
창오 | 10 | 다들 파이팅. 일주일 거의 다 끝났네요!!!!! |
HAN(한아름) | 8 | 이렇게 스터디 참여한 나 자신이 기특하고요! 잘하고있는 것 같습니다!!ㅎㅎ |
nelldklee(이동규) | 8 | 밥먹고 식곤증이.... 그것외엔 좋습니다~! |
Rita | 8 | 저녁이라 살짝 피곤합니다 ㅜㅜ |
Yummy ( 염상권 ) | 8 | 늦었네요 ㅠㅠ |
팀별 토크(~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. 주석에 담아야하는 대상
우리가 주석을 작성한다면 어떤걸 작성하면 좋을까요? 주석은 실제 코드를 포함하여 해당 부분을 이해하는데 더 많은 시간을 할애해야하고, 화면의 일정한 부분을 차지합니다. 그래서 주석 또한 코드의 일부이므로 작성할 때는 분명한 이유가 있어야 합니다. 그럼 작성해도 괜찮은 주석과, 그렇지 않은 주석 사이의 경계를 어떻게 정할 수 있을까요?
1) 감독의 설명을 포함하기
영화에는 감독이 자신의 통찰을 설명하고, 영화가 만들어진 과정을 관람객이 잘 이해하게 도와주는 '감독의 설명'을 담은 감독판 트랙이 있습니다다. 이와 비슷한 방식으로 중요한 통찰을 기록한 주석을 코드에 포함시킨다면, 읽는 사람으로 하여금 해당 코드에 대한 더 높은 이해를 할 수 있게 도와줍니다.
예를 들면
// 이 데이터에서 이진트리는 해시테이블보다 40%정도 빠르다.
// 재귀함수를 이용하면, 기존의 100line이 될수 있는 코드가 단 20줄로 정리된다. 반면 성능은 유의미한 차이가 없다.
JavaScript
복사
이러한 주석은 코드를 읽는 사람에게 코드를 최적화하느라 시간을 허비하지 않게 도와줍니다.
2) 코드에 있는 결함을 설명하기
코드는 지속적으로 진화하며, 그러는 과정 중에 버그를 갖습니다. 이러한 결함을 설명하는 것을 부끄러워할 필요는 없습니다. 숨기는게 문제이지, 결함을 공개적으로 드러내고 수정할 여지를 남겨야 합니다. 예를 들어 개선이 필요할 때 아래와 같이 작성할 수 있습니다.
// TODO: JPEG외 다른 이미지 포맷도 처리할 수 있어야 한다
JavaScript
복사
// AsyncMode should be deprecated
export function isAsyncMode(object: any) {
if (__DEV__) {
if (!hasWarnedAboutDeprecatedIsAsyncMode) {
hasWarnedAboutDeprecatedIsAsyncMode = true;
// Using console['warn'] to evade Babel and ESLint
console['warn'](
'The ReactIs.isAsyncMode() alias has been deprecated, ' +
'and will be removed in React 18+.',
);
}
}
return false;
}
JavaScript
복사
이러한 결함 설명과, 추가로 해야하는 일의 경우 팀내에서 컨벤션을 맞추면 좋습니다.
3) 상수에 대한 설명
상수를 정의할 때는 종종 그 상수가 무엇을 하는지, 그것이 왜 특정한 값을 갖게 되는지 사연이 있는 경우가 있습니다. 예를 들면 아래와 같이 작성된 경우, 주석을 통해 해당 코드에 대한 이해도를 더 높게 가져갈 수 있습니다.
// 합리적인 한계 - 1000개 이상을 구독하는 사람은 기획 논의상 없다.
const MAX_SUBSCRIPTIONS = 1000
// 사용자들은 0.72가 크기/해당도 대비 최선이라고 생각한다.
const IMAGE_QUALITY = 0.72
JavaScript
복사
4) 사람들이 쉽게 빠질 것 같은 함정을 미리 경고하기
예를 들어 사용자에게 이메일을 보내는 써드파티 라이브러리와 함수를 작성했다고 해봅시다. 이 때 아래와 같은 이슈가 있고, 그에 대해 작성해준다면, 함께 개발하는 사람은 1분정도 소요된다는걸 사용자에게 애니메이션이나, 메시지로 알림을 줄 수 있습니다. 그렇지 않다면, 왜 시간이 1분이나 걸리는지 그것이 에러가 아닌지 확인하는 작업을 해야할수도 있습니다.
// 이 함수는 외부 서비스를 호출하여, 이메일을 발송한다 (1분정도 소요)
function sendEmail() {
...
}
JavaScript
복사
회고 (~21:55)
디스코드 닉네임 | - 오늘 코드리뷰 진행하며 느낀 점 & 새롭게 알게 된점
- 스텝3 미션하면서 반드시 반영하고 싶은 인사이트 1가지 |
BO(강보현) | - 바닐라 JS를 컴포넌트화하신 게 신선했어서 너무 재밌게 봤고, 참고 링크를 잘 달아주셔서 저도 참고하여서 다음에는 한번 도전해봐야겠다고 느꼈습니다.
- 다음주에는 꼭 미리 PR을 올릴 수 있도록 해야겠습니다. |
Ysw | 오늘 코드리뷰 진행하며 느낀 점 & 새롭게 알게 된점
- 비슷한 코드여도 신경쓰는 부분이 다르다
- 같은 기능이여도 구현하는 코드가 다른데, 다른 분들의 PR을 보면 재밌게 코드 리뷰를 할 수 있어서 좋았다
- 코드 리뷰 너무 열심히 달아주셔서 고마웠다..
스텝3 미션하면서 반드시 반영하고 싶은 인사이트 1가지
- 코드를 먼저 짠뒤에 강의 코드를 반영해 보자 |
용석 | - 매번 느끼는 거지만, 각자 코드 스타일이 다양한 것을 느낄 수 있었다.
- DOM 조작할 때 className.toggle을 처음 알게 되었다. 앞으로는 자주 사용할듯!
- 주석을 되게 잘 남겨주셨어서 인상 깊었다!(창오님 코드) |
초롱 | - 오늘 코드리뷰 진행하며 느낀 점 & 새롭게 알게 된점
다른 분들의 코드를 더 보려고 하면 얻어가는게 많을거 같다.
- 스텝 3 미션에 반영하고 싶은 인사이트
강의 무조건 보지 말고 일단 짜보고 나서 안 되면 강의를 보는 시도를 해야겠어요. |
치이즈 | -사람마다 모듈화하는 스타일이 다 다름을 느꼈습니다.
-스텝3에서는 강의에 의존하지 않고 혼자 해보려고 노력해봐야겠습니다.
-커밋을 나눠서 가독성을 높여봐야겠습니다! |
김정환 | - 코드 작성 스타일이 다양하다는 것을 알 수 있었습니다
- 항상 어떤 코드가 좋은지 생각하는 게 좋다고 생각했습니다.
- 커뮤니케이션 스피치 능력을 키워야겠다는 생각을 했습니당. ㅜㅜ |
승희 | |
트루워터 | - 의미있는 변수와 함수를 유틸로 빼서 따로 관리하는 방법도 경험했습니다.
- 안전하게 상수를 쓰는 것도 좋은 것 같아요!
- 인스턴스에 대해서 한번 더 배웠습니다.
- 스텝 3에서는 좀 더 의미 있는 함수와 상수를 활용해보려고 합니다. |
최원준 | - 생각보다 실수하기 쉬운 분들에 대해 주석을 달지 않았다는 것을 알 수 있었고, 프레임워크나 외부 라이브러리 없이도 디자인 패턴을 이용해서 코드를 좀 더 깔끔하게 짤 수 있었단 걸 배웠습니다.
- 지적해주신 부분들을 전부 분리해보고 이러한 과정에서 생기는 문제들을 해결해보고 싶습니다. |
주영 | - 주석이 잘 달려 있어서 코드 리뷰하기 편했습니다.(창오님 코드)
- 용석님, 창오님, 상민님 코드 보며 많이 배웠습니다. 감사합니다!
- 용석님께서 step1 PR에 올려주신 상태관리 관련 링크 보고도 많이 배웠습니다. 감사합니다! |
Jay | - 구조 같은 걸 보는 눈을 많이 키워야 할 거 같다. 다른 분들 코드 보면서 내가 막혔던 부분을 해결할 수 있었고, 참고해서 보완해야 할 거 같다.
- 리스트 추가할 때 li 안에 li가 또 생긴 경우가 있었는데, 다른 분 코드를 보면서 해결 방법을 찾았고, 용석님 코드를 보면서 template 자체를 바로 추가될 수 있게 구현해놓으셨어서 한번 더 보고 공부해봐야 겠다는 생각이 들었습니다.
- 코드리뷰 하면서 좋은 코드였다라고 언급된 분들은 다시 읽어보려고 합니다.(창오님: 주석, 주영, 용석) |
쌤(상민) | - 리액트처럼 코드를 짜신 걸 보고 많은 걸 배울 수 있었다. |
창오 | - 용석님과 주영님 코드를 보면서 상태 관리, 모듈화 파트에서 제가 많이 부족하다는 걸 느껴서 다음 스텝에서 꼭 보완하려고 합니다.
- 별개로 소모임에서 주도적으로 이야기 잘 이끌어주시는 상민님, 주영님, 보현님께 감사합니다 |
HAN(한아름) | 1. 느낀 점 & 새롭게 알게 된점
- 조건 문이 여러개로 나눴을 때 return 해주면 뒤에 불필요한 if 문을 연산하지 않는다고 한다. return이 쓰이는 그 위치를 공부해봐야겠다.
2. 반영하고 싶은 1가지
- 한 개 이상 모듈 분리하기!
- 다른 팀의 코드도 한 번씩 다 보기! |
nelldklee(이동규) | 서로 다른 코드를 보면서 내 코드에 적용해보고 싶은 부분이 많다는게 우선 좋았습니다.
step3에선 스스로 많이 고민하고, 다른 분들이 짠 코드를 비교하며 적용해봐야겠습니다. |
Rita | 팀에서 MVC패턴, PubSub패턴으로 작성된 코드를 구경할 수 있어서 신기했습니다! 타입스크립트를 적용하신 분도 계셔서 많이 배웠습니다
제가 작성한 코드에서 함수를 인자로 받는 setMenuList를 구현했는데, 관련해서 클로저/순수함수/함수형에 대해 좀 더 고민해보려고 합니다. |
Yummy ( 염상권 ) | 1. 서로 다른 분들의 코드를 보면서 새로운 개념을 많이 알아가는 것 같습니다. 특히 다른 분들의 코드리뷰를 보는것도 큰 공부가 되는것 같습니다.
2. 파일 목적에 맞게 나누기 그리고 변수명 & 코드 가독성 다시 한번 생각하기.... |
은선 | 기존에 있는 디자인패턴을 어떻게 적용하면 좋을까 고민하는 것도 좋지만, 멘땅에 해딩해보면서 어떻게 하다가 그러한 패턴들이 나오게 되었는지를 고민해보면 더 좋을 것 같네요! 그리고 변수명 짓기는 언제나 쉬워보이지만 가장 어려운 일인 것 같아요 |
4주차 목표
⇒ 스텝3 미션 진행 후 PR 올리고, 팀원 분들과 서로 체크해주기
체크 리스트
스텝3 미션을 진행하고 Pull Request를 규칙에 맞게 보냈는가? (https://makerjun.notion.site/b83f1f42a51a431ebe95cba7a4498dd1 참조)
질문, 배울만한 점 1가지 이상 남기기
컨플릭트 나는 파일은 없는가?
Q&A
체크아웃 & 개선을 위한 피드백
디스코드 닉네임 | 체크아웃 점수
(1~10) | 이유 & 스터디 피드백 | 1. 버퍼 기간을 가진다
2. 이어서 진행하고 마지막 주차에 찐하게 회고하자 |
BO(강보현) | 10 | 사실 오늘 비도 오고 할 일도 너무 많아서 벅찬 상태였는데, 또 같은 공부를 하는 분들과 인터랙션하면서 좋은 기운 받아가는 거 같습니다!
다음 주에는 꼭 PR을 미리 올려서 더 좋은 리뷰들 받아가고 많은 공부할 수 있도록 노력해야 겠습니다! | 1 |
Ysw | 10 | 어색어색 하더라도, 열심히 아이스브래이킹 해주시는 분들이 있어서 즐겁게 스터디 할 수 있었습니다!! 감사합니다 | 2 |
용석 | 9 | 저희팀에 진행을 너무 잘해주시는 분(BO)이 계셔서 항상 즐겁게 스터디하고 가요 !!
ㄴ 용석님 감사합니다... (머쓱) - BO | 2 |
초롱 | 8 | 강의를 듣지 않고 짜신 코드를 봤는데 잘 이해를 못해서 약간 속상하네요. | 1 |
치이즈 | 10 | 다른분들의 코드를 보면서 배워가는게 정말 많습니다! | 1 |
김정환 | 10 | 사실 완벽한 정답이라는 게 없긴 하겠지만 아직도 많이 헷갈리네요 ㅠ 더 좋은 코드를 공부하는 것도 좋지만 커뮤니케이션 하는 능력을 좀 더 키워보고 싶다는 생각이 들기도 합니다!! | 2 |
승희 | 8 | 오랜만에 바닐라 자바스크립트를 사용하면서 프로젝트를 하고 있어서 다시 배우며 몰랐던것들을 배울수 있어서 좋았습니다! | 2 |
트루워터 | 10 | 오늘도 코드리뷰를 통해 많이 배웠습니다! | 2 |
최원준 | 10 | 다른 분들 코드를 보면서 항상 많이 배우는 것 같습니다. | 1 |
주영 | 10 | 오늘 코드 리뷰도 좋았고 진행도 재밌었던 것 같아요. 감사합니다!(BO)
ㄴ 웁스.. 다들 너무 감사합니다! 다음 주에 PR 리뷰하러 출동합니다잉~! - BO | 2 |
Jay | 8 | 다음번엔 더 많이 고민하고 참여하고싶습니다~ 이번에 얻은 것들 반영해서 보완 할 수 있는 시간 가지려고 합니다. | 1 |
쌤(상민) | 10 | 많은 것을 새로 배우고 느낄 수 있는 시간이었습니다. 덕분에 새로 공부할 것을 찾았습니다! | 1 |
창오 | 10 | 조금 피곤한 채로 들어왔는데 팀원분들 열정 덕분에 기분 좋게 체크아웃합니다!
서로 코드 읽고 리뷰 하느라 정신없어서, 팀원 분들과의 이야기는 중간에 어떻게 자연스럽게 나눠야 될지 고민될 때가 있었어요.
최고의 진행자 보현님.
ㄴ 창오님 감사합니다... (머쓱) 다음에 PR리뷰 꼭 해드리러 갈게요!(티켓 하나 발행~!) -BO | 1 |
HAN(한아름) | 9 | 다른 분들 코드를 읽는 것만으로도 공부가 됩니다!
그리고 이해하지 못한 코드를 봤을 때 의욕이 생깁니다! 이 코드를 이해하고 싶어서 더 공부하고 싶어집니다! ㅎㅎ
열심히 코드리뷰 해주신 모든 분들 감사합니다~ | 1 |
nelldklee(이동규) | 준비한 만큼 얻어가는 것 같아요. 다음 주차엔 좀 더 준비하고 고민해서 깨닫을 부분을 많이 얻어가야겠습니다.! | 1 | |
Rita | 10 | 코드리뷰하면서 많이 배웠습니다! | 2 |
Yummy ( 염상권 ) | 10 | 저의 부족함을 많이 알게 되어, 공부할게 많아진 것 같아 좋습니다! 다음 주차도 최대한 열심히!! | 2 |
은선 | |||
다리 | 7 | 준비가 부족해서 염탐만 했지만, 다른 사람들은 어떻게 코드를 짜는지 보는게 매우신기하고 좋았습니다. | 1 |
스터디 안내
일정 소개
11월 16일 화 - OT
11월 24일 수 - 1주차 세션
11월 30일 화 - 2주차 세션
12월 7일 화 - 3주차 세션 (버퍼)
12월 14일 화 - 4주차 세션
