Prečo mi nefunguje tento cyklus FOR – C / C++ – Fórum – Programujte.com

Takže, tu mám program (zo 7. lekcii):

#include <iostream>

using namespace std;

void upravapolozky (int cislopolozky);

void vypis (int pocetpoloziek);

struct katalog

{

	char nazov[20];

	float cenaEUR;

	float cenaSK;

} polozka[40];

int main()

{

	cout << "Vitajte v editore poloziek v katalogu!\nChcete pridat polozky? (A/N)";

	char volba[2];

	char maleano[] = "a";

	char velkeano[] = "A";

	int cislo = 0;

	cin.get(volba, 2);

	cin.get();

	while ((volba[0] == maleano[0])||(volba[0] == velkeano[0]))

	{

		++cislo;

		upravapolozky(cislo);

		cout << "Chcete zadat dalsiu polozku? (A/N) ";

		cin.get(volba, 2);

		cin.get();

	}

	vypis (cislo);

	cin.get();

	cin.get();

	return 0;



}

void upravapolozky (int cislopolozky)

{

	cout.precision(15);

	cout << "Zadajte nazov polozky: ";

	cin.get(polozka[cislopolozky].nazov, 19);

	cin.get();

	cout << endl << "Zadajte cenu polozky v mene EURO: ";

	cin >> polozka[cislopolozky].cenaEUR;

	cin.get();

	polozka[cislopolozky].cenaSK = polozka[cislopolozky].cenaEUR * 30,126;

}

void vypis (int pocetpoloziek)

{

	cout.precision(2);

	if (pocetpoloziek = 0)

		cout << endl << "\t\t\tKatalog je prazdny! Program je ukonceny.";

	else 

	{

		cout << "Cislo\t--\tNazov\t--\t--\tCena EURO\t--\tCena SKK" << endl;

		int i = 0;

		for (int i = 0;i<pocetpoloziek;++i)  //TUTO MI TO ROBÍ PROBLÉMY!!!

		{

			cout << i+1 << "\t\t" << polozka[i+1].nazov << "\t\t" << polozka[i+1].cenaEUR << "\t\t" << polozka[i+1].cenaSK << endl;

		}

		cout << "Koniec vypisu. Chcete niektoru polozku zmenit? (A/N)";

		char volbaedit[2];

		char maleano[] = "a";

		char velkeano[] = "A";

		cin.get(volbaedit, 2);

		cin.get();

		if ((volbaedit[0] == maleano[0])||(volbaedit[0] == velkeano[0]))

		{

			cout << "Zadajte cislo polozky: ";

			int cisloupravy;

			cin >> cisloupravy;

			cin.get();

			upravapolozky(cisloupravy);

			cout << "Upraveny katalog:" << endl;

			vypis(pocetpoloziek);

		}

		else cout << "Program skoncil.";

	}

}

- Na řádku 43 je špatně nspsané číslo - má tam být desetinná tečka, ne čárka.
- Na řádku 48 se neporovnává pocetpolozek s nulou, ale přiřazuje se nula do pocetpolozek (chybí tam jedno rovnítko). Takže pocetpolozek se vynuluje, výsledek je nula, podmínka tedy není splněna a přejde se do else větve.
- Řádek 53 je zbytečný

Aha, díki moc, už si len znormalizujem výstup a vstup a bude to OK. Akým príkazom sa dá vypísať formátovať výpis reálneho čísla? Teda, počet platných číslic, zaokrúhlenie a tak...

Zkus si pohrát s tímto příkladem a bude to jasné:

#include <iostream>

#include <iomanip>



int main()

{

    const double d = 123456.789;



    std::cout << std::fixed << std::setw(20) << std::setprecision(12) << d << '\n';

    std::cout << std::scientific << std::setw(20) << std::setprecision(12) << d << '\n';

}

Aha, takže na klasické vypisovanie čísel (pre človeka zrozumitelné) musím ísť teda cez fixed. Šírku vyhradenú pre číslo si nastavím cez setw a počet desatinných pomocou setprecision. Diki moc!

riadok 22,23:
++cislo;
upravapolozky(cislo);
mozes veselo napisat ako jeden riadok upravapolozky(++cislo); pozri si co je post-inkrementacia a pre-inkrementacia

riadok 48:
if (pocetpoloziek = 0)//tu ti priradi do pocetpoloziek nulu a vyhodnoti to cele ako nulu pretoze vysledok tej operacie je nula cize vysledok podmienky je nepravda.
nahrad to za if (pocetpoloziek == 0)// takto porovnas obsah tej premennej s nulou

riadok 53: vymaz pretoze v riadku 54 robis to iste znova(deklarujes premennu i a inicializujes ju na 0)
riadok 54:
for (int i = 0;i<pocetpoloziek;++i) //TUTO MI TO ROBÍ PROBLÉMY!!!
for (int i = 0;i<pocetpoloziek;i++) //v sumare je to to iste ale radsej si zvykni pouzivat toto


riadok 56:
cout << i+1 << "\t\t" << polozka[i+1].nazov << "\t\t" << polozka[i+1].cenaEUR << "\t\t" << polozka[i+1].cenaSK << endl;
by mal vyzerat
cout << i+1 << "\t\t" << polozka[i].nazov << "\t\t" << polozka[i].cenaEUR << "\t\t" << polozka[i].cenaSK << endl;
pretoze pole indexujes od nuly

... a hraj sa stym a skusaj:)

To milan_m :
> for (int i = 0;i<pocetpoloziek;++i) //TUTO MI TO ROBÍ PROBLÉMY!!!
> for (int i = 0;i<pocetpoloziek;i++) //v sumare je to to iste ale radsej si zvykni pouzivat toto

V čem je i++ lepší než ++i? U základních typů je to jedno a u iterátorů je právě naopak lepší ++i, protože se nemusí dělat kopie objektu.

To ondra.holub : rozdíl mezi preinkrementací a postinkrementací je ten, že preinkrementace nejprve k proměnné přičte 1 a vrátí hodnotu a postinkrementace vrátí hodnotu a až poté přičte 1. Můžeš si to zkusit na následujícím příkladě:

int x = 5;

int y = 5;

cout << "++x = " << ++x; // Vypíše 6

cout << "\ny++ = " << y++; // Vypíše 5

// pořád platí, že x == y (x == 6; y == 6)

To Bald3rr :
No to já vím, jaký je v tom rozdíl. Měl jsem na mysli právě ten for cyklus:
> for (int i = 0;i<pocetpoloziek;++i) //TUTO MI TO ROBÍ PROBLÉMY!!!
> for (int i = 0;i<pocetpoloziek;i++) //v sumare je to to iste ale radsej si zvykni pouzivat toto
Tam je to (jak jsem psal) pro základní typy (jako int apod.) jedno, ale pokud se jedná např. o iterátor, tak je právě chyba preferovat postinkrementaci (jak někdo psal ve výše citovaném doporučení).

Takže abych shrnul co jsem tím myslel: pokud je významově jedno, která varianta se použije, je lepší si zvyknout na prefixovou variantu, protože někdy může být efektivnější a nikdy není horší.

to ondra.holub: sorry tak s tou pre inkrementaciou mas asi pravdu:), mne len zo zvyku pride postinkrementacia taka prirodzenejsia, pretoze to tak pisem stale a prislo mi to uz ako gramaticka ked vydis niekde ypsilon tam kde nema byt. tipujem ,ze ty ked pises takyto jednoduchy cyklus asi tiez davas postinkrementaciu a nelamentujes nad tym:) ale ber to tak ze som nic nepovedal.
a diky za poucenie asi dam na teba a zvyknem si na preinkrementaciu

while ((volba[0] == maleano[0])||(volba[0] == velkeano[0]))  {

	++cislo;

	upravapolozky(cislo);

	...

while ((volba[0] == maleano[0])||(volba[0] == velkeano[0]))  {

	upravapolozky(cislo);

	++cislo;

	...

char volba;

cin.get(volba).get();

while (volba == 'a' || volba == 'A')...