amatt13/BossFight

"Sell()" is a method on PlayerWeapon.

Aperta

#14 aperta il 22 gen 2023

Vedi su GitHub
 (0 commenti) (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

This should be moved to the ShopController or mayby a new buy/sell class. The function should be made general such that any item can be sold trough it.

We should also make sure that the player dosen't receive gold if the method fails. Likewise the player<->item relation should not be delted if the method fails.

Guida contributor

Direzione di ricerca
Identifica l'attuale metodo Sell() in PlayerWeapon, progetta un'interfaccia di vendita generale, spostalo in ShopController, assicura l'integrità transazionale.
Tech stack
csharp
Dominio
game dev
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70