Wprowadzanie liczb do tablicy dynamicznej, zwiekszenie tablicy - C

Question

Witam. Chciałem napisać program, w którym użytkownik wprowadza liczbę do tablicy, a po wpisaniu liczby 0 funkcja wyswietla cala tablice. Poczatkowa wielkosc tablicy powinna wynosic 5. Problem w tym, ze przy dalszych indeksach program wypisuje niepoprawne wartosci. Byłby ktoś w stanie mi podpowiedzieć co mam tu zmienić?

 

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

void fill_array(int *array, int (*size))
{

    int i=0,a,n=1,j;

    while(a!=0)
    {
        if(i == (*size))
        {
            ++n;
            (*size) = n*5;
            printf("%rozmiar %d, \t rozmiar z n+1: %d", (*size), 5*(n+1));

            (array) = realloc((array),(*size)*sizeof(int));
            //for(j = (5*n); j<(5*(n+1)); ++j) array[j]=0; // z tym wierszem przy i = 10 wywala program

        }
        printf("\n%d: \t", i);
        scanf("%d", &a);
        array[i] = a;
        ++i;
    }

}


void print_array(int *array, int (*size))
{
    int i=0;
    for(i; i<(*size); ++i)
    {
        printf("[%d] %d \n",i, array[i]);
    }
}



int main(void)
{
    int *tab=NULL;
    int size = 5;
    tab = calloc(size, size*sizeof(int));
    fill_array(tab, &size);
    print_array(tab, &size);
    free(tab);
    return 0;
}

 

Answer 1

realloc nie zeruje pamięci.

Comments

Ale to ma znaczenie tylko przy tych indeksach, do ktorych scanf nie dociera? Czemu przy wiekszych indeksach, indeksy 0 i 1 nie maja poprawnej wartosci?

---

Prawdopodobnie realloc przenosi twoją tablicę w inne miejsce pamięci.
Wskaźnik array w funkcji fill_array jest zmienną lokalną, dlatego przypisanie w 17 linii kodu nie ma wpływu na wskaźnik tab przekazywany do funkcji.
Musisz użyć wskaźniku na wskaźnik (int **array) i do funkcji przekazać adres wskaźnika tab (&tab)

---

Wprowadzenie tych zmian po wpisaniu 0 wywala program :/

---

Pokaz kod

---

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

int fill_array(int **array, int (*size))
{

    int i=0,a,n=1,j;


    while(a!=0)
    {
        if(i == (*size))
        {
            ++n;
            (*size) = n*5;
            printf("rozmiar %d, \t rozmiar z n+1: %d", (*size), 5*(n+1));

            (*array) = realloc((*array),(*size)*sizeof(int));
            //for(j = (5*n); j<(5*(n+1)); ++j) array[j]=0; // z tym wierszem przy i = 10 wywala program

        }
        printf("\n%d: \t", i);
        scanf("%d", &a);
        array[i] = a;
        ++i;
    }
    return i;

}


void print_array(int **array, int (*size))
{
    int i=0;
    for(i; i<(*size); ++i)
    {
        printf("[%d] %d \n",i, array[i]);
    }
}



int main(void)
{
    int ilosc;
    int *tab=NULL;
    int size = 5;
    tab = calloc(size, size*sizeof(int));
    fill_array(&tab, &size);
    print_array(&tab, &size);
    free(tab);
    return 0;
}

---

Nie poprawiłeś zapisywania i wczytywania danych z tablicy.

Poprawiony kod:
 

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

int fill_array(int **array, int (*size))
{
    int i=0,a,n=1;

    do
    {
        if(i == (*size))
        {
            ++n;
            (*size) = n*5;
            printf("rozmiar %d, \t rozmiar z n+1: %d", (*size), 5*(n+1));

            (*array) = realloc((*array),(*size)*sizeof(int));
            //for(j = (5*n); j<(5*(n+1)); ++j) array[j]=0; // z tym wierszem przy i = 10 wywala program

        }
        printf("\n%d: \t", i);
        scanf("%d", &a);
        (*array)[i] = a;
        ++i;
    }
    while(a!=0);
    return i;
}

void print_array(int **array, int (*size))
{
    int i=0;
    for(; i<(*size); ++i)
    {
        printf("[%d] %d \n",i, (*array)[i]);
    }
}

int main(void)
{
    int *tab=NULL;
    int size = 5;
    tab = calloc(size, size*sizeof(int));
    fill_array(&tab, &size);
    print_array(&tab, &size);
    free(tab);
    return 0;
}

 

---

Wszystko działa. Dziękuję za pomoc :)

Answer 2

Używasz niezdefiniowanej zmiennej a.