-
Notifications
You must be signed in to change notification settings - Fork 47
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
전남대 FE_정서윤 5주차 과제 Step3 #78
Open
yunn23
wants to merge
1
commit into
kakao-tech-campus-2nd-step2:yunn23
Choose a base branch
from
yunn23:step3
base: yunn23
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,11 @@ | ||
# 카카오 테크 캠퍼스 - 프론트엔드 카카오 선물하기 편 | ||
### 5주차 과제 | ||
|
||
- 질문 1. Test code를 작성해보면서 좋았던 점과 아쉬웠던 점에 대해 말해주세요. | ||
- 죻았던 점 : 모든 요소를 일일이 작동시켜보지 않아도 테스트 코드를 통해 검토할 수 있다는 점이 좋은 것 같다. 테스트 코드를 작성하며 자연스럽게 코드가 잘 작성되었는지 확인할 수 있었다. 또, 수정 이후에 다시 모든 컴포넌트를 작동시켜보는 과정을 단축할 수 있어 좋은 것 같다. | ||
- 아쉬웠던 점 : 개발 외에도 테스트 코드를 작성하는데 시간이 들기 때문에 이 부분이 큰 투자인 것 같다. 테스트 코드가 통과해서 넘어갔던 부분에서 예기치 않은 오류가 발생할 수 있으므로 실제로 잘 작동하는지 다시 검토하는 과정을 아예 제외할 수는 없을 것 같다. | ||
- 질문 2. 스스로 생각했을 때 좋을 컴포넌트란 무엇인지 본인만의 기준을 세우고 설명해주세요. | ||
- 좋은 컴포넌트란 재사용이 가능하고 확장성이 좋으며, 되도록 한가지 역할만을 수행하는 컴포넌트인 것 같다. 재사용이 가능하게 함으로써 코드를 효율적으로 작성할 수 있으며, 코드 가독성도 향상된다. 확장성이 좋은 컴포넌트는 같은 내용을 또 다시 작성하지 않고도 속성만을 바꿔 여러 곳에 적용시킬 수 있는 코드를 의미한다. 이는 같은 기능을 수행하는 다른 UI 컴포넌트에 적용시켜 코드의 유연성을 높일 수 있다. 또, 컴포넌트가 한 가지 역할만을 수행하게 하므로써 다른 코드와의 결합도를 낮춰 오류 발생 가능성을 줄이고, 코드의 가독성이 향상된다. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 재사용성에 대해서 한번 생각해보시면 좋을 리뷰 사례가 있어서 공유드려요. #94 (comment) |
||
- 질문 3. 스스로 생각했을 때 공통 컴포넌트를 만들 때 가장 중요한 요소 2개를 선택하고 이유와 함깨 설명해주세요. | ||
- 적당한 확장성 (재사용성과 유연성) : 색상, 너비와 높이, 폰트 크기 등 스타일에 있어서 확장성을 두고, 기능에 있어서는 확장성을 덜 주는게 좋은 것 같다. 여러 곳에서 사용하기 위해서 확장성을 너무 크게 설정하면 그 만큼 예기치 못한 오류가 발생할 수 있으므로 적당히 확장성을 두는 것이 좋다고 생각한다. | ||
- 일관성 : 앱의 전반적인 분위기와 디자인에 맞게 다른 컴포넌트와 조화를 이루는 것 또한 중요하다고 생각한다. 비슷한 기능이면 일관된 UI를 사용하는 것이 사용자의 경험 향상에 도움이 될 것이다. |
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트 코드 작성에 조금 익숙해지다보면 테스트 "코드" 보다는 테스트 자체에 집중해서 한번 공부해보시는게 시간이 드는 부분을 줄여볼 수 있을 거에요.