diasurgical/DevilutionX

Reduce use of globals

オープン

#2,435 opened on 2021/07/20

GitHub で見るコントリビューターガイド
 (21 件のコメント) (0 件のリアクション) (0 人の担当者)C++ (743 件のフォーク)batch import
good first issuehelp wanted

Repository metrics

Stars
 (7,255 個のスター)
PR merge metrics
 (平均マージ 2d 12h) (30d で 15 merged PRs)

説明

The following globals are being referenced a lot where it could instead have been done by reference, changing this will make functions more dependable/testable.

  • Items
  • Lights
  • Missiles
  • Objects
  • QuestData
  • Quests
  • VisionList
  • Players
  • Monsters
  • Towners

コントリビューターガイド

調査方針
リストアップされた各グローバル変数を調べ、その使用箇所を特定し、グローバルに依存する代わりに関数が変数をパラメータ（参照渡し）として受け取るようにリファクタリングします。完了としてマークされているもの（Lights, Missiles, VisionList, Monsters, Towners）からパターンを理解し、残りの項目（Items, Objects, QuestData, Quests, Players）に適用します。すべての呼び出し箇所を更新し、コードがコンパイルできることを確認してください。
技術スタック
cpp
領域
game dev
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60