Pierwszy kalkulator, błąd.

0

Uczę się c++, chciałbym się zapytać co zrobiłem tutaj źle ?
Chciałem trochę na siłę wpakować każdy if w inną funkcję, bo jestem właśnie w tym rozdziale :)

#include <iostream>
using namespace std

int liczba1;
int znak;
int liczba2;

int iloczyn;
int suma;
int roznica;
int iloraz;

	int main()
{
			cout<<"kALKULATOR VER.1.0"
			"Wpisz pierwsza liczbe...";
			cin>>liczba1;
			cout<<"Wpisz znak...";
			cin>>znak;
			cout<<"Wpisz druga liczbe";
			cin>>liczba2;
			cout<<iloczyn,suma,roznica,iloraz;
}

	int iloczyn(int)
	{
			if(znak=="*")
			{
				iloczyn=liczba1*liczba2;
			}
			return iloczyn;
	}

	int suma(int)
	{
			if(znak=="+")
			{
				suma=liczba1*liczba2;
			}
			return suma;
	}

	int roznica(int)
	{
			if(znak=="-")
			{
				roznica=liczba1*liczba2;
			}
			return roznica;
	}

	int iloraz(int)
	{
			if(znak==":")
			{
				iloraz=liczba1/liczba2;
			}
			return iloraz;
	}
3

chyba powinieneś powtórzyć kurs bo to jest delikatnie mówiąc "niepoprawne".

EDIT: jak chcesz podpowiedzi to poszukaj coś w google. Słowa klucze to: switch, declaration, definition, input/output stream. Choć powinieneś to mieć w tym swoim kursie.

0

Wszystko.

Na co te zmienne globalne? Nie ucz się stosować zmiennych globalnych. Te funkcje nie przyjmują żadnych parametrów, tylko operują na zmiennych globalnych. Poza tym są one zrobione bez sensu, bo jak niby rozróżniasz znaki i wiesz co masz wypisać? Herezje po prostu, aż się czytać nie chce.
Idea kalkulatora jest prosta - wczytujesz dwie liczby i znak lub podajesz nazwę działania, które chcesz wykonać. W zależności od znaku/działania wywołujesz konkretne funkcje, do których przekazujesz wczytane zmienne.
Radzę jeszcze raz, na spokojnie przeczytać rozdział o funkcjach i napisać program od nowa.

0

I na dodatek

cout<<iloczyn,suma,roznica,iloraz;

wyswietli tylko zmienna "iloraz". Zeby w ogole to dzialalo, powinienes jeszcze umiescic te 4 zmienne w nawiasie(priorytetowosc operatorow).

Poza tym nigdzie nie wywolales tych funkcji, sa tylko zdefiniowane. Powinienes je jeszcze zadeklarowac "nad" funkcja main(), zeby moc ich uzyc;

1 użytkowników online, w tym zalogowanych: 0, gości: 1