Commit: 290 - OpenXOPS (svn)

X operations（XOPS）に非常に近いFPSゲームを制作・リメイクし、成果物をオープンソースとして公開することを目的としたプロジェクトです。

Commit MetaInfo

• Revision: 290 (tree)
• Zeit: 2022-02-19 11:52:48
• Autor: xops-mikan

Log Message

iniファイルクラスの値取得処理を改善

Ändern Zusammenfassung

• modified: trunk/datafile.cpp
• modified: trunk/datafile.h

Diff

--- trunk/datafile.cpp	(revision 289)
+++ trunk/datafile.cpp	(revision 290)
@@ -1397,20 +1397,22 @@
 //! @param sectionname セクション名（NULL可）
 //! @param keyname キー名
 //! @param defaultvalue デフォルト値
-//! @param errorcode エラーコードを取得するポインタ（NULL可）
-//! @return 値
+//! @param valuestr 値
+//! @param strbuflen 最大文字数（-1で無効）
+//! @return エラーコード
 //! @attention ファイルが開かれていないか キーが見つからなければ、デフォルト値を返します。
+//! @attention 取得した文字列が最大文字数を超えている場合、最大文字数まで切り取ります。
 //! @note エラーコード　1：ファイルが開かれていない、2：値が見つからない
-char* INIFileInterface::GetINIFileString(char *sectionname, char *keyname, char *defaultvalue, int *errorcode)
+int INIFileInterface::GetINIFileString(char *sectionname, char *keyname, char *defaultvalue, char *valuestr, int strbuflen)
 {
 	int state = 0;
-	char readline[256];
-	char buf[256];
+	char readline[512];
+	char buf[512];
 
 	//ファイルが読み込まれていなければ失敗
 	if( inifp == NULL ){
-		if( errorcode != NULL ){ *errorcode = 1; }
-		return defaultvalue;
+		strcpy(valuestr, defaultvalue);
+		return 1;
 	}
 
 	fseek(inifp, 0, SEEK_SET);
@@ -1424,7 +1426,12 @@
 		state = 1;
 	}
 
-	while( fgets(readline, 256, inifp) != NULL ){
+	//データサイズ変数チェック
+	if( strbuflen <= -2 ){ strbuflen = 0; }
+	if( strbuflen == -1 ){ strbuflen = 512; }
+	if( strbuflen > 512 ){ strbuflen = 512; }
+
+	while( fgets(readline, 512, inifp) != NULL ){
 		//コメント行なら無効
 		if( readline[0] == ';' ){ continue; }
 
@@ -1446,7 +1453,7 @@
 			//一文字目が"["なら無効
 			if( readline[0] == '[' ){ break; }
 
-			char readline2[256];
+			char readline2[512];
 			int strcnt = 0;
 			int quotationmode = 0;
 			char str[64];
@@ -1490,15 +1497,20 @@
 			if( strcmp(buf, str) == 0 ){
 				state = 1;
 
+				//データサイズ調整
+				strcpy(buf, &(readline2[strlen(str)]));
+				buf[strbuflen-1] = '\0';
+
 				//値を返す
-				return &(readline2[strlen(str)]);
+				strcpy(valuestr, buf);
+				return 0;
 			}
 		}
 	}
 
 	//値が見つからなければエラーを返す
-	if( errorcode != NULL ){ *errorcode = 2; }
-	return defaultvalue;
+	strcpy(valuestr, defaultvalue);
+	return 2;
 }
 
 //! @brief int値を取得
@@ -1513,9 +1525,11 @@
 {
 	char defaultint[64];
 	char buf[64];
+	int error;
 
 	sprintf(defaultint, "%d", defaultvalue);
-	strcpy(buf, GetINIFileString(sectionname, keyname, defaultint, errorcode));
+	error = GetINIFileString(sectionname, keyname, defaultint, buf, 64);
+	if( errorcode != NULL ){ *errorcode = error; }
 	return atoi(buf);
 }
 
@@ -1531,9 +1545,11 @@
 {
 	char defaultfloat[64];
 	char buf[64];
+	int error;
 
 	sprintf(defaultfloat, "%f", defaultvalue);
-	strcpy(buf, GetINIFileString(sectionname, keyname, defaultfloat, errorcode));
+	error = GetINIFileString(sectionname, keyname, defaultfloat, buf, 64);
+	if( errorcode != NULL ){ *errorcode = error; }
 	for(int i=strlen(buf)-1; i>0; i--){
 		if( buf[i] == 'f' ){ buf[i] = '\0'; }
 		if( buf[i] == 'F' ){ buf[i] = '\0'; }
@@ -1553,6 +1569,7 @@
 {
 	char defaultbool[64];
 	char buf[64];
+	int error;
 
 	if( defaultvalue == false ){
 		strcpy(defaultbool, "false");
@@ -1560,7 +1577,8 @@
 	else{
 		strcpy(defaultbool, "true");
 	}
-	strcpy(buf, GetINIFileString(sectionname, keyname, defaultbool, errorcode));
+	error = GetINIFileString(sectionname, keyname, defaultbool, buf, 64);
+	if( errorcode != NULL ){ *errorcode = error; }
 	for(int j=0; j<=(int)strlen(buf); j++){ if(('A'<=buf[j])&&(buf[j]<='Z')){ buf[j] += 32; } }
 	if( strcmp(buf, "false") == 0 ){
 		return false;

--- trunk/datafile.h	(revision 289)
+++ trunk/datafile.h	(revision 290)
@@ -230,7 +230,7 @@
 	INIFileInterface();
 	~INIFileInterface();
 	bool LoadINIFile(char *fname);
-	char* GetINIFileString(char *sectionname, char *keyname, char *defaultvalue, int *errorcode);
+	int GetINIFileString(char *sectionname, char *keyname, char *defaultvalue, char *valuestr, int strbuflen);
 	int GetINIFileInt(char *sectionname, char *keyname, int defaultvalue, int *errorcode);
 	float GetINIFileFloat(char *sectionname, char *keyname, float defaultvalue, int *errorcode);
 	bool GetINIFileBool(char *sectionname, char *keyname, bool defaultvalue, int *errorcode);