Discussion :

[superspike23]

bonjour a tous.
J essaye de m intitier au c++ mais ca commence bien mal.
j ai ecrit un petit code source mais pleins d erreur que je ne comprends pas.
Si quelqu un peut m indiquer quelles erreurs je fais ca me fera progresser,merci d avance.

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
#include "stdafx.h"
#include <iostream>
class equa2
{
public:
	equa2(float a,float b,float c);
	~equa2();
	float lire(float) const;
	float AfficherDelta() const;
	float CalculerDelta(float sonA,float sonB,float sonC);
	float CalculerS1(float sonA,float sonB,float sonC,float sonDelta);
	float CalculerS2(float sonA,float sonB,float sonC,float sonDelta);
	float AfficherS1() const;
	float AfficherS2() const;
private:
	float sonA,sonB,sonC,sonDelta,sonS1,sonS2;
}
;
 
equa2::equa2(float a,float b,float c) {
	sonA=a;
	sonB=b;
	sonC=c;
}
equa2::~equa2() {}
float equa2::CalculerDelta(float sonA,float sonB,float sonC )
{
	float sonDelta;
	sonDelta=(sonB*sonB)- (4*sonA*sonC);
return sonDelta;
}
float equa2::AfficherDelta()
{
	std::cout << "/ndelta=" << sonDelta << "/n";
}
equa2::AfficherS1()
{
	if(sonDelta>=0)
	{
 
	std::cout << "/,S1="<<sonS1<<"/n";
	};
	if(sonDelta<0)
	{
		std::cout << "/,S1="<<sonS1<<"i/n";
	};
 
}
equa2::AfficherS2()
{
if(sonDelta>=0)
	{
 
    std::cout << "/,S1="<<sonS2<<"/n";
};
if(sonDelta<0)
 
{
std::cout << "/,S1="<<sonS2<<"i/n";
};
}
sonDelta=CalculerDelta(); 
 
sonS1=CalculerS1(sonA,sonB,sonC,sonDelta);
 
sonS2=CalculerS2(sonA,sonB,sonC,sonDelta);
 
equa2::CalculerS1(float sonA,sonB,sonC,sonDelta)
{
	Float sonS1;
sonS1=(((-sonB)-(sqrt(sonDelta)))/(2*sonA));
return sonS1;
}
 
 
 
equa2::CalculerS2(float sonA,sonB,sonC,sonDelta)
{Float sonS2;
sonS2=(((-sonB)-sqrt(sonDelta))/(2*sonA));
return sonS2;
}
 
int _tmain(int argc, _TCHAR* argv[])
{
	return 0;
}

balises [code] rajoutées par r0d. Merci d'y penser la prochaine fois.

[r0d]

Bonjour,

-> Tu devrais séparer le .h et le .cpp. Dans le .h, tu ne met que la déclaration de la classe, dans le .cpp, tu met l'implémentation des fonctions
-> Vérifie bien que les signatures de tes fonctions sont identiques dans la déclaration et dans l'implémentation. Par exemple, ce n'est pas le cas pour la fonction AfficherS1()
-> Lorsque tu écris:

Code :

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

float UneFonction();

ça signifie que cette fonction retourne une valeur. Un flottant en l'occurence. Il faut donc que, dans l'implémentation, ta fonction retourne un flottant.
-> Ensuite, je te conseille de séparer la classe que tu va utiliser, et la fonction main. Le mieux, c'est de mettre la définition de equa2 dans un en-tête (equa2.h par exemple) et son implémentation dans un fichier cpp (equa2.cpp). Ensuite tu fais un fichier main.cpp, dans lequel tu met ta fonction main.

Il y a encore pas mal de trucs, mais fais déjà ça, ce sera un bon début

[poukill]

il y a un ";" après un if de trop aussi !

[koala01]

Salut,

Enfin, et même si je n'ai regardé que la déclaration, les méthodes

Code :

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

1
2
3
float CalculerDelta(float sonA,float sonB,float sonC);
float CalculerS1(float sonA,float sonB,float sonC,float sonDelta);
float CalculerS2(float sonA,float sonB,float sonC,float sonDelta);

ont de quoi me surprendre assez bien, et sont clairement de nature à poser problème...

En effet, le but d'une classe, c'est, en définitive, de regrouper les différentes données - au même titre que ce qui se fait avec les struct du C - et les fonctions "qui vont bien" pour ces données.

Or, il faut savoir que, c'est un problème inhérent aux fonctions, quand tu fournit des arguments dont le nom est le même que celui d'un membre de la classe, l'argument va "cacher" le membre en créant une variable locale à la fonction.

Du coup, quand tu va implémenter ces fonctions, cela te donnera quelque chose sous la forme de

Code :

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

1
2
3
4
5
6
7
8
float equa:: CalculerDelta(float sonA,float sonB,float sonC)
{
   /* parle-t-on de equa::sonA ou du sonA fournit en argument?
    * en fait, on parle du sonA fournit en argument, et on ne changera
    * rien à equa::sonA
    */
   sonA=/*...*/
}

Et je ne suis même pas sûr bien que je puisse me tromper là dessus, que this->sonA puisse faire la différence

En plus, pourquoi fournir des valeurs qui correspondent séparément, justement, aux membres de la classe, au lieu de fournir, carrément, un référence, éventuellement constante, sur la classe (ce qui permettrait d'éviter le problème soulevé plus haut)

[superspike23]

J'ai reecris tout le code en essayant de faire quelque chose de plus propre.
Voila ce que ca donne.

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
53
54
55
56
57
58
59
60
61
62
#include <iostream>
#include <math.h>
 
class equa2
{
public:
	equa2(float a,float b,float c);
	~equa2();
	void AfficherSolution();
 
private:
	float mA,mB,mC,mDelta,mS1,mS2;
 
}
;
equa2::equa2(float a,float b, float c)
{
	mA=a;
	mB=b;
	mC=c;
}
equa2::~equa2() {}
 
void equa2::AfficherSolution()
{
	mDelta= (mB*mB)-(4*mA*mC);
	std::cout << "\n\tDelta="<<mDelta<<"\n";
 
	if (mDelta >=0) 
	{
		mS1= (-mB-(sqrt(mDelta)))/(2*mA);
	    mS2= (-mB+(sqrt(mDelta)))/(2*mA);
		std::cout<<"solution1="<<mS1<<"\tSolution2="<<mS2;
	}
	else 
	{
		mS1= (-mB-(sqrt(-mDelta)))/(2*mA);
	    mS2= (-mB+(sqrt(-mDelta)))/(2*mA);
		std::cout<<"solution1="<<mS1<<"i\tSolution2="<<mS2<<"i\n\n";
	}
 
}
 
 
 
int main()
{
	std::cout<<"Veuillez saisir des valeurs pour a,b et c\n";
	float a,b,c;
	std::cout<<"a= \t";
	std::cin>>a;
    std::cout<<"b= \t";
	std::cin>>b;
	std::cout<<"c= \t";
	std::cin>>c;
 
	equa2 monequa(a,b,c);
	monequa.AfficherSolution();
 
 
	return 0;
}

Maintenant je vais essayer de separer la partie interface utilisateur (cin/cout) de main comme ca je pourra essayer de l adapter a une interface graphique

[Médinoc]

Selon moi, AfficherSolution() devrait être const et ne modifier aucune variable de la classe (utiliser des variables locales aux fonctions).
...Ou alors, renommer la fonction en CalculerSolution().