Discussion :

[xenos]

Bonjour,

La méthode reçoit un pointeur sur un caractère. Elle doit renvoyer la chaîne reçu en ayant remplacé chaque apostrophe ' par deux apostrophes ''.

Malheureusement après quelques testes, il semblerait que ma méthode ne fonctionne pas.

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
char * traiterApostrophe(char * chaine)
{
	char resultat[NBR_LIGNE];
	char * resultatReturn;
	resultatReturn = resultat;
 
	int car, seek = 0, taille = strlen(chaine);
 
	for (car = 0; car < taille; car++)
	{
		resultat[seek] = chaine[car];
		seek = seek + 1;
 
		if (chaine[car] == '\'')
		{
			resultat[seek] = '\'';
			seek = seek + 1;
		}
	}
 
	return resultatReturn;
}

Quelles sont les modifications à y apporter pour que chaque apostrophe de la chaîne d'entrée soit écrit deux fois ? Par exemple : l'avion => l''avion.

D'avance merci pour vos réponses.



xenos

[orfix]

Tu dis vouloir retourner la chaîne passée en argument, mais ce n'est pas ce que tu fais :

Code :

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

return resultatReturn;

retourner l'adresse d'une variable locale est bien sûr incorrect.
règles déjà ça pour le moment et tu y verras plus clair

[xenos]

Tu dis vouloir retourner la chaîne passée en argument, mais ce n'est pas ce que tu fais :

Code :

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

return resultatReturn;

retourner l'adresse d'une variable locale est bien sûr incorrect.
règles déjà ça pour le moment et tu y verras plus clair

Oui, je dis renvoyer la chaine passée en paramètre. Mais la renvoyée modifiée. Donc je copie caractère dans une variable temporaire et lorsque je vois un ' j'écris à la place ''.

Je renvoie alors la nouvelle version de la chaîne passée en paramètre.

Par exemple : l'avion => l''avion.

[publicStaticVoidMain]

Oui, je dis renvoyer la chaine passée en paramètre. Mais la renvoyée modifiée. Donc je copie caractère dans une variable temporaire et lorsque je vois un ' j'écris à la place ''.

Je renvoie alors la nouvelle version de la chaîne passée en paramètre.

Oui mais en meme temps ta variable resultatReturn, tu ne l'utilises pas dans ta fct. Elle ne sert à rien. A mon avis, tu voulais faire

Code :

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

1
2
 
return resultat;

[xenos]

J'ai supprimé resultatReturn. Ca fonctionne, mais lors de la compilation il me dit :

Attention : Cette fonction retourne l'adresse d'une variable locale.

Maintenant que dois je faire pour remplacer les ' par '' ?

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
char * traiterApostrophe(char * chaine)
{
	char resultat[NBR_LIGNE];
 
	int car, seek = 0, taille = strlen(chaine);
 
	for (car = 0; car < taille; car++)
	{
		resultat[seek] = chaine[car];
		seek = seek + 1;
 
		if (chaine[car] == '\'')
		{
			resultat[seek] = '\'';
			seek = seek + 1;
		}
	}
 
	return resultat;
}

[Invité]

Attention : Cette fonction retourne l'adresse d'une variable locale.

Ce warning n'est pas à négliger : tu déclares résultat dans ta fonction. Cela veut dire que c'est une variable locale, et qu'à la fin de la fonction elle n'existe plus. Donc quand tu récupèreras son adresse au niveau du code appelant, il y aura de la daube dedans.

En général pour faire un truc clean, on alloue la chaîne de retour dans l'appelant, et on passe en paramètre de la fonction les chaînes d'entrée et de sortie, avec leurs tailles respectives. Comme ça tu peux utiliser la valeur de retour pour autre chose, comme récupérer un code d'erreur, ou dans ton cas le nombre de guillemets remplacés.

Code :

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

int traiteApostrophe(char *szIn, int iTailleIn, char *szOut, int *iTailleOut);

iTailleOut n'est pas forcément nécessaire si la chaîne de sortie a la même taille que la chaîne d'entrée.

[oswalidos]

je crois ca peut etre la meilleure solution :

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
 
char * traiterApostrophe(char * chaine)
{
	char *resultat=malloc(sizeof(char)*NBR_LIGNE);
 
	int car, seek = 0, taille = strlen(chaine);
 
	for (car = 0; car < taille; car++)
	{
		resultat[seek] = chaine[car];
		seek = seek + 1;
 
		if (chaine[car] == '\'')
		{
			resultat[seek] = '\'';
			seek = seek + 1;
		}
	}
 
	return resultat;
}

[xenos]

Merci pour ton code oswalidos. Mais en le testant, il me rajoute des caractères à la fin. Des ? ... :s

[xenos]

J'ai rajouté une dernière ligne. Qu'il rajoute après avoir copié les caractères un dernier caractère \0 :

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
char * traiterApostrophe(char * chaine)
{
	char * resultat = malloc(sizeof(char) * NBR_LIGNE);
 
	int car, seek = 0, taille = strlen(chaine);
 
	for (car = 0; car < taille; car++)
	{
		printf("%c\n", chaine[car]);
		resultat[seek] = chaine[car];
		seek = seek + 1;
 
		if (chaine[car] == '\'')
		{
			resultat[seek] = '\'';
			seek = seek + 1;
		}
	}
 
	resultat[seek] = '\0';
 
	return resultat;
}

À première vue ça à l'air de fonctionner, je fais encore quelques tests.