Alokace paměti struktury – C / C++ – Fórum – Programujte.com

Opět Vás všechny zdravím. Chtěl bych se zeptat jak se správně alokuje paměť struktur a jejich kopírování, případně práce s pointery.

Můj případ:

typedef struct{
	int 	id;
	BOOL 	active;
	char 	text1;
	char 	text2;
	char 	text3;
}Data;

int main(int argc, char *argv[]){

Data *data_old[10];
Data *data_new[10];

...

napl_struct(data_old);


...

napl_struct(data_new);

...

// nahrazení data_old strukturou data_new - problém:

memcpy(data_old,data_new,sizeof(Data)*10);



return 0;

}

void napl_struct(Data *data[]){
	Data * temp_data = (Data*) malloc(sizeof(Data));
	
	// cyklus:
	// načtení dat ze souboru, vložení do temp_data
	
	//alokace paměti:
		data[file_count] = (Data*) malloc(sizeof(Data));

	//zkopírování dat z temp_data do data[file_count], char pomocí strcpy

	//konec cyklu
	
}

Zkoušel jsem i funkci memmove, ale program píše: *** buffer overflow detected ***

Problém bude v tom, že pole struktur deklaruji o velikost 10, ve skutečnosti je to ale maximální číslo a těch struktur může být různě, třeba 6, 7 ... Chybu mi to ale píše i když jsou obě pole struktur naplněný stejným počtem.

Blbě buď alokuji paměť nebo kopíruji data. Můžete někdo poradit? Díky.

No zaprve nevidim duvod proc alokovat temp, pak do data[file_count] a pak to teprve kopirovat... (zbavujes se v tomdle pripade aspon tempu? Nebo vyrabis memory leak?). Proc kopie pomoci strcpy mas tam jeden znak, tak staci prime prirazeni.. az tam bude retezec, tak to bude neco jinyho.. (jinak bych se nedivil, kdyby tady zase byl segfault)

Magicke konstanty sou taky osklivy ... vsude sama 10 a podobne.. pak si reknes ze chces 12 a musi se vsechno zmenit ... a pak nekde budes mit nahodou stejne velkou magickou konstantu kterou zmenit nechces a hups - chyba na svete...

A k tvemu problemu?

Data *data_old[10]; Data *data_new[10]; jsou pole pointeru - a ty pak zacnes kopirovat jako by tam uz ty struktury byly ulozeny cely rovnou...

#2 KIIV
tempu se nezbavuji :( a asi to předělám a vyhodím ten temp cely, budu rovnou načítat do data_new a data_old. 

konstamty tam mám pomocí #define MAX_FILES 10, ale sem jsem to dal pro přehlednost číslem.

K odpovědi na problém. Měl bych tedy použít:

Data * data_old[MAX_FILES] = (Data*) malloc(sizeof(Data)*MAX_FILES )

?

prave ze MAX_FILES je uz jedna polozka za koncem pole...  je to totiz 10  a posledni prvek je 9 (prvni 0)

takze bud  data_new[0] = (Data*) malloc(sizeof(Data))    a pro zbylych 9 totez..

nebo:  Data * data_new = (Data*) malloc(sizeof(Data)*MAX_FILES);   (akorat bys do funkci, co meni kam se ukazuje, musel predavat   &data_new a ne jen data  ...  a uvnitr samozrejme   Data ** ptr;   *ptr = (Data*) malloc(sizeof(Data)*MAX_FILES);  a tak dale

#4 KIIV
super, díky.  Jsem pro druhou variantu, jen zda jsem to dobře pochopil. malloc bude v main() i v  napl_struct() ?

#5 Meph
nemusi byt nutne... jen sem ukazoval jak se alokuje... v main muze byt prirazeno NULL... pak se preda reference na ten pointer, do nej se alokuje vicero prvku a tak dale... jen se nesmi zapomenout na tu referenci... jinak se jen zkopiruje ta hodnota  NULL, uvnitr se sice zmeni ale mimo funkci se nedostane

#6 KIIV
bezvadný, děkuji za pomoc, jdu to vyzkoušet.

#6 KIIV

ještě jedna drobnost, pokud udělám toto:

Data * data_new = (Data*) malloc(sizeof(Data)*MAX_FILES)

jakým způsobem pošlu pointer nebo adresu na prvek data_new[5] do nějaké funkce? Mám pár funkci, které pracují jen s jednou strukturou z původního data_new[MAX_FILES]

adresu ziskas  &data_new[5] nebo pripadne  (data_new+5)  ...

#9 KIIV
tak pro (data_new+5)->text1 mi to vypíše: error: request for member ‘text1’ in something not a structure or union

a u data_new[0]->text1 je to ok, ale pro data_new[1] mám naoprávněný zápis do paměti

napln_struct(Data ** data){

*data = (Data*) malloc(sizeof(Data)*MAX_FILES);

data[1] -> id = 5; // - chyba !!

}

(*data)[1] pripadne i  data[0][1]