amatt13/BossFight

Split RequestValidator.cs into multiple files with smaller scopes

Aperta

#32 aperta il 28 mar 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C# (1 fork)auto 404
Back Endgood first issue

Metriche repository

Star
 (1 stella)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

  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()

Guida contributor

Direzione di ricerca
Identifica ogni gruppo di metodi (azioni del giocatore, autenticazione, ecc.) e crea file di classe parziale separati o classi di validazione separate. Fai riferimento al namespace e alla struttura del progetto esistenti.
Tech stack
csharp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65