Prosiłbym o pomoc w przypadku tego kodu, czy dobrze zdefiniowałem tablice ?

Question

Prosiłbym o pomoc z tym kodem, a konkretnie chcę to zrobić tak żeby gościu sobie wpisywał liczby do tablicy, potem mógł sizeof albo zmienną c odczytać ilość indeksów a potem wyswietlić całość. Natomiast w ponizszych programie przerywa wpisywanie po 5 liczbach i nie wiem za bardzo o co chodzi ;/ Byłbym bardzo wdzięczny za wskazówki

#include <stdio.h>

int main()
{
	int a,b,c,d,n,rozmiar;
	int tab[rozmiar];
	n = 100000;
	printf("Podaj liczby ciagu, natomiast jeśli bedziesz chcial zakonczyc ciag wpisz 00: \n");
	for(b=0;b<n;b++)
	{
		scanf("%d", &tab[b]);
			if(tab[b]==00)
		{
			break;
		}
	
	}
	d = sizeof(tab[rozmiar]);
	printf("Ilosc elemntow tablicy wynosi %d", d);
	
	for(b=0;b<d;b++)
	{
		printf("%d", tab[b]);
	}
	

	
	return 0;

 

Answer 1

int ..., rozmiar;
int tab[rozmiar];

Chyba zapomniałeś rozmiar przypisać jakąś wartość, bo teraz tablica ma przypadkową wielkość.

d = sizeof(tab[rozmiar]);

Jeśli już, to d = sizeof(tab) / sizeof(int);, ale to wyświetli całą tablicę jak leci. Skoro b po wykonaniu pierwszej pętli będzie zawierać ilość wczytanych liczb, użyj tej wartości w pętli wyświetlającej.

Comments

hmmm teraz to zmienia postać rzeczy, ale rozmiar tablicy w jaki sposób mogę określić ? Skoro chce wpisywać liczby z konsoli a następnie mieć ilosc indeksów, chyba, że tak nie można i muszę wprowadzić ilosc elementów wcześniej a dopiero potem zapisywać liczby poszczególnym indeksom.

---

Jeśli to ma być w C, to użyj funkcji realloc:

int *tab = NULL;
int v, rozmiar = 0;

while(scanf("%d", &v) == 1 && v != 0) {
    tab = realloc(tab, sizeof(int) * (rozmiar + 1));
    tab[rozmiar] = v;
    ++rozmiar;
}

rozmiar będzie zawierać ilość liczb podanych przez użytkownika.

---

Dziękuję :) W sumie zrobiłem to sam po twoich uwagach i rozkminiłem, że jak tab[rozmiar] w tym rozmiar przyjmuje wartosci losowe to w sumie ograniczymy to po pętli, gdzie wpiszemy liczby do tabeli. tab[rozmiar] = tab[b] i w sumie ładnie wypisuje wszystko :)

Napisałem to tak:, ale spróbuję jeszcze twoją wersje :)

#include <stdio.h>

int main()
{
    int a,b,c,d,n,rozmiar;
    int tab[rozmiar];
    n = 100000;
    printf("Podaj liczby ciagu, natomiast jeśli bedziesz chcial zakonczyc ciag wpisz 00: \n");
    for(b=0;b<n;b++)
    {
        scanf("%d", &tab[b]);
            if(tab[b]==00)
        {
            break;
        }
    }
    printf(" Podales %d elementow do tablicy.\n", b);
    d = b;
    tab[rozmiar] = tab[b];
    for(b=0;b<d;b++)
    {
        printf("%d\n", tab[b]);
    }
    
    
    

    

    
    return 0;
}

---

Sizeof zwraca wielkość obiektu (znaną w trakcie kompilacji) w bajtach, a nie liczbę elementów tablicy.

---

Napisałem to tak:,

I błąd nieokreślonego rozmiaru tablicy pozostał. Zrozum jedną rzecz, że jak deklarujesz tablice:

int tab[rozmiar];

to zmienna rozmiar musi mieć ustawioną wartość, by kompilator/program wiedział, jak duża powinna być ta tablica. Skoro pierwsza pętla kręci się n razy, to wypadałoby stworzyć tablicę n-elementową.

---

@Oscar, słuszna uwaga, poprawione.

---

No sprawdziłem, racja totalna po twojej stronie ;/ Dzięki za pomoc