코드가 실행되는 동안 계속 "버스 오류"가 표시됩니까?

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

가 있어야한다 : 당신이 -97하지 않으면, count 배열의 인덱스 fileContent[f]의 가치를 높이기 위해 노력하고있다

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

하는 count 배열의 한계를 초과 할 수 있습니다.

또한, 모두 if 블록 continue에 있는지 확인하고 당신은 당신이 이미 f++을하고있는 for 루프로 모두 if 블록에 명시 적으로 f++을 수행 할 필요가 없습니다.

Answer 2

C 스타일의 배열, 코드의 매직 넘버를 사용하여 모든 곳에서 버퍼 오버 플로우 위험이 있습니다.

이에 코드를 비교 :

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

: (당신은 발생하지 않았다 문자 0을 인쇄하지 않을 경우)

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

또는 당신이 사용할 수있는 모든지도의 내용을 인쇄

독자가 공백과 숫자를 제외하는 코드를 추가하는 연습을하십시오. 힌트 : cctype 헤더를 찾습니다.