Mój pierwszy program w (kalkulator)

[amadeusz575]

Witam. Oto mój pierwszy program w c++. Jest to kalkulator. Wszystkie opinie i krytyki mile widziane.

 

http://przeklej.net/down/60267048986808357610361347964935631d5a5792c4eba321d659725.html

 

Kod:

 

 

 

#include <iostream>
#include <conio.h>
#include <ctime>
#include <cmath>
#include <stdlib.h>
#include <time.h>

using namespace std;

int wybor;
float wynik, l1, l2;

int main()
{
	cout << " || KALKULATOR BY AMADEUSZ V.1.0 BETA || \n " << endl;
	cout << " Wybierz dzialanie: " << endl;
	cout << " 1.Dodawanie \n 2.Odejmowanie \n 3.Mnozenie \n 4.Dzielenie \n 5.Potegowanie \n 6.Pierwiastkowanie \n " << endl;
	cout << " Twoj wybor: ";
	cin >> wybor;
	switch(wybor)
	{
		case 1: 
		cout << " Wpisz pierwsza liczbe: ";
		cin >> l1;
		cout << " Wpisz druga liczbe: ";
		cin >> l2;
		wynik = l1 + l2;
		cout << endl;
		cout << " " << l1 << " + " << l2 << " = " << wynik << "\n\n" << endl;
		getch();
		main();
		case 2: 
		cout << " Wpisz pierwsza liczbe: ";
		cin >> l1;
		cout << " Wpisz druga liczbe: ";
		cin >> l2;
		wynik = l1 - l2;
		cout << endl;
		cout << " " << l1 << " - " << l2 << " = " << wynik << "\n\n" << endl;
		getch();
		main();
		case 3: 
		cout << " Wpisz pierwsza liczbe: ";
		cin >> l1;
		cout << " Wpisz druga liczbe: ";
		cin >> l2;
		wynik = l1 * l2;
		cout << endl;
		cout << " " << l1 << " * " << l2 << " = " << wynik << "\n\n" << endl;
		getch();
		main();
		case 4: 
		cout << " Wpisz pierwsza liczbe: ";
		cin >> l1;
		cout << " Wpisz druga liczbe: ";
		cin >> l2;
		wynik = l1 / l2;
		cout << endl;
		cout << " " << l1 << " / " << l2 << " = " << wynik << "\n\n" << endl;
		getch();
		main();
		case 5: 
		cout << " Wpisz liczbe, ktora chcesz spotegowac: ";
		cin >> l1;
		cout << " Wpisz wykladnik potegi: ";
		cin >> l2;
		cout << endl;
		wynik = pow(l1,l2);
		cout << " " << l1 << " do potegi " << l2 << " wynosi " << wynik << "\n\n" << endl;
		getch();
		main();
		case 6: 
		cout << " Wpisz liczbe, ktora chcesz spierwiastkowac: ";
		cin >> l1;
		wynik = sqrt(l1);
		cout << " Pierwiastek z " << l1 << " wynosi " << wynik << "\n\n" << endl;
		getch();
		main();
	}
	getch();
}		

 

 

[Sopelek997]

Nic przebojowego. Na tym forum jest takich już co najmniej 10. W dodatku dodajesz niepotrzebne biblioteki, wywołujesz rekurencyjne main, czego nie powinno się robić, sztucznie zatrzymujesz konsolę na końcu i powtarzasz te same kawałki kodu.

[Phoenix_PL]

Niepotrzebnie zadeklarowane globalnie zmienne, niepotrzebnie stworzona zmienna "wynik", kod rozsypany (switch wygląda zupełnie kijowo, TABULACJE!). W grom niepotrzebnych bibliotek. No i to co sopel mówi, powtarzanie takiego samego kodu w każdym case. Ćwicz, ucz się dalej. Przynajmniej działa.

[czarek1122]

Jeżeli sam pisałeś to jest dobrze, w każdym innym wypadku nie.

[amadeusz575]

Sam pisałem. I jest to mój pierwszy program. Dziękuję za opinie.

wywołujesz rekurencyjne main

co to znaczy?

 

Z tymi niepotrzebnymi dyrektywami to jest tak, że często zapominam jaka jest potrzebna do różnych funkcji, i we wszystkich programach zapisuje te wszystkie

[Sopelek997]

to znaczy, że wywołujesz main z main

[Phoenix_PL]

To robisz źle, bo zwiększa to wagę i zamula program. Używaj tylko tych, które są potrzebne. Żeby sprawdzić, usuwaj te, które mogą być niepotrzebne - jeśli kompilator walnie błąd, to znaczy że jednak jest potrzebna, jeśli nic się nie stanie - to znaczy że nie jest potrzebna.

 

W tym wypadku wystarczą te:

#include <iostream>
#include <conio.h>
#include <cmath>

Rekurencyjne main, to znaczy wywołujesz funkcję main() w funkcji main(), więc robisz błąd. Zrób to na pętli, zamiast wywoływać w ten sposób funkcje ponieważ może to prowadzić do przeciążenia programu i bugów.

 

@down: Ah, więc tak to działa... Nie myślałem że linker może być na tyle inteligentny

[Sopelek997]

To robisz źle, bo zwiększa to wagę i zamula program.

Nie prawda. Linker nie powinien linkować niepotrzebnych funkcji. Chodzi tylko i wyłącznie o estetykę.

[czarek1122]

Przy takim programowaniu nie ma znaczenia czy program będzie waży o te 8 bitów mniej lub więcej.

[OczkoSX]

Ładnie zrobiłeś (pamiętam początki ), ale:
1. Zamiast wywoływać main(), to wstaw pętle jak mówili poprzednicy
2. Ogarnij estetyke kodu (wcięcia, tabulatory)

Mam dla Ciebie zadanie Tak w ramach ćwiczenia. Zrób menu do programu, że wpisujesz cyfre i nie trzeba klikać enter tylko bezpośrednio cię przenosi do odpowiedniego CASE. Mogę Ci podpowiedzieć, że należy to zrobić znakami ASCII

http://pl.wikipedia.org/wiki/ASCII

. Deklarujesz zmienną char, która będzie pobierała znak z klawiatury instrukcją getch(). Potem robisz case odpowiadające DEX znaku w tablicy ASCII. Przykładowy program(ma za zadanie wyjaśnić o co chodzi):

cout << "[1] Dodawanie" << endl;
cout << "[2] Odejmowanie" << endl;
//itd...
char znak = getch();

switch(znak)
{
    case 49:
    //instrukcje
    case 50:
    /instrukcje i tak dalej...
}

Pamiętaj o zapętleniu menu wyboru (ogólnie całego programu)

[Sopelek997]

lepiej używać literałów typu char

'0'

'1'

[OczkoSX]

Mi zawsze tak lepiej pasowało W końcu jedną funkcję można napisać w wiele sposobów.

[szymeczek31]

Kod jest nie uporządkowany.

Tak szczerze to ja bym walnął tylko #include

Albo jeszcze jak chcesz to zrobić bardziej intuicyjne #include dla zmiennych tekstowych i używać if zamiast switch/case. Zaraz napisze jak bym walnal na dodawanie a ty dodaj na reszte.

#include

#include

using namespace std;

 

int main()

{

string zmienna;

int a,b;

cout << "Podaj liczby";

cin >> a;

cin >> b;

cout << "Dodać je, czy pomnożyć?";

cin >> zmienna;

if (zmienna == "dodaj") cout << a << "+" << b << "=" << a+b << endl;

I tak samo z mnożeniem tylko napisz pomnóż i jako znak zamiast + , daj * i tak ze wszystkim. A teraz sam poćwicz

W tych include 1 bylo string a drogie iostream ale mi usunęlo nwm czemu

[Phoenix_PL]

@szymeczek31 nie rozumiem twojej mowy o "include" a używanie ifa zamiast switcha to poroniony pomysł, bo switch jest lepszy dla wielu opcji oraz szybszy.

[OczkoSX]

@szymeczek31 nie rozumiem twojej mowy o "include" a używanie ifa zamiast switcha to poroniony pomysł, bo switch jest lepszy dla wielu opcji oraz szybszy.

 

Też nie rozumiałem, ale nie pisałem bo myślałem że to takie oczywiste że wyjde na debila Co do ifa to się czasami przydaję i to bardzo, w switchu nie możesz żadnych warunków dać