Programiranje u C++-u - pitanja i odgovori

za ( j = i; ako je "j" manje ili jednako 1 i element polja na poziciji "j-1" veci od vrijednosti varijable pom; umanji "j" za 1)

jasno, petlja ce se izvrsavati sve dok je uvjet zadovoljen

I meni se knjiga ne cini nesto dobra jer kad se vec koriste c++ objekte bilo bi dobro drzati se preporucenog nacina deklariranja varijabli

neposredno prije sto se koriste pogotovo za petlje jer se tako umanjuje njihov scope i smanji mogucnost greske, za for petlju npr.:

for( int index = 0; index < MAX; index--)

sto u ovom slucaju i nebi bilo zgodno jer je program napisan tako da koristi varijablu "i" na vise mjesta.

Takoder nije dobra praksa izostavljati viticaste zagrade pri koristenju petlji i koristenje imena varijabli koji nemaju veze sa njihovom funkcijom

ili sadrzajem, itd..

Hahaha, pa nemoj to raditi čovjeće, stičeš loše navike i prije nego što se naučis programirati kako spada 

Zna se čemu služe vitičaste zagrade i one nisu pravilo ako je riječ o jednoj naredbi. Eventualno, tek preporuka na početničkom nivou. Mislim da je uvijek sve u redu ako programer uredno piše code. Npr.

for(.......)

    naredba;

for(......)

{

    naredba1;   

    naredba2;

}

Znači, jasnim uvlačenjem coda korištenjem tab-a i sl. se uvijek dobije brz pregled. Zna se koji blok naredbi je ispod čega i kada se izvršava, bez obzira na vitičaste zagrade. Štoviše, ako se ovako piše code (uvlačenjem) onda se vrlo lako i detektira greška s vitičastim zagradama.

Ovako,program mi ispisuje prosječnu vrijednost kao cijeli broj,ako dolje kod izračunavanja prosječne vrijednosti ne stavim static_cast.Postoji neki drugi način kako dobiti float broj?Hvala.Trebate napraviti neku datoteku s brojevima pa probajte.

 #include<iostream>#include<fstream>using namespace std;int main() {    ifstream ulazniTok("brojevi.txt");    cout<<"Sadrzaj datoteke:"<<endl<<endl;    float broj;    int suma = 0;    int BrojBrojeva = 0;    while ((ulazniTok >> broj) != 0){       suma+=broj;       BrojBrojeva++;         cout<<broj<<endl;       }    float ProsjecnaVrijednost = static_cast<float>(suma)/static_cast<float>(BrojBrojeva);    cout<<"U datoteci je bilo "<<BrojBrojeva<<" brojeva."<<endl;    cout<<"Njihova prosjecna vrijednost je "<<ProsjecnaVrijednost<<endl;      system("Pause");    return 0;      }

Ima još jedna fora - pošto se radi o castovanju brojeva - jedan int u varijabli pomnožiš sa float brojnim tipom, naravno sa 1:

float prosjecnaVrijednost = (suma * 1.f)/(brojBrojeva);

p.s.

ovdje se radi o varijablama - dakle nazive bi trebalo počinjati sa malim slovom - dakle prosjecnaVrijednost ili brojBrojeva,

nije nikakva obveza, ali nekakva konvencija u c obitelji jezika jest:

camelCasing - ovaj način pisanja naziva se koristi za nazive varijabli i podatkovnih članova klasa,

a PascalCasing - dakle način gdje ime počinje velikim slovom - ProsjecnaVrijednst, BrojBrojeva se koristi kod pisanja naziva funkcija, metoda, svojstava i klasa.

nije obveza - ali je konvencija (sa ciljem jasnoće i preglednosti koda).

Naravno , cast sa double vrijednošću

međutim, manje bitno, onaj način koji si naveo je i najpraktičniji i najviše se koristi:

float prosjecnaVrijednost = (float)suma/brojBrojeva;

a ovo ostalo , sam naveo btw. ako već ima statički brojčani član.

Lapsus - sorry - kako god bilo const, static ili definirano kao define - varijabla ima ime

mislio sam na korištenje brojeva: 

double rezultat = 5./nekiIntBroj;

ovdje ne dolazi do gubitka vrijednosti jer sam broj 5 odredio da je double broj, jer sam ga prikazao kao decimalnog.

my mistake

hajmo ovak, imaš 2 varijable i moraš im zamjeniti mjesta

a = 10;

b= 50;

operator '=' postavlja vrijednost onoga što je desno njemu onom elementu koji je lijevo od njega, ako napišeš a=10;, operator '=' pridodaje vrijednost 10 varijabli a

znači kad bi išao zamjenjivati vrijednosti variable a i b, prema tvome bi ispalo a=b; što bi dalo rezultat da se vrijednost koja ima varijabla b postavi kao vrijednost varijable a, i dobio bi da je a=50; i b=50; što očito nije zamjenjeno

nego, trećom, privremenom varijablom, spremamo vrijednost a (a=10) u varijablu c, napravimo a=b; (a=50, b=50), i sada postavimo orginalnu vrijednost varijable a, koju smo spremili u privremenu varijablu c, na varijablu b, radeći b=c; i dobivamo rezultat a=50, b=10

slobodno sam probaj

pa da,logićno je ,hvala na objašnjenju ;)

Pomoć može, no rješenje ne. Pokaži koliko si napravio i gdje si zapeo.

pomoći ću ti samo funkcijama koje trebaš ubaciti u if, ispunct > provjerava interpunkcije, isalpha > provjerava da li je alfabetic, ostalo možeš sam/a

izvor

edit: uf oprostite, zaboravio sam napomenuti da treba i ctype.h

Uz ovo što ti je Tracer napisao, potrudi se malo oko indentacije - tj. lijeve margine. Ovo što si napravio je nečitljivo - vitičaste zagrade su ti na krivim mjestima - ne vidi se gdje je početak, a gdje kraj bloka. Ako ne znaš sam kako indentirati kôd (zapravo mi nije jasno kako ti je to tako ispalo, s obzirom da većina editora relativno OK stavlja razmake - samo prije } treba stisnuti Enter), onda koristi online beautifiere. Imaš recimo ovaj za JavaScript, ali savršeno može poslužiti i za ovakve jednostavne primjere za C/C++ (iako valja napomenuti da je AFAIK kod C/C++-a nepisana kovencija da se blok vitičastom zagradom otvara u novom redu, dok je kod Jave/JavaScripta da se otvara na kraju tekućeg reda).

Edit - vidim da si u ovom beautifieru za koji sam dao link sam možeš odrediti kamo će ti stavljati vitičanoste zagrade.

Bome nisam ni ja znao.

hmm a stil gdje se takav kratak if natrpa u jedan red zbog meni osobno bolje preglednosti?

edit: hmm čini se da pišem najsličnije horstmannovom stilu x)