キャスト方法について

質問

C言語にてbase64デコード処理を行う独自の関数を作成することになり
putcを使用して出力するソース(以下)はあるのですが、
charの配列にデコード結果を上手く格納することができません。
何か良い方法はありませんでしょうか？

int main(int argc, char **argv, char **envp)
{
FILE *fpin, *fpout;
int c, i = 0;
unsigned char buffer[4];

if( argc < 2 ){
fpin = stdin;
}
if( argc < 3 ){
fpout = stdout;
}
else if(( fpout = fopen(argv[2],OPENMODE)) == NULL ){
return 8;
}

while(( c = getc(fpin) ) != EOF ){
if( c >= 'A' && c <= 'Z' ){
buffer[i++] = c - 'A';
}
else if( c >= 'a' && c <= 'z' ){
buffer[i++] = c + 26 - 'a';
}
else if( c >= '0' && c <= '9' ){
buffer[i++] = c + 52 - '0';
}
else if( c == '+' ){
buffer[i++] = 62;
}
else if( c == '/' ){
buffer[i++] = 63;
}
if( i >= 4 || ( i >= 0 && c == '=' )){
putc( ( buffer[0] << 2 ) | ( buffer[1] >> 4 ), fpout);
if( i >= 3 )putc( ( buffer[1] << 4 ) | ( buffer[2] >> 2 ), fpout);
if( i >= 4 )putc( ( buffer[2] << 6 ) | buffer[3], fpout);
i = 0;
}
}
}

回答No.1

wogota · ベストアンサー

デコード処理自体には特に不審な点は見当たりません。

ただ、fgetc(fpin);のfpinがstdin以外も想定している場合、このソースでは
fopenする箇所がが無いので、argcの比較部分に着目すると解決できるかも
しれません。

また、ストリーム出力部もやや問題が出てきそうです。終端が"=="で
終わる場合、余分なデータが出力されることになりますので、
c="="の場合、ストリーム出力を行ったら速やかにループから抜ける
必要が出てきます。

あと、fcloseが無いので、そのコーディングも必要ですね。

的外れであれば申し訳ありません。