-
Notifications
You must be signed in to change notification settings - Fork 0
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
[그리디] 10월 23일 #10
base: main
Are you sure you want to change the base?
[그리디] 10월 23일 #10
Conversation
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.
p3. 3613 코드 리뷰 완료
안녕하세요 현아님~
문제 푸시느라 정말 고생 많으셨어요!!👍👍
코드에 주석이 잘 달려있어서 코드를 읽기 편했습니다.
몇가지 코멘트를 남기자면, changeType() 함수가 지금 하는 일이 많은 것 같습니다. 조금 분리해주면 좋을 것 같습니다.
또한, islower(), isupper() 등의 함수도 이용해보면 좋을 것 같습니다.
궁금한 점 있으시면 리뷰어를 호출해주세요~
수고하셨습니다!!
bool error_flag = false; | ||
int type = 0; | ||
vector<string> str; |
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.
p3. 굳이 필요하지 않은 변수 선언인 것 같습니다.
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.
11399 코드리뷰 완료
ATM은 문제자체는 해결법이 간단하지만, 시간복잡도 개선의 여지가 있어서
이를 포인트로 잡고 출제한 문제였는데 O(N)만에 훌륭하게 잘 해결해주셨네요!
오늘도 직접 정성껏 풀이하신게 느껴지는 코드 잘 보고 갑니다!
현아님 코드는 항상 새로워서 리뷰어도 배워가는게 많은 것 같아요 ㅎㅎ
그리디 과제 수고많으셨습니다~!
sort(times.begin(), times.end()); | ||
|
||
int sum = 0, tmp = 0; | ||
|
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.
👍👍👍
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.
10610 코드리뷰 완료
안녕하세요 현아님!
제출해주신 코드 잘봤습니다!
준비한 코드와는 또 다른 방식으로 풀이해주셨어요~~👍👍
샘플 코드에서는 정렬을 먼저 한 후에 스트링의 마지막 원소에 접근하여 확인하였는데,
현아님께서는 각 자리수를 더하면서 체크해주셨네요!!
저도 다른 방법이 있다는 걸 배우고 갑니다😊😊
수고하셨어요!!💕💕
내용 & 질문
<기존 제출>