バイナリファイルと16進文字列への相互変換について
解決済
回答 3
投稿
- 評価
- クリップ 0
- VIEW 4,680
画像をバイナリとして開いたのちにそのバイナリ列と16進文字列を相互変換するプログラムを書いています。ここで問題なのが、バイナリ列から16進文字列への変換をしたのち、再びバイナリ列に戻してファイルに書き出すとサイズが半分になってしまいます。どこがおかしいのかわからなくなってしまったためアドバイスをお願いしたいです。参考にしたサイトはこちらです。
ソースを掲載いたします。第一引数からファイル名を読み、そのファイルと同じものをoutputという名前で書き出します。
#include <stdio.h>
#include <fstream>
#include <sstream>
#include <iostream>
#include <string>
#include <string.h>
#include <iterator>
using namespace std;
void BytesFromHexString(const char* str) {
int len = (int)strlen(str);
std::cout << len << std::endl;
unsigned char data[len / 2];
ofstream wf;
wf.open("output", ios::trunc);
for (int i = 0; i < len; i+=2) {
unsigned int x;
sscanf((char *)(str + i), "%02x", &x);
data[i/2] = x;
wf << data[i/2];
}
}
int main( int argc, char *argv[] )
{
std::ifstream ifs(argv[1], std::ios::binary);
if (ifs.fail())
{
std::cerr << "失敗" << std::endl;
return -1;
}
std::string str((std::istreambuf_iterator<char>(ifs)),
std::istreambuf_iterator<char>());
std::string destination;
for(int i = 0; i < (int)str.size(); ++i){
char buffer[2];
snprintf(buffer, sizeof(buffer), "%02x", (unsigned char)str[i]);
destination = destination + buffer;
}
BytesFromHexString((const char*) destination.c_str());
return 0;
}-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
+2
こんにちは。
NULL文字分のバッファを用意していないからですね。
char buffer[2]; // ← 1バイト不足してます
snprintf(buffer, sizeof(buffer), "%02x", (unsigned char)str[i]);投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
0
問題をとらえそこなってしまったようです。大変失礼しました。(またやってしまった...反省)
バイナリーで0x11は
+----+
|0x11|
+----+
それを16進数の文字列に表現すると
+----+----+
|0x31|0x31|
+----+----+
容量が２倍になることに不思議はないのです。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
0
実行環境（コンパイラ）によるかもしれませんが、以下のように修正すれば意図する動作になるかと思います。
// char buffer[2];
char buffer[3]; // 少なくとも2文字+1（終端文字）分を確保
snprintf(buffer, sizeof(buffer), "%02x", (unsigned char)str[i]);2017/03/25 18:08追記
これ以降は、本題のバイナリ操作とは関係なく、誤りを含むので無視してください。
ただし、str.size()は終端文字（\0）を含む場合、質問者の意図した動作をしないようです。
std::string str = "\x00\x01";
int len = str.size(); // 2ではなく0が返る。
すなわち、読み込んだバイナリデータをstd::stringで操作すべきではないと思います。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう！
- ただいまの回答率 90.52%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
