2017-09-27 12 views
-5

Je fais un nettoyage de code et je veux votre AIDE Le code crée des unités et placez-les dans un unitPositionHashmap. En outre, il réduit la valeur de la production en fonction du type d'unité. Je cherche un moyen de réduire la quantité de code et la manière plus intelligente de structurer cette méthode, donc j'évite le code presque dubliqué. The codeComment minimiser plusieurs instructions if?

+0

Première instruction if redondante Pas d'impact sur le code Vous pouvez directement l'enlever –

+0

extraire le test à une méthode, passer les valeurs et les tester en les comparant aux valeurs paramètres – Stultuske

+0

@TavishAggarwal: 'TheCityHashMap.getProduction()' pourrait être nulle – Subhiksh

Répondre

0

Créer une méthode ayant un si les paramètres de bloc et utiliser pour remplir les valeurs des variables (p, valeur ConstantesDuJeu (comme ARCHER), valeur de condition ProductionAmount (comme 10). Vous aurez un si laissé dans la makeUnit

+0

Merci beaucoup –