Discussion :

[afia912]

Ecrire un programme qui lit deux chaînes de caractères, et qui indique leur précédence
lexicographique dans le code de caractères de la machine (ici: code ASCII). On écrira pour cela la
fonction precedence qui récupère les deux chaînes en paramètre et qui retourne 1 si la première
chaîne précède la deuxième, 2 si la deuxième précède la première, 0 si elle sont égale.
Prototype : int precedence(char *,char *)

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
#include<stdio.h>
#include<stdlib.h>
 
int precedence(char*chaine1 , char*chaine2)
{ 
  int i = ;
  int r ;
  for(i = 0 ;(chaine1[i] == chaine2[i]) && chaine1[i],chaine2[i]; i++)
  {
    if (chaine1[i] < chaine2[i])
    {
      return 1;
    }
    else if(chaine1[i] >chaine2[i])
    {
      return 2;
    }
    else return 0 ;
 
  }
  return r ;
 
}
int main ()
{
  int chaine1[] = " ";
  int chaine2[] = " ";
  int r ;
  printf("entrer la premiere chaine \n");
    scanf("%s",chaine1);
  printf("entrer la deusieme chaine \n");
    scanf("%s",chaine2);
   r= precedence(chaine1,  chaine2);
   if ( r == 1)
   {
     printf("la chaine1 precede la chaine2");
   }
   else if(r == 2)
   {
     printf("la chaine1 precede la chaine2");
 
   }
   else printf("la chaine1 est egale a la  chaine2");
 
 
   return 0 ;
}

[esired]

Bonjour.
Tout d'abord, tu n'es pas dans le bon forum pour poser cette question. Le forum C est ici.
Ensuite, il y a plusieurs erreurs dans ton code. Commençons par ta fonction main :

Code :

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

1
2
 
int chaine1[] = " ";

Ce n'est pas ainsi qu'on déclare les chaînes de caractères.

Fonction précédence

Code :

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

1
2
 
int i = ;

Tu as oublié de préciser la valeur initiale de i;

Code :

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

1
2
 
for(i = 0 ;(chaine1[i] == chaine2[i]) && chaine1[i],chaine2[i]; i++)

Pour quelqu'un qui débute en C, tu écris du code que même un programmeur confirmé ne comprendra pas facilement. De plus ce code est erroné.

Donc un bon conseil qui t'évitera beaucoup de déconvenues, relis ton cours et reviens nous voir.

[Sve@r]

Bonjour

Code :

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

for(i = 0 ;(chaine1[i] == chaine2[i]) && chaine1[i],chaine2[i]; i++)

Pour quelqu'un qui débute en C, tu écris du code que même un programmeur confirmé ne comprendra pas facilement.

C'est vrai que jouer ainsi de l'opérateur "virgule" dans l'évaluation c'est assez osé !!! Et comme je ne suis pas certain de mes connaissances concernant les priorités je ne sais même pas si j''évalue l'expression correctement...

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
 
int r;
  for(i = 0 ;(chaine1[i] == chaine2[i]) && chaine1[i],chaine2[i]; i++)
  {
    if (chaine1[i] < chaine2[i])
    {
      return 1;
    }
    else if(chaine1[i] >chaine2[i])
    {
      return 2;
    }
    else return 0 ;
 
  }
return r;
}

Bon ben une boucle qui fait un return dès sa première itération c'est plus trop une boucle quoi. Et sinon il vaut quoi ce "r" que tu renvoies à la fin ? Ce qui me tue le plus, ce n'est pas tant l'erreur c'est surtout que ce genre d'énormité tu ne le vois pas toi-même quand tu te relis. Sans déconner, quand tu écris return r, c'est que tu as un but en tête. Et donc au moment où tu écris cette instruction, quel est ton but précisément ???

[ternel]

question priorité, la virgule, c'est facile, c'est la toute dernière à être évaluée.
ici, la valeur testée de l'expression est chaine2[i]