Fork

がんばれー！

多分、CScoreIni.cs と CStage結果.cs の修正で事足りると思います。 2分しかソースを見てないので本当に多分、ですが。

なお CDTXMania.cs に「tScoreIniへBGMAdjustとHistoryとPlayCountを更新()」 ってのがいますが、ここにはその機能を追加すべきではないと思います。

あ、もう一つのチケットの方も、多分、同じです。

支援ありがとうございます！

ソース流し読みしながらやってみたいと思います。

追加すべきではない、というのも理由がありそうなのでしっかり考えてみます。

早速テスト実装させていただきました。

yyagiさんのお言葉通り、CScoreIni.cs と CStage結果.cs のみの編集となっております。

結構怪しい修正をしたので、挙動不安定などあれば報告お願い致します。

ソース拝見しました。何点かコメントをば。

• int nオート は、フラグとして0か1の値でしか使っていないようですので、bool bオート のほうがよろしいのでは。
• クリア回数をインクリメントするところで、(fromさんが作られている部分の) 他のswitch-case文では、想定外の値が入力された場合に例外をthrowするというお作法にしているようです。DTXManiaのソースの中では、これに合わせた方がよろしいかと。(私個人は通常の開発においてはdefault処理を加えれば十分かと思いますが。)

throwなりdefaultなりを入れた方がよいとコメントする理由は、後でループ回数を変更するような修正が発生するかもしれないためです。うっかりループ回数だけを変更することで気づかないバグを産みがちですが、defaultで拾って何か特別な目立つ処理を入れるなりthrowしておけばすぐ気づくでしょうから。要するに、後々バグができにくいように作りましょうってことです。

で、ソースとは関係ないのですが・・・

個人的にはこの機能自体はIRであまり使って欲しくないなぁと思ってます。クリア回数(のみ)を何かの解禁条件にされてしまうと、普段時間のない私は大変悲しいです・・・。

(fromさんが作られている部分の) 他のswitch-case文では、想定外の値が入力された場合に例外をthrowするというお作法にしているようです。

throw してるのは、this[index] 形式のときだけ……　のはずです。

さすがに「配列」の引数が範囲を超えたら throw しなきゃねって感じで。

他の switch では、実は default はあまり使ってません。

ていうか、ループの値というより種別（enum : int）で case してた箇所もたくさんあったのですが、すべて Reflector によってただの整数に成り下がってますね。（汗

Reflector によって if 文が switch 文に勝手に変えられているところも多々あります。コンチクショー（汗

以上、雑談でした。

あれ？　概要文が消えちゃってる。（汗

復元します。失礼しました。

yyagiさんが指摘してくださった通りにコミットしてみました。

throwというものを知らなかったので調べつつ。例外処理に使う感じですね。

bool変数に書き直し、default処理を加え、throwを使ってみました。仕組みがよくわからないので、間違っていれば指摘お願いします。

fromさんが仰ってくれたのもありますが、勉強ということでこのように編集させていただきました。

>個人的にはこの機能自体はIRであまり使って欲しくないなぁと思ってます。クリア回数(のみ)を何かの解禁条件にされてしまうと、普段時間のない私は大変悲しいです・・・。

ご意見ありがとうございます。

ですが、この実装はしっかり使わせていただこうと思います。もちろん大変な条件にするつもりはありません。

プレイ回数しかカウントされない現状として「曲開始→ESCの繰り返し」という行為をしてほしくないのです。

DMGではかなりの回数が必要だったりするので、そこは反省しています。

修正確認しました。いいと思います。

では修正済み完了とします。

yyagiさんfromさんありがとうございました！

この調子で次もイケるといいなぁ（汗

今更のコメントで恐縮ですが・・・

例えばベース譜面しかない曲データ(ドラム, ギター譜面が含まれない曲データ)を演奏すると、 ベースだけでなくドラム・ギターの ClearCount もインクリメントされるようです。

この場合、ベースの ClearCount だけをインクリメントすべきだと思うのですが、いかがでしょうか。 (PlayCount はそうなっています)