[공예영] 15장, 16장 : 실용주의 장인정신, 소프트웨어 장인으로서의 커리어 #50
Merged
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
yeyounging
requested review from
hyesungoh,
bbbang105,
2unhi,
karnelll,
codrin2,
HoyeongJeon,
yeonjookang,
JeonHeeso and
nebulaBdj
as code owners
hyesungoh
approved these changes
Dec 27, 2024
| > 추상화는 필요할때만, 단순한 소프트웨어 만들기 | ||
| > | ||
|
|
||
| 작업하다보면 지금은 필요하지만 이정도는 필요해지지않을까? 하며 고려하며 만들 때가 있다. 이럴 때면 사실 내가 좀 더 많은 것을 고려하게 됐구나, 하고 성장한 기분이 들기도 했다. 하지만 오히려 이는 소프트웨어의 복잡도를 높이고 유지보수를 어렵게 할 수 있다는 시각을 알게 됐다. 언제든 필요한 기능을 붙일 수 있도록 하는 게 좋은 코드이고 리팩토링이 필요한 코드는 좋지 않은 코드라고 생각했었다. 하지만 어떤 기능이 필요하게 될지 모르기에 단순한 코드가 좋은 코드라는 것을 공감했다. 분석가능한 코드라면 리팩토링도 보다 쉬울 것이다. TDD 기반의 코드라면 안정성 또한 보장할 수 있으니 리팩토링의 부담감도 줄어든다. |
There was a problem hiding this comment.
YAGNI, KISS 와 같은 것들도 생각하면 좋을 거 같아용
You aint gonna need it, keep it simple, stupid
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.