如何摆脱C上的“分段错误”？

Question

我刚开始为 MOOC 编写 C 语言。 第一周的任务是建造马里奥楼梯（使您能够在每个级别结束时到达旗帜顶部的楼梯）。 事件虽然我的程序似乎正在运行，但我在楼梯/金字塔的尽头不断收到“分段错误”。 我想知道是否有人可以向我解释为什么这个程序会出现“分段错误”？

#include <stdio.h>
#include <cs50.h>
#include <string.h>

int main(void)
{
    //Initializing variables : 
    char str[] = "";
    int z;
    int i;
    int n = get_int("How many stairs ?");
    
    //Checking that the number of stairs is correct : 
    
    while (n > 8 || n < 1)
    {
        n = get_int("How many stairs ?");
    }
    
    //Adding each layer of stairs : 
    
    for (i = 0 ; i < n ; i = i + 1)
    {
        
        for (z = 0 ; z <= (n - i) ; z += 1)
        {
            strncat(str, " ", 1);
        }
        
        for (z = 0 ; z <= i ; z += 1)
        {
            strncat(str, "#", 1);
        }
        
        strncat(str, " ", 1);
        
        for (z = 0 ; z <= i ; z += 1)
        {
            strncat(str, "#", 1);
        }
        
        for (z = 1 ; z <= (n - i) ; z += 1)
        {
            strncat(str, " ", 1);
        }
        
        //Going to the next layer of stairs : 
        
        strncat(str, "\n", 1);
    }
    
    //Showing the drawing of the stairs made of "#": 
    
    printf("%s", str);
}

他就是这样打算做的：

$ ./mario
Height: 8
       #  #
      ##  ##
     ###  ###
    ####  ####
   #####  #####
  ######  ######
 #######  #######
########  ########

非常感谢！

Answer 1

strcat 不会更改 char 数组长度，而您的数组只有一个元素。您在 UB 的字符边界之外写入，因此是段错误。您还错误地使用了strncat 函数。无需将所有行放在一个字符串中。

根本不需要最后一个循环

这里确实需要任何字符串函数或字符串。

#include <stdio.h>
#include <cs50.h>

int main(void)
{
    int z;
    int i;
    int n = get_int("How many stairs ?");
    
    //Checking that the number of stairs is correct : 
    
    while (n > 8 || n < 1)
    {
        n = get_int("How many stairs ?");
    }
    
    //Adding each layer of stairs : 

    fputc('\n', stdout);
    
    for (i = 0 ; i < n ; i ++)
    {
        
        for (z = 0 ; z <= (n - i) ; z += 1)
        {
            fputc(' ', stdout);
        }
        
        for (z = 0 ; z <= i ; z += 1)
        {
            fputc('#', stdout);
        }
        
        fputc(' ', stdout);
        
        for (z = 0 ; z <= i ; z += 1)
        {
            fputc('#', stdout);
        }
        fputc('\n', stdout);
    }    
}

https://godbolt.org/z/v1EMn1qsf

但如果你想要字符串和 strncat，你可以使用 VLA

#include <stdio.h>
#include <cs50.h>
#include <string.h>

int main(void)
{
    //Initializing variables : 
    int z;
    int i;
    int n = get_int("How many stairs ?");
    
    //Checking that the number of stairs is correct : 
    
    while (n > 8 || n < 1)
    {
        n = get_int("How many stairs ?");
    }

    char str[n*2 + 3];

    printf("\n");
    
    //Adding each layer of stairs : 
    
    for (i = 0 ; i < n ; i = i + 1)
    {
        str[0] = 0;
        for (z = 0 ; z <= (n - i) ; z += 1)
        {
            strncat(str, " ", 2);
        }
        
        for (z = 0 ; z <= i ; z += 1)
        {
            strncat(str, "#", 2);
        }
        
        strncat(str, " ", n*2+2);
        
        for (z = 0 ; z <= i ; z += 1)
        {
            strncat(str, "#", 2);
        }
        
        
        //Going to the next layer of stairs : 
        
        strncat(str, "\n", n*2+3);
        printf("%s", str);
    }  
}

https://godbolt.org/z/hEq1oGEq6

Answer 2

您的 str 变量没有足够的存储空间。 当 C 到达你的char str[] = ""; 时，它会为数组分配内存，刚好够你传递的文字和Null 字符，在这种情况下，空间的大小将是 1 char。因此，当您开始连接字符时，请将它们添加到您的变量不允许的空格中。 C 也会为程序分配额外的空间，因此您实际上是在编辑为程序保留的内存；这就是为什么分段错误不会更早出现的原因。 你可以保留足够的空间，它会工作的

    char str[160] = "";