Передача динамически выделяемых целочисленных массивов в C

StackOverflow https://stackoverflow.com/questions/894604

Вопрос

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

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

Как и в примере, который я упомянул выше, я хотел бы передать в функцию указатель на целочисленный массив вместе с количеством элементов «num» (или «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;
}
Это было полезно?

Решение

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

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

Другие советы

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


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

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


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

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

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top