Erreur - tableau de structure

Salut,

voile je voudrais transmettre un tableau de structure en paramètre et l'initialiser:

Ma structure + la prototype de la fonction:

Code:

---

1 2 3 4 5 6 7

typedef struct produit produit; struct produit{ char libelle[30]; long stock; };   long recupListeProduit(produit* tab_produit);

---

L'appel de ma fonction:

Code:

---

1 2	produit tab_produit[50]; nb_produit=recupListeProduit(tab_produit);

---

La fonction en elle même:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

long recupListeProduit(produit* tab_produit){   long nb_produit=0,position_deb=0,position_fin=0; FILE *fichier_produits=NULL; produit un_produit;   fread(&un_produit,sizeof(produit),1,fichier_produits); while(feof(fichier_produits)==0){ tab_produit[nb_produit].libelle=un_produit.libelle; tab_produit[nb_produit].stock=un_produit.stock; nb_produit++; fread(&un_produit,sizeof(produit),1,fichier_produits); } }

---

Code block me met l'erreur:

Citation:

---

C:\CodeBlocks\phyto_stock\gest_coop\produit.c||In function `recupListeProduit':|
C:\CodeBlocks\phyto_stock\gest_coop\produit.c|86|error: incompatible types in assignment|
||=== Build finished: 1 errors, 0 warnings ===|

---

C'est la ligne --> tab_produit[nb_produit].libelle=un_produit.libelle;
Si je la met en commentaire, il ne met plus d'erreur.

Merci de m'aider c'est très important.

Depuis quand on fait une affectation de chaine de caractère comme cela :

Code:

---

tab_produit[nb_produit].libelle=un_produit.libelle;

---

Pour copier une chaîne de caractères : strcpy() et strncpy()

Citation:

---

Envoyé par vicenzo

Pour copier une chaîne de caractères : strcpy() et strncpy()

---

Il y a aussi strcat et strncat. Mais les extensions BSDs sont à préférer!

Citation:

---

Envoyé par nicolas.sitbon

Il y a aussi strcat et strncat. Mais les extensions BSDs sont à préférer!

---

copie et concaténation sont deux choses différentes.

Dans le cas de ce post, il s'agit d'une affectation, donc une copie, et donc les fonctions à utiliser sont strcpy() ou strncpy() et non strcat() ou strncat().

De plus dans le cadre du code posté, strcat() aurait un effet désastreux puisque que la chaine de destination n'est pas encore initialisée !

Citation:

---

Envoyé par vicenzo

copie et concaténation sont deux choses différentes.

Dans le cas de ce post, il s'agit d'une affectation, donc une copie, et donc les fonctions à utiliser sont strcpy() ou strncpy() et non strcat() ou strncat().

De plus dans le cadre du code posté, strcat() aurait un effet désastreux puisque que la chaine de destination n'est pas encore initialisée !

---

non non, je persiste et signe :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

char buffer[10]; strcpy (buffer, "chaine");   char buffer2[10]; buffer2[0] = 0; strcat (buffer, "chaine");   char buffer3[10]; strncpy (buffer, "chaine trop longue", 9); buffer[sizeof buffer - 1] = 0;   char buffer4[10]; buffer[0] = 0; strncat (buffer, "chaine trop longue", 9);

---

Citation:

---

Envoyé par vicenzo

De plus dans le cadre du code posté, strcat() aurait un effet désastreux puisque que la chaine de destination n'est pas encore initialisée !

---

strcat ou strcpy sont 2 fonctions dangereuses que personnelement je n'utilise jamais. Il n'y en a pas une meilleure que l'autre.

Enfin avec le code que j'ai posté précédemment on s'apperçoit que cela revient au même de faire un strncpy ou un strncat.

Citation:

---

Envoyé par nicolas.sitbon

Enfin avec le code que j'ai posté précédemment on s'apperçoit que cela revient au même de faire un strncpy ou un strncat.

---

Tu initialises ta chaine de destination avant de faire le strcat... Pourquoi faire :

Code:

---

1 2	buffer[0] = 0; strncat(buffer, "chaine", 6);

---

alors qu'un simple

Code:

---

strncpy(buffer, "chaine", 6);

---

suffit. Pour moi, le strcat()/strncat() est hors de propos du post initial.

Citation:

---

Envoyé par nicolas.sitbon

strcat ou strcpy sont 2 fonctions dangereuses que personnelement je n'utilise jamais. Il n'y en a pas une meilleure que l'autre.

---

La, je suis d'accord avec toi, strcpy() et strcat() sont dangereuses et il faut leur préférer leur version strncpy() et strncat() qui ajoute un contrôle.

Mais pourquoi coder 2 lignes alors tu peux n'en faire qu'une...

ton exemple marche seulement pour des chaines plus petites que le buffer mais voyons ce qui se passe pour une taille plus grande :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include <stdio.h> #include <string.h>   int main (void) { char buffer[10]; strncpy (buffer, "chaine trop longue", 9); puts (buffer);   return 0; }

---

affiche :

Code:

---

chaine tr���8 ��P$�������8 ��P$��

---

le seul moyen de le faire proprement est :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#include <stdio.h> #include <string.h>   int main (void) { char buffer[10]; strncpy (buffer, "chaine trop longue", 9); buffer[sizeof buffer - 1] = 0; puts (buffer);   return 0; }

---

affiche

Code:

---

chaine tr

---

le même code avec strncat :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#include <stdio.h> #include <string.h>   int main (void) { char buffer[10]; buffer[0] = 0; strncat (buffer, "chaine trop longue", 9); puts (buffer);   return 0; }

---

affiche bien

Code:

---

chaine tr

---

on voit même que c'est plus simple puisque qu'il n'y a pas à calculer la position du dernier caractère.