seigot / tetris

A Tetris Game for programming education in Japanese
MIT License
30 stars 107 forks source link

GitAuto: 1回のコールバック関数実行時に1秒制約のルールはあったほうがいいのかどうか #171

Open gitauto-ai[bot] opened 1 week ago

gitauto-ai[bot] commented 1 week ago

Resolves #62

What is the feature

1秒以上の実行時間を持つコールバック関数に対する制限ルールの導入

Why we need the feature

コールバック関数の実行時間が1秒を超えると、ゲームのレスポンスが低下し、ユーザー体験に悪影響を与える可能性があります。これを制限することで、ゲームのパフォーマンスと安定性を向上させることができます。

How to implement and why

  1. コールバック関数の実行時間を測定する機能を追加します。
  2. 実行時間が1秒を超えた場合にペナルティを適用します(例:スコアに減点)。
  3. 必要に応じて、ログを記録し、問題のあるコールバック関数を識別します。

この実装により、パフォーマンスの低いコールバック関数がゲーム全体の動作に影響を与えるのを防ぐことができます。

About backward compatibility

既存のコールバック関数に対しても影響を与えないように、ペナルティの適用はオプションとして設定可能とします。これにより、従来の動作を維持しつつ、新機能を利用できるようにします。

Test these changes locally

git checkout -b gitauto/issue-62-e0adadb9-c84d-42f6-82b2-3c1cb33b5d9a
git pull origin gitauto/issue-62-e0adadb9-c84d-42f6-82b2-3c1cb33b5d9a
gitauto-ai[bot] commented 1 week ago

Committed the Check Run build (3.9) error fix! Running it again...

gitauto-ai[bot] commented 1 week ago

Committed the Check Run build (3.10) error fix! Running it again...

gitauto-ai[bot] commented 1 week ago

Committed the Check Run MSBuild (3.10) error fix! Running it again...

gitauto-ai[bot] commented 1 week ago

Committed the Check Run MSBuild (3.9) error fix! Running it again...