[1dumbpcuser]

Suhteliselt algaja olen ja vajaks väga kiiresti abi struktuuridega.
Ülesandeks on lugeda algandmed failist sisse ja paigutada need struktuuri, mis koosneb täisarvulistest suurustest Kaal ja Kogus. (Indeksi lisasin ise, kuna vaja teada, mitu elementi on.)
Hiljem oleks siis vaja ka neid andmeid sealt struktuurist kätte saada ja kasutada

Kood hetkel selline:

[etk]

[1dumbpcuser]

Sellest sain isegi, lihtsan nii üritasin asja "õppida" proovimise teel.
Selles omistamises oli Teil õigus, need olid tagurpidi

[etk]

ega proovides õpibki, niiet õige mõte

[1dumbpcuser]

Nüüd on vist tiba parem (lisaks prindib ta tulemused ka faili):

Spoiler

# include <stdio.h> # include <stdlib.h> # include <conio.h> /* # define "sv.txt" sv # define "vv.txt" vv */ typedef struct {         int Kaal;         int Kogus;         int idx;         }Pomm;         int main(){ FILE *in; FILE *out; in=fopen("sv.txt","r"); out=fopen("vv.txt", "a"); Pomm *a[1]; a[1]=(Pomm*)malloc(sizeof(Pomm)); int i=0; while(!feof(in)){a[1]=realloc(a[1],(2+i)*sizeof(Pomm));                  printf("a[%d] aadress on %d\t", i, a[i]);                  a[2+i]=malloc(sizeof(Pomm));                  printf("a[2+%d] aadress on: %d\n", i, &a[2+1]);                  fscanf(in,"%d %d\n", &a[i]->Kaal, &a[i]->Kogus);                  printf("a[%d]=%d %d\n",i, a[i]->Kaal, a[i]->Kogus);                  fprintf(out,"a[%d]=%d %d\n", i, a[i]->Kaal, a[i]->Kogus);                  i++; } fclose(in); fclose(out); getchar(); return; }

Kuigi ta prindib kõigi a[i] väärtused, jookseb program kusagil getchar() juures kokku .
Samuti ei ilmunud midagi konsooli (ja jooksis kokku) kui kommenteerisin while-tsüklis kaks esimest printf-lauset välja

[etk]

esiteks, alati kui kasutada malloc, realloc, calloc vms, tuleb free() ka kasutusele võtta, et lõpus vabastada mälu
teiseks, mina isiklikult oleksin kasutanud Pomm *a; mitte *a[1] kuna piisab täitsa ühemõõtmelisest massiivist, seega oleks while esimene lause a = realloc(a, (i+1)*sizeoff(Pomm)); ning while tsüklis mallocit ei kasutaks
mallocist hoiduksin kuna see tagastab tühja sisuga pointeri, realloc aga tagastab pointeri teise suurusega arrayle hoides seal sees oleva info alles. seega kui sa teed realloc arrayle kus on juba 6 elementi sees ja seejärel malloc siis sa sujuvalt kustutad kõik eelnevad ära, kuna sa aga samas tsüklis kohe ka välja faili kirjutad, siis sa ei pruugi seda märgata. Võin eksida muidugi aga selle kontrolliks tee väljund faili trükkimine eraldi for tsükliga, mis käib 0-st i-ni ja kirjutab väljundfaili a[i] sisu. Kui ma eksin, siis hoolimata mallocist peaks ta ilusit sul kõik sisse loetud arvud teises tsüklid väljundfaili kirjutama. Kui ma ei eksi, siis ka kirjutab ainult viimasena sisse loetud info ning kõik teised kohad arrayd viitavad NULL-le, mis võib aga ei pruugi programmi crashida

ja kui võimalik, siis getchar()-i ka mitte kasutada aga antud harjutuse puhul las siis olla. getchari vajadusest saaksid määda kui programmi otse käsurealt(command promt, või terminal, vastavalt siis kas win või unix) käivitad

free(a) tuleks panna peale while tsüklit, see vabastab mälu mis sa oled allokeerinud a-le programmi käigus

kui IDE-d kasutad, mis debugi võimaldab, siis pane selles käima ja käi rida haaval kood läbi ning vaata muutujate sisu, annab ehk aimu mis kus ja kuidas on ning ehk seletab ka seda, miks programm crashib

[Ho Ho]

[etk]

noh, kui arvestada, et dünaamilist mälu soovitasin mina kastuada ei maksa kohe väga kurja häält teha
kuigi jah pean nõustuma, et ise jäin ka seda kohta imelikult silmitsema


kui tahad lihtsamalt siis jäta dünaamiline mälu hetkel kõrvale ja tegele ainult konstantsete suurustega, tehes Pomm a[n] ja jättes pointerid sinnapaika, teeb alguses elu lihtsamaks
õppimise mõttes aga soovitan dünaamilisega edasi jännata kuni hakkama ja aru saad, hiljem on parem.

[mark11]

https://foorum.hinnavaatlus.ee/viewtopic.php?t=92709