base64変換プログラム完成したと思ったのですが、、、

Question

###前提・実現したいこと
入力した文字列をbase64で変換するプログラムを改良したものが完成したと思ったのですが、、、

###発生している問題・エラーメッセージ
以前https://teratail.com/questions/53500にて質問させて頂いた際にご教授頂いたecho文を使い正しい出力かみるという方法を試してみると以下のような現象が起こりました。

$ echo -n 'Hello World' | base64
SGVsbG8gV29ybGQ=  /*正しい*/

$ echo -n 'Hello World' | ./base64
SGVsbG8gV29ybA== /*正しくない*/

$ ./base64
Hello World
SGVsbG8gV29ybGQ= /*正しい*/

ご指摘があったので追記させていただきます。
./base64は自分の作った実行ファイルを起動シています。

echoとパイプラインをつかった出力では誤った値が出てくるのです。通常どおりに起動すると正しい値が表示されるのですが、、、
どうすれば改善するかを教えていただきたいです。

また良ければこのソースコードにダメ出しや改善点を教えてほしいです。
###該当のソースコード
base64.c

#include <stdio.h>
#include "base64.h"

int main(void){
	
	char pre_encode[MAX_INPUT];				//入力値
	fgets(pre_encode, sizeof(pre_encode), stdin);
	
	int length = len(pre_encode);
	
	//decへエンコード
	int dec_encoded[MAX_INPUT];			//エンコード先
	ToDec(pre_encode, dec_encoded, length);
	
	//binへエンコード
	int bin_encoded[MAX_INPUT][bitNum];	//エンコード先
	allDecToBin(dec_encoded, bin_encoded, length);

	//一列にsort
	int sorted[MAX_INPUT*length];
	sort(bin_encoded, sorted, length);

	//6つに分割
	int column_6 = (length*bitNum-1)/6+1;	//商+1 = 行数
	int divided_6[column_6][6];
	divide_6(sorted, divided_6, length*bitNum);

	//文字列に変換
	char post_lib[column_6];
	enLibrary(divided_6, post_lib, column_6);

	//最終整形
	int resultLen = column_6+4-(column_6%4);
	char post_output[resultLen];
	last_encode(post_lib, post_output, column_6);
	
	for(int i=0; i<=resultLen-1; i++){
		printf("%c", post_output[i]);
	}printf("\n");

	return 0;
}

その他のファイルはhttps://github.com/szbhdulla/base64にあります。

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

Comments

[KSwordOfHaste]

最初のechoのテストをしたディレクトリーでwhereis base64としたら何が表示されますか？そこに表示されたものは今作成中のbase64プログラムそのものですか？
->失礼、上の方はlinuxの標準のbase64のことであることがおわかりなのですね？

[szbhdulla]

はい、base64はlinuxの標準のもので、./base64は自分が作ったものを起動シているということはわかっています。

[KSwordOfHaste]

以前の質問でlenをテストしてみることを提案しましたが、ソースが変わっていないように見えます。デバッグされたのでしょうか？（念のため、自分はlength.cしかみていませんのでこのソースだけに問題があるかどうかは分かりません。）

[szbhdulla]

教えていただいたとおりにlenのテストは下のですが誤った出力はされませんでした。申し訳ないですが、何が異なるのかを教えていただけませんでしょうか。

[KSwordOfHaste]

len("") とすると結果は-1になります。この結果は自分には間違っているように見えました。

[szbhdulla]

ありがとうございます。こう解釈シたのですが、誤りでしょうか？私の作ったプログラムはキーボードによる入力を前提とシている。しかしechoはキーボード入力では無いので入力値の文字数が-1されて読み込まれて正確な値が表示されない。こういうことでしょうか？

[KSwordOfHaste]

いえ、単に次のことを指摘しているつもりです。len関数が仕様どおりになっていないのではないかということです。len関数が指定文字列の長さを返すものだとすればlen("")は0を返すべきと考えられます。しかし実際は-1を返すのでlen()関数にバグがあるように見えるというだけです。

[otn]

何故コードの全体を載せないのでしょう？他のサイトを見に行って複数のファイルを見て回る気はしないです。少なくとも私は。

Answer 1

回答ではないのですが、質問に対するコメント欄でのやりとりで自分の意図がうまく伝わっていない気がしたのでここに申し上げたいことを整理させていただきます。

echoとパイプラインをつかった出力では誤った値が出てくるのです。通常どおりに起動すると正しい...(A)

この点について自分は「そうなるかどうかやそうなる理由について」は特にアドバイスできることがありません。

また良ければこのソースコードにダメ出しや改善点を...(B)

自分はこの点についてのみコメントしているつもりです。前回の質問の回答において自分は「lenに問題(バグ)がある」と指摘しました。今回の質問でもlenは修正されていなかったので引き続き指摘しています。指摘の根拠は以下の通りです。

• len()の仕様(わたしがこうであろうと思った内容)

指定文字列の長さ（つまりNUL文字を除いたcharデータの数）を返す関数と思えたのでそう仮定しました。本来質問者さんが意図した仕様がこのとおりでなければ私の指摘は誤りである可能性もあります。

• lenの実装(ソースに基づき実際に動かした結果)

実際に動かすとわかりますが、以下のようなプログラムを動かすと結果として-1が表示されます。

int length = len("");
printf("%d", length);
-> -1が表示される

この結果はlenの仕様が上に書いたとおりなら0であるべきなので「問題がある」と指摘しました。自分が指摘しているのはbase64プログラム全体の動きのことではなくてlen関数単体の動作についてだけなのです。なぜそういう指摘をするのかといえばプログラム全体が正しく動くためには「全ての関数がそれぞれの仕様に従って正しく動くことが必要条件になるから」です。

Answer 2

len()がおかしいというのはあなたの質問で割りと何度も指摘されています。

じゃあなんでechoで正しくうごかなくて、直接起動だとうまくいくように見えるかですが、

実のところ直接起動でも正しく動いていません。
"Hello World"を入力していると思いますが、
渡されているのは"Hello World**\n**"です。

あなたのプログラムは1文字落とすというバグがあるようなので、
"Hello World"を入力したつもりで"Hello World\n"を渡し、
"Hello World\nが一文字落ちてしまって"Hello World"になってうまく動いているように見えるだけです。

Comments

[szbhdulla]

そうだったんですね。改行文字のことを完全にわすれていました。読み込んだ値から改行文字があれば消すようにして再度検討してみます

Answer 3

プログラムは読んでませんが、

$ echo -n 'Hello Worl' | base64
SGVsbG8gV29ybA==

なので、最後の1文字を落とすというバグがあるものと思います。
KSwordOfHasteさんの指摘通りじゃないでしょうか。
前の質問の回答を消化してから、次の質問をするのがいいと思いますよ。