Skip to content

[스택/큐/덱] 2월 26일#2

Open
song122333 wants to merge 4 commits intomainfrom
02-assignment
Open

[스택/큐/덱] 2월 26일#2
song122333 wants to merge 4 commits intomainfrom
02-assignment

Conversation

@song122333
Copy link
Collaborator

###인적사항
학번: 2391035
이름: 한송희

###과제제출
기존제출 2164, 4949, 10757

Comment on lines +7 to +8

Choose a reason for hiding this comment

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

p3. 저희 코드 규칙에 변수는 소문자로 시작하는 camelCase, 상수는 대문자 + snake 표기법을 쓰기로 되어 있어요! 컨벤션에 맞추어 변수는 소문자로 작성해주시면 더 좋을거 같아요!

Choose a reason for hiding this comment

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

죄송합니다. 변수는 소문자 snake_case입니다!

Comment on lines +11 to +12

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.

마지막 올림수까지 반영 잘 해주셨네요!

Choose a reason for hiding this comment

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

전반적으로 잘 푸셨습니다☺️
다만, 현재 github 상에서 주석이 깨져있습니다. 인코딩 문제로 보이는데, IDE에서 인코딩 형식을 바꾸어 해결할 수 있으니 링크 참고해서 설정해보세요! 설정 상 어려운 점이 있으시면 언제든지 편하게 질문해주세요!!

Copy link

@ruruisryu ruruisryu left a comment

Choose a reason for hiding this comment

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

안녕하세요 송희님! 4949번, 2164번 로직 잘 구현해주셨어요! 사소한 코멘트를 드렸으니 참고하셔서 개인적으로 수정해보시면 좋을 것 같아요. 수고하셨습니다~

Comment on lines +10 to +11

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.

큐 자료구조를 적절히 잘 사용해주셨네요!

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.

틀린 표현은 아니지만 로직 상 Q.size() > 1 이 더 적합한 표현일 것 같습니다!

Comment on lines +22 to +26

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 +9 to +12

Choose a reason for hiding this comment

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

while(getline(cin, input)) 으로도 줄일 수 있을 것 같아요!

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