cout<<char(int) 转换上的 C++ EOF

Question

这是我的代码：

#include <cstdlib>
#include <iostream>
#include <cstring>

using namespace std;

int main(int argc, char *argv[])
{
    int duze[26];
    int male[26];
    int n;
    //cout<<int('a')<<endl<<int('A');
    cin>>n;

    char temp;
    for (int i=0; i<27; i++)
        male[i]=0;
    for (int i=0; i<27; i++)
        duze[i]=0;

    while (n>=0)
    {
        cin.get(temp);
        if (temp=='\n') { n--; continue;}
        if (temp==' ') continue;
        if (temp>='a' && temp<='z')
            male[temp-'a']++;
        else if (temp>='A' && temp<='Z')
            duze[temp-'A']++;
    }

    for (int i=0; i<27; i++)
        if (male[i]>0) cout<<char(i+'a')<<" "<<male[i]<<endl;
    for (int i=0; i<27; i++)
        if (duze[i]>0) cout<<char(i+'A')<<" "<<duze[i]<<endl;
    //system("pause");
    return 0;
}

程序计算给定 n 行文本中的字母。不存在的字母被跳过。 当我在控制台中运行它时，它看起来不错，但我知道字符之前有 EOF 字符...... 如何避免？

Answer 1

你有一个buffer overflow（实际上有几个）。您声明了两个大小为 26 的数组（duze 和 male），然后继续将数据写入 27 个索引（0 到 26）。

由于这超出了缓冲区的范围，因此您正在踩踏其他内存，这会导致不稳定、不可预测的行为。 C++ 标准称之为未定义的行为：一旦你这样做了，任何事情都可能发生：你的程序可能会崩溃，它可能会出现像这样的细微错误，它可能看起来正常工作，甚至可能会擦除你的硬盘（虽然这不太可能）。

要解决此问题，请将所有 27 更改为 26；您也可以将数组大小增加到 27，但随后打印出 char(26+'a') 会得到 {，这可能不是您的意图。

Answer 2

数组索引从0 运行到N-1，其中N 是元素的数量：所有for 循环访问1 太多导致未定义的行为。将for 循环上的终止条件更改为i < 26。

另一个小提示，为了简化 duze 和 male 的初始化，您将它们声明为：

int duze[26] = { 0 };
int male[26] = { 0 };

这会将所有元素设置为0，这意味着您可以删除当前执行此操作的两个for 循环。

Answer 3

您应该检查以下内容：

if ( cin.get(temp) )
{
  // read was ok
}
else
{
  // eof or other issue on read
}