-
Notifications
You must be signed in to change notification settings - Fork 0
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
[구현 & 코너케이스] 11월 8일 #8
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p3. 1063번 코드리뷰 완료
안녕하세요, 채원님! 몇 가지 코멘트 남겼으니, 참고해주시면 감사하겠습니다
이번 과제도 하시느라 수고하셨어요🥰🥰
|
||
for (int i = 0; i < cnt; i++) { | ||
string str; cin >> str; | ||
moves.push_back(str); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p2. 따로 vector
에 이동내역을 저장하지 않고 입력 즉시 함수호출을 해도 될 것 같네요! 이러면 공간복잡도가 많이 줄 것 같아요😁
if (move == "R") { | ||
idx = 0; | ||
} | ||
else if (move == "L") { | ||
idx = 1; | ||
} | ||
else if (move == "B") { | ||
idx = 2; | ||
} | ||
else if (move == "T") { | ||
idx = 3; | ||
} | ||
else if (move == "RT") { | ||
idx = 4; | ||
} | ||
else if (move == "LT") { | ||
idx = 5; | ||
} | ||
else if (move == "RB") { | ||
idx = 6; | ||
} | ||
else if (move == "LB") { | ||
idx = 7; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p3. 전역변수의 장점을 살리려면 방향을 결정하는 부분을 함수화해도 좋을 것 같네요!
printf("%c%d\n", kx, ky); | ||
printf("%c%d\n", sx, sy); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cout으로도 char
형 출력이 가능한 것 아시나요?
ios_base::sync_with_stdio(false);
와 같이 실행속도를 높이기 위한 구문을 사용하는 경우에는 printf()
를 사용할 수 없으니, 모르셨다면 한 번 찾아보시는 것도 좋을 것 같아요😃😃
추가제출 확인 완료 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p1. 2607 코드 리뷰 완료
안녕하세요 채원님🤗 리뷰를 너무 늦게 드려 죄송합니다!!!ㅜㅜ 꼭 늦지 않도록 하겠습니다.. 😭😭
문제에서 해결해야 하는 부분을 케이스 별로 잘 나눠서 풀어주셨네요!😁
해당 방법도 좋지만, 개수를 처리해야 하는 부분에 대해선 해결되지 않았어요🥲
참고해주시면 좋을 것 같아 다른 방향성을 코멘트로 달았으니, 확인 부탁 드립니다 !😁😁
수정하신 부분 있으시면 리뷰어로 불러주세요!
수고 많으셨습니다. 감사합니다.😄😄
for (int i = 1; i < s.size(); i++) { | ||
int cnt = 0; | ||
string strvar = s[i]; | ||
int l = strvar.length() - strdef.length(); | ||
// ���� ���� 2 �̻��̸� ��� | ||
if (abs(l) > 1) { | ||
continue; | ||
} | ||
// �������� 0�̸� | ||
else if (l == 0) { | ||
// find�� �˻� | ||
for (int i = 0; i < strdef.length(); i++) { | ||
if (strvar.find(strdef[i]) == string::npos) { | ||
cnt++; | ||
} | ||
} | ||
if (cnt <= 1) { | ||
words++; | ||
} | ||
} | ||
// �������� 1�̸� | ||
else if (l == 1){ | ||
// find�� �˻� | ||
for (int i = 0; i < strdef.length(); i++) { | ||
if (strvar.find(strdef[i]) == string::npos) { | ||
cnt++; | ||
} | ||
} | ||
if (cnt == 0) { | ||
words++; | ||
} | ||
} | ||
else if (l == -1) { | ||
// find�� �˻� | ||
for (int i = 0; i < strdef.length(); i++) { | ||
if (strvar.find(strdef[i]) == string::npos) { | ||
cnt++; | ||
} | ||
} | ||
if (cnt <= 1) { | ||
words++; | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p1. 우선 반례를 살펴볼까요?
AA
ABC
예상 답 : 0
해당 코드 답 : 1
현재는 strdef의 "A"가 strvar에 존재하는 지만 확인하는 중입니다.
이렇게 되면, "A"의 개수에 상관없이 존재하기만 하면 비슷한 단어로 처리되는 문제가 발생합니다!😉
그렇다면, 몇 개가 있는 지를 확인해야 하는 것도 관건이겠네요!
그럼 각 단어를 이루고 있는 알파벳의 개수를 저장하는 배열을 가지고 있다면, "A"가 몇 개 있는 지를 알 수 있겠죠!
더 나아가, 그 배열을 가지고 단어끼리 비교 하면 "A"가 서로 몇 개 있는 지를 비교할 수 있겠죠!
다시 한번 강조 드리지만, 알파벳의 개수는 26개인 것!!!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p1. 문제가 생길 수 있는 위험한 코드가 있습니다!
현재 가장 바깥 반복문에서 i로 돌고 있는데, 안 쪽 반복문도 i로 선언하여 돌고 있네요!
지역변수라서 코드 돌아가는 데엔 큰 문제는 없지만, 만약 나중에 수정을 할 때 이 i가 바깥 i인지 안쪽 i인지
헷갈릴 상황이 올 수 있기 때문에 다른 변수를 선언하여 쓰는 것을 권장 드립니다.😉😉
내용 & 질문
<기존 제출>
<추가 제출>