TCPでのセグメンテーションエラー 配列（？）

Question

TCPを使ってブラックジャックでの対戦ゲームを作っているのですが
サーバー側とクライアント側を作っているのですが、

こちらはクライアントの質問になります。

エラー文が
int new[128];
↑この部分がエラー原因だと思うのですが

この配列の中を1000にすると
Bus error: 10

とエラーが出て

128にすると
Segmentation fault: 11
とでます。
書き込んでいけないところに書き込んでいると言うことはわかっていますが、
どこがいけないのかがわかりません。
よければ教えてくださると嬉しいです。

printf("ヒット: 0 スタンド: 1 ----->");
ここの選択をするとエラーが発生します。

#include <stdio.h>
#include <string.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <errno.h>
#include <stdlib.h>
#include <netdb.h>
#include <fcntl.h>
#include <pthread.h>



int main(int argc,char *argv[]){
    
    struct sockaddr_in server;
    int sock,i,j;
    char buf[128];
   // char buf3[128];
    char bufdh[128];
    int bufd;
    int n,ret,fd,n2;
    int connect0,write0;
    int new[128];
    /*         */
    
    printf("あなたはプレイヤーです\n");
    printf("*******************\n");
    printf("ブラックの説明を行います。\n");
    printf("ブラックジャックとはトランプの山札からカードを引いて足して、ディーラーより21に近づけた方が勝ちとなります。\n");
    printf("しかし21をオーバーするとバーストとなり負けになります。\n");
    printf("ブラックジャックは11,12,13の数字は10扱いされるので今回のゲームはA~10とします。\n");
    printf("Aが出た場合には1 OR 10 かを選ぶことができます。\n");
    printf("初期の手札自動で2つ配布されます。\n");
    printf("\n");
    printf("スタンド  : 最初に配られた二枚のカードのみで勝負する事。\n");
    printf("ヒット    : 優位なハンドを得られるまでカード引き続ける事\n");
    printf("バースト  :  手札の合計が21を超えてしまうこと\n\n\n");
    
    printf("開始するには 0 を押してください\n");
    scanf("%d",&i);
    printf("それでは開始します。\n");
    printf("*******************\n");
    printf("あなたはプレイヤーです\n");
    printf("*******************\n");

    
    int s,r,t,hit,all;
    char *hit2;
    s=rand()%10+1;
    r=rand()%10+1;
    printf("***あなたの手札は %d   %d です***\n\n",s,r );
    if(s==1){
        printf("あなたの手札にAがあるので1か10を選択してください\n" );
        printf("1 OR 10 ----->");
        scanf("%d",&t);
        s=t;
        printf("***あなたの手札は %d   %d です***\n",s,r );
    }
    if(r==1){
        printf("あなたの手札にAがあるので1か10を選択してください\n" );
        printf("1 OR 10 ----->");
        scanf("%d",&t);
        r=t;
        printf("***あなたの手札は %d   %d です***\n",s,r );
        
    }
    all=s+r;
    
    
    
    
    
    

    
    
    char *hostname =argv[1]; /* ホスト名の設定 */
    
    struct addrinfo hints, *res;
    struct in_addr addr;
    int err;
    
    memset(&hints, 0, sizeof(hints));
    
    hints.ai_socktype = SOCK_STREAM;
    hints.ai_family = AF_INET;
    
    if ((err = getaddrinfo(hostname, NULL, &hints, &res)) != 0) {
        printf("error %d\n", err);
        return 1;
        
    }
    
    addr.s_addr = ((struct sockaddr_in *)(res->ai_addr))->sin_addr.s_addr;
    inet_ntop(AF_INET, &addr, buf, sizeof(buf));
    printf("ip address : %s\n", buf);
    freeaddrinfo(res);
    
    
    sock =  socket(AF_INET,SOCK_STREAM,0);
    
    if (sock < 0) {
        perror("socket");
        printf("%d\n", errno);
        return 1;
    }
    
    server.sin_family = AF_INET;
    server.sin_port = htons(12345);
    /* 127.0.0.1   localhost */
    inet_pton(AF_INET, buf, &server.sin_addr.s_addr);
    /*        */
    connect0= connect(sock, (struct sockaddr *)&server, sizeof(server));
    
    
    if (connect0 < 0) {
        
        perror("connect");
        printf("%d\n", errno);
        return 1;
    }
    
    memset(buf, 0, sizeof(buf));
    
    
    
    printf("ディーラーがカードを引いています。少々お待ちください\n");
    n=read(sock,bufdh,sizeof(bufdh));
    if (n < 0) {
        perror("read");
        printf("%d¥n", errno);
        return 1;
    }
    printf("ディーラーの数字が確定しました。次はあなたのターンです。\n\n\n");
    bufd=atoi(bufdh);
    
    while(1){
        if(all>21){
            printf("あなたの手札は%dでバーストしてしまいました。\n",all);
            all=100;
            break;
        }
        printf("現在のあなたのカードの合計は%dです\n",all);
        printf("ヒットしますか？スタンドしますか？\n");
        printf("ヒット: 0    スタンド: 1  ----->");
        
        scanf("%s",hit2);
        hit=atoi(hit2);
        
        
         if(hit==1){
            printf("%dでスタンドします。\n",all);
            break;
        }
        else if(hit==0) {
            
            new[j]=rand()%10+1;
            printf("あなたは%dを引きました\n",new[j]);
            
            if(new[j]==1){
                printf("今引いたカードがAなのでで1か10を選択してください\n" );
                printf("1 OR 10 ----->");
                scanf("%d",&t);
                new[j]=t;
                
            }
            all=all+new[j];
            j++;
            
            
        }
        else{
        printf("0か1を入力してください\n");
        }
        
        
    }
    
    
    if(all>bufd||bufd==100){
        printf("YOU WIN!!!!!\n\n\n");
        if(bufd==100){
            printf("ディーラーはバーストしました\n");
        }
        else{
        printf("ディーラーのカードの合計値は%dでした\n",bufd);
        }

        
    }
    else if(all==bufd||bufd!=100){
        printf("DRAW.....\n");
    }
    else if(all<bufd && bufd!=100){
        printf("YOU LOSE......\n\n\n");
        
        printf("ディーラーのカードの合計値は%dでした\n",bufd);
    }
        
        else if (all==100 && bufd==100){
            printf("お互いにバーストしました\n");
        }

    
    
    
    
    
    
    
    /* 文字送信 */
  /*  write0= write(sock, argv[3], sizeof(argv[3]));
    
    if (write0 < 0) {
        perror("write");
        printf("%d¥n", errno);
        return 1;
    }
    
    n2=read(sock,buf2,sizeof(buf2));
    printf("%s \n",buf2);*/
    printf("ゲームを終了します。お疲れ様でした。\n");
    
    /* TCP セッションの終了 */ close(sock);
    
    return 0;
    
}

Answer 1

new[j]=rand()%10+1;

jには何が入ってるんでしょうか。

---

scanf("%s",hit2);

これもダメですね
入力文字列が入る場所がありません。
hit2を配列にしましょう

Comments

[退会済みユーザー]

ご回答ありがとうございます。
何も定義しなければ’０'が入っているものだと勘違いしていました。
ご指摘ありがとうございます。

[退会済みユーザー]

ちなみになのですが、手札が毎回10と8にしかならないのは何が原因なのでしょうか？

[y_waiwai]

rand関数ででてくる乱数は、とある計算で算出されます（ぐぐれば詳細が出てきます）
ということは、同じ順番で出していけば、常に同じ値が出てくるってことです。

これをどうにかするためには、この乱数の初期値を変えてやる必要があります
srand関数で初期値を変えれますが、これまた同じ初期値ではおなじ乱数がでてきてしまうのは変わりません

[y_waiwai]

で、どうするかというと、現在時刻を読み出して、これを初期値とするって事が行われてますね

https://omitakahiro.github.io/random/random_variables_generation.html

[退会済みユーザー]

ありがとうございます。
#include <time.h>
srand(time(NULL));を追加することで実装できました。
int jについてなんですが、初期化とはどういう処理なのでしょうか？
int j=0で初期化と言えるのでしょうか？

[y_waiwai]

> int j=0で初期化と言えるのでしょうか？

それでいいです。

＃そこで必要とされるのが０かどうかは私にはわかりませんが

[退会済みユーザー]

初期化したところで実行に変わりがありません。
int new[128];
ここに問題があるのではないのでしょうか？

[退会済みユーザー]

このコードはクライアント側なのですが、サーバー側は同じコードでもエラーを吐きません。
何故でしょうか、、、

[y_waiwai]

> scanf("%s",hit2);

これもダメですね
入力文字列が入る場所がありません。
hit2を配列にしましょう

[退会済みユーザー]

すいません、そのコメントを見る前に配列に直して実行したら実装できました。
長々とありがとうございました。

Answer 2

jが初期化されてませんね。

というか、もう少し関数分割した方がいいですよ。
main関数だけで実装しようとするから恐らくご自身でも追いづらい（見通しの悪い）コードになっています。
関数を小分けにするだけでもデバッグ効率上がりますよ。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
初期化というのはintの定義のところで j=0とするということでしょうか？
なるほど。関数化をしたら見やすくなるのですね。
一度してみます。ありがとうございます。