diasurgical/DevilutionX

Reduce use of globals

开放

#2,435 创建于 2021年7月20日

在 GitHub 查看贡献者指南
 (21 条评论) (0 个反应) (0 位负责人)C++ (743 个派生)batch import
good first issuehelp wanted

仓库指标

星标
 (7,255 个星标)
PR 合并指标
 (平均合并 2天 12小时) (30 天内合并 15 个 PR)

描述

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
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60