call a C function from another function !

Version imprimable

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:

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:

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.

28/07/2011, 15h29
Al_th
Un appel de fonction en C est très simple :
Code:

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
28/07/2011, 16h14
jeroman

Le premier élément d'un tableau a l'indice 0, pas 1.
28/07/2011, 18h56
souviron34

et donc il faut faire les boucles comme :

Code:

i < data->nbr_depots

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

Je me permet une remarque...>

Code:

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:

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.

29/07/2011, 10h50
Student_Master

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.
29/07/2011, 11h01
Al_th

Citation:

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.

Citation:

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.

Citation:

Je me permet une remarque...>

Citation:

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 :ccool:
29/07/2011, 11h01
souviron34

Citation:

Envoyé par Student_Master

Mon code marche très bien et sans aucune lacune !

C'est un pur hasard dû à ce que tu as d'autre en mémoire à ce moment, mais les remarques ci-dessus indiquent que ton code NE MARCHE PAS...

Citation:

Envoyé par Student_Master

mais j'ai posté ma question sur la façon comment je pourrai utiliser une fonction qui retourne une liste de clients

client **