char型、char**型の取り扱いについて

Question

装置を制御する関係で
C++プログラムを扱う必要が出てきており、急遽勉強中です。

以下のようなプログラムを作ったのですが、
コンパイルは出来るのですが、実行すると
sprintfのところでSegmentation fault で止まってしまいます。
set_comの取り扱いが正しくないと思うのですが、その先が分からず困っております。

ご教授いただければ幸いです。

int main(int argc, char** argv)
{
 float enegry;
 float threshold;
 char *set_comm;

 enegry = atof(argv[1]);
 threshold = atof(argv[2]);
 sprintf(set_comm, "-setenergy %8.5f %8.5f\n", energry, threshold);
 printf("**energy & threshold : %s\n", set_com);
}

Answer 1

生成する文字列の領域が確保されていません。

#include <cstdio>
#include <cstdlib>

int main(int argc, char** argv) {
  float energy;
  float threshold;
  char *set_comm;

  // 追加:ここから
  char buff[256]; // 文字列格納領域(大きさはテキトー)
  set_comm = buff;
  // 追加:ここまで

  energy = atof(argv[1]);
  threshold = atof(argv[2]);
  sprintf(set_comm, "-setenergy %8.5f %8.5f\n", energy, threshold);
  printf("**energy & threshold : %s\n", set_comm);
}

Comments

[yamatail]

早速回答いただきありがとうございます！
無事に動作いたしました。
サイズも必要なんですね、勉強になりました。

[episteme]

サイズを気にしたくないなら:

#include <sstream>
#include <iomanip>
#include <cstdio>

int main(int argc, char** argv) {
float energy;
float threshold;

energy = atof(argv[1]);
threshold = atof(argv[2]);
std::ostringstream stream;
stream << std::setprecision(5) << "-setenergy "
<< std::setw(8) << energy
<< " "
<< std::setw(8) << threshold;

printf("**energy & threshold : %s\n", stream.str().c_str());
}