0

我试图制作一个循环遍历字符串并计算每个字母使用了多少的程序。问题是我无法让数组正确保存。任何帮助是极大的赞赏。

int main()
{
    string textRad = "";
    int histogram[ANTAL_BOKSTAVER];

    getline(cin, textRad);

    berakna_histogram_abs(histogram, textRad);

    cout << histogram[0] << endl;
    cout << histogram[2];

    return 0;
}

void berakna_histogram_abs(int histogram[], string textRad) 
{
    for(int i = 0; i < ANTAL_BOKSTAVER; i++) 
    {
        histogram[i] = 0;
    }

    for(int i = 0; i < textRad.length(); i++)
    {
        for(int j = 0; j < ANTAL_BOKSTAVER; j++)
        {
            int antal = 0;
            string alfabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";

            if(char(toupper(textRad.at(i))) == alfabet.at(j))
            {
                antal++;
            }
            histogram[j] = antal;
        }
    }
}
4

3 回答 3

1

考虑到 Javid 和 Tjofras 的答案，这是一个完整、更简单、更安全的示例：

#include <iostream>
#include <string>
#include <algorithm>
#include <vector>

void berakna_histogram_abs(std::vector<int>& histogram, const std::string& textRad);

int main() {

    const int ANTAL_BOKSTAVER = 26; //Assumed value.

    std::string textRad;
    std::vector<int> histogram(ANTAL_BOKSTAVER, 0);

    std::getline(std::cin, textRad);

    std::transform(textRad.begin(), textRad.end(), textRad.begin(), toupper);
    berakna_histogram_abs(histogram, textRad);

    std::cout << histogram[0] << std::endl;
    std::cout << histogram[2];

    return 0;
}

void berakna_histogram_abs(std::vector<int>& histogram, const std::string& textRad) {

    static std::string alfabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";

    std::size_t s = alfabet.length();
    for(std::size_t i = 0; i < s; ++i) {
        histogram[i] = std::count(test_string.begin(), test_string.end(), alfabet[i]);
    }
}
于 2013-08-24T20:30:33.727 回答
0

尝试这个：

int main()
{

    string textRad = "";
    int histogram[ANTAL_BOKSTAVER];

    getline(cin, textRad);

    berakna_histogram_abs(histogram, textRad);


    cout << histogram[0] << endl;
    cout << histogram[2];

    return 0;
}

void berakna_histogram_abs(int histogram[], string textRad) {

    for(int i = 0; i < ANTAL_BOKSTAVER; i++)
        histogram[i] = 0;

    string alfabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
    for(int i = 0; i < textRad.length(); i++) {
        for(int j = 0; j < alfabet.length(); j++)
            if(char(toupper(textRad.at(i))) == alfabet.at(j))
                histogram[j]++;
    }

}
于 2013-08-24T19:45:42.100 回答
0

您使用 antal 变量的方式是错误的。每个字母你永远不会得到超过 1 个。这样的事情应该会更好：

void berakna_histogram_abs(int histogram[], string textRad) 
{
    for(int i = 0; i < ANTAL_BOKSTAVER; i++) 
    {
        histogram[i] = 0;
    }

    for(int i = 0; i < textRad.length(); i++)
    {
        for(int j = 0; j < ANTAL_BOKSTAVER; j++)
        {
            string alfabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";

            if(char(toupper(textRad.at(i))) == alfabet.at(j))
            {
                histogram[j]++;
            }
        }
    }
}

我还建议对变量/方法名称使用英文。

于 2013-08-24T19:46:01.847 回答