Discussion :

[kochfet]

Bonjour,
J'ai codé le programme suivant qui saisie 5 produits et les affiche. J'ai un problème avec les deux fonctions de saisie et d'affichage : il ne fonctionne pas. SVP aidez-moi à trouver mes fautes et merci d'avance.

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
 
#include <stdio.h>
#include <stdlib.h>
 
struct Produit
{
    char nom[40];
    float prix;
};
void SaisieStock(struct Produit stock[], int n)
{
    int i;
    for(i=0; i<n; i++)
    {
        //printf("Produit numéro %d : \n",i);
        scanf("nom : %s",&stock[i].nom);
        scanf("prix : %f",&stock[i].prix);
    }
}
 
void affiche_Stock (struct Produit stock[], int n)
{
    int i;
    printf("Les produits sont :\n");
    for(i=0; i<n; i++)
    {
        printf("Produit %d : \n",i);
        printf("nom : %s \n",&stock[i].nom);
        printf("prix : %f \n",&stock[i].prix);
    }
}
 
int main()
{
    struct Produit stock[5];
    SaisieStock(stock,5);
    affiche_Stock(stock,5);
 
    system("PAUSE");
    return 0;
}

[gerald3d]

Code :

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

void SaisieStock(struct Produit stock[], int n)

Tu transmets à cette fonction, ainsi qu'à celle pour l'affichage, une copie de ton tableau. Il faut lui transmettre l'adresse en lieu et place si tu veux pouvoir récupérer dans ton main(); les informations saisies.

[kochfet]

Merci pour la réponse.

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
 
#include <stdio.h>
#include <stdlib.h>
 
struct Produit
{
       char nom[40];      
       float prix;
};
      void SaisieStock(struct Produit* stock[], int n)
      {    
           int i;
           for(i=0;i<n;i++)
           {
                printf("Produit numéro %d : \n",i);    
                printf("nom : ");       
                scanf("%s",&stock[i]->nom);  
                printf("prix : ");
                scanf("%f",&stock[i]->prix);
           }
       }
 
 
 
int main()
{
    struct Produit* stock[3];
    SaisieStock(stock,3);
 
 
  system("PAUSE");	
  return 0;
}

A l'exécution, juste après la saisie du nom du premier produit ,le message suivant s'affiche "Le programme a cessé de fonctionner"

[kochfet]

Merci pour la rapidité de la réponse.

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
 
#include <stdio.h>
#include <stdlib.h>
 
struct Produit
{
       char nom[40];      
       float prix;
};
      void SaisieStock(struct Produit* stock[], int n)
      {    
           int i;
           for(i=0;i<n;i++)
           {
                printf("Produit numéro %d : \n",i);    
                printf("nom : ");       
                scanf("%s",&stock[i]->nom);  
                printf("prix : ");
                scanf("%f",&stock[i]->prix);
           }
       }
 
 
 
int main()
{
    struct Produit* stock[3];
    SaisieStock(stock,3);
 
 
  system("PAUSE");	
  return 0;
}

A l'exécution, juste après la saisie du nom du premier produit ,le message suivant s'affiche "Le programme a cessé de fonctionner"

[gerald3d]

J'ai l'impression que tu ne maitrise pas totalement les pointeurs. Bon, c'est vrai, le sujet n'est pas forcément facilement assimilable.

Code :

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

void SaisieStock(struct Produit* stock[], int n)

Tu récupères ici l'adresse de l'adresse de ton tableau. Ce n'est pas ce que tu recherches. Pour t'aider un peu les[*] sont de trop .

[kochfet]

Oui c'st vrai, j'ai un problème avec les pointeurs. Mais maintenant j'ai un problème plus difficile : j'ai besoin de cet exercice dans une demi heure et je vois pas encore comment le corriger .

[Bktero]

Code :

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

void SaisieStock(struct Produit stock[], int n)

Tu transmets à cette fonction, ainsi qu'à celle pour l'affichage, une copie de ton tableau. Il faut lui transmettre l'adresse en lieu et place si tu veux pouvoir récupérer dans ton main(); les informations saisies.

Je ne suis pas sûr de comprendre cette phrase. C'est bien le tableau qui est passé (ou plus exactement l'adresse de son premier élément).

Il y a surtout un problème avec cette ligne qui émet un warning :

Code :

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

printf("nom : %s \n",&stock[i].nom);

Je pense que le & est en trop.
Il y a le même problème avec les printf(), les & sont en trop.

En les enlevant, le premier code semble fonctionner correctement.

[gerald3d]

Je ne suis pas sûr de comprendre cette phrase. C'est bien le tableau qui est passé (ou plus exactement l'adresse de son premier élément).

Il y a surtout un problème avec cette ligne qui émet un warning :

Code :

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

printf("nom : %s \n",&stock[i].nom);

Je pense que le & est en trop.
Il y a le même problème avec les printf(), les & sont en trop.

En les enlevant, le premier code semble fonctionner correctement.

Trop tôt pour moi ce matin .

Toutes mes excuses pour cette explication qui n'en est pas une . Pour la peine je plussoie aussi Bktero qui a relevé cette erreur impardonnable. J'ai bien mérite une

[Neckara]

Code :

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

void SaisieStock(struct Produit stock[], int n)

Tu transmets à cette fonction, ainsi qu'à celle pour l'affichage, une copie de ton tableau. Il faut lui transmettre l'adresse en lieu et place si tu veux pouvoir récupérer dans ton main(); les informations saisies.

Je plussoie Bktero, en C, les tableaux ne sont pas passé par copie mais par adresse.
Le tableau est implicitement convertit en un pointeur sur son premier élément.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
 
void foo(int tab[])
{
        tab[0] = 5;
}
 
int main(void)
{
        int tab[1] = {4};
        foo(tab);
        printf("%d\n", tab[0]);
        return 0;
}

Affiche 5 et non 4.