2016-12-05 80 views
0

我即将完成我的代码!我可以使用一些帮助,我写了一个程序来计算字符串中的字母数量。在输出我的数据时,我的问题到了最后。说我输入字符串 “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; 
} 
+3

使用正确的工具来解决这些问题是你的调试器。在*堆栈溢出问题之前,您应该逐行执行您的代码。如需更多帮助,请阅读[如何调试小程序(由Eric Lippert撰写)](https://ericlippert.com/2014/03/05/how-to-debug-small-programs/)。至少,您应该\编辑您的问题,以包含一个[最小,完整和可验证](http://stackoverflow.com/help/mcve)示例,该示例再现了您的问题,以及您在调试器。 –

+0

仔细阅读最后一个循环中的变量名称。 – molbdnilo

+0

提高代码的一般质量:如果方法不返回任何内容,请将void,couter(我猜)设置为整数。使用常量122 etc –

回答

1

代码修改最后的循环如以下:

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

并在第一个循环中将位和转换为逻辑和(使用& &)。

顺便说一句,在你的情况下,最好使用像unordered_map这样的STL容器,因为你要建立一个实际的字符直方图。如果打算将频率存储为一般字符,那么分配由所有可能字符索引的数组是很浪费的,因为很多条目最后都是0。代替指数的 “我”

+0

嘿,谢谢!这是一个简单的修复。我不确定你的STL容器是什么意思,我会阅读它。真的很感谢帮助! – eggo

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. 打印。
  2. & - 位与使用& &