配列のポインタについて

質問

C言語で処理が下記のような処理を作成したいと思っております。
関数の引数によって、データを代入する配列を切り替えたいと考えております。
下記のような処理を考えた場合、if文の中・for文のbuffをどのように処理すれば
よいでしょうか？
アドバイスよろしくお願い致します。

int a[100][150];
int b[100][150];

void sample(int flg)
{
int i,j;
unsigned short *buff;

if(flg == 0){
buff = &a;
}else{
buff = &b;
}

for( j=0 ; j<100 ; j++ ){
for( i=0 ; i<150 ; i++ ){
buff[y][x] = data;
x++;
}
x = 0;
y++;
}
}

回答No.1

Wr5 · ベストアンサー

>int a[100][150];
>int b[100][150];

int型の二次元配列ですが…

>unsigned short *buff;

unsigned short型へのポインタで本当に大丈夫ですか？
sizeof(int) == sizeof(unsigned short)の環境でないと変にメモリ破壊を招くことになりますが……。

>for( j=0 ; j<100 ; j++ ){
>　for( i=0 ; i<150 ; i++ ){
>　　buff[y][x] = data;
>　　x++;
>　}
>　x = 0;
>　y++;
>}

このループの開始時のxとyの値はどうなっているのでしょうか？
特に理由がないのであれば、ループカウンタに使っているiとjで
buff[j][i] = data;
としてよいかと思われますがどうでしょう？

最初の型の問題を除けば、そのまま動作するかと思われます。

>buff = &a;
>buff = &b;
は、それぞれ
buff = &a[0][0];
buff = &b[0][0];
の方がよいかも知れませんが…。

回答No.4

Tacosan

int (*buff)[150];
って方針もあることはある.

しかし, x や y はどこにあるんだろう.

回答No.3

Wr5

おおぅ………
＃２さんの指摘通り…ですね。
サイズ不明では次が指せない……。
*(buff + y * 150 + x) = data;
とか。
毎回演算させる羽目になりますな。
# ある程度は最適化されるかも知れませんけど……。

回答No.2

m-take0220

＞ buff[y][x]

このように使いたいのであれば、ポインタの型としてxの部分の要素数(今回の場合は150)を指定しておかないとけいないでしょう。
でないと、buff[1][0]としたときにbuff[0][0]からどれだけ先にある要素を示しているのかがコンパイラにはわかりませんし、今の型ではbuff[0]が既にunsigned shortなので、それに[0]とか指定すると「配列ではないものに添え字がつけられた」とエラーになるはずです。