プログラマ専用SNS ミクプラ

フォルダのパスと画像ファイル名をchar変数に格納し、それらをstrcatで結合した文字列をファイルのパスとして画像を呼び出すプログラムを組んでデバッグしたところアクセス違反が発生してしまいました。

作成したプログラムの大まかな構造は以下の通りです

コード:

#include "DxLib.h"
int img[100];
struct card{
	char *pic;
};struct card data[100];

void ini(){
	int i;
	for(i=0;i<100;i++){
		img[i]=LoadGraph( "pic/null.png" );data[i].pic="";
	}
}

void loadcarddata(){
	data[0].pic="ChronomalyCrystalChrononaut.png";
	img[0]=LoadGraph(strcat("pic/illu/",data[0].pic));//デバッグ停止地点
}

int WINAPI WinMain(HINSTANCE,HINSTANCE,LPSTR,int){
	ChangeWindowMode(TRUE); // ウィンドウモードに設定
	if( DxLib_Init() == -1 ){return -1 ;}//ＤＸライブラリ初期化処理 エラーが起きたら直ちに終了
	SetDrawScreen( DX_SCREEN_BACK ); //描画先を裏画面に設定

	ini();
	loadcarddata();
	WaitKey() ;				// キー入力待ち

	DxLib_End() ;				// ＤＸライブラリ使用の終了処理
	return 0 ;				// ソフトの終了 
}

デバッグ中には自動変数と言う所に表示された変数の中でdata[0].picとimgが赤く表示されていたのでこれらの変数におかしな値が入っているのだと思われますが、どこで変な値が入ってしまっているのかが分かりません。
一体何故アクセス違反が起こってしまっているのでしょうか？また、どうしたらアクセス違反が起こらないように出来るでしょうか？
どなたかよろしくお願いします。

文字列リテラルを書き換えようとしてはいけません。(未定義の動作かもしれません)
すなわち、16行目が間違っています。

wasawasa さんが書きました:また、どうしたらアクセス違反が起こらないように出来るでしょうか？

一時バッファを用いてcharの配列で表現された文字列を結合するか、std::stringを使うといいと思います。

strcat("pic/illu/",data[0].pic)が定数を書き換えようとしてます。
その先を書き換えられるかは環境依存な上に書き変えられても定数領域を破壊してしまうでしょう。ヤバイです。
mallocなどで必要十分なメモリを別に割り当てる必要があります。

あと文の後ろに文を書くのは見逃しやすいので止めたほうが良いと思います。
特に他人は見逃しますので質問には不向きな書き方です。

>文の後ろに文を書くのは見逃しやすいので止めたほうが良い
すみません、次から気をつけます

ソースの方は15~16行目を下記のように書き変えたら無事デバッグ出来ました。ありがとうございます。

コード:

        char rewrite[]="pic/illu/";//書き換え用
        char numeron[256];
	strcpy(numeron,rewrite);
	strcat(numeron,data[0].pic);
	img[0]=LoadGraph(numeron);//エラー発生地点

こうしたほうが分かりやすいですよの例を書いておきます。

コード:

		char filepath[256];
		strcpy( filepath, "pic/illu/" );//パスを設定
		strcat( filepath, data[0].pic );//ファイル名
		img[0] = LoadGraph( filepath ); //読み込み

気をつけるのは、変数名の付け方、定数で良い所は定数をそのまま使うって所でしょうか。インデントも合わせましょう。

プログラマ専用SNS ミクプラ

アクセス違反の原因が分かりません

アクセス違反の原因が分かりません

Re: アクセス違反の原因が分かりません

Re: アクセス違反の原因が分かりません

Re: アクセス違反の原因が分かりません

Re: アクセス違反の原因が分かりません

Re: アクセス違反の原因が分かりません