範囲外のベクトル下付き文字 - 構造体のベクトル

Question

私は以下のコードを持っています。与えられたファイル内の与えられた文字の繰り返し数を数えることになっています。しかし、私はこれを実行しようとすると、範囲外のベクトルの下付き文字を取得します。同じエラーを持つ他の人は、未定義の部分にアクセスしようとしていましたが、これは問題ではないようです。

struct letters 
{ 
    char letter; 
    int repetitions=0; 
}; 


void howManyTimes(const string &output) 
{ 
    ifstream file(output); 
    vector <letters> alphabet; 

    for (int i = 0; i < 'z' - 'a' + 1; i++) 
    { 
     alphabet[i].letter = 'a' + i; 
    } 

    string line; 

    while (file.eof() == 0) 
    { 
     getline(file, line); 
     for (char c : line) 
     { 
      if(c >= 'a' && c <= 'z') 
       alphabet[c - 'a'].repetitions++; 
      else if (c >= 'A' && c >= 'Z') 
       alphabet[c - 'A'].repetitions++; 
     } 
    } 
    cout << alphabet[10].repetitions; 
} 

Answer 1

vector <letters> alphabet; // (1) 

for (int i = 0; i < 'z' - 'a' + 1; i++) 
{ 
    alphabet[i].letter = 'a' + i; // (2) 
} 

(1)空ベクターを作成します。あなたは空ベクトルでインデックスiを使用してアイテムにアクセスしようとするので、明確にあなたのインデックスが範囲のうちある(2)でforループ内

。

まず、ベクターにデータを入力してから、このデータにアクセスする必要があります。

あなたはベクトルに新しいアイテムを追加したい場合は、（あなたが(2)に何を意味するのか、おそらくです）vector::push_backを使用することができます。

Answer 2

同じエラーを持つ他の人は、未定義の部分にアクセスしようとしていましたが、これは問題ではないようです。

それは間違いなくここで問題です：

vector <letters> alphabet; // Now size of the vector is 0. 

for (int i = 0; i < 'z' - 'a' + 1; i++) 
{ 
    // You are trying to access elements 0, 1, 2, ..., which do not exist. 
    alphabet[i].letter = 'a' + i; 
} 

最も簡単な解決策は、適切なサイズを使用して、ベクターを構築することである。

vector <letters> alphabet('z' - 'a' + 1); 

Answer 3

私はあなたのコード場所の一部が表示されませんalphabetは、そこに保存する予定のオブジェクトに対応するように展開されます。 std::vectorは、push_back、insert、emplaceまたは他の同様の方法を使用する場合にのみリサイズされます。 operator[]メソッドを使用して直接アクセスするときはそうしません。

いずれにせよ、このようなタスクでは、std::map<char, int64_t>がきれいになり、の構文を保持できるようにすると、ベクターを使用したいとは思えません。を追加メンテナンスを必要とせずに使用できます。

void howManyTimes(const string &output) 
{ 
    ifstream file(output); 

    map<char, int64_t> alphabet; 

    string line; 

    while (getline(file, line)) 
    { 
     for (char c : line) 
     { 
      if(c >= 'a' && c <= 'z') 
       alphabet[c - 'a']++; 
      else if (c >= 'A' && c >= 'Z') 
       alphabet[c - 'A']++; 

     } 
    } 
    cout << alphabet[10]; 

} 

Answer 4

このプログラムは、（私は宿題:)としてそれを与えたとき、私の生徒たちがどう感じたか、私は思ってしまう）何の質問が求めているため、本当に強力です。あなたは構造体を使用する必要がありますか？のがないと仮定しましょう、そしてまた、我々は、アルファベットの大きさを知っている、と「」最初の文字と「Z」であるとすると、最後の文字であると仮定します。

vector<int> repetitions(26, 0); 
char nextCharacter; 
while(!file.eof()) 
{ 
    cin >> nextCharacter; 
    nextCharacter = tolower(nextCharacter); 
    if(nextCharacter >= 'a' && nextCharacter <= 'z') 
    { 
     repetitions[nextCharacter - 'a']++; 
    } 
} 

手紙を確認するには：

cin >> letterToQuery; 
cout <<"The amount of occurrences of " << letterToQuery <<" is "; 
cout << repetitions[tolower(letterToQuery) - 'a'] << endl; 

あなたは、にコードの変更あなたのアルファベットのサイズがわからない場合：あなたは、その構造体を使用する必要がない場合は、

vector<int> repetitions('last_alphabet' - 'a' + 1, 0); 
... 
if(nextCharacter >= 'a' && nextCharacter <= 'last_letter') 

そして最後に、あなたのコードの変更へ：

struct Letter 
{ 
    char letter; 
    int repetitions=0; 
}; 

vector<Letter> alphabet; 
letter temp; 
for(int i = 0; i < 'last_alphabet' - 'a' + 1; ++i) 
{ 
    temp.letter = 'a' + i; 
    alphabet.push_back(temp); 
} 

// then everything else is a similar structure 
char nextCharacter; 
while(!file.eof()) 
{ 
    cin >> nextCharacter; 
    nextCharacter = tolower(nextCharacter); 
    if(nextCharacter >= 'a' && nextCharacter <= 'last_alphabet') 
    { 
     alphabet[nextCharacter - 'a'].repetitions++; 
    } 
} 

手紙をチェックするために：

cin >> letterToQuery; 
cout <<"The amount of occurrences of " << letterToQuery <<" is "; 
cout << alphabet[tolower(letterToQuery) - 'a'].repetitions << endl; 

お知らせを、あなたは 'Z' と 'last_alphabet' を交換する場合、あなたは現在のアルファベットを取得します。