Discussion :

[lefevrelaumonier]

Bonjour,

Je suis débutant sur C++.
Je me lance dans mes premiers code et je crée une fonction cashFlowPvDiscrete qui prend en input 2 vecteurs, cflowTimes et cflowAmounts ainsi qu'une constante r.

Il semble que j'ai des soucis sur ma fonction le compilateur n'aboutit pas.

L'erreur est sans doute minime (ou pas) mais quelqu'un pourrait il me "corriger" mon erreur afin qu je puisse partir sur une base cohérente pour la suite de mon prog.

Merci à vous tous
Je promets de m'améliorer

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
 
int main()
{
 
double PV;
double cflowTimes[3]={1,2,4};
double cflowAmounts[3]={5,5,100};
double r = 0.05;
 
PV = cashFlowPvDiscrete(cflowTimes, cflowAmounts, r);
 
}
 
double cashFlowPvDiscrete(const vector<double>& cflowTimes,
                          const vector<double>& cflowAmounts,
						  const double& r)
 
{
double PV = 0;
for (int t=0; t <cflowTimes.size(); t++)
	{
		PV += cflowAmounts[t]/ pow(1+r,cflowTimes[t]);
	}
	return PV;
}

[mitkl]

Bonsoir,

ta fonction cashFlowPvDiscrete attend des : cashFlowPvDiscrete(const vector<double>& cflowTimes, const vector<double>& cflowAmounts, const double& r)

or, toi tu passes cflowTimes et cflowAmounts qui sont des double[3].

Pour corriger, tu peux simplement remplacer ces lignes :

Code :

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

1
2
vector<double> cflowTimes = {1,2,4};
vector<double> cflowAmounts = {5,5,100};

Si tu as un compilateur supportant les listes d'initialisation pour un vector ( vector x = {...} ). Sinon il faut faire à l'ancienne :

Code :

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

1
2
3
4
5
6
7
8
9
vector<double> cflowTimes;
cflowTimes.push_back(1);
cflowTimes.push_back(2);
cflowTimes.push_back(4);

vector<double> cflowAmounts;
cflowAmounts.push_back(5);
cflowAmounts.push_back(5);
cflowAmounts.push_back(100);

PS : n'oublie pas la balise [CODE] et les messages retournés par le compilateur la prochaine fois

[Neckara]

Si tu connais la taille des éléments à insérer, il vaut mieux faire vector<double> cflowTimes(nbElement); ou faire un réserve avant d'effectuer les push_back().

On va allouer une nouvelle zone mémoire à chaque push_back, copier toutes les valeurs stockées puis libérer l'ancienne zone mémoire.

[lefevrelaumonier]

wouhou ça fonctionne !
petit moment d'émotion c'est mon premier code c++ avec appel de fonction perso !

Merci à tous pour votre aide
Bonne journée