Un problème de débutant !

Version imprimable

Bonjour à tous, j' essaye de developper mon premier programme en C.
Je me sert des tutoriaux mis à disposition sur developpez.com , mais voila j'ai un souci et je ne comprends pas !

J'utilise Dev-C++ pour developper .

J'aimerai faire un Cryptage selon la méthode de César, mais je n'y arrive pas.

Voila mon code :

Code:

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
/* Déclaration de la fonction sleep  */
void sleep( int nbr_seconds );
/* Déclaration de la fonction cesar  */
void cesar (char *str, int decalage);
 
 
void main( void )
{ 
    char *str ;
    str = "bonjour" ; 
    int decalage = 2;
 
    cesar ( *str , decalage );
 
    while (*str)
    {
          printf ("%c",* str) ;
          str++ ;
    }
 
    int x;
    int wait = 13;
 
    /* Pause pendant un nombre de secondes déterminé. On affiche *
     * un point pour chaque seconde de pause.                   */
 
    printf("Pause pendant %d secondes\n", wait );
    printf(">");
 
    for (x=1; x <= wait; x++)
    {
       printf(".");         /* affichage d'un point */
       fflush(stdout);      /* on force l'affichage du point sur les*/
                            /* machines qui utilisent la mémoire tampon*/
       sleep( (int) 1 );    /* pause d'une seconde */
    }
    printf( "Fin !\n");
 
}
 
 
/* Pause pendant un nombre de secondes déterminé */
void sleep( int nbr_seconds )
{
    clock_t goal;
 
    goal = ( nbr_seconds * CLOCKS_PER_SEC ) +  clock();
 
    while( goal > clock() )
    {
       ; /* loop */
    }
}
 
/* Cryptage selon la méthode de César */
void cesar (char *str, int decalage)
{
  const char alphabet[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j',
                           'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't',
                           'u', 'v', 'w', 'x', 'y', 'z'};
 
  while (*str)
  {
    int i;
 
    for (i = 0; i < 26; i++)
    {
      if (alphabet[i] == *str)
      {
        *str = alphabet[(i+decalage)%26];
        break;
      }
    }
    str++;
  }
}

et quand je compile j'obtient ça :

Citation:

In function `main':
[Warning] passing arg 1 of `cesar' makes pointer from integer without a cast
[Warning] return type of 'main' is not `int'

06/06/2006, 20h22
Nasky
Bonjour et bienvenue sur les forums de Developpez ;)

Enlève l'étoile lorsque tu appelles la fonction cesar()

Code:

cesar(str,decalage);

Et le prototype de main c'est :
Code:

1 2 3 4 5 int main() { // ton code return 0 ; }
Nas'
06/06/2006, 20h33
Jack_serious

Et pour la lisibilite de ton code:
:tagcode:

Bonne soiree !
06/06/2006, 20h35
Ascella

Merci Nasky, j'ai procédé au modif ...

Je n'ai plus de problème de compilation.
Mais quand j'execute mon programme j'aobtient une erreur windows !

cesar.exe a rencontré un problème et doit fermer. Nous vous prions de nous excuser pour le désagrément encouru.
|Débogage| |Envoyer le rapport d'erreur| |Ne pas envoyer|

Merci pour ton aide

Voici le nouveau code :

Code:

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h> 
 
/* Déclaration de la fonction sleep  */
void sleep( int nbr_seconds );
/* Déclaration de la fonction cesar  */
void cesar (char *str, int decalage);
 
 
int main( )
{ 
    char *str ;
    str = "bonjour" ; 
    int decalage = 2;
 
    cesar ( str , decalage );
 
    while (*str)
    {
          printf ("%c",* str) ;
          str++ ;
    }
 
    int x;
    int wait = 13;
 
    /* Pause pendant un nombre de secondes déterminé. On affiche *
     * un point pour chaque seconde de pause.                   */
 
    printf("Pause pendant %d secondes\n", wait );
    printf(">");
 
    for (x=1; x <= wait; x++)
    {
       printf(".");         /* affichage d'un point */
       fflush(stdout);      /* on force l'affichage du point sur les*/
                            /* machines qui utilisent la mémoire tampon*/
       sleep( (int) 1 );    /* pause d'une seconde */
    }
    printf( "Fin !\n");
 
    return 0;
}
 
 
/* Pause pendant un nombre de secondes déterminé */
void sleep( int nbr_seconds )
{
    clock_t goal;
 
    goal = ( nbr_seconds * CLOCKS_PER_SEC ) +  clock();
 
    while( goal > clock() )
    {
       ; /* loop */
    }
}
 
/* Cryptage selon la méthode de César */
void cesar (char *str, int decalage)
{
  const char alphabet[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j',
                           'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't',
                           'u', 'v', 'w', 'x', 'y', 'z'};
 
  while (*str)
  {
    int i;
 
    for (i = 0; i < 26; i++)
    {
      if (alphabet[i] == *str)
      {
        *str = alphabet[(i+decalage)%26];
        break;
      }
    }
    str++;
  }
}

06/06/2006, 20h56
Jack_serious

A mon avis l'erreur provient du fait que tu tentes de modifier une chaine declaree en dur dans le code.

Selon la compilation, ca peut faire planter ton programme.

Essaie d'allouer une chaine par toi meme et de la passer en parametre a ta fonction.
Je ne garanti pas que ton erreur vienne de la, mais c'est possible.
06/06/2006, 21h02
AjJi
tu as écrit:
Code:

1 2 3 char *str ; str = "bonjour" ;
essaye plutot d'allouer de la place pour ta str (assure-toi qu'il y est assez de place pour ce que tu veux faire, et teste aussi l'allocation ;) )

Après c'est plutot un strcpy (ou strncpy) qu'il te faut parce pour l'affectation des chaines de caractère ca ne marche pas avec l'opérateur "=" :)
06/06/2006, 21h48
Ascella

Merci de vos réponses

Citation:

Essaie d'allouer une chaine par toi meme et de la passer en parametre a ta fonction.

cad ?

Citation:

essaye plutot d'allouer de la place pour ta str (assure-toi qu'il y est assez de place pour ce que tu veux faire, et teste aussi l'allocation )

Après c'est plutot un strcpy (ou strncpy) qu'il te faut parce pour l'affectation des chaines de caractère ca ne marche pas avec l'opérateur "="

je ne comprends pas vraiment ...

vous pourriez me montrer ce que ça donne en C :oops:

Merci encore
06/06/2006, 21h56
AjJi

pour allouer de la mémoire pour ta chaine : malloc
pour copier une chaine dans une autre : strcpy

voilà, j'espere que ça t'aidera ;)
06/06/2006, 22h25
Ascella

Merci pour tout ajJi !

hey j'avais pas vu "localisation : Maroc" !!
Je suis Marocain aussi, ça fait plaisir ... :D

choukrane ya sadikki el aziz !

et une dernière petite question ( svp )

mon application semble maintenant tourner correctement.
l'invite de commande se lance, l'appli tourne comme il faut, mais dés que tout mon code est lu, l'invite de commande disparait aussitôt .

Comment faire pour que l'invite de commande ne disparaisse pas ?
06/06/2006, 22h30
Nasky

Avant le return 0, ajoute :

Code:

system("pause");

Nas'
06/06/2006, 23h43
AjJi

Citation:

Envoyé par Ascella

Merci pour tout ajJi !

hey j'avais pas vu "localisation : Maroc" !!
Je suis Marocain aussi, ça fait plaisir ... :D

choukrane ya sadikki el aziz !

et une dernière petite question ( svp )

mon application semble maintenant tourner correctement.
l'invite de commande se lance, l'appli tourne comme il faut, mais dés que tout mon code est lu, l'invite de commande disparait aussitôt .

Comment faire pour que l'invite de commande ne disparaisse pas ?

lol de rien, n'oublie pas le tag :resolu: ;)

Citation:

Envoyé par Ascella

Une petite erreur, quelques améliorations possibles, mais dans l'ensemble, c'est du bon travail. Très bon, même sur certains points, comme la version portable du cesar() (indépendante du charset). Voir quand même les fonctions standards comme strchr()...

Code:

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
#if 0
/* -ed- Je deconseille les prototypes separes pour le fonctions non exportees */
/* Déclaration de la fonction sleep */
void sleep (int nbr_seconds);
/* Déclaration de la fonction cesar */
void cesar (char *str, int decalage);
#endif
 
/* -ed- reorganisation du code. */
/* Pause pendant un nombre de secondes déterminé */
static void sleep (int nbr_seconds)
{
   clock_t goal;
 
   goal = (nbr_seconds * CLOCKS_PER_SEC) + clock ();
 
   while (goal > clock ())
   {
      ;                         /* loop */
   }
}
 
/* Cryptage selon la méthode de César */
static void cesar (char *str, int decalage)
{
   const char alphabet[] =
   {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j',
    'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't',
    'u', 'v', 'w', 'x', 'y', 'z'
   };
 
   while (*str)
   {
      int i;
 
      for (i = 0; i < 26; i++)
      {
         if (alphabet[i] == *str)
         {
            *str = alphabet[(i + decalage) % 26];
            break;
         }
      }
      str++;
   }
}
 
/* -ed-
   void main( void )
 
   main() retourne int. Toujours
 */
int main (void)
{
   /* -ed-
      char *str ;
 
      str = "bonjour" ;
 
      Une chaine n'est pas modifiable.
      Il faut le qualificateur const ou utiliser une tableau de char.
      Si j'en crois la suite, la chaine sera modifiee par la fonction de cesar().
      Il faut donc un tableau.
    */
   char str[] = "bonjour";
   int decalage = 2;
 
   /* -ed-
      cesar ( *str , decalage );
 
      *str est un char.
      Or la fonction attend l'adresse d'un char. C'est probablement :
    */
   cesar (str, decalage);
 
   /* -ed-
      while (*str)
      {
      printf ("%c",* str) ;
      str++ ;
      }
 
      Une facon inutilement complexe d'ecrire
      (\n et '' ajoutes pour la clarete de la presentation)
    */
 
   printf ("'%s'\n", str);
 
 
#if 1
   /* -ed- tres joli, mais hors-sujet, non ?) */
   {
      int x;
      int wait = 13;
 
      /* Pause pendant un nombre de secondes déterminé. On affiche *
         * un point pour chaque seconde de pause. */
 
      printf ("Pause pendant %d secondes\n", wait);
      printf (">");
 
      for (x = 1; x <= wait; x++)
      {
         printf (".");          /* affichage d'un point */
         fflush (stdout);       /* on force l'affichage du point sur les */
         /* machines qui utilisent la mémoire tampon */
         sleep ((int) 1);       /* pause d'une seconde */
      }
   }
   printf ("Fin !\n");
#endif
 
   /* -ed- parce que main() retourne un int : */
   return 0;
}