Discussion :

[Emmanuel Delahaye]

Les chaînes littérales sont en lecture seule. Un bon compilo comme gcc, configuré avec l'option -wWrite-strings, aurait gueulé...

Démonstration :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:28: warning: initialization discards qualifiers from pointer target type
main.c: At top level:
main.c:26: warning: unused parameter 'argc'
main.c:26: warning: unused parameter 'argv'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 3 seconds)
0 errors, 3 warnings

Et si je mets const :

Code :

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

  char const *chaine = "Coucou tout le monde";

Code :

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

1
2
3
4
5
6
 
<...>
Compiling: main.c
main.c: In function `main':
main.c:29: warning: passing arg 1 of `STR_DelBlanks' discards qualifiers from pointer target type
<...>

Savoir configurer son compilateur fait gagner des bits...

[Emmanuel Delahaye]

Code :

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

1
2
3
4
 
  char chaine[255] = "Coucou tout le monde";
  STR_DelBlanks(chaine[255]);
}

Tu fais un peu n'importe quoi là....

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `main':
main.c:29: warning: passing arg 1 of `STR_DelBlanks' makes pointer from integer without a cast
main.c: At top level:
main.c:26: warning: unused parameter 'argc'
main.c:26: warning: unused parameter 'argv'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 2 seconds)
0 errors, 3 warnings

Réfléchis un peu à ce que tu passes à la fonction...

Une fois corrigé :

Code :

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

1
2
3
4
 
Coucoutoutlemonde
 
Press ENTER to continue.

[Solevis]

Merci, je vais essayer de configurer gcc alors. Pour tout dire je faisais seulement :

Code :

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

gcc main.c -o main

[Emmanuel Delahaye]

Merci, je vais essayer de configurer gcc alors. Pour tout dire je faisais seulement :

Code :

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

gcc main.c -o main

Insuffisant : http://emmanuel-delahaye.developpez....tm#cfg_compilo

[Solevis]

Maintenant je fais :

gcc main.c -Wall -Wextra -O -Wwrite-strings -Wstrict-prototypes Wuninitialized
-Wunreachable-code main

Pour le :

Code :

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

1
2
3
 char chaine[255] = "Coucou tout le monde";
  STR_DelBlanks(chaine[255]);
}

C'est une erreur de copier/coller mais comme le compilateur ne retourné rien je ne m'en etais meme pas rendu compte.
Aprés toute ces peripeties et grace a votre aide : le code marche

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 <stdio.h>
#include <stdlib.h>
#include <string.h>
 
void STR_DelBlanks(char *const s)
{
   if (s && *s)
   {
      char const *pr = s;
      char *pw = s;
 
      while (*pr)
      {
         if (*pr != ' ')
         {
            *pw = *pr;
            pw++;
         }
         pr++;
      }
 
      *pw = 0;
   }
}
 
void main()
{
  char chaine[255] = "Coucou tout le monde";
  STR_DelBlanks(chaine);
  printf("%s",chaine);
  getchar();
 
}

Merci pour tout.
Sylvain

[Melem]

Juste une dernière chose : main doit retourner un int :

Code :

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

1
2
3
4
5
6
7
8
9
10
int main()
{
    char chaine[255] = "Coucou tout le monde";
 
    STR_DelBlanks(chaine);
    printf("%s",chaine);
    getchar();
 
    return 0;
}