Discussion :

[Ascella]

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 :

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
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 :

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

[Nasky]

Bonjour et bienvenue sur les forums de Developpez

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

Code :

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

cesar(str,decalage);

Et le prototype de main c'est :

Code :

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

1
2
3
4
5
int main()
{
 // ton code
 return 0 ;
}

Nas'

[Jack_serious]

Et pour la lisibilite de ton code:


Bonne soiree !

[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

[Ascella]

Voici le nouveau 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
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++;
  }
}

[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.

[AjJi]

tu as écrit:

Code :

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

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 "="

[Ascella]

Merci de vos réponses

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

cad ?

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

Merci encore

[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

[Ascella]

Merci pour tout ajJi !

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

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 ?

[Nasky]

Avant le return 0, ajoute :

Code :

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

system("pause");

Nas'

[AjJi]

Merci pour tout ajJi !

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

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

[Emmanuel Delahaye]

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.

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 :

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
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;
}