search

daniLOLZ / ingsw2022-AM45

Progetto ingegneria del software 2022 Gruppo AM45
5 stars 0 forks source link

avoid magic numbers #9

Closed ingconti closed 2 years ago

ingconti commented 2 years ago

if(idWizard !=0 && idWizard != 10 && idWizard != 20 && idWizard != 30) idWizard = 0;

    List<Assistant> assistants = new ArrayList<>();

    for(int cards = 1; cards <= 10; cards++){

pls use a final static..... for your "10"

Zeekuri commented 2 years ago

image We removed every magic number sitting in FactoryWizard. We'll avoid magic numbers without a proper //TODO in the future