メモリで三次元配列を表したいです。

Question

Nはキーボードから入力するものとして、N x N x N の三次元配列を確保し解放するプログラムを作成したいのですが、コンパイル時にエラーが出てしまいます。何が間違っているでしょうか？

コード
#include <stdio.h>
#include <stdlib.h>
int main()
{
    int i, j, p;
    int* x;
    int N;
    printf("N = ");
    scanf("%d", &N);

    x = malloc(N * N * N * sizeof(int));
    if ( x == NULL)
    {
        puts("Failed.");
        return 1;
    }
    for( i = 0; i < N; i++)
        for( j = 0; j < N; j++)
            for( p = 0; p < N; p++)
            x[i][j][p] = i * N * N + j * N + p;
        
    for( i = 0; i < N; i++)
      for( i = 0; j < N; j++ )
        for( i = 0; p < N; p++)
        printf("x[%d][%d][%d] = %d\n", i, j, p, x[i][j][p]);

    free( x );
    return 0;
    }

コンパイル結果
7-1.c: In function ‘main’:
7-1.c:20:17: error: subscripted value is neither array nor pointer nor vector
   20 |             x[i][j][p] = i * N * N + j * N + p;
      |                 ^
7-1.c:25:53: error: subscripted value is neither array nor pointer nor vector
   25 |         printf("x[%d][%d][%d] = %d\n", i, j, p, x[i][j][p]);
      |                                                     ^

Answer 1

【アドバイス】
int y[N][N][N]と宣言した上で、
int (*x)[N][N] = yと代入する方が無難だと思います。

【修正すべき点】
int*xと宣言した上で、
x = malloc(N*N*N*sizeof(int))をすると、
x[ ][ ][ ] = のようには使えないと思います。

【補足】
僕ならscanfを使った後は、必ずfflush関数を実行します

Comments

[thkana]

> scanfを使った後は、必ずfflush関数
scanf()の前にfflush()を行って入力に対応するプロンプトを確実に表示させるということならよいのですが、scanf()の後ですか?
fflush()をstdinに対して行って入力バッファのクリアを意図するものであったなら、それはCの標準で規定される機能ではなく、コンパイラの独自拡張に類するものですからそのつもりで使う必要があります。

[kazuma-s]

N が 200 のとき、sizeof(y) は 32000000バイト。スタックサイズを越える場合が多くて、
Segmentation fault (core dumped) になるでしょう。
int (*x)[N][N] = malloc(sizeof(int)*N*N*N); でヒープ上に領域を確保したほうがよいと思います。

[Egg-Man]

> それはCの標準で規定される機能ではなく、コンパイラの独自拡張に類するものです....

そうなんですね(驚)(*_*)
初知りです。知識不足ですいません

[Egg-Man]

【Kazuma-s】さん

仰る通りです。

[thkana]

ご参考まで
https://www.kijineko.co.jp/迷信-fflushで入力バッファをクリア/

Answer 2

可変長配列をサポートしている Cコンパイラなら、N を入力した後に
int (*x)[N][N] = malloc(N * N * N * sizeof(int));でいいでしょう。
使っているコンパイラは何ですか？

質問のコードは、出力の for文が間違っています。

Answer 3

#include <stdio.h>
#include <stdlib.h>

int get(int* ptr, int N, int depth, int row, int col) {
  return ptr[(depth*N + row)*N + col];
}

void set(int* ptr, int N, int depth, int row, int col, int value) {
  ptr[(depth*N + row)*N + col] = value;
}

int main()
{
    int i, j, p;
    int* x;
    int N;
    printf("N = ");
    scanf("%d", &N);

    x = malloc(N * N * N * sizeof(int));
    if ( x == NULL)
    {
        puts("Failed.");
        return 1;
    }
    for( i = 0; i < N; i++)
        for( j = 0; j < N; j++)
            for( p = 0; p < N; p++)
                set(x,N,i,j,p, 
                    i * N * N + j * N + p);

    for( i = 0; i < N; i++)
      for( j = 0; j < N; j++ )
        for( p = 0; p < N; p++)
            printf("x[%d][%d][%d] = %d\n", i, j, p, get(x, N, i, j, p));

    free( x );
    return 0;
}

Answer 4

x ってのはintのポインタなのに、

x[i][j][p]

こういうことはできません