amatt13/BossFight

Split RequestValidator.cs into multiple files with smaller scopes

Ouverte

#32 ouverte le 28 mars 2023

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)C# (1 fork)auto 404
Back Endgood first issue

Métriques du dépôt

Stars
 (1 étoile)
Métriques de merge PR
 (Métriques PR en attente)

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

Guide contributeur

Direction de recherche
Identifiez chaque groupe de methodes (actions du joueur, authentification, etc.) et creez des fichiers de classe partielle separes ou des classes de validation distinctes. Referencez le namespace et la structure de projet existants.
Stack technique
csharp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65