C:将2维数组分配给另一个数组

Question

int main() {
    FILE *fp = fopen("fileA.txt", "r"); /* read file */
    int i = 0;
    char name[200][100];
    char goods[200][100];
    char qty[200][100];
    char temp[200][100];
    int x = 0;
    int result;
    while (!feof(fp)) {
        fscanf(fp, "%[^,] , %[^,] , %s " , name[i], item[i], qty[i]); /*get file content and store in array */ 
        if (strcmp(item[i], "Football") == 0) { /* only select Football */
            temp[x][x] = qty[i];
            if (x > 0) {
                if (strcmp(temp[x][x], temp[x + 1][x + 1]) > 0) { /*compare who has more football qty */
                    result = x; /*output the person who have more football*/
                }   
            }
            x = x + 1;
        }
    }
    printf("%s is team leader in class.n", name[result]);
    fclose(fp);
    getchar();
    return 0;
}

大家好，我不知道为什么结果不正确。

我想找出谁拥有更多足球并打印出他/她的名字。

在if (strcmp(temp[x], temp[x + 1]) > 0)上似乎有问题我并不清楚使用指针和地址。

文本文件中的内容是：

Alice,Eating,001
Kitty,Football,006
Ben,Swimming,003
May,Football,004

我希望结果是：

Kitty is team leader in class.

谢谢。

Answer 1

您的代码中有多个问题：

• 您不测试文件是否正确打开。

• 您无法与while (!feof(fp)) {正确解析文件。您应该迭代fscanf()返回3，或者最好按行读取输入，然后用sscanf()进行解析。

• 您不告诉 fscanf()要存储到目标数组中的最大字符数。这可能导致无效输入的行为。

• 您不会为每行读取添加i。每行输入覆盖上一个。

• 您没有检查是否有200行。在这种情况下，未定义的行为。

• 您的测试以找到最高数量的足球迷：在这里不需要2D阵列，只需跟踪当前的最大值并在需要时进行更新。

这是一个修改版本：

#include <stdio.h>
int main() {
    FILE *fp = fopen("fileA.txt", "r"); /* read file */
    char buf[300];
    char name[200][100];
    char goods[200][100];
    char qty[200][100];
    int i, qty, max_qty = 0, result = -1;
    if (fp == NULL) {
        fprintf(stderr, "cannot open filen");
        return 1;
    }
    for (i = 0; i < 200; i++) {
        if (!fgets(buf, sizeof buf, fp))
            break;
        if (sscanf(buf, " %99[^,], %99[^,],%99s", name[i], item[i], qty[i]) != 3) {
            fprintf(stderr, "invalid input: %sn", buf);
            break;
        }
        if (strcmp(item[i], "Football") == 0) { /* only select Football */
            qty = atoi(qty[i]);
            if (result == -1 || qty > max_qty) {
                result = i; /*store the index of the person who have more football */
            }
        }
    }
    if (result < 0)
        printf("no Football fan at all!n");
    else
        printf("%s is team leader in class with %d in Football.n", name[result], max_qty);
    fclose(fp);
    getchar();
    return 0;
}

Answer 2

上面的代码尚不清楚您想在此代码块中要做的事情

if ( strcmp(temp [x], temp [x+1]) > 0 ){ /* when matches, accessing temp[x+1] results in undefined behaviour */
                result = x;
}

也为什么char *temp[200][100];存储qty[i]，char *temp就足够了，或者您可以服用char temp[200][100];

这里有些更好，因为需求尚不清楚。

int main() {
        FILE *fp= fopen("fileA.txt","r"); /* read file */
        if(fp == NULL) {
                /* not exist.. write something ?? */
                return 0;
        }
        char name [200][100],goods[200][100],qty[200][100],temp[200][100];
        int x = 0,result = 0, i = 0;
        while ((fscanf(fp, "%[^,] , %[^,] , %s " , name[i], goods [i], qty[i])) == 3) {
                if (strcmp(goods[i] , "Football") == 0){
                        strcpy(temp[x],qty[i]);
                        if ( strcmp(temp [x], temp [x+1]) > 0 ) { /* UB ? */
                                result = x;
                                x+=1;
                        }
                }
        }
        printf("%s is team leader in class. n", name[result]);
        fclose(fp);
        getchar();
        return 0;
}