Les fonctions et les structures

Fermé
driftman123 Messages postés 15 Date d'inscription vendredi 23 mars 2012 Statut Membre Dernière intervention 31 mai 2014 - Modifié par driftman123 le 12/02/2014 à 12:36
driftman123 Messages postés 15 Date d'inscription vendredi 23 mars 2012 Statut Membre Dernière intervention 31 mai 2014 - 12 févr. 2014 à 16:35
Bonsoir, bon voilà j'ai un projet de classe à réaliser qui fait appele à des fonction qui font :
- l'ajout d'un nouveau produit dans une structure nom_prod, qté.
- la recherche d'un produit.
- modification d'un produit.
et j'ai du mal à travailler avec les fonction :
#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
int fonction_ajout()
{
int i, n;
struct etud { char nom[30], prenom[30], date[10]; };
struct etud e[10];
printf("nbr:");
scanf("%d",&n);
for (i=1;i<=n;i++)
{
printf("name_per:"); scanf("%s",&e.nom[i]);
printf("sec_name:"); scanf("%s",&e.prenom[i]);
printf("date:"); scanf("%s",&e.date[i]);
}
}
int main()
{
int choice;
printf ("donnez choix:");
scanf("%d",&choice);
switch (choice): {
case 1:
fonction_ajout();
break;
}
return 0;
}

1 réponse

[Dal] Messages postés 6198 Date d'inscription mercredi 15 septembre 2004 Statut Contributeur Dernière intervention 13 décembre 2024 1 096
Modifié par [Dal] le 12/02/2014 à 14:57
Salut driftman123,

1.

Ne mets pas de & dans scanf pour les tableaux, ils sont déjà traités comme des pointeurs vers le premier élément du tableau, ou vers l'élément indiqué par l'indice.

Si ta fonction a pour prototyle
int fonction_ajout()
elle doit renvoyer un int. Si elle ne renvoie rien, c'est
void
que tu dois indiquer en type de retour. S'agissant des paramètres, tu devrais les spécifier explicitement (cf. la suite sur ce sujet).

Ce sont des étudiants ou des produits que tu traites.. tu es sûr que c'est ton code ?

Si tu n'utilises pas les paramètres de main, c'est
int main(void)
que tu devrais écrire.

Ta structure conditionnelle switch a une erreur de syntaxe, enlève les deux points à
switch (choice):
.

2.

Sur ta fonction.

En définissant
struct etud
et
struct etud e[10];
dans ta fonction, tu perdras automatiquement l'allocation de cet espace mémoire et la variable en quittant ta fonction.

Et du coup, lorsque tu voudras faire tes fonctions de modification ou de recherche, tu n'auras plus rien.

struct etud
devrait être définit en dehors de toute fonction, après tes include (ou dans un fichier .h).

Pour
struct etud e[10];
tu as plusieurs choix, par exemple :

- définir
struct etud e[10];
en dehors de toute fonction (mais c'est alors une variable globale, ce que certains considèrent comme devant être évité, si possible)

- définir
struct etud e[10];
dans main, et passer ton tableau en paramètre de la fonction, avec sa taille,

- définir un pointeur sur
struct etud
, allouer dynamiquement la mémoire nécessaire en fonction des besoins (avec malloc), passer le pointeur en paramètre de la fonction avec sa taille, puis libérer la mémoire lorsque tu n'en as plus besoin (avec free)


Dal
0
[Dal] Messages postés 6198 Date d'inscription mercredi 15 septembre 2004 Statut Contributeur Dernière intervention 13 décembre 2024 1 096
12 févr. 2014 à 14:59
aussi,
#include<stdlib.h>
et
#include<conio.h>
ne sont pas nécessaires dans ton code (et ce dernier, n'est pas un entête standard du C)
0
driftman123 Messages postés 15 Date d'inscription vendredi 23 mars 2012 Statut Membre Dernière intervention 31 mai 2014
12 févr. 2014 à 16:35
Est ce que c'est mieux comme ca ?
#include<conio.h>
#include<stdio.h>
int main(void)
{
int choice;
struct etud {
char nom[30], qte[30], date[10];
};
printf("Pour inserer un produit-----------------: 1\n");
printf("Pour supprimer un produit---------------: 2\n");
printf("Pour modifier les données d'un produit--: 3\n");
printf("Pour afficher un produit----------------: 4\n");
scanf("%d",&choice);
if (choice==1)
{
function_1();
getch();
}
void function_1(struct etud)
{
int i,n;

struct etud e[10];
printf("donnez n:");
scanf("%d",&n);
for (i=1;i<=n;i++)
{
printf("nom_prod:"); scanf("%s",&e[i].nom);
printf("qte:"); scanf("%s",&e[i].qte);
printf("date:"); scanf("%s",&e[i].date);
}
};
0