Question

我正在写一个代码来打印所有存储在2d数组中的字符串，这是在用户完成字符串键入并提及每个字符串的最大长度和总数字符串之后（它最终将字符串和行号一起打印）。问题在于代码实际上将所有字符串存储在2d数组中，并且整个行间距为一整行，即一个完整的空行。代码，预期输出及其提供的输出如下。

代码：

#include <stdio.h>

int main() {
    char s[20][30];
    int i, number_of_strings, length_of_string, j = 0;
    scanf("%d %d", &number_of_strings, &length_of_string);

    for (i = 0; i<number_of_strings; i++) {
        while ((s[i][j++] = getchar()) != '\n' && j<length_of_string)
            s[i][j] = '\0';
        j = 0;
    }

    for (i = 0; i<number_of_strings; i++) {
        printf("i= %d   %s\n", i, s[i]);
    }

    return 0;
}

示例输入：

2 3
raj
jar

预期输出：

i= 0   raj
i= 1   jar

输出结果：

i= 0   

i= 1   raj
i= 2    

i= 3   jar

请纠正我在哪里做错了。

Answer 1

您已经击中one of the many issues with scanf。在这种情况下，scanf("%d %d", ...)在缓冲区上保留了换行符。您可以将其拖到尾随空格中，并在末尾留一个空格。

scanf("%d %d ", &number_of_strings , &length_of_string);

然后，您如何阅读一行很复杂。您可以像这样简化它：

 int c, j;
 for(j = 0; (c = getchar()) != '\n'; j++ ) {
     s[i][j] = (char)c;
 }
 s[i][j] = '\0';

或更简单...

for(int i=0 ; i<number_of_strings ; i++) {
    scanf("%29s", s[i]);
}

并且不需要length_of_string。实际上，这是一个责任，因为您最多只能分配30个字节。同样，number_of_strings可以高于分配的20。最好在输入或内存用完之前进行读取。

#include <stdio.h>

const int MAX_STRINGS = 20;
const int MAX_LENGTH = 30;
int main(){ 
    char s[MAX_STRINGS][MAX_LENGTH];
    int num_strings;
    for(num_strings = 0; num_strings < MAX_STRINGS ; num_strings++) {
        if( scanf("%29s", s[num_strings]) < 1 ) {
            break;
        }
    }

    for( int i = 0 ; i < num_strings; i++){                                                                               
        printf("i= %d   %s\n",i,s[i]);
    }

    return 0;      
}

二维数组输出错误

1 个答案: