Discussion :

[Despak]

Bonjour, comme l'intitulé l'indique je veux transformer une chaine de caractere en majuscule.

Je fais comme il est dis dans les FAQ, mais j'ai une erreur de segmentation ...

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
26
27
28
29
30
31
32
 
  1 #include <ctype.h>
  2 #include <stdio.h>
  3 
  4 
  5 
  6 
  7 void toUpper( char * );
  8 
  9 
 10 int main( int argc, char ** argv ){
 11 
 12         char * test = "ceci est un test" ;
 13         toUpper( test );
 14         printf("%s\n", test );
 15         return 0 ;
 16 
 17 }
 18 
 19 
 20 
 21 void toUpper( char * clef ){
 22 
 23         int i ;
 24         
 25         for( i = 0 ; clef[i] != '\0' ; ++i ){
 26                 clef[i] = toupper(clef[i] );
 27         printf("%c\n", clef[i] );
 28         }
 29          
 30          
 31 }

Le problème doit surement venir du faire qu'on ne peut assigner directement une nouvelle valeur à la chaine de caractere, mais je n'ai pas d'explications.

Merci .

[FunK92]

Salut,

Je pense que ton erreur de segmentation vient du faite du tu n'alloue pas l'espace mémoire suffisante ton pointeur test.

Tu doit soit mettre un tableau statique (ce qui est bof bof):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
...
#define TAILLE_MAX 50
 
int main()
{
 
      char test[50] = "ceci est un test" ;
     ...
}
...

Soit faire une allocation dynamique pour l'utiliser comme un tableau à 1 dimension :

Code :

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

1
2
3
4
5
 
          char * test ;
          test = (char *) malloc ((1+strlen("ceci est un test"))*sizeof(char));
 
          strcpy(test,"ceci est un test");

[Invité(e)]

Bonjour,

Le problème vient de

Code :

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

1
2
 
char * test = "ma chaine";

Quand on écrit ça, on définit un pointeur sur une chaine non-modifiable.

Solution :

Code :

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

1
2
 
char test[] = "ma chaine";

Dans ce cas, on crée un tableau dans lequel on copie la chaine de caractère passé après le signe '='.

[Invité(e)]

tu n'alloue pas l'espace mémoire suffisante ton pointeur test.

test est un pointeur, aucune mémoire n'est allouée.

Code :

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

1
2
3
 
          char * test ;
          test = (char *) malloc ((1+strlen("ceci est un test"))*sizeof(char));

Pas la peine de caster le retour de malloc, et sizeof(char) est défini à 1, donc

Code :

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

1
2
 
          char * test = malloc(1+strlen("ceci est un test"));

suffit.

[ram-0000]

Code :

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

1
2
3
char * test ;
test = (char *) malloc ((1+strlen("ceci est un test"))*sizeof(char));
strcpy(test,"ceci est un test");

Ce code peut être remplacé par

Code :

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

char * test = strdup("ceci est un test");

strdup() fait le malloc et la copie en une seule fois.

Sinon, je pense que la réponse donnée par Mabu ici est la bonne.

[Despak]

Chouette, ça marche !!!

Merci beaucoup pour les exaplications et merci d'y avoir répondu aussi vite .