@HedgeHotDog (ハリネズミ 🌭)posted at 2024-11-27

🔰「バッチファイルでゲーム」：ファイル構造の悩み

Discussion

コマンドプロンプト batch ゲーム制作バッチファイル

改善提案をいただきたいです

バッチファイルでRPG風ゲームを制作中なのですが、ファイル構造をどのようにすべきかがわかりません；；

ゲームの開発経験自体初めてで、右も左もわからず自己流で制作しているため非常に乱雑なファイル構造となってしまっています。

素人のわめきで申し訳ないのですが、もしよろしければアドバイス、添削など頂きたいです

いつか書いているコード自体もいくつか公開するつもりですが、公開用では無いとはいえ今回添付させて頂いているコード内容にもご意見頂ければ…！と思っています
ひどく汚いのはご容赦ください

ファイル構造

//RPGGAME(root)
INDEX.bat(このバッチからほかの色んなバッチをcallしてる)
    //EnemyData
        //Enemy_Status_Slime.bat
    //ItemData
        //ItemData.bat
    //NewGame
        //EnterYourName.bat
        //Player_Status.bat
        //ReadyForPrologue.bat
        //TextFale.bat
    //SaveData
        (ここにセーブデータ格納)
    //Sounds
        //Sounds_Dev.bat
    //Stories
        //MAPS
            //MAPDATA.txt
        //Pologue.bat
    //Systems
        AutoSaveSys.bat
        InventorySys.bat
        LoadSaveData_Utility.bat
        MainSaveSys.bat
        SaveDataDelSys.bat
        SelectSaveData.bat
        ...(多すぎるので省略)

INDEX.bat の中身

@echo off
::@echo on

:BootSector
title INDEX boot standby...
chcp 65001

set AUTOSAVE=false
set MANUALSAVE=false
set NEWGAME=false
set CONTINUE=false


rem 初期化(Initialize)わざわざcdをしなくてもいいようにパスを格納
set CD_EnemyData=%CD%\EnemyData
set CD_ItemData=%CD%\ItemData
set CD_NewGame=%CD%\NewGame
set CD_SaveData=%CD%\SaveData
set CD_Sounds=%CD%\Sounds
set CD_Stories=%CD%\Stories
set CD_Stories_MAPS=%CD%\Stories\MAPS
set CD_Systems=%CD%\Systems
set CD_Systems_Display=%CD%\Systems\Display


rem 起動時に前回のセーブデータを事前に読み込む
rem *ESDファイルのT/F判定のため

if exist "%CD_SaveData%\ESD_1.txt" (
    set ESD_1=true
) 
if not exist "%CD_SaveData%\ESD_1.txt" (
    set ESD_1=false
)
if exist "%CD_SaveData%\ESD_2.txt" (
    set ESD_2=true
)
if not exist "%CD_SaveData%\ESD_2.txt" (
    set ESD_2=false
)
if exist "%CD_SaveData%\ESD_3.txt" (
    set ESD_3=true
)
if not exist "%CD_SaveData%\ESD_3.txt" (
    set ESD_3=false
)


rem UI調整用の空白変数を読み込み(DisplayAdjustAir.bat)
call "%CD_Systems%\DisplayAdjustAir.bat"

rem セーブデータ選択画面のUIにかかわる変数の初期化
call "%CD_Systems%\SelectSaveData_Initialize.bat"

rem 今後のゲーム進行で使用するテキストの読み込み(TextFile.bat)
call "%CD_NewGame%\TextFile.bat"






echo.
echo "INDEX booting now..."
timeout /t 2 /nobreak > nul


:INDEX
title INDEX


rem 音楽の再生(仮)
start /min %CD_Sounds%\SoundsDev.bat


rem メインメニュー画面のラインデータをロードして開始
call "%CD_NewGame%\hashtag_moveline_linedata.bat"
call "%CD_NewGame%\hashtag_moveline_start.bat"



:MAINMENU
call "%CD_Systems%\MainMenu.bat"
if %errorlevel%==10 (goto :NEWGAME)
if %errorlevel%==11 (goto :CONTINUE)
if %errorlevel%==12 (goto :OPTION)
call "%CD_NewGame%\hashtag_moveline_end.bat"



:NEWGAME
set NEWGAME=true
call "%CD_Systems%\SelectSaveData.bat"
if %errorlevel%==41 (goto :MAINMENU)
if %errorlevel%==42 (goto :SaveData_1_NEWGAME)
if %errorlevel%==43 (goto :SaveData_2_NEWGAME)
if %errorlevel%==44 (goto :SaveData_3_NEWGAME)
if %errorlevel%==52 (goto :SaveData_1_NEWGAME_DEL)
if %errorlevel%==53 (goto :SaveData_2_NEWGAME_DEL)
if %errorlevel%==54 (goto :SaveData_3_NEWGAME_DEL)


:SaveData_1_NEWGAME_DEL
call "%CD_Systems%\SaveData_1_Deleter.bat"
goto :SaveData_1_NEWGAME

:SaveData_2_NEWGAME_DEL
call "%CD_Systems%\SaveData_2_Deleter.bat"
goto :SaveData_2_NEWGAME

:SaveData_3_NEWGAME_DEL
call "%CD_Systems%\SaveData_3_Deleter.bat"
goto :SaveData_3_NEWGAME


:SaveData_1_NEWGAME
set ESD_1=true
set ESD_2=false
set ESD_3=false
goto :EnterPlayerName

:SaveData_2_NEWGAME
set ESD_1=false
set ESD_2=true
set ESD_3=false
goto :EnterPlayerName

:SaveData_3_NEWGAME
set ESD_1=false
set ESD_2=false
set ESD_3=true
goto :EnterPlayerName





rem 以下からストーリー開始
rem ***************************************************************************************************************

rem プレイヤーの名前聞く
:EnterPlayerName
for /F "tokens=2" %%i in ('tasklist /FI "WINDOWTITLE eq sounds playing now*" /NH') do (set PID=%%i)
taskkill /f /pid %PID% > nul
call "%CD_NewGame%\EntreYourName.bat"

:ReadyForPrologue
call "%CD_NewGame%\ReadyForPrologue.bat"
if %errorlevel%==18 (goto :Prologue)
IF %ERRORLEVEL%==666 (GOTO :CRITICAL_ERROR)

:Prologue
call "%CD_Stories%\Prologue.bat"
if %errorlevel%==100 (cls& goto :Prologue)& rem エラーコード100予期せぬエラー
if %errorlevel%==777 (goto :ReadyForPrologue)
if %errorlevel%==900 (goto :EOF)& rem 900はセーブして終了のコード 今はEOFだが。



rem 予期しないエラーが発生しました
cls
echo. %ESC%[91mAn unexpected error occurred. [E-100:EOF]
echo. Terminate Systems.%ESC%[0m
echo %errorlevel%
call "%CD_Systems%\ErrorBeepSounds.bat"
pause > nul
goto :EOF
rem ***************************************************************************************************************

:CONTINUE
set CONTINUE=true
call "%CD_Systems%\SelectSaveData.bat"
if %errorlevel%==41 (goto :MAINMENU)
if %errorlevel%==42 (goto :SaveData_1_CONTINUE)
if %errorlevel%==43 (goto :SaveData_2_CONTINUE)
if %errorlevel%==44 (goto :SaveData_3_CONTINUE)

:SaveData_1_CONTINUE
set ESD_1=true
set ESD_2=false
set ESD_3=false
goto :LoadSaveData

:SaveData_2_CONTINUE
set ESD_1=false
set ESD_2=true
set ESD_3=false
goto :LoadSaveData

:SaveData_3_CONTINUE
set ESD_1=false
set ESD_2=false
set ESD_3=true
goto :LoadSaveData

:LoadSaveData
for /F "tokens=2" %%i in ('tasklist /FI "WINDOWTITLE eq sounds playing now*" /NH') do (set PID=%%i)
taskkill /f /pid %PID% > nul
call "%CD_Systems%\LoadSaveDataSystem_Utility.bat"
rem call先でset goto=行きたいラベル先 をやってる
goto %GOTO%



rem このへんはまたこんどやる
:OPTION
for /F "tokens=2" %%i in ('tasklist /FI "WINDOWTITLE eq sounds playing now*" /NH') do (set PID=%%i)
taskkill /f /pid %PID% > nul
cls
echo.+-------------------------------------------------++-------------------------------------------------+
echo.%P%                                                 %P%%P%                                                 %P%
echo.%P%      (A)        DELETE SAVE DATA                %P%%P%      (C)           @ECHO ON                     %P%
echo.%P%                                                 %P%%P%                                                 %P%
echo.+-------------------------------------------------++-------------------------------------------------+
echo.+-------------------------------------------------++-------------------------------------------------+
echo.%P%                                                 %P%%P%                                                 %P%
echo.%P%      (B)           EXIT GAME                    %P%%P%      (D)           SET /P                       %P%
echo.%P%                                                 %P%%P%                                                 %P%
echo.+-------------------------------------------------++-------------------------------------------------+
choice /c ABCDQ
if %errorlevel%==1 (goto :DeleteAllSaveData)
if %errorlevel%==2 (goto :EOF)
if %errorlevel%==3 (@echo on & goto :INDEX)
if %errorlevel%==4 (goto :DEVCOMMAND)
if %errorlevel%==5 (goto :INDEX)

:DeleteAllSaveData
echo. 開発段階用緊急停止pause
echo. 続行しますか？
pause > nul
rem これセーブデータ初期化バッチにつき注意（キャンセルできるようにしてます）
call "%CD_Systems%\SaveDataDeleteSystem.bat"
if %errorlevel%==80 (goto :BootSector)
if %errorlevel%==90 (goto :INDEX)

:DEVCOMMAND
set /p command="command=>"
%command%
goto :DEVCOMMAND

::*******************************************************************************************************
:CRITICAL_ERROR
COLOR 1f
CLS
ECHO.
ECHO.
ECHO. Error code: 666 is a serious error in the main save system, your save data may be corrupted.
ECHO.
ECHO. A recovery file has been generated. The cause may be determined by analyzing the generated logs.
ECHO.
ECHO. Please contact the developer to attempt a backup.
ECHO.
ECHO.
ECHO.
ECHO. E-mail:s***********3@gmail.com
TIMEOUT /T 600 
ECHO. PRESS ANY KEY TO EXIT.
PAUSE > NUL

EXIT
::*******************************************************************************************************



::EOF

もし宜しければ「もっとこうしたほうがいいよ」とか「なんでここはこうしているの？」などなど、ビシバシとコメントしてくださると本当に助かります...!!

@STSynthe posted at 2024-11-27

一つ言えることは変数名やラベル、そしてファイル名に規則性が無くバラツキ（キャメルケースとスネークケースが混在）が見られます。
別に動いているのなら、問題は無いのですが、これらの統一が無いのは個人的にとても気持ちが悪いです。

コマンド名でも大文字小文字の統一が無くコピペしたかのようなコードになってます。

BATにちゃんとした命名規則ってのは記憶してる限り無いですが、他の言語でも似たようなことをしてしまうと悪いクセがつきます。

乱雑なファイル構造

別にファイル構造自体はそこまで悪いとは思えません。
問題はBATで作ってしまってるのが原因です。

とくにクセが悪いで有名なGOTOは色々と嫌われてます。これは混乱の元です。
ただラベルに関してはCALLと組み合わせることで関数モドキができるので、それと組み合わせることで何処にまとめたらいいか自然と理解は出来るかと。

それでもどうまとめたら良いのか分からないのは経験不足としか言いようがないので、オブジェクト指向の強いJavaやC#なんかを経験すると綺麗なまとまりってのは理解出来ます（ただしコードは肥大化します）。

rem このへんはまたこんどやる

このように適当なコメントを付けがちですが、後になって何をするのか分からなくなるのはあなたです。これも混乱を招く要因と思った方が良いです。
具体的に書くか、要点だけでもまとめるクセをつけたほうがよろしいかと。

INDEX.bat

いわゆるこれがメインでしょうが本来「目次」とかの意味ですから普通にRUN.bat、ゲームタイトル.batで良いかと。

1Like

Are you sure you want to delete the question?

🔰「バッチファイルでゲーム」：ファイル構造の悩み

改善提案をいただきたいです

ファイル構造

Your answer might help someone💌