Traduction chaine de caractères

Version imprimable

bonjour à tous,

je suis actuellement en train de réalisé un projet qui consiste a traduire une chaine de caractère passée en parametre avec donc 2 alphabets différents . Nous avons donc au total 3 arguments. je vous met le code si dessous :
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 int translator(char *argv1, char *argv2, char *argv3) { int i; int b; b = 0; while (argv1[i] != '\0') { while (argv2[i] != '\0') { if (argv2[b] == argv3[b]) { my_putchar(argv2[i]); } else if (argv2[b] != argv3[b]) { my_putchar(argv3[i]); } i = i + 1; } b = b + 1; i = 0; } return (i); } int main(int argc, char **argv) { int i; i = 0; translator(argv[1], argv[2], argv[3]); my_putstr(argv[1]); my_putchar('\n'); }
e n'arrive pas a trouvé soit ce qu'il me manque pour que mon programme puisse marché (car mon programme compile), soit ou est l'erreur dans mon programme .
Si j'ai oublié des infos, dites le moi !
Merci d'avance à tous ce qui pourront m'aidé.

cordialement,

nono

08/11/2014, 15h48
Médinoc

Déjà, que sont supposés représenter chaque argument?
Et pourquoi ne sont-ils pas déclarés const, alors que la fonction n'est pas censée les modifier?
08/11/2014, 16h08
diogene

i est non initialisé
08/11/2014, 16h22
nono20013

Citation:

Envoyé par Médinoc

Déjà, que sont supposés représenter chaque argument?
Et pourquoi ne sont-ils pas déclarés const, alors que la fonction n'est pas censée les modifier?

agrv1 représente la chaine a traduire.
argv2 représente l'alphabet 1.
argv3 rreprésente l'alphabet 2.

je n'est pas encore vu bien les const ( je suis débutant), ont m'a dit que ce n'étais pas nécessaire/ obligatoire pour ce programmme .

nono
08/11/2014, 16h23
nono20013

Citation:

Envoyé par diogene

i est non initialisé

si i est initialisé.
i = 0;

Regarde bien a la fin de la fonction translator :')
08/11/2014, 16h53
droggo
Hia,

Citation:

Envoyé par nono20013

si i est initialisé.
i = 0;

Regarde bien a la fin de la fonction translator :')

Ce n'est pas à la fin qu'il faut l'initialiser, mais avant de l'utiliser, ce que tu n'as pas fait :
Code:

1 2 3 4 5 6 7 8 9 10 11 int translator(char *argv1, char *argv2, char *argv3) { int i; int b; b = 0; while (argv1[i] != '\0') { while (argv2[i] != '\0') ...
08/11/2014, 17h20
nono20013

C'est vrai, j'ai pas fais attention merci :')

Bonjour,

Peut être pas "ce qui manque", mais juste des questions que je me suis posées, en comments, en regardant ton 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
int     translator(char *argv1, char *argv2, char *argv3)
{
  int   i;
  int   b;
 
  b = 0;
  //Est ce que tu n'as vraiment besoin que du test du premier element car le i ici est tjrs 0?
  while (argv1[i] != '\0')//i=0
    {
      while (argv2[i] != '\0')
        {
          if (argv2[b] == argv3[b])
            {
              my_putchar(argv2[i]);
            }
  // est ce que la discontinuité du if .. else if .. else ne pose pas de risque?
          else if (argv2[b] != argv3[b])
            {
              my_putchar(argv3[i]);
            }
          i = i + 1;
	}
      b = b + 1;
      i = 0;
    }
  return (i);
}