amatt13/BossFight

Split RequestValidator.cs into multiple files with smaller scopes

Aberta

#32 aberto em 28 de mar. de 2023

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)C# (1 fork)auto 404
Back Endgood first issue

Métricas do repositório

Stars
 (1 estrela)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

  1. Player actions:
  • PlayerCanSellWeapon()
  • PlayerCanEquipWeapon()
  • PlayerCanAttackMonsterWithEquippedWeapon()
  1. User authentication:
  • ValidateUserLoginCredentials()
  1. Chat messaging:
  • ValidateChatMessage()
  • ValidateMaxMessageRequestNumberNotExceeded()
  1. Game mechanics:
  • ValidateVoteForMonsterTier()
  1. Data validation:
  • AllValuesAreFilled()
  1. Database validation:
  • PlayerExists()
  • PlayerClassExists()
  • PlayerIsEligibleForPlayerClassAcquisition()
  • PlayerOwnsPlayerClass()

Guia do colaborador

Direção de pesquisa
Identifique cada grupo de métodos (ações do jogador, autenticação, etc.) e crie arquivos de classe parcial separados ou classes de validação separadas. Consulte o namespace e a estrutura do projeto existentes.
Pilha de tecnologia
csharp
Domain
backend
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.
1-3 horas
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
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65