jak odwolac sie do zmiennych w funckji?

Jak w temacie, mam taki o to program ale w instrukcji switch musiałem dać stałe aby program "działał". Jak to zrobić, żeby użytkownik mógł sam sobie wybierać jakie chce liczby? Próbowałem zadeklarować dodatkowe zmienne a,b i tam dać ale kompilator pokazywał błąd.

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


using namespace std;

int dodaj(int x, int y);
int odejmij(int x, int y);
int pomnoz(int x, int y);
int podziel(int x, int y);

int main()
{
	int wybor;
	int p;

	cout << "program imitujacy kalkulator, wybierz dzialanie:\n1.dodawanie: \n2.odejmowanie: \n3.mnozenie: \n4.dzielenie: ";
	cout << endl;
	cin >> wybor;
		
		switch (wybor)
		{
		case 1: {dodaj(3, 1); break; }
		case 2: {odejmij(11, 7); break; }
		case 3: {pomnoz(1, 3); break; }
		case 4: {podziel(22, 6); break; }
		}

	
		
	return 0;
}

int dodaj(int x, int y)
{
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	int p = x + y;
	return p;
}

int odejmij(int x, int y)
{
	int p;
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	p = x - y;
	return p;
}

int pomnoz(int x, int y)
{
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	return x * y;
}

int podziel(int x, int y)
{
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	if (y != 0)
	{

		return x / y;
	}
	else
	{
		cout << "nie dziel przez 0!\n";
	}
	return 0;
}


 

Jeśli dobrze zrozumiałem, o co Ci chodzi:

// ...
cin >> wybor;
int x, y;
cout << "podaj pierwszą liczbę" << endl;
cin >> x;
cout << "podaj drugą liczbę" << endl;
cin >> y;
switch(wybor)
{
  case 1: {cout << "wynik: " << dodaj(x, y) << endl; break;}
//...

Jeśli kompilator Ci czegos nie przyjmuje, to wklej kod, którego nie chce przyjąć, wraz z błędem, będzie prościej.

// w switch'u pod odpowiednim case przykładowo.
std::cout << "wynik dodawania to: " << dodaj() << std::endl;

// a funkcja dodaj.
int dodaj()
{
    int x = 0;
    int y = 0;
    cout << "podaj pierwsza liczbe:";
    cin >> x;
    cout << "podaj druga liczbe";
    cin >> y;

    return x + y;
}

Pomijając wszystko inne to chodzi o to że tworzysz funkcje z dwoma argumentami a te argumenty wczytujesz dopiero gdy funkcja juz działa.

Najpierw wczytaj liczby (przy okazji uprościsz kod) a potem wstawiaj je jako argumenty do dowolnych funkcji.

A propos problemu
Kalkulator w petli
http://4programmers.net/Forum/C_i_C++/237486-kalkulator_obiektowy_i_prosba_o_ocene?p=1055245#id1055245
http://4programmers.net/Forum/Newbie/174071-kalklulator_konsolowy_-_ocena_kodu_i_wskazowki?p=1088658#id1088658

Zmodyfikowałem funkcje w taki sposób:

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


using namespace std;

int dodaj(int x, int y);
int odejmij(int x, int y);
int pomnoz(int x, int y);
int podziel(int x, int y);

int main()
{
	int wybor;
		

	cout << "program imitujacy kalkulator, wybierz dzialanie:\n1.dodawanie: \n2.odejmowanie: \n3.mnozenie: \n4.dzielenie: ";
	cout << endl;
	cin >> wybor;
	

		switch (wybor)
		{
		case 1: {cout << "wynik dodawania to: " << dodaj() << endl; break; }
		case 2: {cout << "wynik odejmowania to: " << odejmij() << endl; break; }
		case 3: {cout << "wynik mnozenia to: " << pomnoz() << endl; break; }
		case 4: {cout << "wynik dzielenia to: " << podziel() << endl; break; }
		}

	
		
	return 0;
}

int dodaj()
{
	int x = 0;
	int y = 0;
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;		
	return x + y;
}

int odejmij()
{
	int x = 0;
	int y = 0;
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	return x - y;
}

int pomnoz()
{
	int x = 0;
	int y = 0;
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	return x * y;
}

int podziel()
{
	int x = 0;
	int y = 0;
	cout << "podaj pierwsza liczbe:";
	cin >> x;
	cout << "podaj druga liczbe";
	cin >> y;
	if (y != 0)
	{

		return x / y;
	}
	else
	{
		cout << "nie dziel przez 0!\n";
	}
	return 0;
}


 

I kompilator w instrukcji switch widzi taki o to problem error C2660: 'dodaj' : function does not take 0 arguments To co z tymi in x =0 i int y = 0, on tego nie widzi?

int dodaj(int x, int y);
int odejmij(int x, int y);
int pomnoz(int x, int y);
int podziel(int x, int y);

Deklaracje wciąż masz dla funkcji dwuargumentowych. Swoją drogą tak powinno być. Np. nazwa dodaj w żaden sposób nie sugeruje, że pobierze ona dane od użytkownika z wejścia standardowego, jednocześnie coś na nie wypisując.

kq napisał(a):

int dodaj(int x, int y);
int odejmij(int x, int y);
int pomnoz(int x, int y);
int podziel(int x, int y);

Deklaracje wciąż masz dla funkcji dwuargumentowych. Swoją drogą tak powinno być. Np. nazwa dodaj w żaden sposób nie sugeruje, że pobierze ona dane od użytkownika z wejścia standardowego, jednocześnie coś na nie wypisując.

Pousuwałem argumenty tej deklaracji i program działa. z tego co rozumiem to napisałeś, że deklaracja powinna być dla funkcji dwuargumentowych. Dlaczego?

Bo takie funkcje są znacznie bardziej elastyczne. Każda funkcja powinna robić jedną rzecz, ale robić ją dobrze. Pobierająca dane - pobierać, licząca - liczyć. Załóżmy na przykład, że chcesz wykonać Bardzo Trudne Obliczenia™ oraz Szalenie Istotne Komputacje© a potem obliczyć ich iloczyn. Jak to zrobisz za pomocą Twoich funkcji? Wypiszesz je użytkownikowi, żeby przepisał? Ja np. bym wolał wykonać coś takiego:

int bto = bardzoTrudneObliczenia();
int sik = szalenieIstotneKomputacje();
cout << "Poszukiwany iloczyn: " << pomnoz(bto, sik) << endl;

kq napisał(a):

Bo takie funkcje są znacznie bardziej elastyczne. Każda funkcja powinna robić jedną rzecz, ale robić ją dobrze. Pobierająca dane - pobierać, licząca - liczyć. Załóżmy na przykład, że chcesz wykonać Bardzo Trudne Obliczenia™ oraz Szalenie Istotne Komputacje© a potem obliczyć ich iloczyn. Jak to zrobisz za pomocą Twoich funkcji? Wypiszesz je użytkownikowi, żeby przepisał? Ja np. bym wolał wykonać coś takiego:

int bto = bardzoTrudneObliczenia();
int sik = szalenieIstotneKomputacje();
cout << "Poszukiwany iloczyn: " << pomnoz(bto, sik) << endl;

Dziękuję. Tego właśnie w funkcjach nie rozumiem, a chciałbym to opanować. Słowem kluczowym jest tutaj funkcja dwuargumentowa? Bo nawet nie wiem co mam sobie wygooglować żeby znaleźć jakieś materiały co do tego zagadnienia :)