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

jaehyun: CleanCode Chapter 4~8 #16

Merged
merged 1 commit into from
May 29, 2023
Merged

jaehyun: CleanCode Chapter 4~8 #16

merged 1 commit into from
May 29, 2023

Conversation

RePro1202
Copy link
Collaborator

너무 늦어서 죄송합니다..

너무 늦어서 죄송합니다..
@RePro1202 RePro1202 added this to the CleanCode milestone May 4, 2023
@RePro1202 RePro1202 self-assigned this May 4, 2023
@RePro1202 RePro1202 linked an issue May 4, 2023 that may be closed by this pull request
Copy link
Contributor

@fkdl0048 fkdl0048 left a comment

Choose a reason for hiding this comment

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

👍👍👍👍👍

Comment on lines +125 to +129
논의 사항
=============

저는 다른 사람이 가져다 써야 하는 기능을 만들 때 뭐라도 주석으로 설명하고 싶은 마음이 자주 들었습니다..
주석을 특별히 자주 사용하게 되는 경우가 있으신지 궁금합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

책에서 나오는 내용과 같이 변명을 할 때 자주 사용되는 것 같습니다..ㅠ

협업할 경우 특히..

Comment on lines +100 to +106
논의 사항
=============
``` return b*b - 4*a*c ```
수식을 작성할 때 책처럼 가로 공백을 신경쓰시는지 궁금합니다.

``` return (b * b) - (4 * a * c) ```
저는 보통 위 같이 그냥 다 공백을 주고 괄호로 묶어 버리는데 갑자기 궁금하네요..
Copy link
Contributor

Choose a reason for hiding this comment

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

저는 개인적으로 아래가 좀 더 좋은 것 같습니다..?

그렇다고 위가 이상하다는 게 아니라 잘 읽히긴 해서..

Comment on lines +59 to +63
논의 사항
=============
깨끗한 코드는 읽기도 좋아야 하지만 안정성도 높아야 한다고 하지만, 책에서 설명한 것처럼 잘 짜여진
예외 처리를 구성하는건 작은 프로젝트 일수록 우선순위가 밀리는 것 같습니다..
현실적으로 예외 처리를 기능구현 정도의 중요도를 가지고 진행할 수 있을지 어떻게 생각하시는지 궁금합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

아직 작은 프로젝트의 한 부분이라고 생각합니다.. 실제 현업전에 예외처리를 제대로 사용하는 분은 적을 것 같고 팀의 규정에 따라 필요한 부분에 예외가 들어갈 것 같습니다.

미팅때 말씀드린 I/O, 파일, 서버 관련을 저는 가장 많이 다룬 것 같습니다

Comment on lines +55 to +57
논의 사항
=============
외부 라이브러리 등을 사용할 때 내부 구현 코드까지 살펴보신 경험이 있으신지 궁금합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

내부 구현 코드까지 살펴봤지만.. 그렇게 드라마틱한 좋은 경험은 아닌 것 같습니다.

지금도 정보는 쏟아지기 때문에 잘 골라서 습득 해야하는 부분이 있는 것 같습니다.

=============

저는 다른 사람이 가져다 써야 하는 기능을 만들 때 뭐라도 주석으로 설명하고 싶은 마음이 자주 들었습니다..
주석을 특별히 자주 사용하게 되는 경우가 있으신지 궁금합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

협업할 때, 특히 코드 리뷰 하기 전에 난잡한 제 코드를 설명할 때 자주 사용하게 됩니다..

수식을 작성할 때 책처럼 가로 공백을 신경쓰시는지 궁금합니다.

``` return (b * b) - (4 * a * c) ```
저는 보통 위 같이 그냥 다 공백을 주고 괄호로 묶어 버리는데 갑자기 궁금하네요..
Copy link
Collaborator

Choose a reason for hiding this comment

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

저도 아래를 좀 더 좋아합니다. 가로 공백이 없으면 정리되지 않았다고 느껴서 그런 것 같습니다.

=============
깨끗한 코드는 읽기도 좋아야 하지만 안정성도 높아야 한다고 하지만, 책에서 설명한 것처럼 잘 짜여진
예외 처리를 구성하는건 작은 프로젝트 일수록 우선순위가 밀리는 것 같습니다..
현실적으로 예외 처리를 기능구현 정도의 중요도를 가지고 진행할 수 있을지 어떻게 생각하시는지 궁금합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

아직 예외 처리가 필수라고 느껴진 프로젝트를 경험해보지 못했습니다. :0
기능 구현을 할 때 예외 처리를 같이 생각하는 것은 좀 더 성장한 이후에나 할 수 있는 일이지 않을까요.
지금의 저는 예외 처리를 기능 구현 보다 뒷 순위로 생각합니다.


논의 사항
=============
외부 라이브러리 등을 사용할 때 내부 구현 코드까지 살펴보신 경험이 있으신지 궁금합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

발생했던 오류를 도저히 모르겠어서 뒤적거리다 살펴본 경험은 있지만, 큰 도움은 되지 못했습니다.
이해하는데 시간도 어마어마하게 걸릴 것 같더라구요..

Copy link
Collaborator

@hellog2n hellog2n left a comment

Choose a reason for hiding this comment

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

잘 읽었습니당!

=============

저는 다른 사람이 가져다 써야 하는 기능을 만들 때 뭐라도 주석으로 설명하고 싶은 마음이 자주 들었습니다..
주석을 특별히 자주 사용하게 되는 경우가 있으신지 궁금합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

협업을 하게 될 때 코드가 도저히 알아볼 수 없는 하드코드라고 생각될 때, 설명을 덧붙입니다 ㅎ..
저희 같은 경우는 연구를 하게 될 때 나중에 코드를 공개하는 경우가 있는데, 그 때 주석을 자주자주 붙여서 독자(?)들이 이해하기 쉽게 해야하는 것 같기도 합니다 ㅎ

``` return b*b - 4*a*c ```
수식을 작성할 때 책처럼 가로 공백을 신경쓰시는지 궁금합니다.

``` return (b * b) - (4 * a * c) ```
Copy link
Collaborator

Choose a reason for hiding this comment

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

저희는 무조건 가로를 표기합니다!
혹시나 수식 연산에 문제가 생길까봐 좀더 보수적으로 확실하게 묶기도 합니다.

=============
깨끗한 코드는 읽기도 좋아야 하지만 안정성도 높아야 한다고 하지만, 책에서 설명한 것처럼 잘 짜여진
예외 처리를 구성하는건 작은 프로젝트 일수록 우선순위가 밀리는 것 같습니다..
현실적으로 예외 처리를 기능구현 정도의 중요도를 가지고 진행할 수 있을지 어떻게 생각하시는지 궁금합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

저도 예외처리는 파일, 센서 I/O쪽 등만 해보았고, 사실 다른 쪽은 잘 해보지는 않았네요...

다양한 입력출력값이 오고가는 부분에서 하면 좋을 것 같습니다. (예를 들면 null이 들어올 수도 있는 확률이 높은 구간)


논의 사항
=============
외부 라이브러리 등을 사용할 때 내부 구현 코드까지 살펴보신 경험이 있으신지 궁금합니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

살펴보면서 수정까지 했던 경험이 있습니다. 근데 완전 내부까지 좀더 디테일하게는 들어가보지는 않았어요!

@RePro1202 RePro1202 merged commit 04f3bc3 into main May 29, 2023
@RePro1202 RePro1202 deleted the jaehyun/CleanCode branch June 1, 2023 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

<Clean Code> 4장 ~ 8장, 2023-05-04
5 participants