작성한 질문수 1

1

안녕하세요 ㅎㅎ

아 해결하셨네요 ㅎㅎ

네 맞습니다.

잠시 코드리뷰를 드리면요.

1. inVowel의 초기화:

    

   inVowel 변수는 while 루프 안에서 새 문자열을 처리할 때마다 false로 초기화되어야 합니다.

2. 불필요한 부분.

        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

inVowel 를 초기화 하지 않아서 그랬네요 해결했습니다!