base64変換プログラムについて

Question

###前提・実現したいこと
以前https://teratail.com/questions/58293にて質問して警告文は出なくなったのですが、思い通りの表示がでなくて困っています。文字列を10進数にしてそれぞれを8bitの2進数に変換している最中なのですが、

入力文字が1文字の場合は

文字列を入力してください：a
bin_encoded[0][]=01100001

と正確に表示されるのですが、

入力文字が2文字以上の場合は、

文字列を入力してください：abc
bin_encoded[0][]=01100000
bin_encoded[1][]=01100010
bin_encoded[2][]=01100011

と間違った表示がされます。

何度も見直したのですが間違いが全くわかりません。

###該当のソースコード
https://github.com/szbhdulla/base64
に記載があります。

• base64.sh

実行するための命令です(あまり関係ないと思いますが)

• base64.c

メインのソースです

• base64.h

ヘッダファイルです

• length.c

文字列の長さを求める

• ToDec.c

文字列を10進数変換する

• decToBin.c

10進数を2進数へ変換する

###補足情報(言語/FW/ツール等のバージョンなど)
コンパイラ gcc
OS archlinux

Answer 1

一ファイルしかソースをみてないですが少なくともlenに問題があります。

警告文は出なくなったのですが
何度も見直したのですが

机上で見直しを行ってもなかなかバグの原因をみつけられないとうことはよくあることです。それは知識の不足や勘違いに起因するので一通り見ても原因が見つからなければ他の方法でデバッグすべきです。よく行うのはプログラムの途中にデバッグ用のprintf文を入れて変数の中身や関数の戻り値が正しいかを確認するといった手法です。

例えばlen()関数のreturnの直前に引数や戻り値を表示してみてください。そうすればlen()が正しく実装されているのかそれとも期待外の値を返してしまっているのかといったことがわかります。そういうデバッグをある程度やれば全然やらないのに比べてバグが見つけられる可能性はぐんと上がります。

バグの箇所を教わるのは必ずしも悪くはないのですが「警告がでなくなった後に机上で見直した段階で」助けを求めるのは少々早いです。できるだけ自分で前述のようなデバッグをしてみてください。初心者のうちはそういうデバッグ経験がより重要な意味を持つと思います。

Comments

[szbhdulla]

ありがとうございます。
このデバッグは毎回していきます。

Answer 2

c言語の配列定義の場合、定義数[８]としてのアドレスの０～７の処理とします。
int buf[8];とした場合
for (i=0; i<8; i++){
printf("%d\n",buf[i]);
}
ですのでbitNum-1は、7個の配列定義となり1つ足りなくなります。

１字のみであれば、うまくいくかもしれませんが、2文字めの[0]が前の[7]と重なると思います。

Comments

[szbhdulla]

ありがとうございます。
定義自体が間違っていたのですね。
2次元配列に慣れていないので混乱していました。