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

[스택, 큐, 덱] 9월 15일 #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[스택, 큐, 덱] 9월 15일 #2

wants to merge 1 commit into from

Conversation

minji1289
Copy link
Collaborator

내용 & 질문

스택, 큐, 덱 과제 제출합니다!

<기존 제출>

문제 번호 11866, 18115

<추가 제출>

Copy link

@jk0527 jk0527 left a comment

Choose a reason for hiding this comment

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

P2. 11866 코드리뷰 완료

민지님 안녕하세요! 문제 정말 잘 풀어주셨어요!!😎😎
함수화와 출력 관련하여 사소한 코멘트 남겼습니다!!
궁금한 점이 있으면 리뷰어 호출해주세요!! 수고하셨습니다!


cout << "<";

while (!q.empty())
Copy link

Choose a reason for hiding this comment

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

연산을 하는 부분은 함수화해도 좋을 것 같아요. 함수에서 vector에 원소를 순서대로 저장하고
vector를 main에서 받아서 한 번에 출력하면 어떨까요?

Comment on lines +29 to +30
if(!q.empty())
cout << ", ";
Copy link

Choose a reason for hiding this comment

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

< , >를 사이사이에 출력해야 해서 출력 조건이 조금 복잡하게 들어갔네요..! 앞서 말씀드린 것처럼 vector에 저장한 후에 출력하면 좀 더 코드가 깔끔해질 것 같아요!!

Copy link

@junghk0115 junghk0115 left a comment

Choose a reason for hiding this comment

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

p2. 18115 코드리뷰 완료
안녕하세요 민지님!
함수화 관련 코멘트 남겼습니다 참고해주세요!
수고하셨습니다~~

Comment on lines +33 to +52
for (int i = n - 1; i >= 0; i--) {
if (arr[i] == 1) {
//맨 뒤에 push
deq.push_back(card);
}
else if (arr[i] == 2) {
int end = deq.back();
deq.pop_back();
//맨 뒤에서 두번째에 push
deq.push_back(card);
deq.push_back(end);
}
else if (arr[i] == 3) {
//맨 앞에 push
deq.push_front(card);
}

//순서대로 1,2, ..., N.
card++;
}

Choose a reason for hiding this comment

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

p2. 해당 부분은 문제 풀이의 핵심 부분이니 함수화 하시는 것을 권장드립니다!
참고로 코딩테스트에선 대부분 main이 제공되지 않고 solution 함수만 제공되어 정답을 리턴하는 형태이니 함수화에 익숙해지시면 좋을 것 같습니다!

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