dotNETportal.cz

1'

csharp|

xml|

css|

'

Kedy bude dalsi clanok? predsa su to uz 4 mesiace :)

Nemůžu to slíbit, ale budu se snažit, abych ho napsal do konce ledna, nebo někdy začátkem února. Za tu velkou pauzu se čtenářům omlouvám, je to zapříčiněno jednak nedostatkem času, ale abych se nevymlouval jen na čas, jistou roli v tom hraje i moje lenivost... Nicméně pauza už byla opravdu velká, takže se budu snažit další díl napsat co možná nejdříve.

pocuj mohol by si mi skontorlovat jeden jednoduchy program? (je to moj prvy samostatny program taka jednoducha kalkulacka) ale nejde mi to skompilovat pise mi nejaku chybu pri kompilacii nieco so suborom menu.h moze byt ze som aj uvazoval zle no chybu mi vypisuje len v tom subore este to chcem otestovat ale neviem prist na to co za chyba tam je :(

menu.h:

#include <iostream>
using namespace std;
int menu();
int Scitanie(int x, int y);
int Odcitanie(int a, int b);
int Nasobenie(int c, int d);
int Delenie(int e, int f);

int menu();
{
	int volba;
	cout << "****MENU****\n";
	cout << "(1) Scitanie\n";
	cout << "(2) Odcitanie\n";
	cout << "(3) Nasobenie\n";
	cout << "(4) Delenie\n";
	cout << "(0) Koniec\n";
	cin >> volba;
	return volba;
}

delenie.h

int Delenie(int e, int f)
{
	int podiel;
	cout << "Vlozte 2 cisla:\n";
	cout << "Delenec:"; cin >> e;
	cout << "\nDelitel:"; cin >> f;
	if (f == 0)
	{
		cout << "\nDelenie 0 nieje mozne!!\n";
		return podiel = -1;
	}
	else
	{
	return podiel = e/f;
	}
}

nasobenie.h:

int Nasobenie(int c, int d)
{
	int sucin;
	cout << "Vlozte 2 cisla:\n";
	cout << "Cinitel 1.:"; cin >> c;
	cout << "\nCinitel 2.:"; cin >> d;
	return sucin = c*d;
}

scitanie.h:

int Scitanie(int x, int y)
{
	int sucet;
	cout << "Vlozte 2 cisla:\n";
	cout << "Scitanec 1.:"; cin >> x;
	cout << "\nScitanec 2.:"; cin >> y;
	return sucet = x+y;
}

odcitanie.h:

int Odcitanie(int a, int b)
{
	int rozdiel;
	cout << "Vlozte 2 cisla:\n";
	cout << "Cislo 1.:"; cin >> a;
	cout << "\nCislo 2.:"; cin >> b;
	return rozdiel = a-b;
}

kalkulacka.cpp:

#include <menu.h>
#include <scitanie.h>
#include <odcitanie.h>
#include <nasobenie.h>
#include <delenie.h>
using namespace std;

int main()
{
	int menu();
	int f_Koniec == false;
	while (!f_Koniec)
	{
		if (volba < 0 && volba > 5)
		{
			cout << "Neplatna volba!!\n";
			continue;
		}
		switch (volba)
		{
		case 1:
			Scitanie(int x, int y);
			cout << "Sucet:" << sucet << "\n";
			break;
		case 2:
			Odcitanie(int a, int b);
			cout << "Rozdiel:" << rozdiel << "\n";
			break;
		case 3:
			Nasobenie(int c, int d);
			cout << "Sucin:" << sucin << "\n";
			break;
		case 4:
			Delenie(int e, int f);
			cout << "Podiel:" << podiel << "\n";
			break;
		case 0:
			f_Koniec = true;
			cout << "\nKoniec programu....";
			break;
		deafult:
			cout << "Neplatna volba!!\n";
			continue;
		}
	}
	return 0;
}

V rychlosti jsem na to mrknul a těch chyb tam bude určitě více. Doporučuji Vám podívat se na to, jakým způsobem se volají funkce a jak se pracuje s tím, co daná funkce vrací. Jen pro ukázku:

int vysledek = Delenie(10, 2); // toto je spravne

Zápis Delenie(int e, int f) je v tomto případě zcela mimo. Uvědomte si, že funkce něco vrací, Vy to nikam (tu hodnotu) nepřiřazujete. Dále funkci musíte předat parametry, což se určitě nedělá tak, jak se to snažíte dělat.

ano bolo tam viac chyb :) som si to vsimol az neskor pretoze doteraz som pouzival dev c++ a toto som robil vo MC VS a tam je ten vypis trosku iny (pozrel som len na cislo pri "failed") a nevsimol som si tu kopu chyb trosku vysie :) uz pracujem na oprave zredukovane na 4 chyby este prist na to co tam treba spravit a otestovat to :)

takze dostal som sa sem:

#include <iostream>
using namespace std;

int menu(int volba);
int odcitanie (int a, int b);
int nasobenie(int c, int d);
int delenie (int e, int f);
int scitanie(int x, int y);

int main()
{
	int volba;
	int x;
	int y;
	int a;
	int b;
	int c;
	int d;
	int e;
	int f;
	int f_Koniec = false;
	while (!f_Koniec)
	{
		volba = menu(volba);
		if (volba < 0 && volba > 5)
		{
			cout << "Neplatna volba!!\n";
			break;
		}
		switch (volba)
		{
		case (1):
			int sucet;
			cout << "Vlozte 2 cisla:\n";
			cout << "Scitanec 1.:"; cin >> x;
			cout << "\nScitanec 2.:"; cin >> y;
			sucet = scitanie(x,y);
			cout << "Sucet:" << sucet << "\n";
			break;
		case (2):
			int rozdiel;
			cout << "Vlozte 2 cisla:\n";
			cout << "Cislo 1.:"; cin >> a;
			cout << "\nCislo 2.:"; cin >> b;
			rozdiel = odcitanie(a,b);
			cout << "Rozdiel:" << rozdiel << "\n";
			break;
		case (3):
			int sucin;
			cout << "Vlozte 2 cisla:\n";
			cout << "Cinitel 1.:"; cin >> c;
			cout << "\nCinitel 2.:"; cin >> d;
			sucin = nasobenie(c,d);
			cout << "Sucin:" << sucin << "\n";
			break;
		case (4):
			int podiel;
			cout << "Vlozte 2 cisla:\n";
			cout << "Delenec:"; cin >> e;
			cout << "\nDelitel:"; cin >> f;
			podiel = delenie(e,f);
			cout << "Podiel:" << podiel << "\n";
			break;
		case (0):
			{
			f_Koniec = true;
			cout << "\nKoniec programu....";
			break;
			}
		default:
		{
			cout << "Neplatna volba!!\n";
			continue;
		}
	}
	return 0;
}
}

int menu(int volba)
{
	cout << "****MENU****\n";
	cout << "(1) Scitanie\n";
	cout << "(2) Odcitanie\n";
	cout << "(3) Nasobenie\n";
	cout << "(4) Delenie\n";
	cout << "(0) Koniec\n";
	cin >> volba;
	return volba;
}

int scitanie(int x, int y)
{
	return x+y;
}

int odcitanie(int a, int b)
{
	return a-b;
}

int nasobenie(int c, int d)
{
	return c*d;
}

int delenie (int e, int f)
{
	if (f == 0)
	{
		cout << "\nDelenie 0 nieje mozne!!\n";
		return -1;
	}
	else
	{
		return e/f;
	}
}

ale je tu jeden problem vsetko je skompilovane (da sa to spustit hoci...) ale vyskakuje mi po kompilacii jedno varovanie: "(24) : warning C4700: uninitialized local variable 'volba' used"

...a absolutne neviem co to varovanie znamena (predpokladam ze tie funkcie su uz spravne no aspon okrem funkcie menu() kde sa pouziva premenna "volba")

Otázkou je, proč do funkce menu předáváte parametr volba. Ta chyba je způsobena tím, že té funkci předáte proměnnou volba, která ale v tu chvíli ještě nemá žádnou přiřazenou hodnotu. Tu funkci upravte tak, aby nebrala žádný parametr, pak jí nebudete nic předávat a varování bude pryč a v lepším případě to bude dokonce i fungovat.

Jinak ještě k tomu dělení - máte tam všude int, což je celočíselný datový typ, takže pokud funkci předáte například čísla 5 a 2, dostanete špatný výsledek.

ok upravil som to program funguje ako ma ale chcem sa spytat preco ta funkcia musi byt bez parametru? to trosku nechapem a dakujem :)

Myslím si, že by jste se měl učit programovat s jiným jazykem, jak tak koukám na váš kód, tak je jen otázkou času, než si ustřelíte nohu.

Tak ona samozřejmě nemusí být bez parametru, ale v tomto případě je to zbytečné. Funkce vám vrací hodnotu proměnné volba, která ale není nijak závislá na tom, co funkci předáte. Vaše funkce menu vyzývá uživatele k tomu, aby něco zadal a podle toho funkce vrací rozdílné hodnoty. Ty ale nejsou nijak závislé na tom, co funkci předáte jako parametr. Proto tedy ve Vašem kódu není parametr ve funkci volba potřeba.

aha dakujem :) a k tomu predoslemu prispevku (nie vasemu) by som povedal len tolko ze toto bol moj prvy program takze to netreba brat az tak zle (dufam) :)

Samozřejmě, že není třeba brát to až tak zle. Je sice pravda, že v tom kódu bylo dost chyb, ale programování se člověk nenaučí ze dne na den, chce to čas. Proto bych neházel flintu do žita a to, že si máte vybrat na začátek jiný jazyk, o tom by se dalo debatovat. Je hodně lidí, co tvrdí, že C++ je těžkopádné a pro začátečníky se nehodí, nicméně toto jsou beztak jen základy a ty jsou víceméně stejné všude. Krom toho naučíte-li se C++, nebude Vám dělat větší problém přejít třeba na Javu nebo C#.

samozrejme ze najjednoduhsie je zacat pascalom ale kedze som zacal s programovanim dost neskoro (vzdy som si myslel ze ma to na skole naucia no mylil som sa) a teraz by to bolo treba nejako urychlit tak som pascal (ktory sa aj tak nepouziva) preskocil a siel rovno na C++ ja chapem ze zaciatky su tazke ale vzdat sa nemienim :) jednoducho neexistuje zabavnejsia praca nez v hernom priemysle :)

Tak začít s Pascalem rozhodně není nejjednodušší, to se jenom říká, ale taky jsou jednodušší jazyky. Ono jde ale hlavně o to, že C++ má spoustu "featurek", které těď v žádném případě nevyužijete, akorát vás budou otravovat, neříkám, že C++ je špatný jazyk pro začátek, sám jsem s ním začínal a právě proto to mohu zpětně posoudit. Budete používat spoustu věcí, kterým nerozumíte, jenom proto, že tam "mají být". Mně je celkem jedno, jestli u C++ zatím zůstanete, to je koneckonců vaše věc, já vám jen říkám, jak to vidím já.

Co bych vám poradil, když zůstanete u C++?

1) Odnaučte se takto debilně pojmenovávat proměnné, je to strašný zlozvyk, kterého se budete těžko zbavovat.

2)Deklarujte proměnné až ve chvíly, kdy jsou poprvé potřeba.

3)Pokud používáte proměnné jako parametry mohou se jmenovat stejně, je zbytečné dávat jim jiná jména, mám na mysli toto:

//Nevhodně

int scitanie(int x, int y)
{
	return x+y;
}

int odcitanie(int a, int b)
{
	return a-b;
}

int nasobenie(int c, int d)
{
	return c*d;
}

int delenie (int e, int f)
{
	if (f == 0)
	{
		cout << "\nDelenie 0 nieje mozne!!\n";
		return -1;
	}
	else
	{
		return e/f;
	}
}

//Lepší, přehlednější
int scitanie(int a, int b)
{
	return a+b;
}

int odcitanie(int a, int b)
{
	return a-b;
}

int nasobenie(int a, int b)
{
	return a*b;
}

int delenie (int a, int b)
{
	if (a == 0 || b == 0)
	{
		cout << "\nDelenie 0 nieje mozne!!\n";
		return -1;
	}
	else
	{
		return a/b;
	}
}

Tu funkci pro dělení by šlo napsat i kratší, třeba takto:

int delenie (int a, int b)
{
   return (b == 0) ? 0 : a / b;
}

4) U matematických operací je lepší používat typ double místo int.

Neberte to jako nadávání čí tak něco, snažím se vám poradit, jak se v tom neztratit ;)

Tak zatial dakujem za rady :)

V clanku je chyba - trida Programator neni upresnenim tridy Zamestnanec. Je to uplne samostatna trida a zajimalo by mne, co by autor delal, pokud by mel programatora najmout na smlouvu ci od jine firmy a nebo by to byl jeho rodinny prislusnik.

Dedeni je velmi velmi osemetna vec (a zacinajicimi programatory dost precenovana) a zrovna vztah Zamestnanec-Programator by se timhle urcite neresil.

Vezměte v potaz následující fakt - tento seriál je určen začínajícím programátorům, tj. těm, kteří o OOP, dědičnosti a dalších věcech nic neví. Proto tady nehodlám řešit, co by kdyby, čili mi nepřijde adekvátní řešit, zda programátor pracuje na smlouvu o dílo, nebo je to brigádník nebo je z jiné firmy. Článek si klade za cíl osvětlit začínajícím programátorům pojem dědičnost a myslím si, že to článek splňuje. A proto trvám na tom, že v tomto případě je vztah dědičnosti zvolen správně.

Vy jste ale hnidopich, toto je seriál pro začátečníky, moc to řešíte... Jde o to, aby začátečníci pochopily, k čemu dědičnost je.

Navíc chyba to nění:

JE Programátor zaměstnanec? Ano je! Takže dědičnost se zde uplatnit může, ano nemusí, ale není to chyba :-)

No jestli to zacnou chápat z těhle příkladů.....

Z tvrzení Programátor je zaměstnanec vůbec neplyne, že všichni programátoři jsou zaměstnanci - a často vůbec nejsou. Dědičnost v tomhle příkladě je zcela mimo a je to chyba - zkuste to udělat do nějakého IS a následně řešit všechny problémy, co vám z vašeho nesprávného modelu vznikly.

http://stackoverflow.com/questions/57521...

Pokusím se Vám to vysvětlit ještě jednou, zjevně to pořád nechápete (nebo možná ano, ale jenom chcete rýpat). Tento článek neřeší a nezabývá se vývojem IS ani návrhem architektury, ani podobných věcí, které s vývojem IS souvisí. Tento článek vysvětluje na triviálním příkladě co je to dědičnost a jaké jsou její výhody. Čtenář by tedy měl získat představu o tom, jak dědičnost funguje, jaká je její syntaxe, výhody apod.

O tom, jak správně navrhnout a implementovat informační systém článek nepojednává, to by se nakonec ani do jednoho článku nevešlo. Každopádně, když umíte tak dobře kritizovat, pak pro Vás jistě nebude problém o vývoji IS a zvolení správné hierarchie dědičnosti napsat nějaký ten článek. Ostatní si jej určitě rádi přečtou a něčemu se přiučí. A možná Vám ho i doplní o nějakou tu kritiku a Vy pak budete marně vysvětlovat, proč jste to v článku napsal tak a ne onak. Bude to ale zbytečné, vždy se najde někdo, kdo to prostě nebude chtít pochopit a bude pořád "otravovat".

Prominte, ze jsem upozornil na to, ze vami zvoleny priklad nebyl moc vhodny - a vim, proc jsem to napsal. Pokud to povazujete za rypani misto upresneni, je to vas problem.

S kritikou si klidne si posluzte:

http://blog.vyvojar.cz/mstr/archive/2008...

http://blog.vyvojar.cz/mstr/archive/2008...

http://blog.vyvojar.cz/mstr/archive/2008...

Když se ale podíváte a srovnáte můj článek s těmi, které jste psal Vy, musíte vidět ten jasný rozdíl - můj článek vysvětluje, co to dědičnost je, kdežto ve Vašem případě už předpokládáte, že čtenář má již s dědičností zkušenosti a zaměřujete se na hierarchii dědičnosti v rámci informačních systémů.

Proto mi přišlo vhodnější ukázat dědičnost na velmi jednoduchém příkladu. Netvrdím, že ve velkém IS by se to takto řešilo, ale tvrdím, že pro pochopení dědičnosti to stačí.

Pokud máte rád humor, rád bych podotkl, že můj článek se zaměřuje hlavně na to, jak opravit díla čtenářů vašeho článků :-).

Ale vážně: Nelíbil se mi v článku zavedený vztah Zaměstnanec-Programátor - vysvětlujete dědění na špatném příkladu. Nehodnotím, jak ho vysvětlujete. Tvrzení "Programátor je specifickým druhem zaměstnance" je nepravdivé a pro tento vztah platí to, co jste napsal pro okno a dveře. Jen jsem na to upozornil a první reakcí bylo obvinění z hnidopišství :-) - asi nemáte rád jakoukoliv výtku, byť dobře míněnou.

To vůbec není o tom, že by mi vadila výtka nebo nějaká kritika. To je o tom, že já s Vaší výtkou nesouhlasím, proto si obhajuji svůj názor. Kdybych věděl, že máte pravdu, pak bych to uznal, splést se může každý a zároveň nikdo neví všechno.

V tomto případě ale s Vámi nesouhlasím, trvám si na svém, nic víc v tom není. Kdybych v článku vysvětloval, jak správně navrhovat IS, Vaše připomínka by byla na místě a já bych na ní reagoval úplně jinak.

A o hnidopišství jsem já nemluvil, to byl někdo jiný, ale to je vedlejší.

Za to hnidopisstvi se vam omlouvám.

Ta vaše zatvrzelost přiznat, že vaše tvrzení o programátorovi je nepravdivé, je ale úsměvná. Snad si to čtenáři správně přeberou a nebudou dědičnost takto chybně používat - z praxe vím, že je to velmi častá chyba, převážně začátečníků naučených z podobných článků a školy. Obvykle pak používají dědičnost všude, kde jen trochu mohou. Kdysi jsem tím také prošel :-)

Mimochodem tvrdíte, že programátor je zaměstnanec (což může a nemusí být pravda), ale naprogramoval jste tvrzení, že 'každý programátor je zaměstnanec" (což je nepravda). Podle mne chybka, podle vás OK.

IMHO je fér na to čtenáře upozornit (což nijak nesnižuje přinos článku a dalších informací). Zkuste místo slova Programátor použít slovo Řidič, snad už vám to dojde, že tam žádná dědičnost být nemůže. A příště raději volte třeba obecné názvy tříd A a B a nebo se inspirujte tím odkazem na stackoverflow.

Nechme tedy na čtenářích, ať si to přeberou po svém. Zkusím ještě naposledy ve stručnosti shrnout to, co se Vám celou dobu snažím říct.

Přeneseme-li se z programování do reálného života, pak máte pravdu, programátor může, ale i nemusí být zaměstnanec. Vzhledem k tomu, že IS se navrhují pro něco z reálného života, dědičnost zaměstnanec - programátor by mohla být špatně. V článku však neberu v potaz to, jestli programátor pracuje jako nějaký externista, brigádník či něco podobného. Proto mi v tomto případě nepřijde můj příklad jako chybný. A dál už se o tom odmítám bavit, neboť to asi nikam nevede - každý z nás si bude tvrdit své.

Musim říct, že opět velice kvalitní díl, vysvětleno je vše co začátečník potřebuje, já sice dědičnost znám, ale například na syntaxi v C++ jsem byl celkem zvědavý(protože v C++ přiliš neprogramuji), vysvětlil jste to velice dobře, líbí se mi, že stavíte na příkladech z minulých dílů, donutí to více lidi si to pročíst a trochu víc se to naučit :-)

Díky, takový příspěvek do diskuse vždy potěší.