Skip to content
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

[그리디] 3월 26일 #6

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[그리디] 3월 26일 #6

wants to merge 2 commits into from

Conversation

Jiwoon-Yoon
Copy link
Collaborator

@Jiwoon-Yoon Jiwoon-Yoon commented Mar 26, 2024

인적사항

학번: 2276209
이름: 윤지운

과제제출

기존 제출: 1213, 17451
추가 제출: 15662

Copy link

@chaeri93 chaeri93 left a comment

Choose a reason for hiding this comment

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

[그리디 알고리즘 코드 리뷰 완료]

1213(P3), 17451(P3)

안녕하세요 지운님! 코드 리뷰 완료되었습니다👍
함수와 주석도 너무 좋았고, 문제도 깔끔하게 잘 풀어주셨어요!
몇 가지 사소한 코멘트 남겼습니다.
궁금하신 점이 있다면 리뷰어를 호출해주세요!


using namespace std;

const int ALPHA_NUM = 26;

Choose a reason for hiding this comment

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

많이 사용하는 숫자를 따로 상수로 선언해주신 점 좋아요!👍

Comment on lines +20 to +25
// 0도 2로 나누면 나머지 0이라 이 조건문 없으면 출력되기 때문!
if (cnt[i] == 0) {
continue;
}

if (cnt[i] % 2 != 0) {

Choose a reason for hiding this comment

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

두개의 if문을 합치면 더 가독성이 높을 것 같네요!

vector<int>cnt(ALPHA_NUM, 0);
cin >> name;

cout << makePalindrome(name, cnt);

Choose a reason for hiding this comment

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

함수 분리해서 구현하신거 너무 좋네요!

Comment on lines +8 to +17
for (long long i : d) {
if (i > min) {
min = i;
}
else {
min = ((min-1) / i + 1) * i;
// min-1 안해주면 틀림!
// 그냥 min 하면 같은 속도 연속으로 주어졌을 때 오류
}
}

Choose a reason for hiding this comment

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

문제의 조건을 잘 파악해서 최소 속도를 잘 찾아주셨네요!👍

Copy link

@aeyongdodam aeyongdodam left a comment

Choose a reason for hiding this comment

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

15662(P1)
안녕하세요! 지운님!
아직 15662번 코드 제출이 완전하지 않은 것 같습니다!
추후 수정한 후 리뷰어 호출을 부탁드려도 될까요? 감사합니다~

Copy link

@aeyongdodam aeyongdodam left a comment

Choose a reason for hiding this comment

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

15662(P3)
안녕하세요 지운님! 코드 리뷰 완료되었습니다👍
제출 후 리뷰 요청을 부탁드렸는데 해주셔서 감사합니다~!
주석 자세하게 달아주셔서 이해가 쉬웠습니다! 라이브코딩까지 완벽하게 정리하셨네요 최고~
몇 가지 사소한 코멘트 남겼습니다.
궁금하신 점이 있다면 리뷰어를 호출해주세요

vector<int>visited; //회전했는지 검사하는 벡터


void check_rotation(int n, int dir) {

Choose a reason for hiding this comment

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

P3: 앗 함수는 카멜케이스로 쓰는 것이 권장인데 제가 잘못 써서 저때문에 헷갈리셨나봅니다..!
강의자료에도 고쳐놓겠습니다!! 혼란드려 죄송해요ㅜㅜ 개인의 선호이지만 함수는 카멜 케이스로 하는 것을 권장드립니다!


using namespace std;

const int SAW_NUM = 8;

Choose a reason for hiding this comment

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

P3: 상수 선언 너무 좋습니다 최고~

Comment on lines +67 to +76
if (tmp_dir == 1) { //시계방향 회전
//0번 -> 1번, 7번 -> 0번
li[tmp_num] = li[tmp_num].back() + li[tmp_num].substr(0, 7);
//맨 뒤 7번 인덱스 붙이고 0~6번 인덱스 붙임 (0번째위치부터 7자리)
}
else { //반시계방향 회전
//0번 -> 7번, 7번 -> 6번
li[tmp_num] = li[tmp_num].substr(1) + li[tmp_num][0];
// 1~7번 붙여주고 0번 인덱스 붙여줌
}

Choose a reason for hiding this comment

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

P3: 주석을 꼼꼼하게 달아주셔서 보기 편했습니다~!

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.

3 participants