Question

我写了一小段代码，它应该采用一个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];
    }

}

Answer 1

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

首次修复

使用您在循环中寻找的字符终止字符串：

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

第二次修复

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

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

第三次修复

使用正确的string length检查作为循环终止标准：

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

或这三者的任何组合。

一般来说，将字符串声明为固定大小的数组永远不是一个好主意，因此请改用const char *letters = "...";。

Answer 2

你处于无限循环中

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

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

Answer 3

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

1）你可以在循环中包含\ n，但是必须在数组本身中，并且数组大小必须增加1到字母[28]。

2）你保持数组的声明，但for循环将是

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

第二种解决方案，您不必更改数组大小，循环将持续到数组结束

Answer 4

除了其他评论者提到的字符串末尾问题之外，您的代码可能无法为您提供所需的效果，因为您正在使用cout。该输出流会缓冲所有内容，直到您发送换行符为止，因此您需要输入＆＃34;键入＆＃34;将在打印最终换行符时立即显示。

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

Answer 5

如果您使用的是C ++，则应使用std::string。这将跟踪您的长度，但您也可以使用迭代器。

#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];
    }
}

Stack Smashing / BackTrace

5 个答案: