Discussion :

[nono20013]

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 :

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

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

[diogene]

i est non initialisé

[nono20013]

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

[nono20013]

i est non initialisé

si i est initialisé.
i = 0;

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

[droggo]

Hia,

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 :

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

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')
...

[nono20013]

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

[o_Nejma]

Bonjour,

Peut être pas "ce qui manque", mais juste des questions que je me suis posées, en comments, en regardant ton 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
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);
}