[DEBUTANT] Copie de variable avec ou sans CR

Bonjour,

j'ai un problème de débutant et je m'en remets à vous

J'ai une fonction qui a pour but de retirer tous les retour chariot '\r' afin d'avoir une chaîne de caractère sur une seule ligne (je l'envoie par la suite en requête HTTP d'où mon besoin de ne pas avoir de '\r' dedans).

la fonction est la suivante :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

char * format_paramCR(char * param) { int i=0; int lg=strlen(param); char * temp=(char *)calloc(lg, sizeof(char)); for (i=0;i<lg;i++) { if (param[i]=='\r') { temp[i]='z'; } else { temp[i]=param[i]; } }   return temp; }

---

Le code semble fonctionner car quand je demande l'affichage de temp, ceci est correct. Mon problème vient de la récupération de "temp".

J'ai une variable "Desc" qui contient :
"salut
bonjour"

je voudrais donc obtenir : "salutbonjour" par exemple.
Quand je fais : "Desc = format_paramCR(Desc)" ca plante. Je peux le faire avec n'importe quel autre caractère à remplacer mais ça plante avec le CR.
Vous pouvez m'expliquer? Y a t il une solution pour que je puisse récupérer ma variable modifiée?
J'ai regardé la longueur de "temp" et de "Desc" est la même de même que la mémoire allouée pour les deux variables.
Merci

NB : j'essaye de remplacer \r par 'z' pour le moment afin de voir si cela fonctionne.

Salut,

La norme HTTP dit que les sauts de ligne sont CRLF (\r\n), mais en fait les serveurs acceptent aussi LF (\n) comme saut de ligne (edit : ça c'était completement HS)
Pour recapituler :
* LF : sauts de ligne unix
* CRLF : sauts de ligne dos et souvent protocoles web
* CR : sauts de ligne mac

bon, si j'ai bien suivi ton algo, j'ai une chaine "bonjour\rça va ?" je la passe par ta fonction et voila ce qui se passe :

tu alloues temp avec un calloc (chaque case remplie avec un \0) de la taille de ta chaine (pense a ajouter +1 pour le \0 final qui n'est pas compté par strlen)

ensuite, tu remplis ta chaine temp avec des 'z' (??) a chaque fois que tu trouves un '\r'

donc en sortie de boucle, temp vaut {0,0,0,0,0,0,0,'z',0,0,0,0,0,0,0} et c'est ça que tu renvoies.

je suis pas sur d'avoir compris ou tu voulais en venir donc.

donc au final, quand tu traites la chaine, vu que le premier caractère c'est 0, c'est considéré comme une chaine vide.


EDIT : ah, j'avais pas vu le else, ou tu l as ajouté entre temps.

Quoi qu'il en soit, ajouter un 'z' a la place de ton \r, je sais pas si c'est la meilleure solution

Pour ta question tu alloues pas assez de place pour la chaine de sortie ( :p )

Citation:

---

Envoyé par Faiche

Quoi qu'il en soit, ajouter un 'z' a la place de ton \r, je sais pas si c'est la meilleure solution

---

Il le fait pour débugger. Mais t'as raison, quitte à mettre quelque chose à la place, autant mettre un espace...

Tout d'abord merci de répondre aussi vite.

Pour le "else" oui je me suis aperçu entre temps que je l'avais oublié.:oops:

Je vais essayer d'être plus clair sur l'ensemble de mon problème.

Le programme C sur lequel je bute est en fait une DLL. Cette même Dll est appelé par un programme Java. Ma variable "Desc" est passée en paramètre depuis mon programme Java.

On la récupère donc en jstring.
Je fais ensuite une conversion pour pouvoir l'exploiter en langage C.

Code:

---

1 2	const char *Desc = env->GetStringUTFChars( jDesc, 0);

---

Je n'ai donc pas "alloué" de mémoire à ma variable Desc depuis mon programme C. Comment faire pour la modifier et lui permettre d'avoir plus de place?

ce n'est pas là qu'il te faut plus de place, c'est dans temp.

Je ne sais pas trop... Quand je modifie l'allocation mémoire de "temp" cela ne change rien. De plus, si je laisse l'allocation comme elle est et que je demande :

Code:

---

1 2	format_paramCR(Description);

---

il ne plante pas. C'est à partir du moment où je demande "Description = format_paramCR(Description)" qu'il y a un problème.

(post croisé)
Donc, tu dois d'abord allouer un buffer au moins aussi grand que Desc, caractère nul compris (taille minimale : strlen(Desc)+1).

Ensuite, si tu veux juste remplacer des caractères sans changer la longueur de la chaîne, ça va tout seul.
Si tu veux supprimer des caractères sans les remplacer, tu peux regarder ici.

Il y a peut-être une fuite de mémoire, là...
Vérifie que tu ne dois pas libérer Description avant d'y mettre un autre pointeur...
Et n'oublie pas que temp aussi doit se terminer par un caractère nul...

Ce que je ne comprends pas c'est pourquoi lorsque je change le caractère à remplacer par un autre (par exemple on remplace tous les 'b' par des 'c', on ne parle donc plus de '\r') cela fonctionne parfaitement. Il en va de même si je souhaite remplacer tous les '\n'. le retour chariot occupe une mémoire différente des autres caractères? Comment cela se fait il?

Es-tu sûr que dans tous les cas il y a un (ou des) caractère(s) à remplacer ?
De toute façon, on SAIT que ton allocation est trop petite (pas de place pour le caractère nul), et on SAIT que le caractère nul n'est pas copié.
Ce qui entraîne un comportement indéfini.