在c中打印锯齿状数组

Question

所以我正在用 C 制作一个锯齿状数组。我认为填充数组的代码没问题，我遇到的问题是打印。

代码如下

#include <stdio.h>

int *length;
int **row;

int main(int argc, char *argv[]) {

    //make an array that contains the lengths of the columns for each row
    length = malloc(argc * sizeof (int));
    for (int k = 0; k < argc; k++) {
        length[k] = atoi(argv[k]);
       // printf("%d ", lengths[k]);
    }
    row = malloc(argc * sizeof (int));

    // fill the columns
    int fill = 1;
    for (int i = 0; i < argc; i++) {
        row[i] = malloc(sizeof (int) * length[i]);
        for (int j = 0; j < length[i]; j++)
            row[i][j] = fill;

    }

    //print it
    for (int i = 0; i < argc; i++)
        for (int j = 0; j < length[i]; j++)
            printf("%d", row[i][j]);

    return 0;


}

这个程序接受命令行参数，所以如果我输入：

./jagged 1 3 5 1

我应该得到：

1
1 1 1
1 1 1 1 1
1

相反，我的编译器只是说

RUN FAILED

Answer 1

argc == 5，而argv[0] 包含./jagged。

atoi 对于argv[0] 将失败，并且其错误行为未定义。如果程序继续，0 是返回的值，您将存储 0 作为长度之一。这意味着您将执行malloc(sizeof(int) * 0);，这也可能导致麻烦。

要解决这些问题，您有两种选择：

1. 从i = 1; i < argc; i++ 循环以避免argv[0]。
2. 在使用argc 或argv 之前添加一行--argc, ++argv;。

您还应该考虑使用strtol 而不是atoi，因为它具有明确定义的错误行为。

除了您的程序逻辑之外，您只是忘记包含<stdlib.h> 才能使用malloc 和atoi as @Vishal explained。

Answer 2

使用一些限制性编译器标志，例如--pedantic --std=c11 -Wall -Wextra 用于 gcc。这将帮助您找到一些错误，例如缺少自己。

包括stdlib.h，以便您拨打malloc()。

argv[0] 不是您的第一个命令行参数，它是您的二进制文件的名称。所以你必须调整循环。

为了简化没有给出参数的情况，请包含 assert.h 并使用 assert(argc > 1); 检查 main 开头的参数数量。

您对row 的分配不正确，而是依赖于平台，因为row 的元素是int * 类型而不是int。而是分配row = malloc((argc - 1) * sizeof(int *));。

Answer 3

您需要包含stdlib 才能使用malloc 函数并在第一个块之后打印\n

    #include <stdio.h>
    #include <stdlib.h>
    int *length;
    int **row;

    int main(int argc, char *argv[]) {

        //make an array that contains the lengths of the columns for each row
        length = malloc(argc * sizeof (int));
        for (int k = 0; k < argc; k++) {
            length[k] = atoi(argv[k]);
           // printf("%d ", lengths[k]);
        }
        row = malloc(argc * sizeof (int));

        // fill the columns
        int fill = 1;
        for (int i = 0; i < argc; i++) {
            row[i] = malloc(sizeof (int) * length[i]);
            for (int j = 0; j < length[i]; j++)
                row[i][j] = fill;

        }

        //print it
        for (int i = 0; i < argc; i++)
        {
            for (int j = 0; j < length[i]; j++){
                printf("%d", row[i][j]);
            }
            printf("\n");
        }

        return 0;


    }

Answer 4

错误来源

1> argv[0] 一定会失败。

2> 没有为malloc 包含库（需要stdlib.h）。