Dynamiczna tablica (automatycznie zwiększa swoją pojemność) język C

Question

Witam

Próbuję zrobić program, który będzie automatycznie powiększał tablice (buffer) o 5 kolejnych miejsc, czyli użytkownik będzie wprowadzał liczby i jeśli wprowadzi ich więcej niż 5 to tablica się zwiększy, więc będzie 10 elementowa(tak do nieskończoności co 5). Udało mi się to już osiągnąć, aczkolwiek chce również podzielić cały program na funkcje. Pojawił się tu problem, ponieważ jeśli wyświetlanie dam w funkcji wpisywanie to wszystko działa okej, ale jeśli tak jak tutaj podzieliłem je na dwie osobne to jakby po wykonaniu funkcji wpisywanie w main zmienna "buffer" się nie powiększała, tylko dalej była 5 elementowa. Ma ktoś jakiś pomysł ?

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

void wpisywanie(size_t rozmiar, int *buffer)
{
    size_t maximumLength = rozmiar;

    int liczba;
    int i = 0; // zmienna sterujaca

     while(liczba != 0)
    {
        printf("Podaj liczbe: ");
        scanf("%d", &liczba);

        if(i == maximumLength)
        {
            int *newBuffer = realloc(buffer, (maximumLength = maximumLength + rozmiar) * sizeof (*buffer) );
            if (!newBuffer)
            {
                free(buffer);
                perror("blad realloc");
            }
            buffer = newBuffer;
        }

        buffer[i++]=liczba;
    }


}


void wyswietlanie(int *array)
{
    int i = 0;
    while(array[i] != 0)
    {
        printf("%d ",array[i]);
        i++;
    }
}




int main()
{
    const size_t rozmiar = 5; //poczatkowy rozmiar
    int *buffer;
    buffer = (int*)malloc(rozmiar * sizeof(*buffer));//oblicza potrzebną wielkość tablicy. Dla każdej zmiennej int potrzebujemy tyle bajtów, ile zajmuje ten typ danych



    wpisywanie(rozmiar,buffer);
    wyswietlanie(buffer);

    free(buffer);

    return 0;
}

 

Answer 1

Wskaźnik to zwykła zmienna. Musisz przesłać jej adres, aby ją zmodyfikować. Czyli drugi argument funkcji wpisywanie musi być podwójnym wskaźnikiem, a przy wywoływaniu tej funkcji przesyłaj adres wskaźnika.

Comments

Poprawiłem tak jak Pan napisał, ale teraz kiedy chce wprowadzić 3 liczbę od razu wyskakuje błąd.

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

void wpisywanie(size_t rozmiar, int **buffer)
{
    size_t maximumLength = rozmiar;

    int liczba;
    int i = 0; // zmienna sterujaca

     while(liczba != 0)
    {
        printf("Podaj liczbe: ");
        scanf("%d", &liczba);

        if(i == maximumLength)
        {
            int *newBuffer = realloc(buffer, (maximumLength = maximumLength + rozmiar) * sizeof (*buffer) );
            if (!newBuffer)
            {
                free(buffer);
                perror("blad realloc");
            }
            *buffer = newBuffer;
        }

        *buffer[i++]=liczba;
    }


}


void wyswietlanie(int *array)
{
    int i = 0;
    while(array[i] != 0)
    {
        printf("%d ",array[i]);
        i++;
    }
}




int main()
{
    const size_t rozmiar = 5; //poczatkowy rozmiar
    int *buffer;
    buffer = (int*)malloc(rozmiar * sizeof(*buffer));//oblicza potrzebn¹ wielkoœæ tablicy. Dla ka¿dej zmiennej int potrzebujemy tyle bajtów, ile zajmuje ten typ danych



    wpisywanie(rozmiar,&buffer);
    wyswietlanie(buffer);

    free(buffer);

    return 0;
}

 

---

int liczba;
int i = 0; // zmienna sterujaca

while(liczba != 0)
{

Nie używaj zmiennych lokalnych bez nadanej wartości. Takowa zmienna zawiera śmieci i może być powodem błędów. Jeśli chcesz wyjść z pętli po wprowadzeniu zera, zastosuj pętlę do-while.

int *newBuffer = realloc(buffer, (maximumLength = maximumLength + rozmiar) * sizeof (*buffer) );

Używasz złych parametrów funkcji realloc. W pierwszym musi być *buffer, zaś w drugim zamień sizeof(*buffer) na sizeof(**buffer) lub po prostu sizeof(int).

free(buffer);

Tu dokładnie to samo. Powinieneś też zakończyć tutaj cały program. Przecież wtedy zwalniasz całą pamięć i reszta kodu nie ma racji bytu.

*buffer[i++]=liczba;

Operator [] ma wyższy priorytet niż *.

---

Zamykam temat, program jest dobrze napisany. Problem był jedynie na moim komputerze, ponieważ kiedy pracowałem nad programem nie używałem na końcu funkcji free(...) , dopiero kiedy skończyłem program go dodałem. Przez to wskazywało cały czas na te zajęte już adresy i wychodziły błędy. Reset komputera wyczyścił zajęte komórki i teraz program się poprawnie kompiluje. Dziękuję za pomoc, bo faktycznie lepiej to chodzi z pętlą do i zainicjowaną zmienną liczba.

Pozdrawiam

---

Zamykam temat, program jest dobrze napisany. 

No niezbyt, jeśli nie zastosowałeś się do moich porad (mogłem coś pominąć, ale wątpię). 

Przez to wskazywało cały czas na te zajęte już adresy i wychodziły błędy. Reset komputera wyczyścił zajęte komórki i teraz program się poprawnie kompiluje.

Jak to jest możliwe? Wszelkie dzisiejsze systemy operacyjne czyszczą zasoby po zakończeniu procesu. Na jakiej ty pracujesz konfiguracji?