1

我尝试了很多组合，但实际上没有任何效果。时间已经够长了，所以我决定写这个问题。

我只想要一个指向结构的指针数组，以便以后可以通过交换地址轻松对其进行排序。我有一个从文件中获取数据并写入数组的函数。不幸的是，我不可能在函数之外读取这些数据。

我最后一次尝试（我删除了文件操作，因为这些不是问题）：

页眉.h：

#pragma once

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

typedef struct {
    int32_t year;
    uint16_t month;
    uint16_t day;
} Date;

typedef struct {
    char name[16];
    Date date;
    uint32_t number;
} Player;

size_t readData(Player** players);

来源.c：

#include "Header.h"

size_t readData(Player** players) {
    players = NULL;

    players = realloc(players, sizeof(Player*) * 1);
    players[0] = malloc(sizeof(Player));
    strcpy(players[0]->name, "asd");
    printf("$ %s\n", players[0]->name);//OK

    return 1;
}

int main() {
    Player **players = NULL;
    uint32_t elemCount, i;

    elemCount = readData(players);

    printf("> %s", players[0]->name);//BUM - ERROR!

    return 0;
}

我正在使用 Visual Studio Community 2015。我知道用 C 进行编码并没有那么多，但我设法配置了解决方案，而且它似乎不是 imo 的事情。

我将非常感谢您的帮助。然而，我想把它看作是我的代码的翻版。我尝试了一些其他类似 StackOverFlow 的问题的答案，但它们没有帮助。

4

3 回答 3

2

如果函数的参数不仅被读取而且也应该是输出，则必须将指向数据的指针传递给函数。如果您的数据类型是Player**您的参数类型必须是Player ***，那么玩家列表本身就可以在函数内部进行更改。

size_t readData(char* fname, Player*** players) {
                                // ^ players is a input and output parameter
    Player **tempPlayers = *players; // local list of players
    tempPlayers = realloc(tempPlayers, sizeof(Player*) * 1);
    tempPlayers[0] = malloc(sizeof(Player));
    strcpy(tempPlayers[0]->name, "asd");
    printf("$ %s\n", tempPlayers[0]->name);//OK

    *players = tempPlayers; // write back list of players to paramter
    return 1;
}

int main() {
    Player **players = NULL;
    uint32_t elemCount, i;

    char *fileName = NULL; 
    elemCount = readData(&players);
                      // ^

    printf("> %s", players[0]->name);//BUM - ERROR!

    return 0;
}

如果您不想使用***，可以这样做：

Player* *readData(char* fname, Player** players, size_t *size) {
    players = realloc(players, sizeof(Player*) * 1);
    players[0] = malloc(sizeof(Player));
    strcpy(players[0]->name, "asd");
    printf("$ %s\n", players[0]->name);//OK

    *size = 1;
    return players;
}

players = readData( fileName,  players, &elemCount );
于 2016-01-15T17:54:19.620 回答
1

您通过value传递指向函数的指针；原来players在main 没有改变。这与这样做没有什么不同：

#include <stdio.h>

void func(int x)
{
    x = 4;
}

int main()
{
    int x = 0;
    func(x);

    printf("%d\n", x);      // zero
}

我假设是指针让你感到困惑，所以你现在应该看到你的错误了。

要使修改在 之外可见readData，您将需要使用三个间接级别：

size_t readData(Player*** players) {
    *players = malloc(sizeof(Player*) * 1);
    *(players)[0] = malloc(sizeof(Player));
    strcpy((*players)[0]->name, "asd");
    printf("$ %s\n", (*players)[0]->name);//OK

    return 1;
}

int main() {
    Player **players = NULL;
    uint32_t elemCount, i;

    elemCount = readData(&players);

    printf("> %s", players[0]->name);  // prints 'asd'

    return 0;
}

realloc当您知道传递的指针 always时使用没有意义NULL，因此我将其更改为malloc。

附带说明：Visual Studio 非常适合 C 开发，并实现了 C99 甚至 C11 的部分内容（过去 MSVC 一直停留在 C89）。您可以只.c为源文件使用扩展名，这将使编译器假定代码是 C，或者您可以在属性页中显式设置它。

于 2016-01-15T17:56:56.157 回答
1

传递给 readData() 的参数必须是调用者指针 'players' 的地址

readData() 中的每个引用都players必须考虑到这一点。

否则调用者的指针'player'将不会被更新。

然后 main() 函数，对 printf() 的调用试图从地址 0 输出，导致未定义的行为，导致 seg 错误事件。

在 main() 函数中，如果您插入：

printf( "player value; %p\n", (void*)players);

在当前调用 printf() 之前，您将看到players指针仍然包含 NULL。

于 2016-01-16T15:05:11.447 回答