質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.93%

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

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,213

szbhdulla

score 15

前提・実現したいこと

入力した文字列を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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • szbhdulla

    2016/12/15 00:19

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

    キャンセル

  • KSwordOfHaste

    2016/12/15 00:47

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

    キャンセル

  • otn

    2016/12/15 01:24

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

    キャンセル

回答 3

+3

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

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

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

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

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

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

  • lenの実装(ソースに基づき実際に動かした結果)
    実際に動かすとわかりますが、以下のようなプログラムを動かすと結果として-1が表示されます。

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

+2

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/12/15 10:52

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

    キャンセル

0

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

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.93%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る