Я постоянно получаю "Ошибка шины", пока выполняется код?

Question

Цель этого кода - взять файл, который был передан в программу, и сгенерировать частоту букв каждой буквы в файле. В приведенном выше коде я удаляю знаки препинания и преобразовываю их в строчные буквы.

#include<iostream>
#include<string>
#include<fstream>

using namespace std;

int main()
{
    string fileContent = "qr rqh zrxog kdyh eholhyhg lq wkh odvw bhduv ri wkh qlqhwhhqwk fhqwxub wkdw wklv";

    int count[26] =  { 0 }; // an array the size of the alphabet.

    for(int f = 0; f < fileContent.length(); f++) // run til the file end.
    {
            if(fileContent[f] == 32) // to take care of the spaces.
            {
                    f++; // also tried "continue;" and yeild different and also incorrect results.
            }

            if(fileContent[f] >= 48 && fileContent[f] <= 57) //take care of numbers.
            {
                    f++; // tried "continue;"
            }

            count[fileContent[f]]++;

    }

    for(int p = 0; p < 26; p++)
    {
            cout << char(p + 97) << ": " << count[p]  << endl;
    }
return 0;
}

Когда я запускаю этот код, я получаю некоторые точные частоты, и некоторые ужасно неправильные (кажется, что любой другой результат неправильный, но после нескольких букв он затихает в астрономически большие числа). Есть ли способ сделать это лучше? что не так с этим кодом? В соответствии с запросом я добавил еще немного кода (включая строку со случайным числом 100 в нем), так как он был явно недостаточно ясен)

Для большего контекста эта программа предназначена для сдвигового декодера Ceasar, над которым я работаю. Я нахожусь на базовом C++ и буду очень признателен за любые советы от вас более опытных разработчиков. благодарю вас!

Answer 1

В вашей программе это утверждение:

count[fileContent[f]]++;

должно быть:

count[fileContent[f]-97]++; //Assuming that all alphabets are in lowercase

Если вы не делаете -97пытается увеличить значение по индексу fileContent[f] из count массив, который может быть за пределами count массив.

Кроме того, убедитесь, что continue в обоих if блоки и вам не нужно делать f++ явно в обоих if блоки, как в for цикл вы уже делаете f++,

Answer 2

Вы делаете вещи сложным способом: используя массивы в стиле C, магические числа в вашем коде и рискуя переполнением буфера повсюду.

Сравните ваш код с этим:

#include <string>
#include <iostream>
#include <map>
using namespace std;

int main()
{
    string fileContent = "qr rqh zrxog kdyh eholhyhg lq wkh odvw bhduv ri wkh qlqhwhhqwk fhqwxub wkdw wklv";
    map<char, int> counts;

    for (char ch : fileContent)
        ++counts[ch];

    for (char ch = 'a'; ch <= 'z'; ++ch)
        cout << ch << ": " << counts[ch] << '\n';
}

Или, чтобы распечатать все содержимое карты (если вы не хотите печатать 0 для букв, которых не было), вы можете использовать:

for (auto& item : counts) 
    cout << item.first << ": " << item.second << '\n';

Упражнение для читателя, чтобы добавить в код, чтобы исключить пробелы и числа. Подсказка: посмотрите заголовок cctype.