如何使函数返回指向数组结构的指针

Question

我正在使用C中的一个函数，它应该返回一个指向数组结构的指针。 结构是

struct corr{
  char nome[128];
  char cognome[128];
  char password[10];
  float importo;
};
typedef struct corr correntista;

返回指向此结构的指针的函数是

correntista *max_prelievo(FILE *fcorrentisti, FILE *fprelievi){
    correntista *corr_max[2];
    corr_max[0] = (correntista *)malloc(sizeof(correntista));
    corr_max[1] = (correntista *)malloc(sizeof(correntista));
    /*.........*/
    return *corr_max;
}

在主程序中，我想以下列方式打印返回的值：

*c_max = max_prelievo(fc, fp);
printf("Correntista con max prelievi:\n");
printf("Nome: %s\n", c_max[0]->nome);
printf("Cognome: %s\n", c_max[0]->cognome);
printf("Max prelievo: %f\n\n", c_max[0]->importo);

printf("Correntista con max versamenti:\n");
printf("Nome: %s\n", c_max[1]->nome);
printf("Cognome: %s\n", c_max[1]->cognome);
printf("Max versamento: %f\n", c_max[1]->importo);

但只有第一个结构c_max[0]具有预期值。 c_max[1]有垃圾值。 我的节目应该改变什么？

Answer 1

有几种解决方案，但它们都需要对功能的运行方式进行全面检查。 也许最好的解决方案是考虑scanf工作原理; 它不返回任何对象，而是仅对传递给它的参数指向的对象进行操作。 因此：

void max_prelievo(correntista *corr_max, FILE *fcorrentisti, FILE *fprelievi){
    /* NOTE: You should make a habit of separating your allocation from this logic;
     *       it makes debugging and testing so much easier... */

    corr_max[0] = (correntista) { .nome = "Michael",
                                  .cognome = "Anderson",
                                  .importo = 42.0 };

    corr_max[1] = (correntista) { .nome = "Undefined",
                                  .cognome = "Behaviour",
                                  .importo = -1.0 };


    /*.........*/
}

int main(void) {
    correntistia c_max[2] = { 0 };
    FILE *f1 = fopen(...), *f2 = fopen(...);

    max_prelievo(c_max, f1, f2);

    printf("Correntista con max prelievi:\n");
    printf("Nome: %s\n", c_max[0].nome);
    printf("Cognome: %s\n", c_max[0].cognome);
    printf("Max prelievo: %f\n\n", c_max[0].importo);

    printf("Correntista con max versamenti:\n");
    printf("Nome: %s\n", c_max[1].nome);
    printf("Cognome: %s\n", c_max[1].cognome);
    printf("Max versamento: %f\n", c_max[1].importo);
}

什么scanf 不会返回是一个整数，表示成功。 也许，如果你将来要使用返回值，你可以使用类似的模式吗？ 看看光明的一面：这里没有malloc或free ，你没有机会泄漏任何记忆; 代码尽可能简单。

---

或者，尽管看起来很可怕，如果你必须使用这样的模式，那么对malloc较少调用将很好地为你服务。 考虑这样的事情：

correntista *max_prelievo(FILE *fcorrentisti, FILE *fprelievi) {
    correntista *corr_max = malloc(2 * sizeof *corr_max);

    corr_max[0] = (correntista) { .nome = "Michael",
                                  .cognome = "Anderson",
                                  .importo = 42.0 };

    corr_max[1] = (correntista) { .nome = "Undefined",
                                  .cognome = "Behaviour",
                                  .importo = -1.0 };

    /*.........*/

    return corr_max;
}

main入口点几乎与我之前给出的例子相同......除了一件事： 别忘了free那段记忆！ :(

Answer 2

记住你拥有的是指针数组，所以当你退出函数时，数组超出了范围。 为此目的指向指针，如下所示。

correntista **max_prelievo(FILE *fcorrentisti, FILE *fprelievi){
    correntista **corr_max = malloc(sizeof(correntista *) * 2);
    corr_max[0] = malloc(sizeof(correntista));
    corr_max[1] = malloc(sizeof(correntista));
    /*.........*/
    return corr_max;
}

电话应该是

correntista **c_max = max_prelievo(fc, fp);