Discussion :

[wedge.tm]

Bonjour,
je dois utilisé le cryptage par la méthode césar (décalage de n lettres dans l'alphabet) et mon programme (qui doit utiliser des fichiers) me rajoute des caractères après la chaine crypté correctement (selon le nombre de caractères : moin de 6 ou plus de 16).
Je vous copie mon fihier source ici et j'attend vos commentaires et suggestions !

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
 
#include<stdlib.h>
#include<stdio.h>
 
int main(void)
{
  FILE * f;
  FILE * g;
  f=fopen("cesarclr.txt","r");
  g=fopen("cesarcrt.txt","w");
  char clr[60],crt[60];
  int i,n;
 
  if(f==NULL || g==NULL)
  {printf("erreur");}
  else
  {
    fgets(clr,80,f);
    printf("chaine lu : %s\n",clr);
    n=strlen(clr);
    for(i=0;i<n;i++)
    {
      if((clr[i]!=0)&&(clr[i]!=' '))
      {
        crt[i]=clr[i]-0x03;
        if (crt[i]<'a')
        {crt[i]=crt[i]+26;}
      }
      else 
      {crt[i]=clr[i];}
    }
    fprintf(g,"%s",crt);
    printf("chaine crypte : %s\n\n",crt);
  }
  fclose(f);
  fclose(g);
 
  system("pause");
  return 0;
}

je pense donc que mon programme doit planter ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
for(i=0;i<n;i++)
{
if((clr[i]!=0)&&(clr[i]!=' '))
{
crt[i]=clr[i]-0x03;
if (crt[i]<'a')
{crt[i]=crt[i]+26;}
}
else
{crt[i]=clr[i];}
}

Merci d'avances pour vos réactions, ceci permettrai de me débloquer car ce programme et le début d'une suite de différents autres programmes, qui eux, fonctionne correctement.

Merci encore!

[granquet]

met ton code entre des balises ... code ...

sinon:

Code :

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

1
2
3
4
 
char clr[60],crt[60];
[...]
fgets(clr,80,f);

ça te choque pas?

[wedge.tm]

C'est à dire entre balise?

[Emmanuel Delahaye]

C'est à dire entre balise?

Merci de (re)lire les règles du forum...



Tu peux éditer ton post avec et mettre les balises code avec

[wedge.tm]

j'avais lu mais j'avais pas capter comment on faisait!
sinon j'ai résolu le probleme a l'instant!
il falai se servir de EOF dans la boucle for
merci quand meme!
j'aurais apris le système de code ^^

[Lunixinclar]

Salut,

Code :

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

1
2
3
4
5
6
7
8
      if((clr[i]!=0)&&(clr[i]!=' '))
      {
        crt[i]=clr[i]-0x03;
        if (crt[i]<'a')
        {crt[i]=crt[i]+26;}
      }
      else 
      {crt[i]=clr[i];}

Sur if((clr[i]!=0)&&(clr[i]!=' '))
Le caractère analysé ne sera jamais un NULL parce que la longueur de la chaîne est retrouvée avec un n=strlen(clr); par conséquent la vérification est inutile. Les instructions du else aussi.

'A' ne deviendra pas 'X', mais '>'. je me trompe?

De plus, un modulo serait plus joli pour ce ROT3.

[wedge.tm]

slt,

merci de ta réponse mais le 'A' ne devient pas '<' (de toute façon le problème est résolu et rendu !)
et il faut quand meme testé le pointeur car dans la boucle for ce n'est pas le dernier caractère qui arête la chaine mais la lecture du suivant.
c'est pour ça qu'il faut le retesté dans un if ensuite!

voila merci quand même c'est sympa d'avoir répondu au sujet!
bye et a bientot je crois qu'au deuxième semestre de mon dut en geii je fais du c++ ^^