ばばぬきプログラムについて

質問

下記のようなプログラムで4人のコンピュータにババヌキをさせるプログラムを組んでる最中なんです。
　decklistまでは表示できてもshufflelistではコンパイルは通っても実行するとデバックが起きてしまって困っています。while文をなくすとちゃんと返り値を持って表示はできるんですがwhile分で繰り返したとたんデバックが起きるんですがその理由がわかりません。どうして無理でしょうか？ご教授願います。

decklistは整列されリストに保存した山札の関数。
shufflelistはランダムで保存していく関数。
personの関数はdecklistのほうで試したところコンパイルは通るんですがこちらもデバックが起きてしまいます。

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<time.h>
#define N 53

//デッキのカードの構造体
struct card{
int t;
struct card *next;
}*deck;

//カードを配った後のそれぞれのプレイヤーの情報（仮
struct player{
int card;
struct player *nextcard;
struct player *nextturn;
};

//関数定義
struct card *talloc(void);
struct player *lalloc(void);
struct card *decklist(void);
struct card *shufflelist(void);
struct player *person(struct player *P);
void displist(void);

int main(void){
struct player *A,*B,*C,*D;
struct player *p;
p=person(A);
while(p!=NULL){
printf("%d ",p->card);
p=p->nextcard;
}
return 0;

}

//card構造体のセルの確保
struct card *talloc(void){
return (struct card *)malloc(sizeof(struct card));
}

//player構造体のセルの確保
struct player *lalloc(void){
return (struct player *)malloc(sizeof(struct player));
}

//カードを切る前の整列された山札の関数
struct card *decklist(void){
int i,add=1,count=0;
struct card *p;
deck=NULL;
for(i=0;i<53;i++){
if(count==4){
add+=1;
count=0;
}
p=talloc();
p->t=add;
p->next=p;
count++;
p->next=deck;
deck=p;
}
return deck;
}

//山札をシャッフルした後の山札の関数
struct card *shufflelist(void){
int i,a,count=0,r;
struct card *d,*p,*q,*shuffle;
p=decklist();
shuffle=NULL;

while(p!=NULL){
srand(time(NULL));
r=(int)rand()%(N-count);
for(i=0;i<r;i++){
p=p->next;
}
q=talloc();
q->t=p->t;
d=p->next;
p->next=d->next;
q->next=shuffle;
shuffle=q;
count++;
p->next=p;
}

return shuffle;
}

//プレイヤーへカードを配るための関数
struct player *person(struct player *PERSON){
struct player *a;
struct card *library;
int i;
PERSON=NULL;
library=decklist();
a=lalloc();
while(a!=NULL){
a->card=library->t;
for(i=0;i<4;i++){
library=library->next;
}
a->nextcard=a;
a=PERSON;
}

a->nextcard=NULL;
while(a!=NULL){
printf("%d ",a->card);

}

return PERSON;

}

回答No.6

7o8 · ベストアンサー

とりあえず、カードを抜く処理について、抜いたカードの前後に対する
処理の一例として理解いただければ、いいかと思います。

が、実はシャッフルであれば、もっとお手軽にできます。
そう、カードの交換です。
------------------------------------------------------------
//山札をシャッフルした後の山札の関数
struct card *shufflelist3(void){
int i,j,cnt,r1,r2,t;
struct card *p,*p1,*p2;

srand(time(NULL));
cnt=rand()%53+30;

p=decklist();
for(j=0;j<cnt;j++){
p1=p2=p;
r1=(int)rand()%53;
r2=(int)rand()%53;
if(r1==r2) continue;
for(i=0;i<r1;i++) p1=p1->next;
for(i=0;i<r2;i++) p2=p2->next;

// r1番目とr2番目の入れ替え
t=p1->t;
p1->t=p2->t;
p2->t=t;
}
return p;
}
------------------------------------------------------------

で、気付いたかもしれませんが、ババ抜きにおいて山札は枚数が決まっていて、
カードが配布されたら０になるべきものであるので、配列でOKだったりします。
ポインタ構造は不定枚数の時に効果を発揮しますが、なにぶん、10枚目！といった際に
10枚目を探すことになりますので、例えば p[9]と一発で指定できる配列には
かないません。
私はプログラムは一回作ったら必ず見直すようにしています。
配列版にすると全体的な影響が大きすぎるので、プログラムは提示しませんが
上記よりものすごくシンプルになったりします。
＃時間がない、といっている中、同じところばかり見ていてすみません。
＃ただ、バグが発生しやすいCはプログラムの見直し、及び、確実な仕様の理解は必須です。

> 自分の頭がコンピュータ側としての理解の仕方に合わせれてないことを痛感しました。
> もうちょっと練り考えて見たいと思います。

そんなにコンピュータ側にあわせる必要もないです。
前に提示した私のサンプルはポインタのリスト構造が後ろに伸びていく、ただ
それをイメージしただけで、コンピュータ側にあわせたわけではないです。
カードを抜く処理が言葉だけでは伝わらなかったので、サンプルを作ろうと思ったのですが
ちょっと思うようにいかず、若干ですが、リスト構造作成の方に手を入れさせて頂きました。
私としてはカードを抜く処理が伝われば、それでいいです。

で、他処理ですが、

・プレーヤーにカードを配る処理
　→A,B,C,Dはグローバル変数にしてしまってもいいのではないでしょうか？
　　その上で、これからもらうカードとしてa,b,c,d、及び、その手前をaa,bb,cc,ddとして
　　山札から1枚引く関数を実行、カードが無くなったらループから脱出、って感じに
　　すればいいでしょう。
　　山札から1枚引く関数は前に提示した関数を参考にすればいいかと思います。

・ゲーム実行
　4人の内3人はコンピュータでしょうか？
　ババ抜きは本来は相手の表情等を伺っての戦略が必要ですが、流石にそういった処理は
　難しいので、何も戦略がないとすれば、単に乱数を使用しカードを抜いていく、といった
　処理をすればいいだけとなります。
　これも山札から1枚抜く処理が主立ったものになるかと思います。
　言葉で書くより面倒な処理は多いかとは思いますが、慣れればすぐにできるようになります。
　がんばってみてください。

お礼

助言ありがとうございます。
　確かに枚数が決まってる処理には不定なリストよりも配列のほうが直接ほしい値に接続できるのが利点でここでは走するべきでした。自分がリストを使おうと思ったのは最初の整列された山札からシャッフルの山札を作るときに配列だとランダムで配列を指定してそこのあたいをマタ別の配列に入れるときランダムで指定する配列の値がかぶってしまうのがきになりリストで実現するほうがイイと思ったのでリストにしたのがおっしゃるとおりむしろ難易度を上げてる気がしてなりませんね。。。。
　関数での使い方が凡庸性が高すぎて逆に混乱してしまうことが多々あり山札で一枚引かせる関数などその動作でどこまで細かく分けるのかまだよくわからないことが多いので確認していきたいと思います。
　ゲーム実行ではこのばばぬきプログラムはパソコンにババヌキの処理をやらせたいので作ろうとしました。設定はよくありがちな４人くらいが一般的かと思いそうしました。
　戦略というよりは個人的につけてみたい機能として「ジョーカーがあるとある動作をする」っという人間らしい考え方を組み込みたいと思ってますが時間がないのでそこまで手が出せるかわからないのが現状です。なので今は短的にプログラムの完成を目指して生きたいと思ってます。
　リストでの実現はこれで勉強になるのですが、ただ起源もあり、配列の場合のプログラムが探したところあったのでとりあえずこちらを優先的にプログラムを完成させ、リストでのプログラムも後々完成させたいと思っています。そちらの法に力を入れたいのでこの質問版はとりあえず〆させていただきます。
　勝手ながらプログラムそのものを変えてしまって7o8さんにはご迷惑かけました。すみません。
ただ今後に生かせるようにがんばっていきます！！ありがとうございました。お世話になりました。