2016-12-05 3 views
0

Я пытаюсь подсчитать частоту букв в строковом вводе. У меня проблемы, и я точно не знаю, что я делаю неправильно. Я получаю странные результаты, как смайлики. Я хочу, чтобы мой код, чтобы сделать что-то вроде этого, если строка «abcdc» а - 1 б - 1 с - 2 d - 1Частота подсчета символов в строке C++

 #include <iostream> 
using namespace std; 




class moose 
{ 
    char inputbuffer[132]; 
    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') { 
      index = inputbuffer[i]; 
      countbuffer[index]++; 

     } 
     for (i = 0; i < count; i++) 
     { 
      cout << inputbuffer[i] << " - " << countbuffer[index] << endl; 
     } 
    } 




    return 0; 
} 
+0

выключено тема: просто ответил на аналогичный вопрос. Такой подход может быть полезен. http://stackoverflow.com/a/40967365/4581301 – user4581301

ответ

0

У вас есть несколько вопросов, с кодом:

  1. Метод moose::charinput() содержит дополнительный звонок для gcount(). Вам нужно только это заявление: count = cin.gcount();. Не должно быть и count--;.
  2. Состояние внутри moose::charcount() должен быть

    if (inputbuffer[i] >= 'a' && inputbuffer[i] <= 'z') 
    
  3. moose::charcount() должны иметь две различные петли:

    char moose::charcount() 
        { 
         for (int i = 0; i < count; i++) 
         { 
          if (inputbuffer[i] >= 'a' && inputbuffer[i] <= 'z') { 
           index = inputbuffer[i]; 
           countbuffer[index]++; 
          } 
         } 
         for (i = 0; i < count; i++) 
         { 
          cout << inputbuffer[i] << " - " << countbuffer[inputbuffer[i]] << endl; 
         } 
         return 0; 
        } 
    
  4. countbuffer должен быть объявлен вне int для печати правильно

+0

^^ Это ничем не отличается от моего исходного кода. Это не совсем то, что я хочу. если, например, я набираю «aaaaaa», он будет выходить a-1 a-2 a-3 a-4 a-5 ect, я пытаюсь просто вывести a-6. спасибо за помощь, хотя – eggo

+0

Если это ваше намерение, массив с счетчиками должен быть 'counter [26]', вы должны считать символы типа 'counter [inputbuffer [i] - 'a']' и печатать как 'cout < <(char) ('a' + i) << "-" << count [i] << endl; 'Кстати, ваш код не учитывает символы верхнего регистра, не нужны ли вам? – GMichael

+0

Да, они мне понадобятся, а также счет 0-9. Я пытался заставить мой мозг окунуться в то, как я буду выводить свои строчные массивы в первую очередь. Я считаю, что считать верхние и цифры, я создам еще два цикла логики. Я не совсем уверен, что вы имеете в виду с помощью счетчика [26] и счетчика [inputbuffer [i] - 'a'], но я получаю некоторые идеи, снова ценю руководство! – eggo