Question

该程序应该读取文本并计算元音和辅音的数量。它应该忽略任何非字母字符。结果应该是这样的：

Enter your text: I have to TURN this..in before midnight!!
 a,  e,  i,  o,  u,  y
 1,  3,  5,  2,  1,  0
There are 19 consonants.

但我的代码的结果是：

Enter your text: I have to TURN this..in before midnight!!
 a,  e,  i,  o,  u,  y
 1,  3,  4,  2,  0,  0
There are 31 consonants.

我不知道发生了什么!!这也是一个任务，我必须使用所有这些功能，我不能添加或删除它们！我读了几个其他方法来计算和显示数字但不幸的是模板被给出了......

#include <iostream>
#include <iomanip>
#include <string>
#include <vector>

using namespace std;

// FUNCTION PROTOTYPES GO HERE:
void init_vectors(vector<char> & vowels, vector<int> & frequencies);
string read_text(const string & prompt);
bool is_alphabetic(const char character);
void create_list(const string & str_text, vector<char> & vec_text);
bool is_member(const vector<char> & list, char character);
int find_index(const vector<char> & list, char character);
int compute_vowel_freqs(const vector<char> & text, const vector<char> & vowels, vector<int> & freqs);
void display_characters(const vector<char> & characters, const int colwidth);
void display_freqs(const vector<int> & freqs, const int colwidth);

int main()
{
    vector<char> vowels;
    vector<int> freqs;
    string input;
    vector<char> text;
    int consonants(0);

    const int COLUMNWIDTH = 2;
    init_vectors(vowels, freqs);
    input=read_text("Enter your text: ");

    create_list(input, text);
    compute_vowel_freqs(text, vowels, freqs);

    display_characters(vowels, COLUMNWIDTH);
    display_freqs(freqs, COLUMNWIDTH);

    consonants = compute_vowel_freqs(text, vowels, freqs);
    cout<<"There are "<< consonants<< " consonants."<<endl;
    return 0;
}

void init_vectors(vector<char> & vowels, vector<int> & frequencies)
{

    for (int i(0); i<6; i++) //i is loop variable
    {
        frequencies.push_back(0);
    }
    vowels.push_back('a');
    vowels.push_back('e');
    vowels.push_back('i');
    vowels.push_back('o');
    vowels.push_back('u');
    vowels.push_back('y');
}

string read_text(const string & prompt)
{
    string phrase;
    cout<<prompt;
    getline(cin,phrase);
    return phrase;
}


bool is_alphabetic(const char character)
{
    bool alphabet;
    if ((character > 'a' && character < 'z')||(character > 'A' && character < 'Z'))
    {
        alphabet = true;
    }
    return alphabet;
}

void create_list(const string & str_text, vector<char> & vec_text)
{
    for( int i = 0 ; i < str_text.length() ; i++)
    {
        if(is_alphabetic(str_text[i]))
            vec_text.push_back(str_text[i]);
    }
}

bool is_member(const vector<char> & list, char character)
{
    bool vowel;
    for (int i(0); i<list.size(); i++)
    {
        if (character == list[i])
        {
            vowel=true;
        }
    }
    return vowel;
}

int find_index(const vector<char> & list, char character)
{
    int index = -1;
    for(int i=0; i<list.size(); i++)
    {
        if(character == list[i])
        {
            index = i;
            break;
        }
    }
    return index;
}

int compute_vowel_freqs(const vector<char> & text, const vector<char> & vowels, vector<int> & freqs)
{
    int num_cons(0);
    for(int i = 0 ; i < text.size() ; i++)
    {
        int index;
        if(is_member(vowels, text[i]))
        {
            index = find_index(vowels , tolower(text[i]));
            freqs[index]++;
        }
        else
            num_cons++;
    }

    return num_cons;
}

void display_characters(const vector<char> & characters, const int colwidth)
{
    for(int i=0; i<characters.size(); i++)
    {
        cout<<setw(colwidth)<<characters[i];
        if((i+1)<characters.size())
        {
            cout<<",";
        }
    }
    cout<<endl;
    return;
}

void display_freqs(const vector<int> & freqs, const int colwidth)
{
    for(int i=0; i<freqs.size(); i++)
    {
        cout<<setw(colwidth)<<freqs[i];

        if((i+1)<freqs.size())
            cout<<",";
    }
    cout<<endl;
    return;
}

Answer 1

如果在测试任何代码之前编写了数百行代码，那么你一定会失败。从小而简单开始，一次添加一点复杂性，在每一步测试，永远不会添加到不起作用的代码。

您应该在编写这些函数时逐一测试这些函数。这是第一个问题：

bool is_alphabetic(const char character)
{
  bool alphabet;
  if ((character > 'a' && character < 'z')||(character > 'A' && character < 'Z'))
    {
      alphabet = true;
    }
  return false;
}

此始终会返回false，因此不会将任何内容识别为文本。

修改

第二个问题：is_member具有完全相同的错误，使用相同的解决方案。

修改

第三个问题：我没有注意到is_alphabetic中的这一行：

if ((character > 'a' && character < 'z')||(character > 'A' && character < 'Z'))

你正在使用'＆gt;'和'＆lt;'何时应使用“＆gt; =”和“＆lt; =”。根据这个功能，'a'和'z'不是字母。

看，您仍在尝试测试并修复此程序作为一个整体。你必须测试零碎的。在main中选择一个位置，然后打印出应该为其分配值的每个变量。此功能可能很有用：

void printVector(const vector<char> &V)
{
  for(vector<char>::const_iterator citr=V.begin(); citr!=V.end(); ++citr)
    cout << *citr;
  cout << endl;
}

然后检查结果。如果变量不包含它应该包含的内容，那么高于该点的内容就是行为不端。将问题追溯到一个功能，好的东西进入，但出现了一些不好的事情。修复它，然后再次查看输出。我不能强调这一点：不要试图一次解决所有问题。

Answer 2

你的is_member总是假的，所以没有char是元音。它导致0计数。

bool is_member(const vector<char> & list, char character)
{
    bool vowel = false;   # this fix is not obligatory, I just made code look clearer
    for (int i(0); i<list.size(); i++)
    {
        if (character == list[i])
        {
            vowel=true;
        }
    }
    return vowel;  # this should be fixed
}

顺便说一下，同样的问题出在函数'is_alphabetic'中。这总是错误的。

C ++向量和字符串，计数元音

2 个答案: