0

我有一个二维指针数组（指向字符串）

 char *result[7000][14];

我想编写一个函数，返回每个“行”中的第一个字符串。

这是我尝试过的：

char *getRownames (int a, int b, char *matrix[a][b])
{
    char *rownames[a];
    for(int i=0;i<a;i++){
        rownames[i] = malloc(strlen(matrix[i][0])+1);
        strcpy(rownames[i],matrix[i][0]);
    }

    return *rownames;
}

进而

 char *names = getRownames(7000, 14, result);

我收到一条错误消息，指出 getRowNames 的类型冲突。仍然习惯于 C 并且不得不分配我自己的内存。

4

2 回答 2

1

这里有几个问题

  • 您的 return 语句是错误的（它应该只是行名，而不是 *rownames）。反正我不会那样做。
  • 我没有看到您的其余代码，但是如果您不初始化*result[][0]，您很可能会在 strlen 调用中出现段错误。
  • 我会避免尝试在该大小的堆栈上返回一个指向数组的指针（不要返回指向尚未 malloc 的局部变量的指针），所以我会传入数组并让函数填写它你。如果你有一个指向该数据大小的指针，即char *rownames=malloc(a*sizeof(char *)); 你会没事的。

所以我用我的测试代码做到了这一点：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>

void getRownames (int a, int b, char *matrix[a][b], char* rownames[a])
{
    int i;
    for(i=0;i<a;i++){
        //printf("%d\n",strlen(matrix[i][0]));
        rownames[i] = malloc(strlen(matrix[i][0])+1);
        strcpy(rownames[i],matrix[i][0]);
    }
    //strlen(matrix[i][0])
    //return &rownames[0];
}

int main(void) {
    char *result [700][14];
    int i=0;
    for(i=0;i<700;i++){
    result[i][0]="abcd0";
    }
    char *rownames[700];
    getRownames(700,14,result,rownames);
    printf("I finished");
    printf("%s",rownames[0]);
    printf("%s",rownames[1]);
    printf("%s",rownames[2]);
    printf("%s",rownames[3]);
}
于 2012-04-17T19:47:43.870 回答
0

你有一些事情发生在这里。

函数声明/原型需要为其数组和矩阵具有固定大小。*

char *getRownames (int a, int b, char *matrix[a][b])

不会工作，因为编译器不知道a或b在编译您的程序时。它需要是

char *getRownames (int a, int b, char *matrix[7000][14])

如果您知道数组将是那个大小。然后你不需要a或根本不需要b。如果您希望能够将不同大小的矩阵传递给函数，那完全是另一回事。

*（注意编译器允许你省略数组的第一维：char *matrix[][14]或char *array[]）

接下来，您需要将 malloc 的返回值转换为 char*，因为 malloc() 返回 void*：

rownames[a] = (char*)malloc(strlen(matrix[i][0])+1);

顺便说一句，它应该rownames[i]在你的循环中。:-) 因为i是你的循环变量。

最后，看起来您想要返回一个 char* 数组，但return *rownames只会返回数组中的第一个值。同样，如果您知道数组的大小，将现有数组传递给函数并让它填充值会更容易。否则，您必须 malloc 数组才能返回。

char *result[7000][14];
char *firstRows[7000];
//... other code that fills in these values
getRownames(7000, 14, result, firstRows);

void getRownames (int a, int b, char* matrix[7000][14], char* returnrows[7000])
{
    for(int i=0;i<a;i++){
        returnrows[i] = (char*)malloc(strlen(matrix[i][0])+1);
        strcpy(returnrows[i],matrix[i][0]);
     }
}
于 2012-04-17T19:55:42.987 回答