diasurgical/DevilutionX

Refactor devilution.DefaultClass::CheckCursMove()

开放

#1,995 创建于 2021年5月17日

 (2 条评论) (0 个反应) (0 位负责人)C++ (743 个派生)batch import
good first issuehelp wanted

仓库指标

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

描述

I've selected devilution.DefaultClass::CheckCursMove() for refactoring, which is a unit of 421 lines of code. Addressing this will make our codebase more maintainable and improve Better Code Hub's Write Short Units of Code guideline rating! 👍

Here's the gist of this guideline:

  • Definition 📖 Limit the length of code units to 15 lines of code.
  • Why❓ Small units are easier to analyse, test and reuse.
  • How 🔧 When writing new units, don't let them grow above 15 lines of code. When a unit grows beyond this, split it in smaller units of no longer than 15 lines.

You can find more info about this guideline in Building Maintainable Software. 📖


Good luck and happy coding! :shipit: :sparkles: :100:

贡献者指南