Discussion :

[Invité]

Bonjour à tous,je vais essayer d'être le plus clair possible
je suis en train de créer un programme de SGBD en langage C.J'ai donc crée une structure avec un paramètre indice qui me permet de savoir si une marque est disponible ou non.
Je travaille donc avec des fonctions,et là viens mon problème.
je demande à l'utilisateur qu'elle marque il veut supprimer, puis je souhaite comparer dans la basse de données ( un fichier texte ).
Si elle y est je place un * pour indiquer qu'elle n'est plus disponible.
Mon appel de fonction dans mon main est le suivant:

Code :

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

suppression_piece (&nb_piece,tab);

avec nb_piece=0
et ma fonction est la suivante:

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
void suppression_piece (int* adr_nb_casse, casse tab[])
{
    int i;
    char marque[N];
    FILE * f;
    f=fopen("D:\Documents and Settings\denicod\Desktop\SGBDpiece.txt","r");
    i=0;
    printf("Saisir le marque de la voiture a suprimer \n");
    scanf("%s",marque); 
 
    for(i=0;i < *adr_nb_casse;i++)
    {    if ((marque ==tab[i].marque)&& (tab[i].indice == ' ')) 
            {
                tab[i].indice='*';
             }  
 
          else
            {
                printf("Marque non présente dans la BD \n");
             }
    return;
}

Problème: le if n'est pas exécuter et je ne comprend pas pourquoi.
Merci d'avance pour votre aide

[Médinoc]

Tu ne peux pas comparer deux tableaux de caractères avec == (ça compare leurs adresses à la place).
Utilise strcmp().

PS: Tu dois aussi doubler tes backslashes, dans la chaîne de caractères.

[Invité]

Merci Médinoc
J'ai donc modifier comme suivant:

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
void suppression_piece (int* adr_nb_casse, casse tab[])
{
    int i,a,b;
    char marque[N];
    FILE * f;
    f=fopen("D:\Documents and Settings\denicod\Desktop\SGBDpiece.txt","a+");
    i=0;
    printf("Saisir le marque de la voiture a suprimer \n");
    scanf("%s",marque);
    for(i=0;i < *adr_nb_casse;i++)
 
    {
         a=strcmp(marque,tab[i].marque);
         b=strcmp(tab[i].indice,' ');
 
           if ((a==0)&(b==0))
            {
                tab[i].indice='*';
                fprintf(f,"%c",tab[i].indice);
                printf("suppresion reussi");
             }
 
          else
            {
                printf("Marque non présente dans la BD \n");
             }
    }
    return;
}

Mais alors là je ne rentre pas dans la boucle for
Un idée de mon erreur? je suis sur que c'est un petit truc...

[Médinoc]

• Ton scanf("%s", marque) nu est vachement dangereux, mais ça ne devrait pas empêcher d'entrer dans la boucle à moins que tu tapes plus de (N-1) caractères.
• Ton fopen() est toujours faux, mais ça ne devrait pas empêcher d'entrer dans la boucle non plus.
• Ton deuxième strcmp() ne devrait même pas compiler (ou au moins, pas compiler sans warning). strcmp() marche pour des tableaux de caractères, pas des caractères seuls.
• if ((a==0)&(b==0)) devrait en théorie utiliser && au lieu de &, mais en pratique ici ça ne devrait faire aucune différence.

[Invité]

Comme je suis débutant je suis très preneur de conseil afin d'avoir la meilleur programmation possible,donc merci pour tes conseils.
J'ai donc modifier mon programme celons tes remarques:

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
void suppression_piece (int* adr_nb_casse, casse tab[])
{
    int i,a;
    char marque[N];
    FILE * f;
    f=fopen("D:\\Documents and Settings\\denicod\\Desktop\\SGBDpiece.txt","r+");
    i=0;
    printf("Saisir le marque de la voiture a suprimer \n");
    fgets(marque,sizeof(marque),stdin);
 
 
    for(i=0;i<*adr_nb_casse;i++)
 
    {
         a=strcmp(marque,tab[i].marque);
 
           if ((a==0)&&(tab[i].indice==' '))
            {
                tab[i].indice='*';
                fprintf(f,"%c",tab[i].indice);
                printf("suppresion reussi");
             }
          else
            {
                printf("Marque non présente dans la BD \n");
             }
    }
    fclose(f);
    return;
}

[Médinoc]

Attention, contrairement à scanf(), fgets() enregistre le \n.
Il faut donc le supprimer d'abord.

Généralement, on fait des fonctions comme ça pour:

• Supprimer le \n du buffer s'il est là
• "lire" le reste de la ligne sinon (pour éviter de parasiter les saisies suivantes)

Code C :

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

1
2
3
    printf("Saisir le marque de la voiture a suprimer \n");
    fgets(marque, sizeof(marque), stdin);
    fclean(marque, stdin);

Pour le reste, je ne comprends pas ce que tu fais sur le fichier. Pour l'instant, ton code l'ouvre en lecture/écriture (et en mode texte), et modifie sur-place son premier caractère.