Debbug pour débutant

**BFH** · 26/11/2005, 10h47

Bonjour, je suis en train d'ecrire un petit programme que j'essai de compiler au fur et a mesur pour elever les erreurs d'ecriture.
Malheureusement pour moi j'ai des warning et des erreurs que je ne sais pas resoudre.
Quelqu'un pourrait il m'aider a les resoudre et m'expliquer aussi l'erreur?

Code :

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

Bibliotheque *nouv=SaisieLivre();

Warning initialization makes point from integer without a cast

Code :

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

1
2
3
char *choix;
Bibliotheque *courant;
if (strcmp(courant->*choix,courant->suiv->*choix)>0)

syntax error before *

**scol30** · 26/11/2005, 11h51

pourrais tu s'il te plait nous montrer l'implémentation de Bibliotheque.

Ensuite strcmp va te comparer 2 char*.
Dans ton cas tu dois surement vouloir faire

Code :

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

strcmp(courant->???; choix);

ou enfin quelque chose comme ça.

Donc montres nous l'architecture de Bibliotheque puis on verra mireux quoi faire.

Stef

**diogene** · 26/11/2005, 12h38

Code :

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

strcmp(courant->*choix,...

L'utilisation de -> suppose que courant est un pointeur sur une structure (ou une union). Derrière le -> doit figurer le nom d'un champ de la dite structure. Il ne peut donc PAS y avoir un opérateur de déréférencement *

Code :

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

char *choix;

A noter que ce choix n'a rien à voir de toute façon avec celui figurant dans l'expression courant->...

**Emmanuel Delahaye** · 26/11/2005, 12h46

Envoyé par BFH

Code :

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

Bibliotheque *nouv=SaisieLivre();

Warning initialization makes point from integer without a cast

Ca signifie que le compilateur considère que la fonction retourne un entier (int, probablement), ce qui ne rentre pas dans un pointeur, évidemment (pas le bon type). Le comportement est indéfini.

Il manque probablement une définition d'interface avant l'appel (définition de fonction, prototype séparé...)

Une regle simple : ce qui est utilisé doit avoir été déclaré au préalable.

Code :

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

1
2
3
char *choix;
Bibliotheque *courant;
if (strcmp(courant->*choix,courant->suiv->*choix)>0)

syntax error before *

Des '*' en trop. Il ne faut pas programmer au hasard, mais apprendre les regles syntaxiques du langage. Pour atteindre un élément de structure, on fait

Code :

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

1
2
3
4
5
6
7
8
9
 
   struct x
   {
      int a;
   };
 
   struct x y;
 
   y.a = 123;

Si on a un pointeur de structure, on fait :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
   struct x
   {
      int a;
   };
 
   struct x y;
   struct x *p = &y;
 
   (*p).a = 123;

Vu le coté lourd de l'expression, le langage C a défini une écriture plus compacte :

Code :

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

1
2
 
   p->a = 123;

**BFH** · 26/11/2005, 17h49

En fait désole c'était juste une faute de frappe!

j'appelais la fonction "SaisieLivre" mais j'avais créé la fonction "Saisie_Livre"

Sinon quand j'enleve les * des mon second bug:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void Trier(Bibliotheque *tete)
{
	char *choix;
	Bibliotheque *courant=tete;
	Bibliotheque *suivant=tete;
	choix =Choix_Trie();
	while(suivant!=NULL)
	{
        if(strcmp(courant->choix,suivant->choix)>0)
        {
            courant->droite=suivant;
        }
/*j'ai pas mis la suite*/

ça me retourne comme erreur :
structure has no member named `choix'

Je comprends pas pourquoi ça bug, la fonction Choix_Trie retourne une chaine de caracteres qui est soit 'Auteur' soit 'Titre' Et ces 2 mots sont 2 champs de la Bibliotheque.

**Emmanuel Delahaye** · 26/11/2005, 18h26

Envoyé par BFH

Code :

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

1
2
3
4
 
	Bibliotheque *courant=tete;
<...>
        if(strcmp(courant->choix,suivant->choix)>0)

structure has no member named `choix'

Montre la definition de 'Bibliotheque'.

**BFH** · 26/11/2005, 20h38

Voici ma definition de bibliotheque:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
struct livre
{
	char Titre[50];
	char Auteur[20];
	char MEdition[10];
	int Annee;
	int NbPages;
	int Prix;
	struct livre *gauche;
	struct livre *droite;
};
 
typedef struct livre Bibliotheque;

**Emmanuel Delahaye** · 26/11/2005, 20h49

Envoyé par BFH

Voici ma definition de bibliotheque:

Ben oui, je ne vois pas de 'choix'.

Or tu as écrit

Code :

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

if(strcmp(courant->choix,suivant->choix)>0)

Y'a un bug...

**BFH** · 26/11/2005, 22h33

En fait, c'est pas si simple:
Ma Bibliotheque contient les champs 'Auteur' & 'Titre'

Ma fonction 'Trier' fait appel a la fonction 'Choix_Trie' or 'Choix_Trie fait ça:

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
char * Choix_Trie(void)
{
	char choix;
	char *chaine = malloc(7);
 
	printf("Souhaitez vous effectuer un tri par Titre ou par Auteur (t/a) ?\n");
 
	scanf("%c",&choix);
 
	if (choix!='a' || choix!='t')
	{
		printf("Erreur de saisie de choix de trie !\n");
		Choix_Trie();
    }
	else
	{
		if (choix == 'a')
			{
			strcpy(chaine,"Auteur");
			}
		if (choix == 't')
			{
			strcpy(chaine,"Titre");
			}
	}
	return chaine;
}

Donc

Code :

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

choix =Choix_Trie();

devrais faire que choix=Auteur ou Titre

Mais non apparement...

**Médinoc** · 26/11/2005, 22h53

Code :

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

1
2
printf("Erreur de saisie de choix de trie !\n");
      Choix_Trie();

Tu es obligé de faire ça en récursif? Utilisé ainsi, c'est générateur de bugs et difficile à maintenir... une boucle serait plus indiquée...

**Smortex** · 26/11/2005, 23h40

Arg... heu ... Salut

Non, le C ne te permet pas d'utiliser une chaine de caractère pour manipuer le membre xxx d'une structure (certains langages le permettent (script shell Unix, PHP, ...) mais pas le C). Le plus simple dans ton cas est de retourner le choix de l'utilisateur (donc un char, qui vaudra 'a' ou 't') et selon cette valeur, faire des comparaisons en conséquence...

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
char Choix_Trie(void) {
  char choix = '\0';
  while ((choix != 'a')&&(choix != 't')) { /* Récursif ou pas, comme tu préfère... */
    scanf("%c", &choix); /* scanf convient pour lire un char */
  }
  return choix ;
}
 
[...]
 
  switch (Choix_Trie()) {
    case 'a':
      if(strcmp(courant->Auteur,suivant->Auteur)>0) { ... }
      break ;
    case 't':
      if(strcmp(courant->Titre,suivant->Titre)>0) { ... }
      break;
  }

Bon, ca ca fonctionne, mais niveau évolutivité, c'est plutot bancale... Si ce programme est pour un TP, ca devrait être acceptable, mais pour une application qui devra évoluer, il vaudrait mieux définir des procédures de comparaisons dont l'adresse serait renvoyées par la fonction de choix du trie pour être utilisée avec un appel de qsort par exemple.

**BFH** · 27/11/2005, 10h39

Vu que c'est pour un tp ça me suffit largement. Merci a toi!
Par contre j'ai des erreurs que je ne comprends pas :

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
Bibliotheque Trier(Bibliotheque *tete)
{
	char *choix;
	Bibliotheque *courant=tete;
	Bibliotheque *reference=tete;
	while(courant!=NULL)
	{
        switch (Choix_Trie()) /*ICI*/
        { 
            case 'a': 
            if(strcmp(courant->Auteur,reference->Auteur)>0)
            {
                courant->droite=Trier(courant->droite); /*ICI*/
            }
            else
            {
                if(strcmp(courant->Auteur,reference->Auteur)<0)
                {
                    courant->gauche=Trier(courant->gauche);/*ICI*/
                }
                else
                    courant->droite=Trier(courant->droite);/*ICI*/
            }
            break ; 
            case 't': 
            if(strcmp(courant->Titre,reference->Titre)>0)
            {
                courant->droite=Trier(courant->droite);/*ICI*/
            }
            else
            {
                if(strcmp(courant->Titre,reference->Titre)<0)
                {
                    courant->gauche=Trier(courant->gauche);/*ICI*/
                }
                else
                    courant->droite=Trier(courant->droite);/*ICI*/
            }
            break ;
        } 
    }
    return *courant;
}

Pour la premiere ça me retourne : switch quantity not an integer

Pour toutes les autres : incompatible types in assignment

Pourquoi?

**hegros** · 27/11/2005, 12h30

Envoyé par BFH

Vu que c'est pour un tp ça me suffit largement. Merci a toi!
Par contre j'ai des erreurs que je ne comprends pas :

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
Bibliotheque Trier(Bibliotheque *tete)
{
	char *choix;
	Bibliotheque *courant=tete;
	Bibliotheque *reference=tete;
	while(courant!=NULL)
	{
        switch (Choix_Trie()) /*ICI*/
        { 
            case 'a': 
            if(strcmp(courant->Auteur,reference->Auteur)>0)
            {
                courant->droite=Trier(courant->droite); /*ICI*/
            }
            else
            {
                if(strcmp(courant->Auteur,reference->Auteur)<0)
                {
                    courant->gauche=Trier(courant->gauche);/*ICI*/
                }
                else
                    courant->droite=Trier(courant->droite);/*ICI*/
            }
            break ; 
            case 't': 
            if(strcmp(courant->Titre,reference->Titre)>0)
            {
                courant->droite=Trier(courant->droite);/*ICI*/
            }
            else
            {
                if(strcmp(courant->Titre,reference->Titre)<0)
                {
                    courant->gauche=Trier(courant->gauche);/*ICI*/
                }
                else
                    courant->droite=Trier(courant->droite);/*ICI*/
            }
            break ;
        } 
    }
    return *courant;
}

Pour la premiere ça me retourne : switch quantity not an integer

Pour toutes les autres : incompatible types in assignment

Pourquoi?

Parce que choix_trie envoi un char *. switch ne marche pas avec les chaines de caractere

**@drien** · 27/11/2005, 12h51

J'ai pas tout lu mais je crois que si tu écrit *Choix_Trie() dans ton switch ça va marcher

**BFH** · 27/11/2005, 14h02

Merci c'etait ça!
Mais là j'en ai marre j'ai passé mon w-e dessus et ça ne fonctionne pas!

Voila mon code complet :
Definition de la structure

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
struct livre
{
	char Titre[50];
	char Auteur[20];
	char MEdition[10];
	int Annee;
	int NbPages;
	int Prix;
	struct livre *gauche;
	struct livre *droite;
};
 
typedef struct livre Bibliotheque;

Fonction qui definie le type de tri qu'on va faire sur une bibliotheque :

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
char * Choix_Trie(void)
{
	char choix;
	char caractere;
 
	printf("Souhaitez vous effectuer un tri par Titre ou par Auteur (t/a) ?\n");
	scanf("%c",&choix);
 
	if (choix!='a' || choix!='t')
	{
		printf("Erreur de saisie de choix de trie !\n");
		Choix_Trie();
    }
	else
	{
		if (choix == 'a')
			{
			caractere='a';
			}
		if (choix == 't')
			{
			caractere='t';
			}
	}
	return (caractere); /*Premiere ERREUR*/
}

Voila ma fonction qui trie la bibliotheque (en fonction du choix de trie qui est fait grace a la fonction Choix_Trie :

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
Bibliotheque Trier(Bibliotheque *tete)
{
	Bibliotheque *reference=tete;
	Bibliotheque *nouveau=tete->droite;
	switch(*Choix_Trie())
	{
        while(reference!=NULL)
        { 
            case 'a': 
            if(strcmp(nouveau->Auteur,reference->Auteur)>0)
            {
                reference->droite=Trier(nouveau);/*ERREUR*/
            }
            else
            {
                if(strcmp(nouveau->Auteur,reference->Auteur)<0)
                {
                    reference->gauche=Trier(nouveau);/*ERREUR*/
                }
                else
                    reference->droite=Trier(nouveau);/*ERREUR*/
            }
            break ; 
            case 't': 
            if(strcmp(nouveau->Titre,reference->Titre)>0)
            {
                reference->droite=Trier(nouveau);/*ERREUR*/
            }
            else
            {
                if(strcmp(nouveau->Titre,reference->Titre)<0)
                {
                    reference->gauche=Trier(nouveau);/*ERREUR*/
                }
                else
                    reference->droite=Trier(nouveau);/*ERREUR*/
            }
            break ;
        } 
    }
    return *nouveau;
}

Parmi tout ce code je ne sais pas si mon arbre fonctionne reelement!

De plus, j'ai plein d'erreur
Premiere erreur [Warning] return makes pointer from integer without a cast

Les autres : incompatible types in assignment

**@drien** · 27/11/2005, 14h16

Le problème c'est que ta fonction Choix-Trie retourne un CHAR alors que dans l'entête de ta fonction tu dis qu'elle retrourne un char *

Code :

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

char Choix_Trie(void) {...}

devrait aller mieux

Par contre repasse à switch(Choix_Tri) du coup

Il faut bien comprendre les pointeurs avavnt de les utiliser

**Emmanuel Delahaye** · 27/11/2005, 14h39

Envoyé par BFH

Merci c'etait ça!
Mais là j'en ai marre j'ai passé mon w-e dessus et ça ne fonctionne pas!

Voila mon code complet :
Definition de la structure

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
struct livre
{
	char Titre[50];
	char Auteur[20];
	char MEdition[10];
	int Annee;
	int NbPages;
	int Prix;
	struct livre *gauche;
	struct livre *droite;
};
 
typedef struct livre Bibliotheque;

Fonction qui definie le type de tri qu'on va faire sur une bibliotheque :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
char * Choix_Trie(void)
{
	char choix;
	char caractere;
 
	printf("Souhaitez vous effectuer un tri par Titre ou par Auteur (t/a) ?\n");
	scanf("%c",&choix);
 
	if (choix!='a' || choix!='t')
	{
		printf("Erreur de saisie de choix de trie !\n");
		Choix_Trie();

Un appel récursif ? En mesure tu bien tous les effets ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
    }
	else
	{
		if (choix == 'a')
			{
			caractere='a';
			}
		if (choix == 't')
			{
			caractere='t';
			}
	}
	return (caractere); /*Premiere ERREUR*/

Gné ? Si la fonction doit retourner un caractère, le type du retour doit être int et non adresse de char....

Code :

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

Voila ma fonction qui trie la bibliotheque (en fonction du choix de trie qui est fait grace a la fonction Choix_Trie :

Code :

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

1
2
3
4
5
Bibliotheque Trier(Bibliotheque *tete)
{
	Bibliotheque *reference=tete;
	Bibliotheque *nouveau=tete->droite;
	switch(*Choix_Trie())

Et si elle retourne int, il suffit alors d'écrire

Code :

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

	switch(Choix_Trie())

**BFH** · 27/11/2005, 14h47

Merci a tous pour avoir passé beaucoup de temps sur mes bugs!
Mention spéciale pour Emmanuel Delahaye Franchement merci a toi pour le temps que tu as du passer a lire et a corriger mes erreurs!

Encore merci a tous!

Note: oui j'ai bien mesuré ce que faisait la recurence de la fonction Choix_Trie : Si qq rentre une lettre innapropriée ça lui retourne un message d'erreur et ça lui demande de re rentrer soit 'a' soit 't'.

**Emmanuel Delahaye** · 27/11/2005, 15h15

Envoyé par BFH

Note: oui j'ai bien mesuré ce que faisait la recurence de la fonction Choix_Trie : Si qq rentre une lettre innapropriée ça lui retourne un message d'erreur et ça lui demande de re rentrer soit 'a' soit 't'.

Je te parles d'appel récursif, pas de récurrence... Personellement, je traite ce genre de problème en itératif. La récursion mal maitrisée, c'est dangereux... Il faut aussi voir quelle sera la valeur réellement retournée en cas d'erreur... La récursion, c'est une certaines façon de raisonner... Parfois, ça surprend...

Debbug pour débutant

C

Discussions similaires

Partager

Partager