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

[구현 & 코너케이스] 09월 20일 #4

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

Conversation

uommou
Copy link
Collaborator

@uommou uommou commented Sep 26, 2022

내용 & 질문

ex. 제출합니다!

<기존 제출>

1316, 2840, 8911, 5397

<추가 제출>

@uommou uommou changed the title [09월 20일] 구현 & 코너케이스 [구현 & 코너케이스] 09월 20일 Sep 26, 2022
@Dong-droid
Copy link

p3. 5397 코드 리뷰 완료
먼저 수고하셨습니다!
stack으로 구현하기가 좀 까다로운 문제였는데 너무 잘 푸신 것 같습니다!
다른 풀이 방법으로 list를 사용할 수도 있어요 왜냐하면, list의 erase와 insert 함수는 O(1)의 시간이 걸려서 안전합니다!
리뷰를 남겼으니 참고하시기 바랍니다!!

Comment on lines +10 to +17

while (!front.empty()) {
front.pop();
}
while (!back.empty()) {
back.pop();
}

Choose a reason for hiding this comment

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

p3. 초기화할 필요가 없는 것 같아요! 매 s 마다 스택이 생성되기 때문입니다

front.pop();
}

string password;

Choose a reason for hiding this comment

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

p3. 변수 선언은 stack 선언할 때 같이 묶어서 해주면 좋을 것 같습니다.
수고하셨습니다!!!!

Copy link
Member

@dooli1971039 dooli1971039 left a comment

Choose a reason for hiding this comment

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

p3. 2840 코드 리뷰 완료
안녕하세요 채원님~
문제 푸시느라 정말 고생 많으셨어요!!👍👍
코드가 깔끔하서 읽기가 정말 편했습니다. 정말 문제를 잘 풀어주셨어요.
getWheel의 코드가 길어지니까, 좀 더 간단하게 하기 위해 37번째 줄부터 결과를 만드는 부분을 함수화를 시키도 좋을 것 같습니다.
궁금한 점 있으시면 리뷰어를 호출해주세요~
수고하셨습니다!!

Copy link

@flowersayo flowersayo left a comment

Choose a reason for hiding this comment

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

p2
1316 코드리뷰 완료

안녕하세요 서현님!

코드 흐름이 정말 좋아서 리뷰드리기가 정말 편해요🥺👍
사소한 커멘트 몇개 남기고 갑니당😚
p1 코멘트는 꼭 수정해주셨으면 하는 부분이고, p2/p3는 참고만해주셔도 괜찮습니다!
피드백 반영하고 싶으시면 코드 수정하시고 저 리뷰어로 호출해주세요~!
이번 구현 코너케이스 과제도 정말 수고 많으셨습니다!🥰

}
temp = s[i];
}
return true;

Choose a reason for hiding this comment

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

p2. 특정 알파벳을 이미 방문한 경우와 연속된 문자열 아닌 경우를 먼저 처리하여 return 하면 인덴테이션이 깊어지지 않을 수 있어요!

Copy link

@jaeseo222 jaeseo222 left a comment

Choose a reason for hiding this comment

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

p1. 8911 코드 리뷰 완료

안녕하세요 채원님🤗

방향 배열을 쓰지 않고도 너무 깔끔하게 잘 구현해주셔서 놀랐습니다!!!🤩🤩
문제에서 뭘 요구하는 지 꼼꼼하게 잘 캐치 해주신 점이 느껴졌습니다☺️☺️

그런데, 8911과 비슷한 방향 구현 문제들과 비롯해 참고하시면 좋을 것 같아
방향 배열 에 관련한 피드백 남겼습니다!!

수고 많으셨습니다. 감사합니다.

Comment on lines +9 to +38
vector<int> point(4, 0); // �� �� �� �� ���

for (int i = 0; i < s.length(); i++) {
if (s[i] == 'F') {
point[dir % 4]++;
point[(dir + 2) % 4]--;
}
else if (s[i] == 'B') {
point[dir % 4]--;
point[(dir + 2) % 4]++;
}
else if (s[i] == 'L') {
dir += 3;
}
else if (s[i] == 'R') {
dir++;
}

if (point[0] > top) {
top = point[0];
}
if (point[1] > right) {
right = point[1];
}
if (point[2] > bottom) {
bottom = point[2];
}
if (point[3] > left) {
left = point[3];
}

Choose a reason for hiding this comment

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

p2. 조건에 따라 방향을 잘 구현해주셨네요!!✨✨
현재 방법도 매우 좋지만, 방향은 방향 배열로 구현해주시면 구현이 편해질 거에요! 보통, 방향 배열은 아래처럼 많이들 표현합니다!

//북 동 남 서
int dx[4] = { 0, 1, 0, -1 };
int dy[4] = { 1, 0, -1, 0 };

위와 같이 선언해주시면 방향 배열의 인덱스를 활용해 회전을 구현할 수 있습니다. (채원님께서 구현해주신 것과 비슷한 느낌으로요!)
예를 들어, 북쪽은 0이니까 방향은 dx[0], dy[0] 가 되겠네요.
현재 위치가 (0, 0) 일 때 북쪽으로 한 칸 이동한다고 할 때,
현재 x좌표(0)+dx[0](0) , 현재 y좌표(0)+dy[0](1) 를 한다면 (0, 1) 가 되겠네요!

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.

5 participants