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

[10주차] 08. 기능 이동 (8.5~끝) #9

Open
minkukjo opened this issue Nov 24, 2021 · 2 comments
Open

[10주차] 08. 기능 이동 (8.5~끝) #9

minkukjo opened this issue Nov 24, 2021 · 2 comments
Labels

Comments

@minkukjo
Copy link
Contributor

진도

  1. 기능 이동

방식

  • 코멘트로 자유롭게 느낀점 (좋았단 점) 을 적어주시면 됩니다.
  • 질문, 다른 좋은 참고자료, 실무 적용 사례 등을 공유해주셔도 좋습니다.
@minkukjo minkukjo added the share label Nov 24, 2021
@minkukjo minkukjo changed the title [10주차] 08. 기능 이동 (8.4~끝) [10주차] 08. 기능 이동 (8.5~끝) Nov 24, 2021
@minkukjo
Copy link
Contributor Author

minkukjo commented Dec 6, 2021

느낀 점

  • 8.5과 8.6은 당연한 이야기 였다. 특히 8.6은 왜 당연한 이야기를 구구절절 했을까? 싶은 의문..
  • 8.7과 8.8은 아주 좋았다. 특히 8.8의 반복문을 파이프라인 바꾸기는 하면서 혼자 감탄을 했다!
  • 8.9는 왠만하면 주석보다는 죽은 코드를 제거하는게 좋다~ 정도로 읽고 끝났다.

총평

  • 8.8의 반복문 파이프라인 바꾸기는 진짜 지린다.

@donghoon-song
Copy link

8.5 인라인 코드를 함수 호출로 바꾸기

8.6 문장 슬라이드하기

8.7 반복문 쪼개기

각각의 반복문으로 분리해두면 수정할 동작 하나만 이해하면 된다.

반복문을 두 번 실행하는 게 병목이라 밝혀지면 그때 다시 하나로 합치면 된다. 하지만 이런 경우는 매우 드물다.

반복문을 복제해놓고 중복코드를 제거한다.

8.8 반복문을 파이프라인으로 바꾸기

8.9 죽은 코드 제거하기

코드가 더 이상 사용되지 않는다면 지워야 한다. 남기고 싶다면 어느 리비전에서 삭제했는지를 커밋 메시지로 남긴다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants