2013-11-26 3 views
0

В моем коде есть инструкция if else, которая проверяет, является ли входной текст строкой, заполненной альфа-символами.C++ cout печатает одну и ту же строку несколько раз

Код работает, но часть cout << original << "\n"; распечатывает результат 5 раз. Я думаю, что корень проблемы лежит в пределах линии for (std::string::iterator it=original.begin(); it!=original.end(); ++it), в частности бит ++it.

Ниже приведен код:

#include <cstdio> 
#include <cstdlib> 
#include <algorithm> 
#include <string> 
#include <locale> 
#include <iostream> 
using namespace std; 

int main(int nNumberofArgs, char* pszArgs[]) 
{ 
    std::locale loc; 
    std::string original; 
    std::cout << "Welcome to the English to Pig Latin translator!\n"; 
    std::cout << "Type a word you wish to translate:\n"; 
    std::getline(std::cin, original); 
    std::cout << "Your word: " << original << "\n"; 
    for (std::string::iterator it=original.begin(); it!=original.end(); ++it) 
    { 
     if (original.length() > 0 && std::isalpha(*it,loc)) 
     { 
      std::string word; 
      std::transform(original.begin(), original.end(), original.begin(), ::tolower); 
      cout << original << "\n"; 
     } 
     else 
     { 
      std::cout << "Please enter a valid word." << std::endl; 
     } 
    } 

    system("PAUSE"); 
    return 0; 
} 

Эта ссылка скриншот моего выхода CLI: http://gyazo.com/5b9cea385794fecc39ed578b539a84c3

ответ

1

Ваш неправильный код. Он проверяет каждый символ, а не всю строку.

Изменить его:

bool alphaString = true; 
for (std::string::iterator it=original.begin(); it!=original.end(); ++it) 
    { 
     if (! std::isalpha(*it,loc)) 
     { 
      std::cout << "Please enter a valid word." << std::endl; 
      alphaString = false; 
      break; 
     } 
    } 
if (alphaString) { 
    std::transform(original.begin(), original.end(), original.begin(), ::tolower); 
    cout << original << "\n"; 
} 
4

Это печать пять раз, потому что «привет» длиной пять символов. Ваш цикл for работает один раз для каждого символа в строке.

+0

Ах, хорошо! Причина, по которой задействован цикл for, заключается в том, что это единственный пример, в котором я знаю, как выполнить часть isalpha в коде. Если бы я сделал это без цикла for и просто использовал оператор if/else и использовал '... && isalpha (original)), то компилятору Dev-C++ это не нравится. – RoyalSwish

+0

Если вы хотите увидеть, является ли вся строка альфа-цифрой или аналогичной, см. Этот ответ http://stackoverflow.com/questions/2926878/determine-if-a-string-contains-only-alphanumeric-characters-or- пространство –

Смежные вопросы