Primerjanje stringa in izpis v C++

Si že ugotovil kakšno vrednost (če jo sploh ima) ima izdatoteke pred prvim if stavkom, takoj za cin << ime?

Pa kaj novega (kar že ne veš) preveriš v drugem if stavku?

Če bi bolj pozorno pogledal kodo pred prvim if stavkom, bi ugotovil, da iz datoteke, ki jo odpreš sploh nič ne prebereš in potem primerjaš prazen string ("") s tistim kar si vnesel!

Ne bi namesto dveh if stavkov uporabil kar if - else, saj je jasno, da če ni od nič različno (!=0) je enako nič (==0). Zakaj bi za to porabili še kaj procesorskega časa?

Ne razumes
1. sem popoln n00b
2. to je sam tko za test
zadeva dela sam nekaj manjka in nikjer ne zvoham kaj

if (strcmp (izdatoteke,ime).....!=0....)

tukaj v if stavku nekaj manjka.....kaj? nimam pojma

LP Brane

No evo sem koncno pogruntal kako zadeva deluje

void iskanje (){

char ime[255]="";
char izdatoteke[255]="";

ifstream vhod ("blackbook.txt");

cout << "Vnesi ime: ";
cin >> ime;

while (strcmp (ime, izdatoteke)!=0 && vhod >> izdatoteke);
if (strcmp (ime, izdatoteke)!=0) {
cout << "Tega podatka ni na voljo. Sorry!" << endl;
}

if (strcmp (ime, izdatoteke)==0) {
cout << izdatoteke << " ";
for (int counter=0; counter<5; counter++) {
vhod >> izdatoteke;
cout << izdatoteke << " ";
}
cout << endl;
}
}

komentarji??

CU Brane

Zadeva sicer deluje, vendar pa bi se dalo veliko reči glede optimizacije.

Pri branju spremenljivk preverjaj kaj si prebral (predvsem za overflowi, čeprav so v tvojem primeru zelo malo verjetni). In ne razmetavaj preveč s prostorom (char[255]).

Pri prvi while zanki: Prvič primerjaš prazen string in prebran podatek, šele nato izdatoteke prebereš. Nepotreben cikel.
Potem bereš kar vse po vrsti. Iz zgornjih postov je razvidno, da sekvenčno iščeš ime, ki je v seznamu datoteke, se pravi, da te zanima samo prvi niz v vsaki vrstici. Zakaj po prebranem prvem nizu ostanek vrstice preprosto ne zavržeš (in tako zmanjšaš število nepotrebnih ciklov), npr. z getline() metodo!?
Pa while zanka bi se ti s kakšno optimizacijo zaciklala, ker bi program po preverjanju prvega člena ugotovil, da false && nekaj == false in se drugega parametra (branja spremenljivke) sploh lotil ne bo.

Pri zadnjih dveh if stavkih: Kot sem že v prejšnjem postu rekel, drugič preverjaš že znano. Pa daj že uporabi ta if - else. Koda je s tem preglednejša in hitrejša.
Pa pri drugem if stavku čisto nepotrebno kompliciraš z izpisovanjem vrstice oz. opravljaš nepotrebno. Vrstico prebereš z getline() metodo in nato celo izpišeš (saj je formatirano že v datoteki iz katere prebereš?).

Pa kakšno preverjanje konca datoteke (EOF) tudi ne bi škodilo ...


Nauči se pisati lepo in hitro kodo, ker s slabo kodo boš nezmožen napisati kaj bolj kompleksnega od tega, ki si ga podal.

Fantje, to je blo vprasanje iz kiberpipine c++ sole, tam se nismo prisli do dinamicne alokacije. pocasi je treba napredovat, da se ljudem ne zameri

Tako je BigWhale

drugace sem ze u startu povedal da sem POPOLN zacetnik

ampak nekje je treba zacet a ne
mislim da ni nobeden tukaj ze od rojstva znal programirat
(ali pa??)



LP Brane