call a C function from another function !

**Student_Master** · 28/07/2011, 13h52

Bonjour à Tous,

J'aimerais savoir si vous pouvez m'aider à faire un appel à une fonction qui permet de retourner une liste de clients aléatoirement.

Fonction 1:

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
42
43
44
45
46
47
48
49
50
51
52
void  affect_client_to_days(JOURS *days, donnee *data, usine1 *usine, client *cust, int TotDemCust)
{
    int j=0;
    int t;
    int jr;
    int dem;
    int it=0;
    int nbr_c=1;
    srand(time(NULL));
    int nombre_aleatoire = 0;
 
    for (jr=1; jr<=data->nbr_periodes; jr++)
        {                 
                   days[jr].TotDemCust = 0;
                   days[jr].nbr_clients= 0;
                   while (((usine->dem_jour[jr]+usine->stock[jr-1]) > TotDemCust) && (it < data->nbr_clients))
                   {
                         nombre_aleatoire = rand()%data->nbr_clients+1;
                         j=nombre_aleatoire;
                         if((cust[j].Nbvisites < cust[j].frequence)&& (days[jr].cust[j]==0)) 
                                  {
                                          days[jr].nbr_clients++;
                                          cust[j].Nbvisites++;
                                          days[jr].cust[j]=1;
                                          days[jr].listcust[days[jr].nbr_clients]=j;
                                          days[jr].TotDemCust+=cust[j].demande;
                                          printf("affecter le client numero %d a la periode %d\n",nombre_aleatoire, jr);
                                  }
 
                   it++;
                   }
                   it=0;
                        if(usine->dem_jour[jr] > days[jr].TotDemCust)
                                  {
                                          j=1;
                                          while ((j <= data->nbr_clients) || (usine->dem_jour[jr] > days[jr].TotDemCust))
                                          {
                                              if((cust[j].Nbvisites < cust[j].frequence)&& (days[jr].cust[j]==0))
                                              {
                                                  days[jr].nbr_clients++;
                                                  cust[j].Nbvisites++;
                                                  days[jr].cust[j]=1;
                                                  days[jr].listcust[days[jr].nbr_clients]=j;
                                                  days[jr].TotDemCust+=cust[j].demande;
                                              }
                                              j++;
                                          }
                                   }
         }
         usine->stock[jr]=usine->stock[jr-1]+days[jr].TotDemCust-usine->dem_jour[jr];
         it=0;
}

J'ai besoin maintenant d'utiliser cette liste de clients dans une autre fonction qui permet de les classer par ordre décroissant en fonction de leur demande (sachant que les quantités de demandes se trouvent dans un fichier texte.

Fonction 2 :

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 rechercher_max_quantite(donnee *data, client *cust)
{                     
    int j = 1;
    int i;
    double max;
    int quantite, num_client;
    for (i=1; i<=data->nbr_depots; i++) 
    {
        max = cust[i].demande;
		for (j = data->nbr_depots; j <= data->nbr_clients + data->nbr_depots; j++)
		{
            if (cust[i].demande < max)
            {
               max = cust[i].demande;
               quantite = i;
               num_client = j - data->nbr_depots;
            }
        }
    }
        if((j - data->nbr_depots - 1) == data->nbr_clients)
             printf("Le client %d a la quantite suivante %d\n", num_client, max);
}

Je ne sais pas c'est quoi la syntaxe qui permet de faire appel d'une fonction et l'utiliser dans une autre fonction.

Merci pour vos remarques et vos réponses.
Cdls.

**Al_th** · 28/07/2011, 15h29

Un appel de fonction en C est très simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int fctA(int a, intb){
    //dosomestuff
    return val;
}
 
void fctB(){
    int nb1=0;
    int nb2=0;
    int RetourDeFonctionA = fctA(nb1,nb2);
}

Tu appelles ta fonction avec son nom, et en paramètre des valeurs que tu veux passer. Si la fonction a un retour, tu peux l'affecter quelque part.

Edit : De ce que je vois, aucune de tes deux fonctions n'ont de valeur de retour, j'imagine donc que tu passes ta liste en pointeur, et qu'elle est modifiée comme cela

**jeroman** · 28/07/2011, 16h14

Le premier élément d'un tableau a l'indice 0, pas 1.

**souviron34** · 28/07/2011, 18h56

et donc il faut faire les boucles comme :

Code :

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

i < data->nbr_depots

**Vodsky** · 29/07/2011, 09h50

Bonjour,
ton code c'est un peu une usine a gaz

!

Je me permet une remarque...>

Code :

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

srand(time(NULL));

ne doit en general n'etre appeler qu'une seul fois dans ton programme, et je doute que tu appel cette fonction (affect_client_to_days()) qu'une seule fois

EDIT:

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
for (jr=1; jr<=data->nbr_periodes; jr++) //<<<<<<< jr = 1; !!!
        {                 
                   days[jr].TotDemCust = 0; // <<<<
                   days[jr].nbr_clients= 0; //<<<< days[1] = 0; days[0] = ?;
                   while (((usine->dem_jour[jr]+usine->stock[jr-1]) > TotDemCust) && (it < data->nbr_clients))
                   {
                         nombre_aleatoire = rand()%data->nbr_clients+1;
                         j=nombre_aleatoire;
                         if((cust[j].Nbvisites < cust[j].frequence)&& (days[jr].cust[j]==0)) 
                                  {
                                          days[jr].nbr_clients++; //<<< days[0] = ? + 1;
                                          cust[j].Nbvisites++;
                                          days[jr].cust[j]=1;
                                          days[jr].listcust[days[jr].nbr_clients]=j;
                                          days[jr].TotDemCust+=cust[j].demande; // <<< days[0] += (du_caca_la_encore)
                                          printf("affecter le client numero %d a la periode %d\n",nombre_aleatoire, jr);
                                  }
 
                   it++;
                   }
                   it=0;
                        if(usine->dem_jour[jr] > days[jr].TotDemCust) //<<< la encore tu pari sur la chance pour l'index 0
                                  {
//...

Comme le précise jeroman l'index de tes tableaux commence a 0. ton code est boiteux ici, tu risque de ne pas avoir le comportement attendu pour le premier tour.

**Student_Master** · 29/07/2011, 10h50

Bonjour,

Mon code marche très bien et sans aucune lacune ! je suis ravi de voir vos remarques mais j'ai posté ma question sur la façon comment je pourrai utiliser une fonction qui retourne une liste de clients et les trier dans une autre fonction en fonction de leur demandes c'est tout.

Cdls.

**Al_th** · 29/07/2011, 11h01

Mon code marche très bien et sans aucune lacune ! je suis ravi de voir vos remarques mais j'ai posté ma question sur la façon comment je pourrai utiliser une fonction qui retourne une liste de clients et les trier dans une autre fonction en fonction de leur demandes c'est tout.

Le premier élément d'un tableau a l'indice 0, pas 1.

Si tu écris/lis des cases 1 a n+1 d'un tableau, ton code peut marcher "sans lacunes aucunes". Mais la derniere case de ton tableau n'est pas "référencée" comme faisant partie de ton tableau. Donc tu t'exposes a ce que cette case soit modifiée a ton insu sans que tu ne le sache.

Je me permet une remarque...>

srand(time(NULL));

ne doit en general n'etre appeler qu'une seul fois dans ton programme, et je doute que tu appel cette fonction (affect_client_to_days()) qu'une seule fois

Là encore, ca n'est pas une "erreur" a proprement parler qui ferait planter ton code, mais je t'invite a lire ce sujet, qui explique pourquoi il ne faut pas appeler srand plusieurs fois.
http://www.developpez.net/forums/d11...nombre-dappel/

Quand a ta question, le premier post y répond