Crashowanie programu przez zawartą w nim losowość liczb?

Question

Witam Was, pracuje z książką "Przewodnik dla początkujących C++" i trafiło sie tam zadanie przy dziale losowosc by utworzyc program rozdajacy 5 kart uztykownikowi i jest ciąg dalszy ale mniejsza narazie z nim iz występuje problem nastepujacy: Program przeważnie wywala, ale zdarzy się, że zadziała normalnie i pokaże wylosowaną talię 5 kart :p. Proszę Was o pomoc, w czym moze byc problem - naciskam na to że jestem początkujący w c++ i rozwiazanie tego problemu moze okazac sie banalne haha :D Ale i tak z góry dziękuje ;) Poniżej wstawiam kod:

#include <iostream>
#include <time.h>
using namespace std;

int main()
{
    string tk[14];
    tk[0]="AS";
    tk[1]="2";
    tk[2]="3";
    tk[3]="4";
    tk[4]="5";
    tk[5]="6";
    tk[6]="7";
    tk[7]="8";
    tk[8]="9";
    tk[9]="10";
    tk[10]="JOPEK";
    tk[11]="DAMA";
    tk[12]="KROL";
    tk[13]="JOKER";

    srand(time(NULL));
    cout<<"-----------------------------------"<<endl;
    cout<<"GRA W POKERA"<<endl;
    cout<<"-----------------------------------"<<endl;

    int karta1, karta2, karta3, karta4, karta5;
    karta1=rand()%14+1;
    karta2=rand()%14+1;
    karta3=rand()%14+1;
    karta4=rand()%14+1;
    karta5=rand()%14+1;
    string rk[5];  // rozdane_karty
    rk[0] = tk[karta1];
    rk[1] = tk[karta2];
    rk[2] = tk[karta3];
    rk[3] = tk[karta4];
    rk[4] = tk[karta5];

    cout<<"TWOJA TALIA KART: "<<'\t'<<rk[0]<<'\t'<<rk[1]<<'\t'<<rk[2]<<'\t'<<rk[3]<<'\t'<<rk[4]<<'\t'<<endl;






}

 

Answer 1

karta1=rand()%14;
karta2=rand()%14;
karta3=rand()%14;
karta4=rand()%14;
karta5=rand()%14;

 

Answer 2

Nie potrzebnie do 

rand()%14

dodajesz jeszcze 1

Answer 3

Zauważ, że losujesz z zakresu 0-14, a w twojej tablicy tk ostatni element to 13.

Usuń poprostu rand()%14+1

Comments

Tak jest Panowie, właśnie zanim zajrzałem na forum sam odkryłem błąd. Wiedziałem, że błąd bedzie banalny, i tak dzięki za pomoc, trzymajcie się, pozdrawiam ;)

---

Fajnie.. a jak będzie 10 kart do rozdania (wiem... nie poker... ) to będziesz klepał 10 razy tę samą linię? Może by pętlą spróbować?

---

Siedzi już w funkcji  Ale generalnie nie skupiam sie na dalszej czesci pokera, chcialem tylko zgodnie z zadaniem zainicjowac rozdawanie