C++ Beginner - While 반복 반복 첫 번째 반복

Question

전체

그래서 나는이 문제에 대해 정말로 두뇌를 포기하고 있습니다. 필자는 사용자 지정 문자열에서 공백/모음/문자를 계산해야하는 프로그램 섹션을 가지고 있습니다. 이것은 "이미 수업에서 다루었던 것만 사용할 수 있기 때문에 아무도하지 않을 방법을 가르쳐줍니다"라는 과제 중 하나입니다. 그래서 나는 사용자에게 sentinel char (이 경우에는 '#')로 끝나는 텍스트를 입력하게했습니다. 루프는 센티넬이 발생할 때 빠져 나가는 것과 관련하여 멋지게 작동하지만 문자열 [0]에 두 번 반복됩니다. 여기에 코드입니다 : 그래서

i = 0; 
characterToBeProcessed = userInputText.at(i); 

while (characterToBeProcessed != LOOP_SENTINEL) 
{ 
    fout << characterToBeProcessed; 

    // Convert to lowercase 
    characterToBeProcessed = 
     static_cast<char> (tolower(characterToBeProcessed)); 

    // Increment character counters 
    switch (characterToBeProcessed) 
    { 
     case 'a': 
     case 'e': 
     case 'i': 
     case 'o': 
     case 'u': 
      totalVowelCount++; 
      totalCharacterCount++; 
      break; 
     case ' ': 
      totalSpaceCount++; 
      totalCharacterCount++; 
      break; 
     default: 
      totalCharacterCount++; 
      break; 
    } 
    characterToBeProcessed = userInputText.at(i++); 
} 

이 때 프롬프트에서 입력 :

"Please input a text to be analyzed, ending with the # character: " 

Hi there, my friend!# 

출력은 : 나는이 프로그램 출력을 .at에 대한 문자를 했어

Below is the text entered by the user: 

HHi there, my friend! 

Total characters: 21 
Total vowels: 5 
Total blank spaces: 3 

(0) 및 .at (1), 그리고 그 날 올바른 문자를 줄, 난 그냥 루프를 두 번 첫 번째 char 반복하고 그 두 번째 시간 후에 잘 작동하는지 알아낼 수 없습니다. 그렇지 않으면 count/output은 첫 번째 문자가 복제되는 것을 제외하고는 정확합니다. 감사의 마음을 크게 전할 것입니다.

Answer 1

다른 사람들도 말했듯이 이런 종류의 문제를 해결하는 올바른 방법은 디버거를 사용하는 것입니다. 그것은 많은 시간과 많은 시간을 절약 할 수 있습니다.

characterToBeProcessed = userInputText.at(i++); 

하지만 당신의 while 루프 전에, 당신이 할 :

그러나 어떤 경우에

, 당신의 오류는 while 루프의 끝에서, 당신은이 작업을 수행 할 것입니다

characterToBeProcessed = userInputText.at(i); 

당신의 문제는 사용자가 매번 i을 증가시키지 않아서 자연스럽게 관찰 된 동작을 유발한다는 것입니다.

Answer 2

첫 번째 위치에서 문자는 다음 while 루프를 사용해야하는 경우

i = 0; 
characterToBeProcessed = userInputText.at(i); 
             ^^^^^^ 

while (characterToBeProcessed != LOOP_SENTINEL) 
{ 
    //... 
    characterToBeProcessed = userInputText.at(i++); 
              ^^^^^^^^ 
} 

그것은 또한

i = 0; 

while ((characterToBeProcessed = userInputText.at(i++)) != LOOP_SENTINEL) 
{ 
    //... 
    // remove the next statement 
    // characterToBeProcessed = userInputText.at(i++); 
} 

이 문장처럼 보일 수 두 번 읽어

totalCharacterCount++; 

각 라벨 아래에 사용됩니다. 스위치 문 앞에 또는 뒤에 스위치 문 외부에 배치하는 것이 좋습니다. 예 :

totalCharacterCount++; 

switch (characterToBeProcessed) 
{ 
    case 'a': 
    case 'e': 
    case 'i': 
    case 'o': 
    case 'u': 
     totalVowelCount++; 
     break; 
    case ' ': 
     totalSpaceCount++; 
     break; 
    default: 
     break; 
}