widelands-dev team mailing list archive

[Toni Förster <toni.foerster@xxxxxxxxx>]

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.