diasurgical/DevilutionX

Refactor devilution.DefaultClass::CheckCursMove()

Aberta

#1.995 aberto em 17 de mai. de 2021

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)C++ (743 forks)batch import
good first issuehelp wanted

Métricas do repositório

Stars
 (7.255 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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:

Guia do colaborador

Direção de pesquisa
Examine a função CheckCursMove() em Source/cursor.cpp da linha 244 a 692 e refatore a em unidades menores de no máximo 15 linhas cada, garantindo que a lógica seja preservada.
Pilha de tecnologia
cpp
Domain
game dev
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70