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월 22일 #4

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

[브루트포스] 9월 22일 #4

wants to merge 2 commits into from

Conversation

minji1289
Copy link
Collaborator

@minji1289 minji1289 commented Sep 22, 2022

내용 & 질문

제출합니다!

<기존 제출>

1544

<추가 제출>

2858

@kwakrhkr59
Copy link

[추가제출 확인 완료]
안녕하세요 민지님~ 추가제출 확인 완료되셨습니다😊
이번 과제도 수고 많으셨습니다:)

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.

p2. 1544 코드 리뷰 완료

안녕하세요 민지님 🦕🦕 코드리뷰를 늦게 드려 죄송합니다 ㅜㅜ
너무 잘 풀어주시고, 함수화로 코드도 깔끔할 뿐더러 주석도 작성해주셔서 리뷰 드리기 너무 좋았습니다!!
간단한 코멘트 달았으니, 확인 부탁 드립니다 !
수정하신 부분 있으시면 리뷰어로 불러주셔도 좋습니다! 수고 많으셨습니다.
감사합니다.🤗🤗

Comment on lines +11 to +15
for (int i = 0; i < q.size(); i++) {
str += q.front();
q.push(q.front());
q.pop();
}

Choose a reason for hiding this comment

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

p2. 같은 로직인데 조금은 더 간편히 구현 가능할 것 같아요!

시계 방향으로 회전하기 위해선, 맨 앞 한 글자를 떼서 맨 뒤에 붙여야 하네요! 이에 딱 맞는 자료구조인 큐로 문자열을 잘 가공해주셨어요!!!😄😄
큐도 좋지만, string 클래스에는 <,>,==,+ 등과 같은 연산자들을 사용할 수 있다는 것을 상기해볼까요?
또한, 문자열 일부를 지우는 함수도 있어요!
그럼, string 자체에서도 시계 방향으로 회전이 가능하겠네요!

Comment on lines +47 to +72
bool same = false;
//문자열이 같은지 확인하는 bool 변수

for (int j = 0; j < v.size(); j++) {
for (int k = 0; k < s.size(); k++) {
string compstr = makeStr(q);

if (compstr == v[j]) {
same = true;
break;
}

q.push(q.front());
q.pop();
}

if (same) {
break;
//같은 문자열이 queue에 있으면, vector에 문자열 추가 x
}
}

if (!same) {
v.push_back(s);
//같은 문자열이 없으면 vector에 문자열 추가!
}

Choose a reason for hiding this comment

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

참고해주시면 좋을 것 같아서 남깁니다!
중복을 허용하지 않는 자료구조가 있었죠! 서로 다른 단어를 담기에 아주 적합하겠어요. 이 자료구조는 찾으려는 요소가 존재하는지 확인하는 함수도 제공되기 때문에 활용해보셔도 좋을 것 같아요! 샘플 코드로 준비했으니, 참고해주셔도 좋을 것 같습니다 🤩🤩

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