0

我正在尝试编写一个将文本文件中的单词转换为猪拉丁语的程序。我得到了将文本文件的单词分开的代码，但我现在在尝试整理它们时遇到了麻烦。当我运行此代码时，它总是打印所有单词的第一个索引，而不是与 if 语句匹配的单词

void wordpro(string sent)
{
  string word;

  istringstream iss(sent, istringstream::in);
  while (iss>> word)
  if (word[0] == 'a'||'e'||'i'||'o'||'u'||'A'||'E'||'I'||'O'||'U')
  {
    cout<< word[0] <<endl;
    }
}
4

2 回答 2

10 
if (word[0] == 'a'||'e'||'i'||'o'||'u'||'A'||'E'||'I'||'O'||'U')

这不是||C++ 中的工作方式。但这并不意味着上面会导致编译错误。不，从编译器的角度来看这是正确的；唯一的问题是它没有做你想做的事！相反，条件将始终为true。这就是为什么它会打印代码中所有单词的第一个字符。

为了得到你想要的，你必须写成||：

if (word[0] == 'a'|| word[0] == 'e'|| word[0] ==  'i' || ... so on)

也就是说，您必须分别比较每个字符。这肯定是令人恼火的。

C++11 已将您从这里拯救出来，因此您可以std::any_of用作：

//C++11 only

std::string const v = "aeiouAEIOU";
if (std::any_of(v.begin(), v.end(), [](char c){ return word[0] == c;})

或者您可以std::find用作：

//C++03 and C++11 both

if ( std::find(v.begin(), v.end(), word[0]) != v.end() )

它比前一个短一点。此外，这也适用于 C++03！

或者您可以std::count用作：

//C++03 and C++11 both

if ( std::count(v.begin(), v.end(), word[0]) )

这甚至更短。

或者您可以std::string::find用作：

//C++03 and C++11 both

if ( v.find(word[0]) != std::string::npos)

哪个是最短的！

阅读文档以了解他们每个人的真正作用，以及为什么它适用于您的案例：

希望有帮助。

于 2013-01-27T15:19:48.323 回答
0

上述使用很多的建议word[0] == 'a' || word[0] == 'e' || ...是正确的解决方案，以“修复”您编写的内容，它会按照您对单个 if 语句的期望进行。

你可以这样做：

 switch(word[0])
 {
    case 'a':
    case 'e':
    case 'i':
    ... /// you'll have to fill the rest in yourself. 
        cout << word[0]; 
        break; 
 }

或者你可以使用老式的 C 风格strchr：

if (strchr("aeiouAEIOU", word[0]) != NULL)
{
    cout << word[0];
}

可能还有六种其他解决方案。这一切都取决于你喜欢什么“风格”。我可能会选择交换机作为我的第一选择。

于 2013-01-27T15:26:22.647 回答