C++中的字符串溢出？这种持续不断的嘟嘟声很奇怪

Question

我正在开始开发垃圾文件生成器，但由于某种原因，如果我使用大量数字，它会无限发出蜂鸣声，直到文件完成，我认为 ascii 表中的某处有一个 \a 字符，或者它溢出了并导致错误蜂鸣声。有人想解释一下为什么这东西对我尖叫吗？

#include <string>
#include <iostream>
#include <cstdio>
#include <cstdlib>
#include <fstream>
#include <time.h>
#include <windows.h>
#define print cout<<

using namespace std;

int numberof,i;
char charvalue;
string charlist,filename;

int main()
{
    srand (time(NULL));
    print "What do you want the name of your file to be?(with a .txt extension)\n";
    getline(cin,filename);
    print "\nHow many characters do you want?\n";
    cin>>numberof;

    for(numberof>0;numberof!=0;numberof--)
        {
        i = rand() % 255 + 32;
        charvalue=i;
        charlist=charlist+charvalue;
        print charlist;
        }

    ofstream writefile(filename.c_str());
    writefile<<charlist;
    writefile.close();
    ShellExecute(NULL, "open", filename.c_str(), NULL, NULL, SW_SHOWNORMAL);
    return 0;
}

看起来此时字符最终已经正常显示，但它只将其中的 1/4 写入文本文件。有人知道为什么吗？

Answer 1

i = rand() % 255 + 32;

您可能希望它是这样的：

i = rand() % (255-32) + 32;

你也真的想要摆脱这个：

#define print cout<<

按照目前的情况，当（不是如果）给你作业评分的人决定杀了你时，他（她？）几乎肯定会被判无谋杀罪理由是这是自卫。

Answer 2

你想说

i = rand() % 223 + 32;

Answer 3

我认为 ascii 表中某处有一个 \a 字符

，我相信它位于位置 7。

另外，这个：

for(numberof>0;numberof!=0;numberof--)

应该是这样的：

 for(; numberof > 0; numberof--)

条件在中间，并且不需要任何初始化，因此在 for 循环的开头有一个空语句。

另外，可打印的 ASCII 字符范围仅为 32 到 126，因此您应该这样写：

i = rand() % 95 + 32;

另外，下面的代码效率极低，因为它每次都会生成一个新的字符串对象：

charlist=charlist+charvalue

改为这样做：

charlist.push_back(charvalue);

Answer 4

事实上，您每次都打印整个字符列表，这意味着只要您在输出字符串中生成至少几个 \a 字符（其几率为'太糟糕了），除了不断的蜂鸣声之外，你什么也得不到。

Answer 5

删除 print charlist 这样就不会打印出响铃字符。