Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sangbooom
requested review from
hyesungoh,
100Gyeon,
Dongkyuuuu,
gutenLEE,
metacode22,
Orchemi,
Seojunhwan,
sjoleee and
sumi-0011
as code owners
100Gyeon
approved these changes
Sep 1, 2024
Comment on lines
+25
to
+26
| - 더 경험이 적은 프로그래머에게 코드를 자세히 설명할 수 있어야 된다. 그렇지 않다면 우연에 기대고 있을 확률이 있다. | ||
| > 자세히 설명한 적은 없었고 대부분 간단한 설명과 함께 참고했던 레퍼런스를 공유했었다. 경험이 적을 뿐 대부분 뛰어난 실력을 가지고 계셔서.. |
| 가장 빠른 알고리즘이 언제나 좋은 알고리즘은 아니다. 입력값의 규모가 작다면 단순한 삽입 정렬도 퀵 정렬과 비슷한 성능을 냄 | ||
| 그니까 '성급한 최적화' 조심하자. 어떤 알고리즘을 개선하느라 우리의 귀중한 시간을 투자하기 전에 그 알고리즘이 정말로 병목인지 먼저 확인하는게 좋다. | ||
|
|
||
| > 코드 리뷰에서 흔히 일어나는 상황이라 생각한다. 이론적으로는 성능상 이점(수행시간 줄어듬)은 있긴한데.. 입력값의 규모가 작아서 눈에 띄는 이점은 없고 오히려 복잡도가 증가하는 경우가 더러 있다 (이런 경우 있지 않나요..?) |
There was a problem hiding this comment.
맞아요.. 이런 논쟁을 하다가 결국 코드 리뷰에서 결론이 안 나면 프론트엔드 위클리까지 가져가곤 합니다🤯
| 2. 코드와 테스트를 함께 | ||
| 3. 나중에 테스트 === 테스트 하지 않음 | ||
|
|
||
| 45년차가 30년동안 테스트 코드를 써왔는데 나중엔 테스트를 쓰지 않고도 테스트에 대해 생각할 수 있게 됨 ㄷ ㄷ |
Comment on lines
+102
to
+103
| 디버깅 정보는 공격 매개체 될 수 있음 | ||
| > 종종 웹사이트에서 브라우저 콘솔에서 데이터 다 보여주는 경우 있음 |
There was a problem hiding this comment.
(이건 좀 다른 이야기지만) 오늘의집 콘솔에 있는 고양이 귀여워요
There was a problem hiding this comment.
서핏은 개발자 도구에 프론트엔드 개발자 채용 공고를 띄워두더라고요 ㅋㅋㅋ
hyesungoh
approved these changes
Sep 3, 2024
Comment on lines
+19
to
+22
| > 운 좋게 동작하고 있는 코드가 있었다. 시간은 한정적으로 주어진 상황에서 그 코드를 유지보수 해야된다면 리팩토링을 할 것인가? 아니면 요구사항 코드를 어떻게든 동작하도록 욱여넣을 것인가.. | ||
| > 리팩토링 한다면 이전에 동작하던 기능들이 사이드이펙트로 인해 동작하지 않을 수 있다. 그렇다고 구덩이를 보고 그냥 지나칠 것인가? 누군가는 밟고 넘어질 수 있다.. 그게 먼 미래의 나일수도 있다.. | ||
| > 요구사항 코드를 어떻게든 동작하게 만들고 주석만 달아놓는다면 오랫동안 쳐다보지 않을 확률이 높다.. | ||
| > 구덩이를 발견한 사람이 그 구덩이를 메우자.. 보이스카우트 규칙과 비슷 |
Comment on lines
+102
to
+103
| 디버깅 정보는 공격 매개체 될 수 있음 | ||
| > 종종 웹사이트에서 브라우저 콘솔에서 데이터 다 보여주는 경우 있음 |
There was a problem hiding this comment.
서핏은 개발자 도구에 프론트엔드 개발자 채용 공고를 띄워두더라고요 ㅋㅋㅋ
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.