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.

[Invité]

Pour être un petit peu plus clair si c'est possible
j'ai une autre fonction qui écrit sous forme de structure dans le fichier avec comme premier élément:

Code :

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

tab[i].indice=' '

Mon but avec la suppression n'est pas vraiment de supprimer mais de changer cette valeur par '*' afin qu'avec une autre fonction je puisse faire la différence entre les * et les espaces.
En gros quand il y a un espace la marque est présente dans le fichier donc je la lie quand il y a * la pièce ne doit pas être lu.

[Médinoc]

Je ne vois pas le rapport avec le fichier.
Normalement, quand on bosse sur un fichier, il y a deux écoles différentes:

1. Chaque opération agit directement sur le fichier (chargement/traitement/sauvegarde, copie-en-flux/renommage, etc.)
2. Le programme charge le fichier en mémoire, effectue les opérations en mémoire quand le demande l'utilisateur, puis sauvegarde toute la mémoire dans le fichier.

Et toi, dans ton opération de suppression, tu sembles tenter de faire un truc hybride: Tu as un tableau de marques en mémoire, mais ta suppression agit à la fois sur la mémoire et sur le fichier. C'est anormal.

[Invité]

Je pense que c'est cette notion que j'ai mal compris.
Mon but et juste de demander a l'utilisateur une marque,de vérifier si elle est présente et si oui remplace l'espace par un *.
Je vais continuer à cherché et SURTOUT à comprendre ce que je fais.
Merci en tout cas pour ton temps

[Sve@r]

Mon but et juste de demander a l'utilisateur une marque,de vérifier si elle est présente et si oui remplace l'espace par un *.

Bonjour
C'est ce qu'on appelle une suppression "logique". Tu places une marque associée à l'information "supprimée et ensuite, les fonctions d'affichage n'affichent pas l'information qui a une marque. C'est d'ailleurs ce qui se passe avec la "corbeille".
Avantages: rapidité dans la suppression, possibilité aussi de faire du "undelete"
Inconvénients: perte d'espace disque, possibilité d'avoir un taux "informations obsolètes/informations présentes" trop décalé, nécessité de programmer une fonction destinée à supprimer réellement les informations marquées (en bdd on nomme ça une "réorganisation" et Postgres l'appelle "vacuum").