Funkcja - Lokata bankowa

Question

Witam, mam takie o to zadanie:

 

Stworzyć funkcję wyznaczającą wartość lokaty (tablica dwunastu miesięcy) przy lokacie na sumę W z kapitalizacją odsetek (e%) w skali 1, 2, 3...10 miesięcy (parametr s). Lokatę można stworzyć tylko na jeden rok. Deklaracja funkcji: float kredyt(float W, float e, int s, int *stanLok). StanLok to tablica opisująca stan lokaty w kolejnych miesiącach

 

Udało mi się stworzyć takie coś, lecz program się wykrzacza:

#include <iostream>
#include <math.h>

using namespace std;

float W, e;
int s, stanLok[12];

float kredyt(float W, float e, int s, int stanLok[])
{
	if (s == 1)
		return W;

	else
		return W + e*W;
}

int main()
{
	cout << "Podaj sume pieniedzy na lokacie: ";
	cin >> W;
	cout << "Podaj procent lokaty: ";
	cin >> e;
	cout << "Podaj ilosc miesiecy lokaty (maksymalnie 12): ";
	cin >> s;
	if (s < 1 && s > 12)
	{
		cout << "Podales zla ilosc miesiecy!";
	}
	else
	{
		stanLok[s] = W + e*W;
		kredyt(W, e, s,stanLok);
	}

	for (int i = 0; i < s; i++)
		cout << "Stan lokaty dla " << i+1 << " miesiaca wynosi: " << stanLok << endl;


	system("pause");
	return 0;

}

 

Answer 1

W funkcji liczysz wartość tylko dla jednego miesiąca i zwracasz ten wynik przez `return` a masz w funkcji policzyć dla wszystkich miesięcy (od 1 do 12 - czyli potrzebna jakaś pętla) i wyniki zapisywac w stanLock.

---

Przy wypisuwaniu zapominasz, że stanLock to tablica więc masz wypisywać stanLock[ i ] a nie stanLock.

Comments

#include <iostream>
#include <math.h>

using namespace std;

float W, e;
int s, stanLok[12];

float kredyt(float W, float e, int s, int stanLok[])
{
	if (s == 1)
		return W;

	else
		return W+kredyt(e/100*W);
}

int main()
{
	cout << "Podaj sume pieniedzy na lokacie: ";
	cin >> W;
	cout << "Podaj procent lokaty: ";
	cin >> e;
	cout << "Podaj ilosc miesiecy lokaty (maksymalnie 12): ";
	cin >> s;
	if (s < 1 || s > 12)
	{
		cout << "Podales zla ilosc miesiecy!" << endl;
	}
	else
	{
		for (int i = 0; i < s; i++)
		{
			kredyt(W, e, s, stanLok);
			stanLok[i] = W*e / 100 + W;
			cout << "Stan lokaty dla " << i + 1 << " miesiaca wynosi: " << stanLok[i] << endl;
		}

	}
	system("pause");
	return 0;

}

Przerobiłem na takie coś, ale funkcja wymaga więcej argumentów i nie wiem jak wcisnąć je tam.

---

Prawdę mówiąc nic takiego nie zrobiłeś. 

Po to przekazujesz `stanLok` do `kredyt` aby w `kredyt` zapisywać wyniki od razu do `stanLok` a ty to wciąż robisz w `main`

Nie wiem po co robisz `return` jak i tak tej zwracanej informacji nie wykorzystujesz. Masz wyniki wrzucać od razu do `stanLok` a nie zwracać je przez `return`.

W `main` wykonujesz pętlę `for` a powinieneś ją wykonywac w `kredyt`. Masz w `main` wywołać `kredyt` tylko raz a nie wielokrotnie w pętli. A dopiero po wywołaniu `kredyt` będzie Ci potrzebna pętla `for` aby jedynie wypisywać wartości z `stanLok`.

W `main` masz pobrać dane od użytkownika, wywołać tylko raz `kredyt` i na koniec wypisać wyniki. Cała robota musi się odbyć wewnątrz `kredyt`.

---

http://ideone.com/hX2JEJ

---

ps. parametr s (czyli kapitalizacja odsetek w skali X miesięcy) nie oznacza na ile miesięcy jest otwarta lokata tylko co ile miesięcy są dopisywane odsetki. Możesz mieć lokatę na 12 miesięcy a odsetki mogą być dopisywane co miesiąc albo co 2 miesiące lub 3 miesiące, itd.