Porovnání dvou řetězců v poli – C / C++ – Fórum – Programujte.com

Ahoj,
včera jsem zde zakládal téma ohledně semestrální práce. Byla mi ohodnocena jako už tak vyhovující, ale pořád by se dalo něco zlepšit. Dostalo se mi takovéto odpovědi od učitele:

"int jeVPoli(tPamatka **polePamatek, tPamatka *pamatka); - myšlenka OK, ALE!!!, neporovnávejte však adresy řetězců, ale celé řetězce. Nevím co bylo přesně vaším záměrem, ale pokud byste si ukládal do pole kopie záznamů tento způsoby by nebyl funkční."

Nějak mu nerozumím, co tím myslel. Abych vás uvedl do problému. tPamatka je struktura, která vypadá takto:

typedef struct pamatka
{
	char pamatka[21];
	tGPS gps;
	struct pamatka *dalsi;
} tPamatka;

Funkce jeVPoli je napsaná takto:

int jeVPoli(tPamatka **polePamatek, tPamatka *pamatka)
{
	int i;

	for(i = 0; i < 3; i++)
	{
		if(polePamatek[i]->pamatka == pamatka->pamatka)
		{
			return 0;
		}
	}
	return 1;
}

Jejím úkolem je prohledat pole památek o maximální velikosti 3 památek (mám najít 3 nejbližší památky s daným GPS souřadnicím) a zkontrolovat, zda se v něm památka nachází. Reakce učitele tedy je (pokud dobře rozumím), že porovnávám adresy dvou památek místo jejich řetězců. Mě to však není jasné. V "ifu" porovnávám památku na pozici "i" s jejím názvem s památkou předanou parametrem a jejím názvem.

Díky za rady  

#1 avalagne
fce strncmp

#2 crazy
To je vše co tím myslel?

#1 avalagne
Ucitel od teba chcel aby si porovnaval retazce pomcou strcpy() pripadne podobnou funkciou. 

Dovod je nasledovny:

char name1[] = "meno";
char name2[] = "meno";

if(name1 == name2)puts("a) toto sa nevypise\n");
else puts("a) toto sa vypise\n");


if( !strcmp(name1,name2) )puts("b) toto sa vypise\n");
else puts("b) toto sa nevypise\n");

Kedze ti program funguje, tak zrejme pri vytvoreni pamiatky sa uz nemenijej adresa (a ani adresa nazvu).

Takze teoretycky na porovnavanie pamiatok staci porovnat ich adresy a nazov bude len na vypis.

int jeVPoli(tPamatka **polePamatek, tPamatka *pamatka){
	for(int i = 0; i < 3; i++)
		if(polePamatek[i] == pamatka)
			return 0;
		
	return 1;
}

#4 vitamin
Už tomu rozumím. Nejspíš chtěl tedy použít funkci na porovnání řetězců jak píšeš. Tvoje varianta co máš dole:

int jeVPoli(tPamatka **polePamatek, tPamatka *pamatka){
	for(int i = 0; i < 3; i++)
		if(polePamatek[i] == pamatka)
			return 0;
		
	return 1;
}

mi nefunguje. Spustí to nějakej breakpoint v heapu nebo co to píše. Přes strcmp by se mi to líbilo, ale taky to nějak nejede:

for(i = 0; i < 3; i++)
	{
		porovnej = strcmp(polePamatek[i]->pamatka, pamatka->pamatka);
		if(porovnej == 0)
		{
			return 0;
		}
	}
	return 1;

Toto to hlásí:

Chyba: Unhandled exception at 0x58E21F50 (msvcr110d.dll) in UJC_SemestralniPrace_V1.exe: 0xC0000005: Access violation reading location 0x00000000.

Nevíte někdo prosím co by mohlo tuto chybu vyřešit? Přistupuju někam do nepřidělené paměti, ale nevim proč najednou...

#6 avalagne
pred tim si porovnaval jen pointery... ale strcmp se musi podivat kam ukazujou... a pokud tam je nejaky null pointer tak uz je to docela problem.. (a ani naznak kontrol nedelas)

(a null pointery vubec nekontrolujes)