problème appel de fonction

Version imprimable

1 pièce(s) jointe(s)

Bonjour à tous ,j'ai créer un programme permettant de demander combien l'utilisiateur veut entrer d'entiers et le pgm en fait la somme.
Mais comme je ne comprends pas bien les appels de fonction ,j'ai voulu essayer.
Mais le programme ne marche pas.
Pouvez-vous me dire ce qui cloche svp ??
Merci à tous.

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
 
 
#include<iostream>
#include<cstdlib>
using namespace std;
int somme()
{   
           int nb,d;
           int som=0;
           cout << "entrer une entier" << endl;
           cin>>nb;
           som=som+nb;
           cout << "la somme est " << som << endl;
           cin >> d;
}
 
 
int main()
{ 
           int nb,t;
           int n;
           int cpt=1;
           cout << "entrer d'entiers coulez vous entrer ? " << endl;
           cin >> n;
           cout << "enter un entier " << endl;
           cin >> nb;
                while(cpt<n)
                {           cpt=cpt+1; 
                            cout << "entrer un autre entier " << endl;
                            cin >> nb;
                            cout << "la somme est "<< somme() << endl;
                            }
                            return EXIT_SUCCESS;
}

A chaque appel de somme (), la somme est réinitialisé à 0 (som = 0). Il faudrait au moins passer som en argument de somme pour que l'addition ait réellement lieu.

Légères modifications... (mais il y aurait d'autres choses à changer)

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
#include<iostream>
#include<cstdlib>
using namespace std;
int somme(int som)
{   
           int nb,d;
           // int som=0;
           cout << "entrer une entier" << endl;
           cin>>nb;
           som=som+nb;
           // fait doublon avec main ()
           // cout << "la somme est " << som << endl;
           cin >> d; // Je n'en vois pas l'intérêt. Faire une pause ?
           return som;
}
    
 
int main()
{ 
           int nb,t;
           int n;
           int cpt=1;
           int som = 0;
           cout << "entrer d'entiers coulez vous entrer ? " << endl;
           cin >> n;
           // On ne fait rien de nb pas la suite.
           // cout << "enter un entier " << endl;
           // cin >> nb;
                while(cpt<n)
                {           cpt=cpt+1; 
                            // Inutile
                            // cout << "entrer un autre entier " << endl;
                            // cin >> nb;
                            som = somme (som);
                            cout << "la somme est "<< som << endl;
                            }
                            return EXIT_SUCCESS;
}

En C++, je dirais plutôt un appel de ce genre, avec un passage par référence:

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
#include<iostream>
#include<cstdlib>
using namespace std;
 
// Le "int &" dans le prototype signale un passage par référence.
void somme(int &som)
{   
	int nb;
	cout << "Entrer un entier :" << endl;
	cin >> nb;
	som += nb;
	//J'ai viré le reste en-dessous
}
 
 
int main(void)
{ 
	int n;
	int cpt=1;
	int som = 0; //som DOIT être initialisé à zéro ici
	cout << "Combien d'entiers voulez-vous entrer ? " << endl;
	cin >> n;
 
	while(cpt<n)
	{
		cpt++; 
 
		somme(som); //som sera modifié par le passage par référence
		cout << "la somme est "<< som << endl;
	}
	return EXIT_SUCCESS;
}

Mais bon, il est possible que pepsi33 n'ait pas encore vu les différents modes de passage de paramètre en C++...

03/10/2007, 22h38
aoyou

Citation:

Envoyé par Médinoc

Mais bon, il est possible que pepsi33 n'ait pas encore vu les différents modes de passage de paramètre en C++...

C'est bien possible...
04/10/2007, 16h03
pepsi33

Effectivement ,je n'ai pas encore tout vu.
Mais merci beaucoup quand mème car je comprend déjà beaucoup mieux mème si il me reste encore beaucoup de boulot.
Encore merci.

Rebonjour ,j'ai encore une petite question (désolé).
En fait ,je dois créer un programme qui affiche un menu et qui demande à l'utilisateur de choisir l'opération choisie!En suite ,il affiche une confirmation.
J'ai utilisée un appel de fonction mais ,encore une fois ,cela ne marche pas.Ca affiche tout le temps "vous avez choisi l'exercice sur la soustraction".
Qu'est-ce qui cloche svp ??

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
 
#include<iostream>
#include<cstdlib>
using namespace std;
 
void menu(int b)
{
    cout<<"1.Exercices sur l'addition"<<endl;
    cout<<"2.Exercices sur la soustraction"<<endl;
    cout<<"3.Exerices sur la multiplication"<<endl;
    cout<<"choississez votre exo et taper le chiffre correspondant "<<endl;
    cin>>b;
 
}
 
 
 
int main(void)
{
    int b,a;
    menu(b);
    if(b=2)
        {   cout<<"vous avez choisi l'exercice sur la soustraction"<<endl;
         }
    else 
        if(b=1)
          {   cout<<"vous avez choisi l'exercice sur l'addition"<<endl;
           }
         else
           if(b=3)
           {  cout<<"vous avez choisi l'exercice sur la multiplication"<<endl;
            }
    return EXIT_SUCCESS;
}

04/10/2007, 16h28
BainE

Bonjour,

c'est normal, ton parametre de fonction est ne peux etre modifié ! (je parle de b). C'est comme ca, les parametres d une fonction retrouve leur valeur d origine a la fin de l execution de la fonction? (pour plus d info je t invite a lire la doc du site)

il faut que ta fonction retourne le resultat, ou bien passé par un pointeur ou une reference.

PS : pas sur que ce soit pertinant de passer une reference a la place d un int, car les 2 doivent faire la taille d un registre non ? donc pas de gain en performance.

C'est bon j'ai trouvé avec switch!Ca sonne ça :

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
 
 
#include<iostream>
#include<cstdlib>
using namespace std;
 
void menu(int b)
{
    cout<<"1.Exercices sur la soustraction"<<endl;
    cout<<"2.Exercices sur l'addition"<<endl;
    cout<<"3.Exerices sur la multiplication"<<endl;
    cout<<"choississez votre exo et taper le chiffre correspondant "<<endl;
    cin>>b;
 
}
 
 
 
int main()
{
    int b,a,p;
    menu(b);
    switch(b)
    {
    case 1 :cout<<"vous avez choisi l'exercice sur la soustraction"<<endl;
    break;
    case 2 :cout<<"vous avez choisi l'exercice sur l'addition"<<endl;
    break;
    case 3 :cout<<"vous avez choisi l'exercice sur la multiplication"<<endl;
    break;
}
    cin>>p;
}

04/10/2007, 16h40
pepsi33

Ah non ,ce programme ne marche pas :il affiche tout le temps "vous avez choisi l'exercice sur l'addition"!
Pourquoi ?
Merci beacoup à tous de m'aider.
04/10/2007, 16h41
yan
Bonjour
Code:

1 2 void menu(int b)
Avec cette méthode, b ne sera pas modifié.
Il faut soit

Code:

void menu(int &b)

soit

Code:

int menu();
04/10/2007, 16h48
pepsi33

Merci ,j'ai trouvé!Mème problème pour switch (int &b)!
Merci beaucoup à tous.
04/10/2007, 16h50
yan
Citation:

Envoyé par pepsi33

Merci ,j'ai trouvé!Mème problème pour switch (int &b)!
Merci beaucoup à tous.

personnellement je préfère la seconde solution
Code:

1 2 int menu()
Qui va te renvoyer le code du menu choisie

Voila j'ai fini mon programme mais j'ai un autre soucis :Le programme affiche tout le temps 0 et 5 pour a et c alors qu'il devrait générer deux chiffre entre 0 et 10 !!
Ou est le problème SVP ?
Merci beaucoup.

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
 
#include<iostream>
#include<cstdlib>
using namespace std;
 
void menu(int &b)
{
    cout<<"1.Exercices sur l'addition"<<endl;            /*On affiche ici les différents choix possibles du menu*/
    cout<<"2.Exercices sur la soustraction"<<endl;
    cout<<"3.Exerices sur la multiplication"<<endl;
    cout<<"choississez votre exo et taper le chiffre correspondant "<<endl;
    cin>>b;                                              /*On demande à l'utilisateur de choisir son exercice*/
 
}
 
int genOperande(int &max)
{
    max=(int )(10.0*rand()/RAND_MAX);     /*cette fonction retourne un entier compris entre 0 et 10*/
}
 
 
 
int main()    /*Fonction principale*/
{
    int x,p,a,c,d;
    menu(x);
    switch(x)                             /*Equivalent de Selon-Que en Exalgo*/
    {
    case 1:cout << " vous avez choisi l'exercice sur l'addition " << endl;
    genOperande(a);
    genOperande(c);
    cout <<" a = "<< a <<" et c = "<< c<< endl;
    cout<<"combien font a+c"<<endl;
    cin>>d;
    while(d!=a+c)
    {cout<<"Perdu"<<endl;
    cout<<"retente ta chance "<<endl;
    cin>>d;}
    cout<<"Bravo"<<endl;
    break;
    case 2:cout << " Vous avez choisi l'exercice sur la soustraction " << endl;
    genOperande(a);
    genOperande(c);
    cout <<" a = "<< a <<" et c = "<< c<< endl;
    cout<<"combien font a-c"<<endl;
    cin>>d;
    while(d!=a-c)
    {cout<<"Perdu"<<endl;
    cout<<"retente ta chance "<<endl;
    cin>>d;}
    cout<<"Bravo"<<endl;
    break;
    case 3:cout << " Vous avez choisi l'exercice sur la la multiplication " << endl;
    genOperande(a);
    genOperande(c);
    cout <<" a = "<< a <<" et c = "<< c<< endl;
    cout<<"combien font a*c"<<endl;
    cin>>d;
    while(d!=a*c)
    {cout<<"Perdu"<<endl;
    cout<<"retente ta chance "<<endl;
    cin>>d;}
    cout<<"Bravo"<<endl;
    break;
    default :cout << " Ce chiffre ne fait pas partie du menu " << endl;  /*C'est-à-dire si x est dif de 1,2 ou 3*/
    }
}

04/10/2007, 17h34
Médinoc

Il doit y avoir une entrée dans la FAQ sur le fait qu'il faille initialiser rand() une fois par processus...

Tu peux utiliser la fonction srand() au début du programme pour initialiser rand(). Tu peux demander la valeur de la "graine" à l'utilisateur, ou si tu veux vraiment être imprévisible, lui passer la valeur retournée par time()...