작성
·
266
·
수정됨
0
안녕하세요 예제는 잘 통과하는데 제출하면 틀렸다고 나오는데 반례를 못찾겠어요ㅠㅠ아래는 모음과 자음이 3번연속으로 반복되는지 확인하기 위한 코드인데 맞는지 의심이 됩니다!
if (isVowel(s[i - 1]) == isVowel(s[i]))
{ cnt++; }
else { cnt = 1; }
답변 2
1
안녕하세요 ㅎㅎ
아 해결하셨네요 ㅎㅎ
네 맞습니다.
잠시 코드리뷰를 드리면요.
inVowel
의 초기화:
inVowel
변수는 while
루프 안에서 새 문자열을 처리할 때마다 false
로 초기화되어야 합니다.
불필요한 부분.
if(s.length() == 1){
if(isVowel(s[0])){
printAcc(s);
이부분은 뺄 수 있습니다.
제가한번 수강생님 코드를 다듬어 봤는데요. ㅎㅎ
#include <bits/stdc++.h>
using namespace std;
bool isVowel(char ch) {
return ch == 'a' || ch == 'e' || ch == 'i' || ch == 'o' || ch == 'u';
}
void printNot(const string& s) {
cout << "<" << s << "> is not acceptable.\n";
}
void printAcc(const string& s) {
cout << "<" << s << "> is acceptable.\n";
}
int main() {
string s;
while (true) {
cin >> s;
if (s == "end") break;
bool ret = true, inVowel = false;
int cnt = 1;
for (char ch : s) {
if (isVowel(ch)) {
inVowel = true;
break;
}
}
for (int i = 1; i < s.length() && ret; i++) {
if (isVowel(s[i]) == isVowel(s[i - 1])) {
cnt++;
} else {
cnt = 1;
}
if (cnt == 3) ret = false;
if (s[i] == s[i - 1] && s[i] != 'e' && s[i] != 'o') {
ret = false;
}
}
if (!inVowel || !ret) {
printNot(s);
} else {
printAcc(s);
}
}
return 0;
}
이렇게 하면 더 좋을 것 같습니다.
다듬은 부분 설명을 좀 하자면요.
bool isVowel(char ch) {
return ch == 'a' || ch == 'e' || ch == 'i' || ch == 'o' || ch == 'u';
}
이부분을 if문 분기처리를 하셨는데 이렇게 간단하게 더 짤 수 있습니다.
bool ret = true, inVowel = false;
int cnt = 1;
for (char ch : s) {
if (isVowel(ch)) {
inVowel = true;
break;
}
}
처음에 거를 수 있는 것은 이렇게 밖으로 빼내어서 처리를 하면 더 깔끔합니다.
문자열 순회가 O(N)이라고 했을 때 두번 순회하면 O(2N)이긴 하지만 상수라 별 차이가 없습니다.
효율적으로 짜는 것은 좋지만 어느정도 시간초과가 안나는 범위라면 비효율적으로 + 깔끔하게 하는 것이 좋습니다.
나머지 부분은 수강생님께서 짜신 print부분의 모듈화 등을 조금 다듬은 수준입니다.
함수를 밖으로 빼서 하는 등 초기화 말고 다른 부분들은 모두 잘 짜셨습니다.
또 질문 있으시면 언제든지 질문 부탁드립니다.
좋은 수강평과 별점 5점은 제게 큰 힘이 됩니다. :)
감사합니다.
강사 큰돌 올림.
0
헉.. 친절하게 답변해주셔서 감사합니다!! 열심히 수강할게요 ㅎㅎ