Ne mogu da utvrdim u cemu je problem

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Evo mucim muku sa ovim programom vec neko vreme...radi se o programu koji treba da deli random karte...radi normalno i kao sto ocekujem ali posle veceg broja ponavljanja pojavi se bag...

1. #include <iostream>
2. #include <cstring>
3. #include <cmath>
4. #include <cstdlib>
5. #include <ctime>
6. using namespace std;
7.  
8.  
9. void izvuci_kartu();
10. int random_broj(int x);
11. int prva_slobodna(int g);
12.  
13. int brojkarata=52;
14. char* redna_karta[13]={"as","dva","tri","cetri","pet","sest","sedam","osam","devet","deset","zandar","dama","kralj"};
15. char* znak_karte[4]={"herc","karo","pik","tref"};
16. bool spil[52];
17.  
18.  
19. int main() {
20. int n;
21. srand(time(NULL));
22.  
23. cout << "Koliko karata zelite da izvucete (0 za izlaz): " << endl;
24. while(true) {
25. cout << endl;
26. cin >> n;
27. if (n==0) break;
28. for (int i=0;i<n;i++) izvuci_kartu();
29.  
30. }
31.  
32.  
33. return 0;
34. }
35.  
36.  
37. void izvuci_kartu() {
38. int znk,br,n,x;
39. n=random_broj(brojkarata--);
40. if (brojkarata==0) {
41.     brojkarata=52;
42.     for (int l=0;l<52;l++) spil[l]=false;
43.     cout << "---------------------------" << endl << "Ponovo mesam" << endl << "---------------------------" << endl;
44. }
45. x=prva_slobodna(n);
46. br= x % 13;
47. znk=x / 13;
48. cout << redna_karta[br] << " - " << znak_karte[znk] << endl;
49. }
50.  
51. int random_broj(int x){
52. return rand() % x;
53. }
54.  
55. int prva_slobodna(int g) {
56. int o=-1;
57. g++;
58. while (g-- > 0) {
59.     o++;
60.     while (spil[o]) o++;
61. }
62. spil[o]=true;
63. return o;
64. }

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

A bag se ogleda u cemu?

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Nakon veceg broja izvlacenja jednostavno se ukoci i nece da nastavi da radi

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Mozda se prepuni bafer. Dodaj sledecu liniju u kod.
Include-uj cstdio i dodaj sledecu liniju:

1. fflush(stdout);

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

NIx Car ::Mozda se prepuni bafer. Dodaj sledecu liniju u kod.
Include-uj cstdio i dodaj sledecu liniju:

1. fflush(stdout);

• 3Ovo se svidja korisnicima: NIx Car, ThePhilosopher, vladasimovic
  
  Registruj se da bi pohvalio/la poruku!

Nemoj da mixujes C I/O funkcije sa C++ I/O strimovima!

Ne valja ti algo u "prva_slobodna" funkciji, dok vezbas za debagovanje mozes koristiti recimo assert funkciju, pogotovo tamo gde indexiras neki niz:

0. #include <cassert>
1. ...
2. int prva_slobodna(int g) {
3. int o=-1;
4. g++;
5. while (g-- > 0) {
6.     o++;
7.     while (spil[o]) o++;
8. }
9.  
10. // out of bounds check
11. assert( o >= 0 );
12. assert( o < 52 );
13.  
14. spil[o]=true;
15. return o;
16. }

• 0Niko još nije pohvalio poruku.
  Registruj se da bi pohvalio/la poruku!

Uspeo sam da popravim kod hvala morando
Bio si upravu...postavio sam assert funkciju i video da je greska u tome sto ta funkcija vraca vrednost preko 51, a to nastaje zato sto sam ja postavio da mi postavi vrednosti niza spil na false pre nego sto se izvrsi funkcija prva_slobodna...

Nisam ni znao za assert...hvala za savet koristicu je ubuduce