Skip to content

#26 : Week6_지윤이티 - #30

Merged
ziy00n merged 5 commits into
mainfrom
jiyoon
Jun 14, 2023
Merged

ziy00n merged 5 commits into
mainfrom
jiyoon

Conversation

@ziy00n

@ziy00n ziy00n commented May 24, 2023

Copy link
Copy Markdown
Contributor

image

image

@plum-king plum-king left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

적록색약의 경우, 대부분 잘 풀어주신 것 같은데 일부 수정이 필요하여 리뷰 달아두었습니다. 범위는 모두 n으로 맞춰주시고 if문 하나 더 추가하셔야 되는 부분까지 수정해주시면 답 제대로 출력될 것 같아요! n단 논법도 플로이드 알고리즘 잘 사용하셔서 풀어주신 것 같습니다!! 3중 for문이 돌아가는 횟수도 잘 넣어주신 것 같아요! 문제 푸시느라 고생하셨습니다 :)

Comment thread 이티지윤/10026_적록색약.py Outdated
Comment on lines 16 to 17

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

graph = [list(input().rstrip()) for _ in range(n)]

이런 식으로 범위를 n+1 말고 n으로 수정해주시면 될 것 같아요! 아래에서 for문 돌 때도 마찬가지로 범위를 n으로 맞춰서 수정해주시면 될 것 같아요!

Comment thread 이티지윤/10026_적록색약.py Outdated
Comment on lines 43 to 45

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if (n > newX >= 0) and (n > newY >= 0):

이 부분 추가해주셔서 newX와 newY에 대한 조건이 만족할 때만 해당 코드를 실행할 수 있도록 제어해주도록 수정하시면 좋을 것 같아요!

Comment thread 이티지윤/10026_적록색약.py Outdated
Comment on lines 58 to 59

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

R과 G의 위치가 바뀐 것 같아요!

@plum-king plum-king left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

랜선 자르기는 배운 이분 탐색 내용 잘 사용해서 풀어주신 것 같고, 회의실 배정은 그리디 알고리즘의 개념 사용해서 잘 풀어주신 것 같아요! 문제 푸시느라 고생하셨습니다 :)

@ziy00n
ziy00n merged commit 17f87a3 into main Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants