Discussion :

[faulk]

Bonjour à tous.
J'ai crée un petit programme de gestion de stock. Dans mon code j'ai une liste de produit. Produit est une structure :

Code :

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

1
2
3
4
5
6
7
 
struct produit
{
       char code[10];
       char libelle[50];
       double quantite ;
} ;

Liste est aussi une structure contenant un en entier et un tableau de produit:

Code :

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

1
2
3
4
5
struct liste
{
       produit t[nb_max];
       int nb;
};

Tout le monde suit pour l'intant
Ensuite je vais manipuler cette liste (ajout, suppression de produit, affichage de la liste,..)
Pour cela, je veux en debut de programme initialiser la liste à vide, cad nb=0, et sans produit, donc le tableau vide.
J'ai mis cette fonction que j'apelle en debut de programme dans mon main.

Code :

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

1
2
3
4
void init(liste l)
{
     l.nb=0;
}

Voila dans le main je fais donc juste un init ( l ) ; (j'ai declarer l juste avant).

Mon probleme c'est que lorque je vais afficher la liste des le debut du programme, celle-ci n'est pas vide. Elle est meme rempli, avec des caractère spéciaux, l'affichage (qui affiche tous les produits) fait meme planter le programme. Je ne comprend. Je pense que ca viens de l'intitialisation de la liste.
Voilà, j'èspère avoir été le plus clair possible. Merci de me dire ce qui cloche dans mon programme.

[Enayem]

Quand tu créé un tableau statique, ton compilateur ne fais que reserver de l'emplacement mémoire pour ton tableau, cette emplacement mémoire n'est pas mis à zero et peu tres bien contenir des valeurs.

Toujours initialiser les variables avant de les utiliser.

D'ailleurs vu que tu utilises le C++, je pense que tu devrais gerer ton projet avec des objets plutot que des structures, où la c'est dans le constructeur que tu poura initialiser tes tableau.

[faulk]

Ok, merci, c'est ce un peu que je pensais, mais comment l'initialiser a vide le tableau ?

[Matthieu Brucher]

Pourquoi ne pas utiliser std::vector<produit> qui fait exactement ce dont tu as besoin tout en étant éprouvé et fiable ?

[faulk]

Parceque que je ne connaissais pas, lol. Je vais essayer. Merci

[Matthieu Brucher]

D'ialleurs, ton init prend en argument une liste l par valeur, donc la vraie liste l ne sera jamais modifiée, et donc jamais initialisée

[Laurent Gomila]

Sinon pour répondre à ta question :

Code :

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

1
2
3
4
void init(liste l)
{
     l.nb=0;
}

Va travailler sur une copie de ta liste.

Pour travailler sur l'original il faut la passer par référence :

Code :

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

1
2
3
4
void init(liste& l)
{
     l.nb=0;
}

EDIT : oups grillé

[Matthieu Brucher]

ben non, tu donnes la solution, pas moi