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
|
P3. 20923 코드 리뷰 완료 |
Dong-droid
reviewed
Oct 3, 2022
Comment on lines
+6
to
+10
|
|
||
| int n, m; | ||
| string final_winner; | ||
| deque<int> dodo, suyeon, do_ground, su_ground; | ||
|
|
There was a problem hiding this comment.
P3. 클린 코드를 위해 전역 변수 사용을 지양해주시기를 부탁 드립니다! 😸 🐈
Dong-droid
reviewed
Oct 3, 2022
| if (do_ground.front() == 5) return 'd'; | ||
| if (su_ground.front() == 5) return 'd'; | ||
| if (do_ground.size() && su_ground.size() && dodo.front() + suyeon.front() == 5) return 's'; | ||
| return 0; |
Dong-droid
reviewed
Oct 3, 2022
Comment on lines
+41
to
+65
| if (winner == 'd') { | ||
| while (!su_ground.empty()) { | ||
| //수연이 그라운드 카드가 남을떄까지 뒤에서부터 빼서 뒤로 넣기 | ||
| dodo.push_back(su_ground.back()); | ||
| su_ground.pop_back(); | ||
| } | ||
| while (!do_ground.empty()) { | ||
| //본인 그라운드 카드 넣기 | ||
| dodo.push_back(do_ground.back()); | ||
| do_ground.pop_back(); | ||
| } | ||
|
|
||
| } else if (winner == 's') { | ||
| while (!do_ground.empty()) { | ||
| suyeon.push_back(do_ground.back()); | ||
| do_ground.pop_back(); | ||
| } | ||
| while (!do_ground.empty()) { | ||
| //본인 그라운드 카드 넣기 | ||
| suyeon.push_back(su_ground.back()); | ||
| su_ground.pop_back(); | ||
| } | ||
|
|
||
| } | ||
|
|
There was a problem hiding this comment.
P3. 카드를 합치는 부분은 함수화 하는 것을 제안드립니다!!
합쳐지는 source 카드와 destination 카드 덱만 다를 뿐 동일한 일을 하고 있지 않나요?
함수화하면 더 🌷 한 코드가 될 것 같아요!
flowersayo
reviewed
Oct 4, 2022
| for (int i = 0; i < n; i++){ | ||
| ans = max(ans, dp[i]); //가장 긴 길이찾기 | ||
| } | ||
|
|
There was a problem hiding this comment.
참고하기 이전 코드를 올려주셨다고 했는데 수정 이후 코드가 올라온 것 같네요..!
다시 올려주시면 한번 더 피드백 드리겠습니다~! 지금 코드는 정말 완벽해요🥰
There was a problem hiding this comment.
안녕하세요! 11053번은 제대로 제출한게 맞고, 20923번이 백준에서 틀리게 나오는건데 착각했네요!!
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.
내용 & 질문
이후에 블로그 글 참고한 코드를 다시 제출을 해서 백준에선 맞게처리는 되었으나, 참고하기 이전 코드가 왜 틀린지 모르겠어서 이전 코드로 제출했습니다..!!
사정 상 제 컴퓨터 사용을 못해서, 깃허브 페이지 자체에서 파일을 올리게 되어 폴더안에 파일을 넣지 못했고, 형식이 좀 다를 수 있을것 같습니다.. 검토 받은뒤에 수정해서 머지하도록 하겠습니당
<기존 제출>
<추가 제출>