C言語　行列　代入/表示

C言語で、入力された数字に基づいた行列(文字列)を出力させたいのですが、数字を入力すると

zsh: segmentation fault

と出てしまいます。何がいけないのでしょうか？
printf?それとも代入がうまくいっていない...?
基礎的な部分で間違えているのかなと思います。
調べようにも具体的すぎて解決策は見つけられませんでした。
関係ないincludeとかがたくさんありますが、後々使うものなので、無視してください。

C
1#include<stdio.h>
2#include<stdlib.h>
3#include<time.h>
4#define MAX1 10
5#define MAX2 50
6
7int main(){
8  
9  char yama[MAX1][MAX2];
10  int a, b, i, j;
11  srand((unsigned)time(NULL));
12
13  printf("How many stacks and coins?\n");
14  scanf("%d %d", &a, &b);
15  for(i = 0;i < a;i++){
16    for(j = 0;j < b;j++){
17      yama[i][j] = 'X';
18    }
19  }
20  for(i = 0;i < a;i++){
21    for(j = 0;j < b;j++){      
22      printf("%s\n", yama[i][j]);
23    }
24  }
25}

例えば、
2 4
と入力すると、
XXXX
XXXX
と表示されるようなプログラムを目指しています。
よろしくお願いします。

行動規範の内容に同意します

回答3件

ベストアンサー

yama[i][j]はcharですので、printf()のフォーマット指定子は%cを使います。

参考：フォーマット指定子一覧

c
1for(i = 0;i < a;i++){
2  for(j = 0;j < b;j++){
3    printf("%c", yama[i][j]);
4  }
5  printf("\n");
6}

ただ、上記の方法はprintf()を無駄に呼びすぎです。
yama[i]の末尾にNull文字\0をつけて文字列にし、%sで表示するとよいでしょう。

c
1for(i = 0;i < a;i++){
2  for(j = 0;j < b;j++){
3    yama[i][j] = 'X';
4  }
5  yama[i][j+1] = '\0';  // 末尾にNull文字を追加して文字列にする
6}
7for(i = 0;i < a;i++){
8  printf("%s\n", yama[i]);
9}