Question

我试图创建一个循环字符串的程序，并计算每个字母的使用量。问题是我不能让数组正确保存它。非常感谢任何帮助。

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;
        }
    }
}

Answer 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]);
    }
}

Answer 2

试试这个：

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]++;
    }

}

Answer 3

你使用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]++;
            }
        }
    }
}

我还建议使用英语作为变量/方法名称。

无法让数组正确更新

3 个答案: