Discussion :

[Flo1763]

Bonjour à tous,

Etant nouveau sur ce forum et débutant en langage C, je fais appel à votre expérience pour m'aider.
Je cherche (dans le cadre de mes études) à créer un programme qui traite des informations sur des CDs.
La procédure qui me pose problème est celle qui permettrait d'afficher les caractéristiques d'un CD que nous avons dans la liste.
Il se trouve que lorsque je lance le programme, il affiche sans fin les caractéristiques du cd qu'il trouve, comme
si il ne sortait plus de la boucle alors que la condition est validé.
Voici la partie de mon programme qui pose problème (je m'excuse par avance pour la qualité de 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
void afficherCD (char titre[40], t_cd liste [], int *nb_cd) {
int i=0; //indice correspondant au i-eme cd de ma liste
int OK=1; // condition de fin de boucle
int R; // résultat de ma fonction strcmp : 0=même titre ; 1=titre différent
 
do
  {
    R=strcmp(titre, liste[i].titre);
   if (R == 0){
    OK=1;
    printf(" %s\n %s\n %d\n %s\n %s\n\n",liste[i].titre, liste[i].artiste, liste[i].annee, liste[i].genre, liste[i].fichier_pochette);//Ensemble des caracteristiques du cd
      }
   else
    i=i+1;
    OK=0;
  }
 
while ((i<=*nb_cd)||(OK=!1));
 
if (OK==0){
  printf("CD NON TROUVE\n\n");
  }
 
}

Je pense que ce sont mes conditions qui ne sont pas bonnes... pourtant le while est bien l'inverse logique du "jusqu'à" algorithmique non ?

Merci par avance.

Bonne soirée!

[Kannagi]

Il faut faire attention OK=!1 , c'est une affectation et non une comparaison.
Sinon c'est fortement déconseillé dans la plupart des langage de prog d'utiliser les majuscules pour les variables.
(Il existe les balises code sur le forum )

[Matt_Houston]

Il y a tellement d'incohérences dans ce morceau de code qu'on ne va pas toutes les relever, mais j'ajoute aux remarques de Kannagi :

• Comment la fonction est-elle appelée ?
• Que vaut *nb_cd ?
• Pourquoi nb_cd est-il un pointeur ?
• Pourquoi un do ... while (plutôt qu'un for ou un while) ?

Il manque probablement des accollades à la suite du else : OK est mis à zéro quoiqu'il arrive. Alternativement tu pourrais return dès que tu as trouvé l'élément recherché, et donc supprimer ce else qui deviendrait implicite.

[Flo1763]

Oui merci pour la remarque je sais que mon code est de mauvaise qualité (je débute).

Je ne comprend pas : l'opérateur "=!" vérifie bien qu'une variable (ici mon "OK") est différente d'une valeur (ici 1) non ?

Je travaille sur le fait de remplacer cette boucle do ... while par une boucle for.

Merci en tout cas pour vos quelques conseils, j'avance, doucement certes ... mais surement!

[Matt_Houston]

Nous savons que tu débutes et je ne t'en fais nullement le reproche. Je t'expose simplement les choses comme elles sont, à savoir que le code est truffé d'erreurs que nous allons t'aider à appréhender dans l'ordre.

L'opérateur de test d'inégalité est !=. Or tu as écrit =!. Ton code reste toutefois valide car OK=!1 a bien une signification en C, mais pas celle que tu attends : tu demandes au compilateur d'affecter !1 (non « vrai », soit « faux » : 0) à la variable OK, puis de tester la valeur de l'expression d'affectation, qui se trouve être celle affectée soit : 0 de type int.

OK=!1 renvoie donc toujours « faux ».


Afin d'activer la coloration syntaxique et la conservation de l'indentation, peux-tu comme te l'as demandé Kannagi éditer ton message initial et placer le listing de ton programme entre balises [code] ? Il y a un bouton pour cela (si tu as Javascript activé pour les domaines DVP) sur la page de rédaction du message.

[Flo1763]

En effet, je n'avais pas remarqué la différence entre "!=" et "!=" ! Merci
Et désolé pour la notification [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
 
void afficherCD (char titre[40], t_cd liste [], int *nb_cd) {
int i=0; //indice correspondant au i-eme cd de ma liste
int OK=0; // condition de fin de boucle
int R; // résultat de ma fonction strcmp : 0=même titre ; 1=titre différent
 
do{R=strcmp(titre, liste[i].titre);
    if (R == 0){
    OK=1;
    printf("%d\n",OK);
    }
    else{
    i=i+1;
    OK=0;}
    }
 
while ((i<=*nb_cd)||(OK!=1));
 
 
if(OK==1){
printf(" CD TROUVE : %s\n",liste[i].titre);
}
else
  if (OK==0){
  printf("CD NON TROUVE\n");
  }
}

Bon j'ai essayé de simplifier tout ca mais j'ai vraiment du mal à comprendre comment fonctionne cette boucle "Do.. While"
Dans ce cas là, je n'obtiens que des 1 non-stop si je rentre le meme titre et mon programme plante si je rentre un titre différent.
Et si je supprime la ligne 10 (printf("%d\n",OK) alors il ne maffiche plus rien dans les 2 cas comme si la boucle n'était pas éxécuté!

Merci pour votre aide

[Flo1763]

Bon petite évolution j'ai réussi à obtenir quelque chose à l'affichage en reprenant le code à 0

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
 
void afficherCD (char titre[40], t_cd liste [], int *nb_cd) {
int i=0;
int ok=0; // ok=0=FAUX et ok=1=VRAI
 
do
{
    if(titre == liste[i].titre){
        ok=1;
        printf(" %s %s %d %s %s\n",liste[i].titre, liste[i].artiste, liste[i].annee, liste[i].genre, liste[i].fichier_pochette);;}
    else{
        i=i+1;
    }
}
while ((i<=*nb_cd)||(ok!=1));
 
if (ok==0){
    printf("CD NON TROUVE\n");
  }
}

Le problème maintenant est qu'il ne m'affiche que le bon titre, par contre pour le reste YOLO... il me donne des données appartenant aux autres cd.

Merci en tout cas !

J'avance doucement mais surement

[edgarjacobs]

Hello,

Pour comparer des char * (des chaînes de caractères), il faut utiliser (entres autres), strcmp() ! Ce que tu ne fais pas à la ligne 8 de ton dernier code.