Discussion :

[Mygirl]

Bonjour,
Alors mon problème va sûrement vous paraître trivial mais je souhaite créer une fonction qui retire le premier caractère d'une chaine de caractères. J'ai créer les fonctions suivantes :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
int taille_chaine (char * chaine)
{
int taille = 0;
char caract = 0;
 
do
{
caract = chaine[taille];
taille++;
}
while(caract != '\0'); // On boucle tant qu'on n'est pas arrivé à l'\0
 
taille--; // On retire 1 caractère de long pour ne pas compter l'\0
 
return taille;
}
 
/*******************************************************/
 
char * reste_mot (char * chaine)
{ 
char * new_chaine;
int i, j;
int n;
 
n = taille_chaine(chaine);
printf("nombre de caracteres dans word : %d \n",n);
new_chaine = (char*) malloc (n*sizeof(char));
system("pause");
 
 
for (i=0;i<n;i++)
{ for (j=1;j<n+1;j++)
{ new_chaine[i] = chaine[j];
}
}
printf ("Maintenant votre mot est : %s \n",new_chaine);
return new_chaine;
 
}
 
Voici mon main :
 
int main(int argc, char *argv[])
{
char * reste;
reste = reste_mot("bonjour");
 
free(reste); 
 
 
system("PAUSE");    
return 0;
}

Le problème est que lorsque j'affiche new_chaine, je n'ai rien d'afficher. Alors je vous remercie à l'avance pour toutes les solutions que vous pourrez m'apporter.
Merci
Mygirl

[Sanguko]

Je te laisse chercher, l'erreur est là

for (i=0;i<n;i++)
{ for (j=1;j<n+1;j++)
{ new_chaine[i] = chaine[j];
}
}

Tu peux également relire les règles du forum, et notamment la partie présentation du code.

[Emmanuel Delahaye]

Bonjour,
Alors mon problème va sûrement vous paraître trivial mais je souhaite créer une fonction qui retire le premier caractère d'une chaine de caractères. J'ai créer les fonctions suivantes :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `reste_mot':
main.c:27: warning: implicit declaration of function `printf'
main.c:28: warning: implicit declaration of function `malloc'
main.c:29: warning: implicit declaration of function `system'
main.c: In function `main':
main.c:45: warning: passing arg 1 of `reste_mot' discards qualifiers from pointer target type
main.c:47: warning: implicit declaration of function `free'
main.c: At top level:
main.c:42: warning: unused parameter 'argc'
main.c:42: warning: unused parameter 'argv'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 4 seconds)
0 errors, 7 warnings

Supair...

• Manque les headers
• Les paramètres de main() ne sont pas utilisés
• Le nom de la fonction est abscons.
• La fonction devrait accepter les chaine non modifiables. ajouter const : char *reste_mot (char const *chaine)
• Pour 'taille_chaine()' tu peux utiliser strlen()
• Revoir l'algorithme de copie. Il n'y a aucune raison d'avoir 2 boucles imbriquées. Manque le 0 final... Pourquoi réinventer la roue ? strcpy() convient. Il suffit de démarrer au bon endroit...
• new_chaine = (char *) malloc (n * sizeof (char)); Mieux vaut écrire du code simple et sûr que du code inutilement compliqué...
  

  Code :

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

  1
  2
  3
     char *new_chaine = malloc (n); /* +1 -1 */
     if (new_chaine != NULL)
     {

Je propose ceci :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *str_supp_first (char const *chaine)
{
   char *new_chaine = NULL;
   if (chaine != NULL)
   {
      size_t const n = strlen (chaine);
 
      new_chaine = malloc (n);
      if (new_chaine != NULL)
      {
         strcpy (new_chaine, chaine + 1);
      }
   }
   return new_chaine;
}
 
int main (void)
{
   char *reste = str_supp_first ("bonjour");
   if (reste != NULL)
   {
      printf ("Maintenant votre mot est : '%s' \n", reste);
      free (reste), reste = NULL;
   }
 
   return 0;
}

Code :

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

1
2
3
4
 
Maintenant votre mot est : 'onjour'
 
Press ENTER to continue.

Pose des questions si tu ne comprends pas.

[stephl]

Quand vous copiez des chaînes, il vous faut aussi copier le '\0' à la fin. En outre, il ne faut pas utiliser deux boucles mais une seule:

Code :

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

1
2
3
4
new_chaine = (char*) malloc ((n+1)*sizeof(char)); /*+1 pour le \0 à la fin*/
...
for (j=0;j<n+1;j++)
 new_chaine[j] = chaine[j];

[Mygirl]

Ah oui, je pense que ceci serait préférable :

Code :

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

1
2
3
4
 
for (i=0;i<n;i++)
{ new_chaine[i] = chaine[i+1];
}

En tout cas merci beaucoup pour la réponse, et désolée pour le code trop long.

[Stupeflip]

strcpy() convient. Il suffit de démarrer au bon endroit...

Bonjour, pourrais-tu donner plus de précisions sur le strcpy ? Est-il possible de commencer à copier par exemple à partir du 4e caractère ?

Code :

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

1
2
3
4
5
6
7
 
		i=3; // On initialise à 4-1 pour l'incrémentation i++
		do
		{
			i++;
			msg[i-4]=buffer2[i];
		}while(buffer2[i]!='\0');

Je fais cela pour l'instant mais c'est pas super optimisé (y a un while en plus)

Merci d'avance

[Emmanuel Delahaye]

Ah oui, je pense que ceci serait préférable :

Code :

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

1
2
3
4
 
  for (i=0;i<n;i++)
    { new_chaine[i] = chaine[i+1];
    }

Nettement mieux. Attention au 0 final...

[Emmanuel Delahaye]

Bonjour, pourrais-tu donner plus de précisions sur le strcpy ? Est-il possible de commencer à copier par exemple à partir du 4e caractère ?

J'ai donné un exemple.

[Stupeflip]

Merci beaucoup

[souviron34]

il y a encore plus rapide

Code :

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

1
2
3
 
memcpy ( new_chaine, &(chaine[1]), (size_t)(strlen(chaine)-1) );
newchaine[strlen(chaine)-1] = '\0' ;

[Emmanuel Delahaye]

il y a encore plus rapide

Code :

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

1
2
3
 
memcpy ( new_chaine, &(chaine[1]), (size_t)(strlen(chaine)-1) );
newchaine[strlen(chaine)-1] = '\0' ;

Plus rapide avec 2 strlen() ? J'avoue ne pas comprendre où tu veux en venir. Que peut-il y avoir de plus simple et de plus rapide que

Code :

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

strcpy (new_chaine, chaine + 1);

déjà cité ?