C++ のsprintfの使い方

Question

引数に、ファイルパスOutFileNameと数numberを指定して、
その数の分ファイルを「ファイルパス_xxx」という名前で作るというプログラムを作りたいのですが、

sprintfでファイル名を作る際、
SaveFileName = sprintf("%s_0%2i", OutFileName, index)
のところで
intからchar*への無効な変換です
とエラーが出ます。

どこが悪いのでしょうか。
お詳しい方、アドバイスいただければ幸いです。

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char** argv)
{

  FILE *fp;

  char *OutFileName;
  char *SaveFileName;
  int number;
  int index = 0;

    OutFileName = argv[1];
    printf("%s\n", OutFileName);
    number = atoi(argv[2]);


    for (index = 0; index < number; index++)
    {
        SaveFileName = sprintf("%s_0%2i", OutFileName, index);
        fp = fopen(SaveFileName, "w");
        for (int i = 0; i < 5; i++) fprintf(fp, "%i\t%i\n", i, i+i); // write file
　    printf("\n");
   　 fclose(fp);
    }
  return 0;
}

Comments

[int32_t]

sprintf()のマニュアルを読んだり、「sprintf 使い方」「sprintf サンプルコード」などでウェブ検索してみましたか?

Answer 1

        char SaveFileName[FILENAME_MAX];
        snprintf(SaveFileName, FILENAME_MAX, "%s_0%2i", OutFileName, index);

• ライブラリの使い方で問題が出たら、まずはマニュアルを読んだりサンプルコードを探して自分のコードとの違いを見つけましょう。
• そもそも sprintf() はたいへん危なっかしい関数なので、snprintf() などを使いましょう。

Comments

[yamatail]

初歩的な問題に親切にアドバイス頂きありがとうございます。snprintf()すら知りませんでした。感謝いたします。

Answer 2

[別解] C++っポい版:

#include <iostream>
#include <iomanip>
#include <sstream>
#include <fstream>
#include <string>

int main(int argc, char** argv) {

  const std::string OutFileName = argv[1];
  int number = std::stoi(argv[2]);

  for ( int index = 0; index < number; ++index ) {
    std::ostringstream stream;
    stream << OutFileName << "_" << std::setfill('0') << std::setw(3) << index;
    std::string SaveFileName = stream.str();

    std::ofstream file(SaveFileName);
    for (int i = 0; i < 5; i++) {
      file << i << '\t' << i+1 << std::endl;
    }
  }
}