ゲームの盤を表示させたいが、意図した表示にならない問題について

Question

以下のコード内のコメントのようなものを作りたいのですが、文字化けと、なぜか横ではなく縦に表示されてしまいます。
アドバイス等をお願い致します。
OSはmacOSでAtomで作業しており、コンパイラは最初から入っているものを使用しています。

#include <stdio.h>

// こんな盤を作りたい
//   1 2 3
// A * * *
// B * * *
// C * * *

int main() {
  unsigned short board[3][4][30] = {
    {'A', "＊", "＊", "＊"},
    {'B', "＊", "＊", "＊"},
    {'C', "＊", "＊", "＊"}};
    char y = '1';
    char i = 0;
    char j = 0;

    for (i = 0; i < 3; i++) {
      printf("  %c  ", y);
      y++;
    }
    printf("\n\n");

    for (i = 0; i < 4; i++) {
      for (j = 0; j < 4; j++) {
        if (board[i][4]) {
          printf("\n");
        }
        printf("%c", board[i][j]);
      }
    }

    return 0;
}

Answer 1

ボードのデータについて、ひとマスごとを一文字だけとする場合、
char型でchar board[3][4]にして、各要素を全て' 'で囲みます。

ひとマスごとを、文字列にしたい場合は、
char型でchar board[3][4][30]にして、各要素を全て" "で囲みます。
そのうえ、出力時はprintfのフォーマット指定には"%s"を使います。

また、

if (board[i][4]) {
          printf("\n");
        }

だと、指定した行の次の行の頭が0以外の時に改行する。
になってしまっているので毎回改行してしまいます。
行末で改行したい場合は、文字を出力した後に
if(j==3)で改行するとかにすると、4文字目の後に改行を行うようになります。

Comments

[DreamTheater]

そもそも if (board[i][4]) { はバッファー・オーバーランなので、amiyaさんご提示の
if (j == 3) に変更すべきですね。

[gyoruo]

ご回答ありがとうございます！
後々全角記号1文字を入れたいのですが、バイト数が2以上になってしまうため、やはり3次元配列でで行うしかないのでしょうか？

[gyoruo]

また、配列の箱の個数を掛け算すると256を超えるのでshort型にしたのですが、単体で見て256を超えなければcharでも大丈夫ということでしょうか？

Answer 2

こんにちは。

元コードについて

  unsigned short board[3][4][30] = {
    {'A', "＊", "＊", "＊"},
    {'B', "＊", "＊", "＊"},
    {'C', "＊", "＊", "＊"}};

そもそも３次元目の30に深い意味がなさそうですし、shortで宣言する必要もありません。
将来的に全角文字を入れる予定ということであれば、２次元のポインタ配列にしてしまっては如何でしょうか？
というかコピペして気づきましたが元コードのアスタリスクが全角文字ですね。

  char *board[3][4] = {
    {"A", "＊", "＊", "＊"},
    {"B", "＊", "＊", "＊"},
    {"C", "＊", "＊", "＊"}};

Comments

[gyoruo]

ご回答ありがとうございます！
2次元のポインタでできるかしてみますね！
アスタリスクは全角で表示させたいのでそうしました。
あと、[30]はミスで、全角3バイトなのでA＝1バイト＋\0の2バイト＋＊＝（全角3バイト+\n4バイト）*３の計[14]でした。{"A", "＊", "＊", "＊"}
全角にしなければならない場合は配列の箱の数＝バイト数なのでおかしくなると思ったので3次元でしてました。

[DreamTheater]

gyoruoさん
そもそも大きな勘違いをされているようです。
配列要素の合計と配列の型は無関係ですし、３次元目がー１時限目のサイズであるわけでもありません。
char *bouard[4] = {"A", "＊", "＊", "＊"};
は、要素数が４のポインタ配列の各要素を文字列定数Aと*で初期化しています。
この配列を３セットにしたのが私が回答したコードです。

配列及びポインタについてもう少し勉強されることをお勧めします。

[gyoruo]

ご回答ありがとうございます！
配列要素の合計と配列の型は無関係というご指摘でわかった気がします。。。
もう一度、ポインタと配列について勉強しなおします。。。