Discussion :

[chercheur111]

Bonjour,

Voici mon code:

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
 
#include<stdio.h>
#include<string.h>
 
char * inverser(char *  ch)
{
    int i;
	int n = strlen(ch); 
	char lettre; 
    for(i=0;i<n/2;i++)
	{
	    lettre = ch[i];
        ch[i] = ch[n-i-1];
        ch[n-i-1] = lettre;
		printf("%c \n",ch[i]);
    }
    return ch;
}
int main()
{
    char * ch="bonjour";
    printf("la chaine inversee de bonjour est %s\n",inverser(ch));
 
    return 0;
}

au moment de l'exécution, j'ai eu le message suivant:

Code :

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

1
2
3
 
$ ./inverser
Segmentation fault (core dumped)

Avez-vous une idée ?
Merci.

[foetus]

Il faut

• Faire attention aux déclarations simple comme char* ch= "bonjour";
• Vérifier ton paramètre s'il n'est pas NULL
• Vérifier ton retour s'il n'est pas NULL

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
#include<stdlib.h>
#include<stdio.h>
#include<string.h>
 
 
char* inverser(char* ch) {
    if (ch == NULL) { return NULL; }
 
    int i = 0, n = strlen(ch);
    char lettre = '\0';
 
    for(i=0; i < (n / 2); i++) {
        lettre = ch[i];
        ch[i] = ch[n-i-1];
        ch[n-i-1] = lettre;
        printf("%c \n",ch[i]);
    }
 
    return ch;
}
 
 
int main() {
    char* ch = (char*) malloc(sizeof(char) * 8);
    memcpy(ch, "bonjour", 8);
    ch[7] = '\0';
 
    printf("la chaine inversee de bonjour est %s\n", inverser(ch));
 
    free(ch);
 
    return 0;
}

[olivier_u]

Il faut

• Faire attention aux déclarations simple comme char* ch= "bonjour";

Je plussois allégrement.
Dans ce genre de déclarations, certains compilateurs (comme celui sur lequel je travaille) met la chaîne "bonjour" directement dans le code objet et fait pointer le pointeur dessus. Cela fait un pointeur qui cible une zone où tu n'as pas le droit d'écrire. D'ailleurs, certains compilateurs et/ou certaines options t'obligent à utiliser const avant le char * pour montrer que tu n'as pas le droit de modifier la zone.
Avec la solution de foetus, ça doit marcher.

[Bktero]

Dans ce genre de déclarations, certains compilateurs (comme celui sur lequel je travaille) met la chaîne "bonjour" directement dans le code objet et fait pointer le pointeur dessus. Cela fait un pointeur qui cible une zone où tu n'as pas le droit d'écrire.

C'est, je pense, le comportement de la plupart des compilateurs.

D'ailleurs, certains compilateurs et/ou certaines options t'obligent à utiliser const avant le char * pour montrer que tu n'as pas le droit de modifier la zone.

Avec gcc, il y a un warning pour se prémunir de ce genre de soucis : http://gradot.wordpress.com/2013/05/...trings-de-gcc/

[chercheur111]

Merci bien pour vos réponses.

[Médinoc]

À noter que le malloc() n'est pas nécessaire ici, un petit char ch[]="bonjour"; suffit (déclare un buffer de caractères initilialisé plutôt qu'un pointeur vers une chaîne littérale).

Et si tu utilises gcc, je plussoie l'usage de -Wwrite-strings.

PS: On ne caste pas le retour de malloc() en C.