Передача динамически распределенных целочисленных массивов в C - PullRequest
0 голосов
/ 21 мая 2009

На этом сайте я прочитал пример " Передача многомерных массивов в C ".

Это отличный пример использования массивов символов, и я многому у него научился. Я хотел бы сделать то же самое, создав функцию для обработки динамически размещаемого одномерного целочисленного массива, а после этого создать другую функцию для обработки многомерного целочисленного массива. Я знаю, как сделать это как возвращаемое значение для функции. Но в этом приложении мне нужно сделать это в списке аргументов функции.

Так же, как в примере, который я упомянул выше, я хотел бы передать указатель на целочисленный массив функции вместе с количеством элементов "num" (или "row" и "col" для функции 2D-массива , так далее.). Я получил переработанную версию другого примера здесь, но я не могу заставить это работать, попробуйте, как я мог бы (строки кода, которые являются новыми или изменены, из этого примера, отмечены). Кто-нибудь знает, как это решить?

#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;
}

Ответы [ 2 ]

1 голос
/ 21 мая 2009

В этом коде довольно много проблем. Посмотрите на следующее:

#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;
}
0 голосов
/ 21 мая 2009

Здесь вам не хватает размера строки:


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

должно быть что-то вроде:


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

где раньше вы использовали длину данной строки в качестве размера строки (также strlen(t)+1 должно быть в скобках, хотя эффект тот же, потому что sizeof(char) равен 1)

...