Since bunnybot refused to merge, I had another Idea, currently we enter the 
if-clause for every amount > 0. But we only want it to enter when: 0 < amount 
<= 6:

if (amount > 0 && amount <= 6) {
        if (amount <= 2) {
                totalchance += 6;
        } else if (amount <= 4) {
                totalchance += 4;
        } else {
                totalchance += 2;
        }
}


Wouldn't this be better?
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1818073-worker/+merge/363851
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/bug-1818073-worker.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to