Skip to content

Conversation

@kimeunsom
Copy link
Collaborator

###인적사항
학번 : 2271018
이름 : 김은솜

###과제제출
기존제출 : 10815, 14500, 16401, 2343, 3079
추가제출 :

Copy link

@chae-jpg chae-jpg left a comment

Choose a reason for hiding this comment

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

[이분 탐색 코드 리뷰 완료]

10815(P3), 16401(P3)

안녕하세요 은솜님! 코드 리뷰 완료되었습니다👍
주석 처리도 좋고, 문제도 깔끔하게 너무 잘 풀어주셨어요!
도전 문제까지 모두 풀어주셨네요... 너무 멋집니다👍
몇 가지 사소한 코멘트 남겼습니다.
궁금하신 점이 있다면 리뷰어를 호출해주세요!

Comment on lines +15 to +22

Choose a reason for hiding this comment

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

포인터를 움직이는 조건을 잘 찾아주셨어요!

Choose a reason for hiding this comment

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

이분 탐색을 위해서는 자료가 정렬되어 있어야 한다는 성질을 잘 이용해주셨네요!👍

Comment on lines +25 to +36

Choose a reason for hiding this comment

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

포인터를 움직이는 조건을 잘 찾아주셨어요!

Comment on lines +13 to +14

Choose a reason for hiding this comment

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

과자의 길이가 최소와 최대가 될 때를 각각 잘 찾아주셨네요!

Choose a reason for hiding this comment

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

매개변수 탐색으로 문제를 잘 풀어주셨네요!

Copy link

@sawoll sawoll left a comment

Choose a reason for hiding this comment

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

[이분탐색 필수 구현문제 코드리뷰 완료]
(P3)14500
안녕하세요 예린님, 14500번 문제 리뷰 완료했습니다. 문제 조건에 집중하여 아주 잘 풀어주셨어요! 간단한 코멘트 남겨드렸으니 확인 부탁드립니다. 수고하셨습니다.😊

Comment on lines +67 to +69
Copy link

Choose a reason for hiding this comment

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

return 뒤에 있는 코드를 ()괄호로 묶어 주시면 더 좋을 것 같아요.😊

Comment on lines +53 to +58
Copy link

Choose a reason for hiding this comment

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

ㅜ 모양 테트로미노를 고려한 코드를 잘 구현해주셨네요! 훌륭합니다.😍

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.

4 participants