在这个非常简单的程序中，什么会导致“信号 SIGABRT”错误？

Question

我一直在将练习作为函数和它们的一些界面。目前我只做了一个练习，就是在一些更大的行中删除第一次出现的“我们”行。有一个人有同样的问题和类似的情况（strcpy causing Thread 1: signal SIGABRT），但我不是在使用函数 strcpy 时遇到错误，而是在我们返回 switch-case 构造之后，虽然这是必须连接的。

#include <iostream>
using namespace std;

void task1_C() {
    const int N = 10;
    char S[N];
    char rem[3] = "we";
    cout << "Input your line (should consist of 10 characters): ";
    cin >> S;
    char *p = strstr(S, rem);
    if (p) {
        strcpy(p, p + strlen(rem));
        cout << S << endl;
    }
}

void task1_Cpp() {

}

void task2_C() {

}

void task2_Cpp() {

}

void task3_C() {

}

void task3_Cpp() {

}

void task4_Cpp() {

}

void task5_Cpp() {

}

int main(){
    int input;
    cout << "Input 1-5 for C++ functions, 11-13 for C functions." << endl;
    cin >> input;
    switch(input){
        case 1: task1_Cpp(); break;
        case 2: task2_Cpp(); break;
        case 3: task3_Cpp(); break;
        case 4: task4_Cpp(); break;
        case 5: task5_Cpp(); break;
        case 11: task1_C(); break;
        case 12: task2_C(); break;
        case 13: task3_C(); break;
    }
    return 0;
}

输出：

Input 1-5 for C++ functions, 11-13 for C functions.
11
Input your line (should consist of 10 characters): sweweratwe
sweratwe
Program ended with exit code: 9

Answer 1

您的程序具有未定义的行为，显示的输入已经在该行中

cin >> S;

数组S 的长度为10，输入由10 个字符组成，但>> 的char* 重载需要一个额外的字符来存储字符串的空终止符。这意味着数组必须至少有大小11。事实并非如此，您的程序会越界访问数组并导致未定义的行为，这意味着程序可能会或可能不会崩溃或执行任何操作。

您不应该使用char 数组在 C++ 中存储字符串。您应该改用std::string。在 C 语言中，您不应该使用 cin >>（因为它没有），您应该只学习/教授 C 或 C++，而不是混合使用。您不会在 C++ 中使用 strcpy 和 strstr。两种语言认为好的风格完全不同。

>> 的 char* 重载甚至将从 C++20 中的语言中删除。取而代之的是一个通过引用char[N] 数组的重载，然后它将在N-1 字符之后停止读取，以避免像这样的缓冲区溢出。在您在这里直接使用S（这是一个数组）的情况下，您的程序将跳过读取最后一个字符，因为它会超出数组的范围，在 C++20 中，但至少这个 undefined 的直接来源行为将不再存在。

此外，如果源和目标重叠，strcpy 的行为未定义。而是在 C++ 中使用 std::copy 或在 C 中使用 memmove。