char*型変数の扱いが分からない

Question

はじめに、質問内容が錯綜気味であることをお許しください。問題の切り分け自体に困難を覚えている状態で、藁をもすがりたいという思いで質問させて頂きます。

Visual Studioを使って他人が書いたC++のプログラムを読んでいるのですが、その中で、**「char*型の引数`filename`が名前のファイルを新しく作って、書き込み処理をして保存する」**という目的で、以下のように`ofstream`を使った関数があります:
```
void createFile(const char* filename){
  ofstream file;
  file.open(filename);
  //以下、ファイルの書き込み処理
}
```
このスニペットでは、**もし引数に指定した新規ファイル名と同じ名前のファイルが該当のディレクトリに存在する場合、そのファイルの中身を上書きしてしまう**、というものでした。
これを避けるために、よくあるように**「同名のファイルが作成されようとしている場合、その新規ファイルのファイル名について、そのファイル名の末尾に数字を付け足すことで上書きを回避する」という機能を付けようと思いました**。そのために、[このサイト]()を真似して、まず最初に引数の`filename`と名前が一致するファイルがディレクトリに存在するかチェックして、存在しなければそのまま通して、存在するならば名前が一致しなくなるまで`filename`の中身に数字をつけ足す、というコードを実装しようとしました:
```
void createFile(char* filename){
  // filenameと一致するファイルが存在するかチェック
  ifstream fileCheck;
  fileCheck.open(filename);
  for(int i=1;;i++){
    if(!fileCheck.fail()){//同名ファイルが既に存在していた場合
      //名前が一致するファイルがなくなるまでfilenameの中身を更新
      *filename = *filename + '_' + i;
    }else{//同名ファイルが存在しない場合
      fileCheck.close();
      break;
    }
  }
  
  ofstream file;
  file.open(filename);
  //以下、ファイルの書き込み処理
}
```
引数`filename`の中身を書き換えることになるので、データ型を`const char*`からconstを取り外して`char*`単体に変更しました。
**しかし、`*filename = *filename + '_' + i;`の個所で、データ型に関するエラーが発生します。**

**(事情があり、いま元のエラーメッセージの原文を表示することが出来ません。申し訳ありません。)**

**私の憶測ですが、`filename` が`char*`型にもかかわらず、`char`型である`*filename`の中に複数の`char`型および`int`型の値を連結してできた`string型`の値を代入している、ということでしょうか？**


`char*`型というデータ型について理解できていないのが原因だと思い、調べたのですが、よくわかりません。以下に、私が行った試行錯誤を示します:
 - はじめ、`char*`を単に「`char`型のポインター」だと思い、「`char`型なら『文字』であって、『文字列』ではないから、そもそも`filename`に『ファイルの名前という文字列』を代入するのは不可能ではないか？」と思いました。しかし、このソースコードでこの`createFile`関数を使っている場面では、普通に`string`型の値を放りこんでおり、混乱しました。
 - あるサイトの解説によると「`char*`型は`string`型が出来る前の昔の記法」ということだったので、`string`型として使ってよいのかと思いました。
 - しかし、上のエラーのように、`*filename`の中にstringを放り込もうとすると、怒られます。

何が間違っているのでしょうか？

大変説明不十分で錯綜気味な質問で申し訳ないですが、ご回答頂けると幸いです。

Accepted Answer

皆様ご回答ありがとうございます。
皆様のご回答を拝見したところ、ポインタ、配列、文字、文字列周りの基礎知識の不足からきた間違いのようですので、その辺りの勉強して参りたいと思います。今回の問題の理解のために必要な勉強個所が分かり、大変な収穫でした。

なお、今回は急ぎの事もあり、y_waiwai様のご回答にあるように、char*型で渡されたfilenameを関数の内部でstringに変換して処理する、というやり方を取ることにしました。

ベストアンサーですが、みなさまのご回答があまりにも有難く甲乙つけがたいため、申し訳ありませんが、今回は無しということにさせて頂きます。(これがマナー違反でしたら申し訳ありません。)
(代わりになるか分かりませんが、皆様全てのご回答に高評価を押させて頂きます)

重ねまして、皆様ご回答ありがとうございました。

Answer

filenameのポインタの領域は、データサイズがどれだけあるのか不明なので使用しないほうが良さそうです。filenameに追記するとエラーになると思います。

例えば、新規領域を用意してファイル名を新たに作るのが良いと思います。

```C
void createFile(char* filename){
  // filenameと一致するファイルが存在するかチェック
	char temp_name[1024];
	char * filename_kouho = filename;
	for(int i=1;;i++){
		ifstream fileCheck(filename_kouho);
		if(fileCheck.is_open()){
			//新しいファイル名の領域を用意する。
			//初期化する。
	  		memset(temp_name,0, 1024);
			//元の名前の長さを取得する。
	  		int len = strlen(filename);

			//元のファイル名をコピーする。
	  		strncpy(temp_name, filename, len);

			//元のファイル名の後ろに、数字を入れる。
	  		sprintf(&temp_name[len], "%d",i);

			//次のチェック候補とする。
	  		filename_kouho = temp_name;
      		//*filename = *filename + '_' + i;
		}else{//同名ファイルが存在しない場合
			fileCheck.close();
			printf("%s
", filename_kouho);
			break;
		}
	}

	ofstream file;
	file.open(filename_kouho);
	file.close();
	printf("create file
");
  
  //以下、ファイルの書き込み処理
}

```

Answer

> あるサイトの解説によると「char*型はstring型が出来る前の昔の記法」

そのココロは、昔とは全然違うということであって、お互いに入れ替えが効くということではないでしょう。例えば、昔は時計はネジを巻いていたけどいまは電池で動くから、新しい時計にネジを付けたらなにか動くかというとそんなことは無いし、昔の時計に電池を突っ込んでもなにも起こらないです。(実際には、stringのデータをchar*型に渡したり、char*型をstring型に暗黙に変えてしまう仕組みはありますけれど)

> char*を単に「char型のポインター」だと思い

いや、それは間違ってないですよ。char*は単にchar型へのポインタです(char型**への**という読み方?呼び方?を強く推奨します)。ただ、「単にポインター」があるという言い方は、「単なるポインターではない」なにものかがあると考えているのでしょうか? そんなものはないと思いますが。
> char型なら『文字』であって、『文字列』ではない

その通りですが、では文字列ってなに? 文字の配列で、終端に'\0'がついているもの。配列は、要素となる型のデータが密に、順に並んでいるものですから、C/C++では、配列をその先頭の要素へのポインタを以て扱います。先頭の位置がわかれば、2番目、3番目...と順に辿って配列全体にアクセス出来ますからね。なので、char*型で文字列を扱える、ということになります。
ついでにここで確認しておくと、文字型というのは「(少なくとも)文字に割り当てられた範囲の数値を扱うことができる整数型」だったりします。

> 私の憶測ですが、filename がchar*型にもかかわらず、char型である*filenameの中に複数のchar型およびint型の値を連結してできたstring型の値を代入している、ということでしょうか？

これは間違い。string型の値が最初からあって、それにchar型やchar*型を絡めた演算をするなら結果がstring型になることはあります。しかし、それにはstring型が絡むことが必要であって、string型なんて全然ないところに突如string型が生まれることはありません(その意味では、質問で'string型'に言及しているところは全て誤りです。そのプログラムにstring型は登場していないので)。

char*型の値に参照演算子*を作用させたら、それはchar*が指している単独のchar型の値になります。
`*filename /* char型 */ = *filename /*char型*/ + '_' /* char型 */ + i /* int型 */;`
右辺は単なる数値計算に過ぎません。値としてはint型の範囲を取りうるので、コンパイル時のオプションによっては左辺のchar型を溢れてしまう可能性があるので警告ぐらいは出るかもしれませんが、通常特に文句を言われること無くコンパイルを通ると思います(Javaだとchar型に対するint型データの代入はエラーになりますけどね)。本当に「エラー」になったのですか?
で、この左辺が*filenameになっているあたりがやりたいことに対して全然合っていないです。
*filenameってのはつまりfilename[0]と等価なわけで、その式はfilenameの最初の文字をゴニョゴニョといじっているだけ。それはどう考えてもやりたいことと違うでしょう。

まず、ポインタの扱いについては特訓が必要に思えます。
それとは別に、string型の扱いもちゃんと知っておいたほうがいい、というところでしょうか。

聞かれていない気もしますが、どうするかと言う話てあれば
Cであれば、
```C
void createFile(const char* filebody){
  char filename[MAX_PATH]; //MAX_PATHはOSで許されるファイル名の最大長
  int i=0
  snprintf(filename,MAX_PATH,"%s_%d",filebody,i);
  File* fp=fopen(filename,"r");
```
とでもするし、
c++であれば
```C++
void createFile(const string filebody){
  string filename;
  int i=0;
  filename = filebody + '_' + std::to_string(i);
  std::ofstream file;
  file.open(filename);
```
などとすればよいのではないかと思います。

Answer

`*filename = *filename + '_' + i;` これは、「ファイル名の末尾に数字を付け足す」コードではありません。 createFile の呼び出し元で、char filename[] = "abc.txt" だったとすると、 *filename は、filename[0] で 'a'、すなわち 0x61 です。 '_' は 0x5f ですから、i が 0 の場合、 *filename + '_' + i は、0x61 + 0x5f + 0x00 で 0xc1 です。 filename は "\xc1" "bc.txt" になりました。文字コードが Shift-JIS だったら "ﾁbc.txt" です。次にトライするのが 0xc1 + 0x5f + 0x01 で 0x122 です。 filename[0] は char なので 0x22、すなわち '"' になります。 fillename は、""bc.txt" になりました。 **追記** やりたいことはこういうことでしょう。 ```C++ #include #include // ifstream, ofstream #include // to_string using namespace std; void createFile(const string& filename) { string fname = filename; int i; for (i = 1; i < 1000; i++) { ifstream fileCheck(fname); if (fileCheck.fail()) break; fname = filename + '_' + to_string(i); } if (i == 1000) return; ofstream file(fname); file << "abcdefg "; } int main() { createFile("abc"); } ``` 最初に実行すると、abc ができて、次に実行すると abc_1 ができます。ファイルは 1000個までしかできないように制限しています。

Answer

char* で渡された文字列は基本、後で変更はできない、と思っときましょう。
C++なら、stringが使えるので、それに変換してからどーこーするのがいいでしょう

関連した質問