Частота символов

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

И конвертировать битовый ANDS в логический НСР в первом цикле (использование & &).

Кстати, в вашем случае лучше использовать контейнер STL, например unordered_map, так как вы на самом деле собираете гистограмму символов. Если вы собираетесь хранить частоту для нескольких символов в целом, нецелесообразно выделять массив, индексированный всеми возможными символами, так как в конце будет много записей.

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. Print "я" вместо индекса.
2. & - побитовое И, используйте & &