Programiranje u C++-u - pitanja i odgovori

hahhaha, ovo zvuči kao da je 1984. :)

Čini mi se da imaš volje baviti se programmingom. To je uz intuitivnu logiku najvažnije. Jezik je čak manje bitan , bez ovog dvoje ništa od svega. SAVJET :: postani što prije friend s kompajlerom. ČITAJ debug informacije , tamo sve piše. Npr ovo gore C:\Users\David\Dropbox\C++\NoSpace\main.cpp|9|error: invalid conversion from 'const char*' to 'char'

odmah ti govori da u toj liniji radiš konverziju iz stringa() const char*  to char. Ako znaš da je tako brzo ćeš zaključiti da je problem double parentheses  " " jer njima iniciraš string. Kad znaš da je tako ,preostaje ti samo da ideš na  '  ' koji iniciraju char.

Dalje , znam da te zbunjuju kobase tipa C:\Users\David\Dropbox\C++\NoSpace\main.cpp||In function 'int main()':|

C:\Users\David\Dropbox\C++\NoSpace\main.cpp|14|error: no match for 'operator!=' in 'input[i] != space'|

C:\Users\David\Dropbox\C++\NoSpace\main.cpp|14|note: candidates are:|

c:\mingw\bin\..\lib\gcc\mingw32\4.6.2\include\c++\bits\postypes.h|223|note: template<class _StateT> bool std::operator!=(const std::fpos<_StateT>&, const std::fpos<_StateT>&)|.................

To ti kompajler samo u najsitnije detalje definira grešku nabrajajući tebi potrebno i nepotrebno.

Slikovito template <class InputIterator>vector ( InputIterator first, InputIterator last, const Allocator& = Allocator() );

u prijevodu za tebe samo znaci da mozes konstruirati recimo vector<int> prvi(drugi.begin(),drugi.end()); 

Malo sam ti pojednostavnio ali uglavnom ne boj se tih kobasa i čitaj dbg.info i praksom ćeš naučiti izdvajati samo ono bitno za rješenje greške. Vježbaj čitati debug info recimo da kopiraš dio nejasnoća iz romana koji ti ispuca kompajler-> pa na google -> open par linkova -> nađi sebi najrazumljivije objašnjenje -> ispravi gešku(). Radi to stalno , pitaj ovdje , vježbaj i sam ćeš naučti čitati te ispravljati veliki dio grešaka.

Imaš dosta propusta. Ono što sam vidio je sljedeće:

1. Stavi radije biblioteku <cstdio> umjesto <stdio.h>, čisto zato što je stdio.h C biblioteka, a cstdio C++ biblioteka, a ti pišeš C++ program.
2. Nepotrebno je stavljati "space" znak kao prvi član, nepotrebno je uopće upisivati znakove u polje u liniji u kojoj ga deklariraš da bi ga poslije mjenjao preko gets funkcije.
3. Varijabla "space" ti nije baš potrebna.
4. Varijabli "output" moraš dati vrijednost prije daljnjih operacija.
5. Ti imaš polje od 32 znaka i ponavljaš petlju 32 puta. Ako upišeš u input polje recimo "Ovo je string", kompjuter će to polje vidjeti kao "Ovo je string_+4354#"%4Gr243GfR" te će sve znakove zbrajati ukoliko nisu "space". Tebi treba ovo:

for(i=0; i < strlen(input); i++)

strlen funkcija broji broj znakova u stringu (str_ing len_gth) te se nalazi u biblioteci <cstdlib> za C++ i <stdlib.h> za C. Evo i cjeloviti kod:

#include <iostream>
#include <cstdio>
#include <cstdlib>
using namespace std;

int main()
{
   int i;
    char input[32];
   char output = 0;
    gets(input);
    cout << input << endl;
    for(i = 0; i < strlen(input); i++)
    {
       if(input[i] != ' ')
        {
            output = output + input[i];
        }
    }
    cout << output;
    return 0;
}

!!!

Koja je početna vrijednost varijable 'i'?

Zar nije deflautna vrijednost varijable 0?

Memorijska adresa?

U Basicu i još nekim jezicima jest. U C-u i C++-u nije. Čitaj što domagoj kaže.

Ukratko, problem je u pogrešno postavljenoj while petlji, umjesto nečeg ovakvog:

while (R[1]<0 || R[1]>13) {       
      cout<<" Unesite vrijednost druge karte:";
      cin>>R[1];
}

bi trebalo ići nešto ovakvo:

do {
      cout<<" Unesite vrijednost druge karte:";
      cin>>R[1];
}
while (R[1]<0 || R[1]>13);

Tako ćeš zajamčiti da se sadržaj petlje izvrši barem jedanput.

Kôd ti inače ima nekih neurednosti koje sam koliko toliko ispeglao. A tu je problem i ogromne monolitne main funkcije koja radi sve. Razbij to na više manjih funkcija, bit će se lakše snalaziti u svemu tome. Dobri kandidati za to su ti:

• onaj veliki if blok koji testira želi li korisnik sam upisati karte, ili želi da se nasumično generiraju. To bi se moglo razbiti u dvije funkcije koje bi pozivao iz maina.
• bubble sort petlja
• i sve one provjere kombinacija karata koje ispisuju vrijednost korisnikovih karata. To možeš ili sve pobacati u jednu veću zasebnu funkciju, ili svaku provjeru u vlastitu funkciju.

Još ću samo primijetiti da bubble sort ne sortira i boje. Sortiraju se samo vrste karata, tako da nakon sortiranja izgubiš vezu karata sa njihovim bojama (tj. boje ostanu na istim mjestima u polju, dok vrijednosti karata ne ostanu). To se lako omakne jer si razdvojio te dvije vrijednosti u dva različita polja. Ja bih napravio strukturu:

struct karta
{
    int karta;
    int boja;
};

I onda polja te strukture, itd...

Krivo si shvatio što se tu događa.

S i R su dva polja koja zajedno predstavljaju korisnikove karte, svaki ima 5 članova (za 5 karata).

'S' su boje za svaku od 5 karata

'R' su vrijednosti za svaku od 5 karata

npr.

R={kralj, as, dama, kralj, 10}

S={pik, pik, karo, herc, tref}

Malo je teže snaći se u nekim njegovim dizajnerskim odlukama, ali kako je čovjek lijepo rekao da je početnik nećemo mu to zamjerati previše.

Hvala vam svima na pomoci i savjetima kao sto rekoh pocetnik sam pa se ne snalazim najbolje :/

Naravno, zašto bih se ja mučio oko ičije zadaće, pa koliko ona trivijalna bila...

Hm, ovo nije jos nitko predlozio koliko mi se cini, a ako i je, rijetko se predlozi, ali jesi li probao koristiti debugger i tocno vidjeti sto se dogada? Ako ga ne znas koristiti, ovo ti je prilika da sto prije naucis, vjeruj mi jako se isplati, a i trebat ce ti u buducnosti cesto puta.

Ovisi koji IDE koristis. Code::Blocks, DevCpp, Eclipse...?

Slažem se. Debugger se ovdje često zanemaruje, a imam osjećaj i da ga dosta ljudi ne zna koristiti.

Prvo nam moraš reći koji IDE koristiš za programiranje. Neke stvari što se debuggiranja tiču su praktički univerzalne, ali detalji variraju od IDE-a do IDE-a.

Osnove su ti svugdje iste, postaviš breakpoint, označiš varijable čije vrijednosti želiš pratiti, i ideš korak po korak kroz program tražeći na kojoj liniji program počne raditi gluposti.

Daj molim te napravi kako sam ti već napisao ponovovo

Ajde da ne čačkaš po dizajnu promjeni 57.liniju u ovo

int S[5] = {9,9,9,9,9};  /*iniciraj matricu uglavnom  vecim brojevima od vrijednosti    boja*/

i 56.liniju u

int R[5] = {19,19,19,19,19};  /*iniciraj matricu uglavnom  vecim brojevima od vrijednosti    karata*/
bez da išta ostalo diraš ,cijeli kod ostaje isti. Nije ti davao ovu ili onu kartu jer prije incijalizacije matrica u whlile uvjetima barataš sa za tebe garbage values , ne znaš šta je u tom trenu na recimo R[0] , R[1]......

while (S[0]<0 || S[0]>3) {
      cout<<" Unesite boju prve karte:";
        cin>> S[0];

imaš S[0] < 0 , možda trenutna vrijednost na memorijskoj adresi na tvom kompjuteru je u tom trenu 2.1234 , i main ide jer je zadovoljen uvjet a da se tebe nije ni pitalo.

INICIRAJ obje matrice R i S i mora ti raditi.

U ovom slučaju nema indicija za potrebom  debuggera.

svejedno , kod mene

Continuing...
Program exited normally.
Debugger finished with status 0

savjet:

Još danas se riješi devC++ -a i stavi Code::Blocks 10.05

devC++ je stvar prošlosti već godinama.

Učini tako , bit ćeš iznenađen , to je moderan free compiler

EDIT

 moderan IDE a ne compiler , u svakom slučaju učini to.