Nefunkční cyklus s funkcí strtok – C / C++ – Fórum – Programujte.com

Ahoj, používám funkci strtok ve while cyklu pro oddělování jednotlivýcj tokenů. Vím, že funkce strtok "vycucává" z původního řetězce její obsah - proto všechny pole překopíruju do jiných (viz kód). Problém mám ale s tím, že "Token" se vypíše pouze jednou... pokud ale zakomentuju určitý řádek v kódu (viz kód), vše proběhne jak má... netušíte, kde je chyba?

char poleA [F_LENGTH];
char vstupniPole [strlen(input)+1];
strcpy(vstupniPole, puvodniPole);
	
char *token;
token = strtok(vstupniPole, "|");
char detekce [strlen(token)+1];
while (token != NULL){

	printf("Token: %s\n", token);
	strcpy(detekce, token);
	if (isupper(detekce[2]) != 0){
		char dalsiPole [strlen(poleA)+1]; 
		strcpy(dalsiPole, poleA);
		char *token2;
		token2 = strtok(dalsiPole, "|"); // pokud toto zakomentuji, vse funguje jak ma
	}
	token = strtok(NULL, "|");
}

puvodniPole má alokovanou paměť? Kde se naplní poleA? Smyčka je zřejmě nekonečná, hodnota proměnné token se v jejím těle nemění.

hu

puvodniPole je vstupni parametr a pamět už má alokovanou (funkce je delší, vytáhl jsem jen ten usek, který nefunguje). Předpokládejme, že poleA je už naplněné.. takhle jak to mám napsané, by se mi druhý token samozřejmě nevypsal, protože poleA nic neobsahuje...

Při výpisu je vidět, že while cyklus proběhne tak jak má JEN když neskočí do podmínky.. ta podmínka ho jakoby breakne...

#1 MucoVerde

strtok je funkcia ktora modifikuje povodne pole (doplna znak '\0') a ktomu si uchovava adresu posledneho nacitaneho tokenu v globalnej premennej. Ty si vytvoris lokalnu premennu na stacku vo vnutri cyklu a potom inicializujes touto lokalnou premennou strtok(). strtok() si uchova adresu tejto lokalnej premennej, lenze hned za tymto volanim opustas scope a uvolni sa lokalna premenna. Pouzivaj malloc()/calloc() a free().

edit: alebo radsej nepouzivaj strtok().

edit2: tak som si ten kod prezrel lepsie a tytam mas viac problemov. Funkcia strtok nie je stavana na pouzivanie s viacerimi poliami sucasne, na to su funkcie ako strtok_s(win) a strtok_r(linux)

a taky pokud bude prvni token nejak vyrazne kratsi nez dalsi tak je tam hned buffer overflow...

Asi už jsem nejen hluchej, ale i slepej. To s tím cyklem odvolávám.

hu

Upravil jsem zdrojový kód tak, aby bylo přehlednější, jaký mám problém. Po spuštění programu by mi měl vypsat

Token abC

Proslo abC

Token vCd

Token dfF

Proslo dfF

Ovšem vypíše mi pouze

Token abC

Proslo abC

Co se týče strtok_s, tak to by stačilo pouze nahradit za současný strtok, či jak?

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


int main(void){
	char puvodniPole[50];
	puvodniPole[0] = 'a';
	puvodniPole[1] = 'b';
	puvodniPole[2] = 'C';
	puvodniPole[3] = '|';
	puvodniPole[4] = 'v';
	puvodniPole[5] = 'C';
	puvodniPole[6] = 'd';
	puvodniPole[7] = '|';
	puvodniPole[8] = 'd';
	puvodniPole[9] = 'f';
	puvodniPole[10] = 'F';
	puvodniPole[11] = '\0';
	
	char poleA[10];
	poleA[0] = 'd';
	poleA[1] = 'd';
	poleA[2] = '|';
	poleA[3] = 'a';
	poleA[4] = 'a';
	poleA[5] = '|';
	poleA[6] = 'd';
	poleA[7] = '\0';
	char vstupniPole [strlen(puvodniPole)+1];
	strcpy(vstupniPole, puvodniPole);
	
	char *token;
	token = strtok(vstupniPole, "|");
	char detekce [strlen(token)+1];
	while (token != NULL){

		printf("Token: %s\n", token);
		strcpy(detekce, token);
		if (isupper(detekce[2]) != 0){
			
			char dalsiPole [strlen(poleA)+1]; 
			strcpy(dalsiPole, poleA);
			printf("Proslo: %s\n", token);
			char *token2;
			token2 = strtok(dalsiPole, "|"); // pokud toto zakomentuji, vse funguje jak ma
		}
		token = strtok(NULL, "|");
	}
  return 0;
}

#8 MucoVerde
Nie nestaci, viac tu: http://msdn.microsoft.com/en-us/library/ftsafwz3(v=vs.80).aspx

retazec mozes napisat nasledovne: 

char str[50] = "abcdefgh";

#9 vitamin
OK. A takhle zapsaný to teda nemůže být? Nemůžu prostě použít víckrát strtok, i když se jedná o dvě jiná pole?

#10 MucoVerde
Mozes, ale nebude to robit to co chces :)

#11 vitamin
OK, koukám na ten strtok_s, ale moc ho nechápu ... má tři parametry, první je řetězec, ze kterého chci dostat ty tokeny, druhý parametr je oddělovací znak a třetí by mělo být k uložení pozice.. ten třetí moc nechápu.. jak by to vypadalo, kdybych tuhle funkci použil na muj kod?

#12 MucoVerde
Ved v tom linku mas aj priklad.

#13 vitamin
Pokud použiju to co je v příkladu, tak mi to nejde ani přeložit (collect2.exe: error: ld returned 1 exit status undefined reference strtok_s)

int main(void){
	char puvodniPole[50];
	puvodniPole[0] = 'a';
	puvodniPole[1] = 'b';
	puvodniPole[2] = 'C';
	puvodniPole[3] = '|';
	puvodniPole[4] = 'v';
	puvodniPole[5] = 'C';
	puvodniPole[6] = 'd';
	puvodniPole[7] = '|';
	puvodniPole[8] = 'd';
	puvodniPole[9] = 'f';
	puvodniPole[10] = 'F';
	puvodniPole[11] = '\0';
	
	char poleA[10];
	poleA[0] = 'd';
	poleA[1] = 'd';
	poleA[2] = '|';
	poleA[3] = 'a';
	poleA[4] = 'a';
	poleA[5] = '|';
	poleA[6] = 'd';
	poleA[7] = '\0';
	char vstupniPole [strlen(puvodniPole)+1];
	strcpy(vstupniPole, puvodniPole);
	
	char *token;
	char *dalsiToken;
	token = strtok_s(vstupniPole, "|", &dalsiToken);
	char detekce [strlen(token)+1];
	while (token != NULL){

		printf("Token: %s\n", token);
		strcpy(detekce, token);
		if (isupper(detekce[2]) != 0){
			
			char dalsiPole [strlen(poleA)+1]; 
			strcpy(dalsiPole, poleA);
			printf("Proslo: %s\n", token);
			char *token2;
			//token2 = strtok(dalsiPole, "|"); // pokud toto zakomentuji, vse funguje jak ma
		}
		token = strtok_s(NULL, "|", &dalsiToken);
	}
  return 0;
}

#14 MucoVerde
použij strtok_r

Změnil jsem to na strtok_r a furt to vyhazuje error...

undefined_reference to strtok_r ... pak něco s bad reloc address 0x20 in section ".eh_frame"

final link failed: Invalid operation

collect2.exe: error: ld returned 1 exit status

Error 1

U kódu který jsi sem vložil nejsou žádné include hlavičkových souborů. Nechybí ti tam #include "string.h" ?

hu

To mě napadlo jako první, ale string.h tam právě že mám.

Takhle vypadá celý kód:

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


int main(void){
	char puvodniPole[50];
	puvodniPole[0] = 'a';
	puvodniPole[1] = 'b';
	puvodniPole[2] = 'C';
	puvodniPole[3] = '|';
	puvodniPole[4] = 'v';
	puvodniPole[5] = 'C';
	puvodniPole[6] = 'd';
	puvodniPole[7] = '|';
	puvodniPole[8] = 'd';
	puvodniPole[9] = 'f';
	puvodniPole[10] = 'F';
	puvodniPole[11] = '\0';
	
	char poleA[10];
	poleA[0] = 'd';
	poleA[1] = 'd';
	poleA[2] = '|';
	poleA[3] = 'a';
	poleA[4] = 'a';
	poleA[5] = '|';
	poleA[6] = 'd';
	poleA[7] = '\0';
	char vstupniPole [strlen(puvodniPole)+1];
	strcpy(vstupniPole, puvodniPole);
	
	char *token;
	char *dalsiToken;
	token = strtok_r(vstupniPole, "|", &dalsiToken);
	char detekce [strlen(token)+1];
	while (token != NULL){

		printf("Token: %s\n", token);
		strcpy(detekce, token);
		if (isupper(detekce[2]) != 0){
			
			char dalsiPole [strlen(poleA)+1]; 
			strcpy(dalsiPole, poleA);
			printf("Proslo: %s\n", token);
			char *token2;
			//token2 = strtok(dalsiPole, "|"); // pokud toto zakomentuji, vse funguje jak ma
		}
		token = strtok_r(NULL, "|", &dalsiToken);
	}
  return 0;
}

#18 MucoVerde
čim to kompiluješ? gcc mi to přeloží v pohodě

Mám to udělaná make, ve kterým mam klasicky gcc abc.c -o abc

ale v cmd, mi to zobrazí jen cc ab.c -o abc ?není to chyba?

Jinak to gcc jsem stáhnul v rámci MinGW

C++ Builder 2010 nepřeloží, nezkousne char vstupniPole [strlen(puvodniPole)+1];. Nemohu vyzkoušet.
<string.h> vkládáš. Zkus jednoduchý projekt ve kterém bude jen řetězec a na něj použitý strtok_r.:

hu

Tak jsem to zkusil jen čistě s strtok_r a opět Error

c:/mingw/bin/../lib/gcc/mingw32/4.7.2/../../../../mingw32/bin/ld.exe: final link
failed: Invalid operation
collect2.exe: error: ld returned 1 exit status
make: *** [matice] Error 1

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


int main(void){
	char puvodniPole[50];
	puvodniPole[0] = 'a';
	puvodniPole[1] = 'b';
	puvodniPole[2] = 'C';
	puvodniPole[3] = '|';
	puvodniPole[4] = 'v';
	puvodniPole[5] = 'C';
	puvodniPole[6] = 'd';
	puvodniPole[7] = '|';
	puvodniPole[8] = 'd';
	puvodniPole[9] = 'f';
	puvodniPole[10] = 'F';
	puvodniPole[11] = '\0';
	
	char poleA[10];
	poleA[0] = 'd';
	poleA[1] = 'd';
	poleA[2] = '|';
	poleA[3] = 'a';
	poleA[4] = 'a';
	poleA[5] = '|';
	poleA[6] = 'd';
	poleA[7] = '\0';
	
	char *token;
	char *dalsiToken;
	token = strtok_r(puvodniPole, "|", &dalsiToken);
	char detekce [strlen(token)+1];
	while (token != NULL){

		printf("Token: %s\n", token);
		token = strtok_r(NULL, "|", &dalsiToken);
	}
	return 0;
}

#22 MucoVerde
strtok_r a strtok_s su nestandartne funkcie ktore kompilator nemusi podporovat. MinGW nepodporuje ani jednu z nich.

#23 vitamin
aha, nevedel jsem, ze to mingw nepodporuje... v tom pripade tady je par implementaci, jak si funkci muzes implementovat...

http://stackoverflow.com/questions/12975022/strtok-r-for-mingw

Tak SUPER, použil jsem odkaz od Crazyho a už se to "točí" 

Vysvětlil by mi tu ještě někdo, proč to tak vlastně funguje? V podstatě jsem jen přidal prázdnej char a už to funguje, nechápu moc to kouzlo, díky kterému strtok_r funguje a strtok ne?