チャーの頻度

Question

私は自分のコードを完成させることに近いです！私はいくつかの支援をすることができた、私は文字列の文字の数を数えるプログラムを書いた。私の問題は最後に、私のデータを出力するときに来ます。私は、文字列 "AAAABBBC" と入力しますと言う

A-4 B-3 C-1

する必要があります私の予想出力の代わりに私が手 C-4 C-3 C-1

任意の助けをいただければ幸い、

#include <iostream> 
using namespace std; 




class moose 
{ 
    char inputbuffer[122]; 
    char countbuffer[122]; 
    long count; 
    short index = 0; 

public: 
    char charcount(); 
    char charinput(); 
    char initialize(); 

}; 

int main() 
{ 
    moose obj; 
    obj.initialize(); 
    obj.charinput(); 
    obj.charcount(); 
    system("pause"); 

} 


char moose::initialize() 
{ 
    for (int i = 0; i < 122; i++) 
     countbuffer[i] = 0; 

    return 0; 

} 

char moose::charinput() 
{ 
    cout << "Enter your text and I'll read your characters" << endl; 
    cin.getline(inputbuffer, 132); 
    cin.gcount(); 
    count = cin.gcount(); 
    count--; 
    return 0; 
} 

char moose::charcount() 
{ 
    for (int i = 0; i < count; i++) 
    { 
     if (inputbuffer[i] >= 'a' & inputbuffer[i] <= 'z' || inputbuffer[i] >= 'A' & inputbuffer[i] <= 'Z' || inputbuffer[i] > '0' & inputbuffer[i] <= '9'){ 
      index = inputbuffer[i]; 
      countbuffer[index]++; 

     } 


    } 


    for (int i = 0; i <= 122; i++) { 
     if (countbuffer[i] > 0) 
     { 
      cout << char(index) << " - " << int(countbuffer[i]) << endl; 
     } 
    } 


    return 0; 
} 

Answer 1

以下の私のコードは次のように最後のループを変更します。

for (int i = 0; i <= 122; i++) { 
    if (countbuffer[i] > 0) 
    { 
     cout << char(i) << " - " << int(countbuffer[i]) << endl; 
    } 
} 

最初のループでビット単位のANDと論理ANDを変換します（& &）。

ところで、実際に文字のヒストグラムを作成しようとしているので、あなたの場合は、unordered_mapのようなSTLコンテナを使用する方がよいでしょう。一般的に数文字の頻度を格納しようとすると、多くのエントリが最後に0になるため、すべての可能な文字でインデックスされた配列を割り当てるのは無駄です。

Answer 2

char moose::charcount() 
{ 
    for (int i = 0; i < count; i++) 
    { 
     if (inputbuffer[i] >= 'a' && inputbuffer[i] <= 'z' || inputbuffer[i] >= 'A' && inputbuffer[i] <= 'Z' || inputbuffer[i] > '0' && inputbuffer[i] <= '9'){ 
      index = inputbuffer[i]; 
      countbuffer[index]++; 

     } 


    } 


    for (int i = 0; i <= 122; i++) { 
     if (countbuffer[i] > 0) 
     { 
      cout << char(i) << " - " << int(countbuffer[i]) << endl; 
     } 
    } 


    return 0; 
} 

1. インデックスの代わりに印刷する "I"。
2. & - ビット-AND、このような問題を解決する最適なツールは、あなたのデバッガです& &