0

我写了一小段代码,它应该采用一个 char 数组,让它看起来像是计算机正在输入文本。很简单,对吧?但是当我运行它时,终端告诉我:

*** stack smashing detected ***: ./TYPE terminated
======= Backtrace: =========
/lib/i386-linux-gnu/libc.so.6(__fortify_fail+0x45)[0xb759aeb5]
/lib/i386-linux-gnu/libc.so.6(+0x104e6a)[0xb759ae6a]
./TYPE[0x80486a9]
/lib/i386-linux-gnu/libc.so.6(__libc_start_main+0xf3)[0xb74af4d3]
./TYPE[0x8048591]
======= Memory map: ========
08048000-08049000 r-xp 00000000 08:01 1580147    /home/jeremy/Desktop/programming/cpp/Jumping into C++/TYPE
08049000-0804a000 r--p 00000000 08:01 1580147    /home/jeremy/Desktop/programming/cpp/Jumping into C++/TYPE
0804a000-0804b000 rw-p 00001000 08:01 1580147    /home/jeremy/Desktop/programming/cpp/Jumping into C++/TYPE
08a30000-08a51000 rw-p 00000000 00:00 0          [heap]
b7449000-b744b000 rw-p 00000000 00:00 0 
b744b000-b7467000 r-xp 00000000 08:01 4195157    /lib/i386-linux-gnu/libgcc_s.so.1
b7467000-b7468000 r--p 0001b000 08:01 4195157    /lib/i386-linux-gnu/libgcc_s.so.1
b7468000-b7469000 rw-p 0001c000 08:01 4195157    /lib/i386-linux-gnu/libgcc_s.so.1
b7469000-b7493000 r-xp 00000000 08:01 4198259    /lib/i386-linux-gnu/libm-2.15.so
b7493000-b7494000 r--p 00029000 08:01 4198259    /lib/i386-linux-gnu/libm-2.15.so
b7494000-b7495000 rw-p 0002a000 08:01 4198259    /lib/i386-linux-gnu/libm-2.15.so
b7495000-b7496000 rw-p 00000000 00:00 0 
b7496000-b763a000 r-xp 00000000 08:01 4198264    /lib/i386-linux-gnu/libc-2.15.so
b763a000-b763c000 r--p 001a4000 08:01 4198264    /lib/i386-linux-gnu/libc-2.15.so
b763c000-b763d000 rw-p 001a6000 08:01 4198264    /lib/i386-linux-gnu/libc-2.15.so
b763d000-b7640000 rw-p 00000000 00:00 0 
b7640000-b7718000 r-xp 00000000 08:01 8786914    /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16
b7718000-b7719000 ---p 000d8000 08:01 8786914    /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16
b7719000-b771d000 r--p 000d8000 08:01 8786914    /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16
b771d000-b771e000 rw-p 000dc000 08:01 8786914    /usr/lib/i386-linux-gnu/libstdc++.so.6.0.16
b771e000-b7725000 rw-p 00000000 00:00 0 
b773e000-b7742000 rw-p 00000000 00:00 0 
b7742000-b7743000 r-xp 00000000 00:00 0          [vdso]
b7743000-b7763000 r-xp 00000000 08:01 4198254    /lib/i386-linux-gnu/ld-2.15.so
b7763000-b7764000 r--p 0001f000 08:01 4198254    /lib/i386-linux-gnu/ld-2.15.so
b7764000-b7765000 rw-p 00020000 08:01 4198254    /lib/i386-linux-gnu/ld-2.15.so
bffc0000-bffe1000 rw-p 00000000 00:00 0          [stack]
abcdefghijklmnopqrstuvwxyzAborted (core dumped)

我是 C++ 新手(我有 C 背景),我不知道堆栈粉碎或回溯是什么。如果你能帮助我,那将有很大帮助!这是代码:

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <unistd.h>

using namespace std;

void type(char letters[]);

int main(){

    char letters[27] = "abcdefghijklmnopqrstuvwxyz";
    system("clear");
    type(letters);

    return 0;
}

void type(char letters[]){

    unsigned int wait = 30000000;

    system("clear");
    for(int i = 0; letters[i] != '\n'; i++){
        usleep(wait)
        cout << letters[i];
    }

}
4

5 回答 5

2

C++ 中的字符串应为空终止,即它们的最后一个字符是\0. 在您的代码中,循环不会在字符串末尾终止,letters因为您正在寻找\n该字符串中实际不存在的字符。

第一次修复

用您在循环中查找的字符终止字符串:

char letters[28] = "abcdefghijklmnopqrstuvwxyz\n";

第二次修复

\0通过查找实际存在的字符串结尾字符来终止循环:

for(int i = 0; letters[i] != '\0'; i++)

第三次修复

使用适当的字符串长度检查作为循环终止标准:

int len = strlen(letters);
for(int i = 0; i < len; i++)

或者这三者的任意组合。

通常,将字符串声明为固定大小的数组绝不是一个好主意,因此请const char *letters = "...";改用。

于 2014-12-26T21:09:12.920 回答
1

你处于无限循环中

您正在寻找'\n'不存在的。你应该寻找空字符\0

for(int i = 0; letters[i] != '\0'; ++i){
    usleep(wait)
    cout << letters[i];
}
于 2014-12-26T20:53:19.310 回答
0

除了其他评论者指出的字符串结尾问题之外,您的代码可能不会给您想要的效果,因为您正在使用 cout。该输出流会缓冲内容,直到您发送换行符为止,因此在打印最终换行符时,您的“打字”将立即出现。

您可以通过打印到 cerr 或在应该打印每个字符后刷新 cout 来获得一次一个字符的效果。

于 2014-12-26T21:11:30.350 回答
0

您可以在此处使用的两种解决方案之一。

1)您可以在循环中包含 \n,但这必须在数组本身中,并且数组大小必须增加 1 到字母 [28]。

2)您保留声明它的数组,但 for 循环将是

for(int i = 0; letters[i]; i++){
   cout << letters[i];
}

第二种解决方案,您不必更改数组大小,循环将一直持续到数组末尾

于 2014-12-26T21:06:13.657 回答
0

std::string如果你使用 C++,你应该使用。这将为您跟踪长度,但您也可以使用迭代器。

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <unistd.h>

using namespace std;

void type(const string &);

int main(){

    string letters = "abcdefghijklmnopqrstuvwxyz";
    system("clear");
    type(letters);

    return 0;
}

void type(const string &letters)
{
    unsigned int wait = 30000000;

    system("clear");
    for(string::const_iterator it = letters.begin(); it != letters.end(); it++)
    {
        usleep(wait)
        cout << *it;
    }
}

虽然,如果您愿意,您仍然可以通过索引访问字符串的字符,例如:

void type(const string &letters)
{
    unsigned int wait = 30000000;

    system("clear");
    for(size_t i = 0; i < letters.length(); i++)
    {
        usleep(wait)
        cout << letters[i];
    }
}
于 2014-12-26T22:03:42.067 回答