ошибка с++

[tolikprankster]

Решил попробывать написать покер.))))
Всё работает вроде нормально кроме функции по выдаче карт игроку(newcards). Работает через раз. 3 варианта работы программы:
1. Раздаёт нормально и выводит на эран.
2. Работает но не выводит карты игрока(послеследующие функции работают)
3 Вылетает.
Можете помочь разобраться в чём дело, предпологаю что проблема где-то здесь:
card pack::getcard()
player::newcards(pack& a)

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

class card{
	char suit;
	char rank;
	int id;
public:
	card(int c=-1);
	void delcard()
	{
		id=-1;
		suit='0';
		rank='0';
	}
void	showcard();
};

card::card(int c){
	id=c;
	if (id==-1)
	{
		suit='0';
		rank='0';
		return;
	}
	suit= char((id-1)/13);
	rank=char(id%13);
	switch (suit){
	case 0: suit=3;break;
	case 1: suit=4;break;
	case 2: suit=5;break;
	case 3: suit=6;break;
	}
	switch (rank){
	case 1: rank='2';break;
	case 2: rank='3';break;
	case 3: rank='4';break;
	case 4: rank='5';break;
	case 5: rank='6';break;
	case 6: rank='7';break;
	case 7: rank='8';break;
	case 8: rank='9';break;
	case 9: rank='T';break;
 	case 10: rank='J';break;
	case 11: rank='Q';break;
	case 12: rank='K';break;
	case 0: rank='A';break;
	}

	
}
void card::showcard(){
	cout<<suit<<rank<<" ";
	}

class pack{
	static const int maxcards=52;
	card cards[maxcards];
	int pos;
	
public:
	void showpack();
		pack();
	void shuffle();
	card getcard();
};

card pack::getcard()
{
	pos--;
	return cards[pos];
}
pack::pack(){
	pos=52;
for(int i=0; i<maxcards;i++)
	cards[i]=i+1;
}
void pack::shuffle(){
	card c;
	srand(time(NULL));
	for(int i=0;i<maxcards;i++)
	{int r;
		r=rand() % 52+1;
		c=cards[i];
		cards[i]=r;
		cards[r]=c;
	}
}
void pack::showpack(){
for(int i=0;i<52;i++)
{if (i%13==0) cout<<"\n";
	cards[i].showcard();
}
cout<<"\n";
}


class player{
	static int count;
	card pocket[2];
	int besthand;
	int money;
public:
	player();
	void newcards(pack& a);
	void spcards();
	void showcount()
	{cout<<count;}
};

player::player()
{
	count++;
	besthand=0;
	money=0;
	pocket[1].delcard();
}
void player::newcards(pack& a) //выдать карты игроку
{	
	pocket[0]=a.getcard();
	pocket[1]=a.getcard();
}
void player::spcards()  //показать карты игрока
{
	pocket[0].showcard();
	pocket[1].showcard();
	cout<<"\n";
}

int player::count;
int main()
{player n;
	pack a;
	a.showpack();	//вывод колоды на экран
	a.shuffle();	// тасуем колоду
	cout<<"\n";
	a.showpack();	//вывод
	n.newcards(a); //выдача карт игроку
	n.spcards();	//вывод карт игрока
	n.showcount();	//вывод кол-ва игроков
	getch();
}

[Abstraction]

1) Что говорит отладчик? Как вариант, что говорит отладочная печать, выводящая pos на каждом getcard()?
2) Воля Ваша, но shuffle() странен. r принимает значения от 1 до 52, а индекс cards может меняться от 0 до 51... гм...
3) Ужас из switch в конструкторе card можно заменить на:

const char suits[] = {3, 4, 5, 6};
if(id<-1 || id>=52) exit(EXIT_FAILURE); 
suit = suits[(id-1)/13];
rank = "A23456789TJQK"[id%13]; //Ага, можно и так
suit

[tolikprankster]

Сообщение от Abstraction

1) Что говорит отладчик? Как вариант, что говорит отладочная печать, выводящая pos на каждом getcard()?
2) Воля Ваша, но shuffle() странен. r принимает значения от 1 до 52, а индекс cards может меняться от 0 до 51... гм...
3) Ужас из switch в конструкторе card можно заменить на:

const char suits[] = {3, 4, 5, 6};
if(id<-1 || id>=52) exit(EXIT_FAILURE); 
suit = suits[(id-1)/13];
rank = "A23456789TJQK"[id%13]; //Ага, можно и так
suit

Спасибо за замечания и полезные советы. Проблема действительно была в shuffle(),
switch заменю)

[tolikprankster]

Возник ещё один вопрос по динамическому выделению памяти. Суть в следующем.
В начале функции game() создается массив игроков(player), в конце он удаляется. В конструкторе и деструкторе данного класса переменная cplayer соответственно инкреминтируется и декриментируется. Однако с каждым запуском функции game(), cplayer уменьшается на 1. Тоесть если count=5, то конструктор вызывается 5 раз, а деструктор 6.


class player{
public:
static int cplayer;
};
void game(int count)
{
player* n=new player[count];
cout<<player::cplayer<<"\n";
createpack();
showpack();
shuffle();
desk* d = new desk;
deal(n,count);
for(int i=0;i<count;i++)
n[i].spcards();
d->flop();
d->showcom();
besthand(n[2],d);
delete d;
delete [count]n;
getch();
}

[tolikprankster]

Проблема была в besthand(n[2],d);
создал конструктор копии, но всё равно здесь "besthand(n[2],d);" вызывается обычный конструктор. Почему?

player:: player(const player& a)
{	
	bhand=a.bhand;
	money=a.money;
	pocket[0]=a.pocket[0];
	pocket[1]=a.pocket[1];
}
player:: player()
{	
	cplayer++;
	bhand=0;
	money=0;
	pocket[0].clearcard();
	pocket[1].clearcard();
}

[Abstraction]

1) Почему переменная player::cplayer public?
2) Почему она не инициализируется при объявлении?
3) " delete [count]n;" Вероятно, имелось в виду delete[] n;
4) Я вижу только один вывод "cout<<player::cplayer<<"\n";". Расставьте их в разных точках функции для более точной диагностики.

[Abstraction]

создал конструктор копии, но всё равно здесь "besthand(n[2],d);" вызывается обычный конструктор. Почему?

Судя по Вашему описанию, проблема ровно обратная: при создании копии Вы не увеличиваете cplayer, а деструктор-то для неё тоже будет вызван...

[tolikprankster]

Сообщение от Abstraction

Судя по Вашему описанию, проблема ровно обратная: при создании копии Вы не увеличиваете cplayer, а деструктор-то для неё тоже будет вызван...

ой протупил. Спасибо. Всё работает!)