인프런 커뮤니티 질문&답변

odeul36님의 프로필 이미지
odeul36

작성한 질문수

10주완성 C++ 코딩테스트 | 알고리즘 코딩테스트

2-H

2-H 4659 반례를 찾아주세요 ㅠㅠ

작성

·

266

·

수정됨

0

안녕하세요 예제는 잘 통과하는데 제출하면 틀렸다고 나오는데 반례를 못찾겠어요ㅠㅠ아래는 모음과 자음이 3번연속으로 반복되는지 확인하기 위한 코드인데 맞는지 의심이 됩니다!

if (isVowel(s[i - 1]) == isVowel(s[i]))

{ cnt++; }

else { cnt = 1; }

http://boj.kr/5dd372c77cac4654bd661ebeb5d37f34

답변 2

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점은 제게 큰 힘이 됩니다. :)

감사합니다.

강사 큰돌 올림.

odeul36님의 프로필 이미지
odeul36
질문자

헉.. 친절하게 답변해주셔서 감사합니다!! 열심히 수강할게요 ㅎㅎ

0

odeul36님의 프로필 이미지
odeul36
질문자

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

odeul36님의 프로필 이미지
odeul36

작성한 질문수

질문하기