Discussion :

[mangobango]

Bonjour,

Je suis dans les phases finales de mon projet... mais bon sang j'aimerai pouvoir m'en sortir, parce que là, c'est assez.

Dans mon programme j'alloue fréquemment des zones pour des chaines de caractères. C'est parce que le SDK de géoconcept fonctionne comme ça. (parfois j'utilise des strings, mais je souhaite apprendre, donc manipuler les chaines de caractère ça me semble une bonne chose).

Passé les affres des différentes syntaxes pour l'initialisation (pas sûr d'avoir bien saisi quand même), je me suis dis qu'il serait bien d'avoir des fonctions de concaténation et de copie qui s'occupent toutes seules de la mémoire à allouer, respectivement strcat_d() et strcpy_d() (_d pour daniel...)

Les voici:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
char * strcat_d(char** but, const char* source)
/**permet de concatener sans s'occuper de la memoire... j'espere*/
{
	if(source	== NULL) return *but;
	if(but		== NULL) return *but;
	if(*source	== NULL) return *but;
	if(*but		== NULL) return *but;
 
	UINT lenFinale	= strlen(*but) + strlen(source) + 1;
	char * butSauf	= *but;
	*but		= NULL;
	*but		= new char[lenFinale];
	strcpy(*but, butSauf);
	delete butSauf;
	butSauf		= NULL;
	strcat(*but, source);
	return *but;
}
 
char * strcpy_d(char** but, const char* source)
/**permet de copier sans s'occuper de la memoire... j'espere*/
{
	if(source	== NULL) return *but;
	if(but		== NULL) return *but;
	if(*source	== NULL) return *but;
	if(*but		== NULL) return *but;
 
	UINT lenFinale	= sizeof(source);
	char * butSauf	= *but;
	delete butSauf;
	butSauf		= NULL;
	*but		= NULL;
	*but		= new char[lenFinale];
	strcpy(*but, source);
	return *but;
}

Je les teste avec le prog suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void main()
{
	int i = 0;
	while(i<10000000)
	{
		char *mama	= new char;
		mama[0]		= 0;
//		strcpy_d(&mama, "hahahaihioyuiohoha");  //  --1--
		strcat_d(&mama, "patatipatata")             // --2--
		printf("%s", mama);
		delete  mama;
		i++;
	}
	return;
}

Quand je commente 1 OU 2 ça marche impecc et sur 10000000 boucles, ya pas de fuite.
Quand je décommente les deux ça plante sur la 2ème fonction et ce quel que soit l'ordre. Le déboggueur me dit que c'est sur le "delete butSauf;" de la fonction qui vient en 2. (je pense qu'il faudrait un [] dans ce cas là)

Si je fais "char *mama = new char[];" puis "delete [] mama" (ce qui me semblerait logique) strcpy() plante mais pas strcat(). Le déboggueur me signale que c'est sur "delete [] mama;" que ça accroche dans ce cas là.

Je crois comprendre qu'il y a un problème avec mes "[]". Quelqu'un voit-il comment résoudre ce problème? J'ai envisagé de faire des tests sur la taille de *butSauf (l'objet pointé) pour savoir si je dois utiliser "delete [] ..." ou "delete tout court" mais ça ne me parait pas correct.

Je fais vraiment ça pour apprendre, je pense pouvoir utiliser des strings, mais je voudrai comprendre. Si vous pouvez m'aider, j'en serais heu-reux ^^.
Puis si ces deux fonctions pouvaient marcher impecc, mon prog en serait globalement heureux lui aussi!

Merci pour nous deux alors
Daniel

[loufoque]

Genre t'as

UINT lenFinale = sizeof(source);

Et tu te demandes pourquoi ça marche pas ?
Compare avec ton code pour strcat_d, ça devrait être évident...

Après y'a d'autres problèmes comme par exemple des mélanges entre new/delete et new[]/delete[].
Sans parler du fait que ce n'est pas optimal du tout.

De toutes façons, c'est hyper mal conçu comme truc, et très crade. Si tu veux apprendre, tu ferais mieux de coder ton propre type string.

[mangobango]

Genre t'as

Citation:
UINT lenFinale = sizeof(source);

Et tu te demandes pourquoi ça marche pas ?
Compare avec ton code pour strcat_d, ça devrait être évident...

Héhé, genre, ouais, avant de faire ça j'ai qd mm testé. Je mets pas le code qui ma permis de tester, ça serait insultant!
strlen me renvoie le nombre de caractères sans le nul final.
sizeof(...) me renvoie la longueur totale.
Du coup, quand j'alloue de la mémoire, il me faut bien prendre en compte le caractère de fin, donc soit strlen(...) + 1, soit sizeof(...) non? si?

Après y'a d'autres problèmes comme par exemple des mélanges entre new/delete et new[]/delete[].

Je crois aussi que c'est le principal problème.

Sans parler du fait que ce n'est pas optimal du tout.

ça c'est l'étape d'après!

De toutes façons, c'est hyper mal conçu comme truc, et très crade.

Mince, moi qui croyais que c'était le plus beau code du monde, me voilà déçu!

Si tu veux apprendre, tu ferais mieux de coder ton propre type string

ça c'est encore après! tu crois pas?

en te remerciant,
Daniel

[JolyLoic]

strlen me renvoie le nombre de caractères sans le nul final.
sizeof(...) me renvoie la longueur totale.

char s1[] = "Abcdefgh";
char *s2 = s1;

sur une machine où sizeof(void*) == 4, on aura sizeof(s1) == 9, mais sizeof(s2) = 4. s2 a comme type char*, et rien d'autre, et à part un astuces comme de parcourir la chaîne pour trouver un zéro final (ou savoir comment est implémenté new[]), il n'y a pas moyen avec un sizeof de savoir combien d'éléments sont contenus après l'adresse pointée par un pointeur.

[droggo]

Hui,

Héhé, genre, ouais, avant de faire ça j'ai qd mm testé. Je mets pas le code qui ma permis de tester, ça serait insultant!

Pour qui ?

[mangobango]

(insultant pour quiconque aurait un meilleur niveau que moi , environ 99.9% des gens ici)

Alors je vois la coquille! Pour finir ce projet je vais me rabattre sur les strings de la biblio standard.

Merci à nouveau pour l'aide,
( plus tard je me pencherai sur implémenter mes strings moi même, mes tables de hashages et un compilateur customizé ^^ )

Aller, ciao
Dani