0

このサイトの「C で多次元配列を渡す」の例を読みました。

これは char 配列を使用した優れた例であり、多くのことを学びました。動的に割り当てられた 1 次元整数配列を処理する関数を作成し、その後、多次元整数配列を処理する別の関数を作成して、同じことを行いたいと考えています。関数への戻り値としてそれを行う方法を知っています。しかし、このアプリケーションでは、関数の引数リストでそれを行う必要があります。

上記の例のように、整数配列へのポインターを要素数「num」(または 2D 配列関数の場合は「row」と「col」など) と共に関数に渡したいと思います。 ）。ここで他の例の作り直されたバージョンを入手しましたが、これを機能させることができません。試してみてください (その例からの新しい、または変更されたコード行はマークされています)。これを解決する方法を知っている人はいますか？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define ELEMENTS 5
void make(char **array, int **arrayInt, int *array_size) { 
    int i;
    char *t = "Hello, World!";
    int s = 10; // new
    array = malloc(ELEMENTS * sizeof(char *));
    *arrayInt = malloc(ELEMENTS * sizeof(int *));  // new
    for (i = 0; i < ELEMENTS; ++i) {
        array[i] = malloc(strlen(t) + 1 * sizeof(char));
        array[i] = StrDup(t);
        arrayInt[i] = malloc( sizeof(int)); // new
        *arrayInt[i] = i * s; // new
    }
}
int main(int argc, char **argv) {
    char **array;
    int  *arrayInt1D; // new
    int size;
    int i;
    make(array, &arrayInt1D, &size); // mod
    for (i = 0; i < size; ++i) {
        printf("%s and %d\n", array[i], arrayInt1D[i]); // mod
    }
    return 0;
}
4

2 に答える 2

0

そのコードには非常に多くの問題があります。以下をご覧ください。

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

#define ELEMENTS 5

/*
 * A string is an array of characters, say char c[]. Since we will be creating
 * an array of those, that becomes char *(c[]). And since we want to store the
 * memory we allocate somewhere, we must be given a pointer. Hence char
 * **(c[]).
 *
 * An int doesn't require a complete array, just int i. An array of those is
 * int i[]. A pointer to those is then int *(i[]).
 */
void
make(char **(chars[]), int *(ints[]), size_t len)
{
    static char hw[] = "Hello, World!";
    size_t i = 0;

    /*
     * Allocate the memory required to store the addresses of len char arrays.
     * And allocate the memory required to store len ints.
     */
    *chars = malloc(len * sizeof(char *));
    *ints = malloc(len * sizeof(int));

    /* Fill each element in the array... */
    for (i = 0; i < ELEMENTS; i++) {
        /* ... with a *new copy* of "Hello world". strdup calls malloc under
         * the hood! */
        (*chars)[i] = strdup(hw);
        /* ...with a multiple of 10. */
        (*ints)[i] = i * 10;
    }
}

int
main(void)
{
    /* A string c is a character array, hence char c[] or equivalently char *c.
     * We want an array of those, hence char **c. */
    char **chars = NULL;
    /* An array of ints. */
    int *ints = NULL;
    size_t i = 0;

    /* Pass *the addresses* of the chars and ints arrays, so that they can be
     * initialized. */
    make(&chars, &ints, ELEMENTS);
    for (i = 0; i < ELEMENTS; ++i) {
        printf("%s and %d\n", chars[i], ints[i]);
        /* Don't forget to free the memory allocated by strdup. */
        free(chars[i]);
    }

    /* Free the arrays themselves. */
    free(ints);
    free(chars);

    return EXIT_SUCCESS;
}
于 2009-05-21T19:29:53.603 に答える
0

ここに行のサイズがありません：


arrayInt[i] = malloc( sizeof(int)); // new

次のようにする必要があります。


arrayInt[i] = malloc( row_len * sizeof(int)); // new

以前は、指定された文字列の長さを行サイズとして使用していました (これもstrlen(t)+1括弧で囲む必要がありますが、sizeof(char)1 であるため効果は同じです)

于 2009-05-21T19:25:20.443 に答える