c言語 配列 返り値

質問

０～２５５の数を２進数に変換し、各８ビットずつに格納するというを作りたいのですが、
以下のような場合、reutrnで配列を返すにはどうしたらよいのですか？

int bina(int r2,int g2,int b2){
int i;
char binary[23]={0};

for(i=0;b2>0;i++){//２進数に変換する
binary[i]=b2%2;
b2=b2/2;
}
for(i=8;g2>0;i++){
binary[i]=g2%2;
g2=g2/2;
}
for(i=16;r2>0;i++){
binary[i]=r2%2;
r2=r2/2;
}

printf(" 2進数 = ");
for(i=0;i<24;i++){
printf("%d", binary[23-i]);
}
printf("\n");
return binary[23];
}

int main(void)
{
r=124,g=235,b=30
char sStr1[23];

sStr1[23]=bina(r,g,b);
for(i=0;i<24;i++){
printf("%d"sStr1[i]);
}
return(0);
}

回答No.1

kanemoto_s

呼び出し側で確保した配列を、引数に配列へのポインタを入れて渡すのが定石です。
多くの開発現場ではreturnはエラーコードの通知に使います。

まだ、ポインタや配列の記述方法が分かってない様子。ポインタと配列の基礎を学習してみてください。
関数内で確保した配列は、関数終了時にメモリが解放されるので配列のポインタで戻してはいけません。たまにstatic変数のポインタを返す人がいますが、マルチスレッドにするとエラーが出るので、バグが発見しづらい悪いコードです。

あと、人に見せる前にはコンパイルしておきましょう。文法エラーが散見されます。

int bina(int r2,int g2,int b2,char binary[]){
// char binary[23]={0}; →不要

binary[i]=b2%2; // 同じことですが →　binary[i] = b2 & 1;
b2=b2/2; // 同じことですが →　b2 = b2 >> 1;
return binary[23]; // 配列要素の24番目のcharをコピーして返しても意味がない。

char sStr1[23]; // 配列要素は24個なので → char sStr1[24];
sStr1[23]=bina(r,g,b); // → bina(r,g,b,sStr1);

return(0); // 一般的にreturnに括弧は書かない