diff options
author | Johannes E. Krause <j.k@eclipso.de> | 2019-03-12 13:09:33 +0100 |
---|---|---|
committer | PeterN <peter@fuzzle.org> | 2019-03-13 07:47:32 +0000 |
commit | ba3d7122dfcbbee750dad6559b8c354bec873505 (patch) | |
tree | 6837bfffcf06a5d148e1821d580201a0ace8b5ab | |
parent | dea7f078f453fd80a8c399cde9ad998f59d5ace8 (diff) | |
download | openttd-ba3d7122dfcbbee750dad6559b8c354bec873505.tar.xz |
Cleanup: Remove questionable syntax in station rating calculation
-rw-r--r-- | src/station_cmd.cpp | 30 |
1 files changed, 14 insertions, 16 deletions
diff --git a/src/station_cmd.cpp b/src/station_cmd.cpp index f7eb353a0..ec4f6739e 100644 --- a/src/station_cmd.cpp +++ b/src/station_cmd.cpp @@ -3375,27 +3375,25 @@ static void UpdateStationRating(Station *st) byte waittime = ge->time_since_pickup; if (st->last_vehicle_type == VEH_SHIP) waittime >>= 2; - (waittime > 21) || - (rating += 25, waittime > 12) || - (rating += 25, waittime > 6) || - (rating += 45, waittime > 3) || - (rating += 35, true); - - (rating -= 90, ge->max_waiting_cargo > 1500) || - (rating += 55, ge->max_waiting_cargo > 1000) || - (rating += 35, ge->max_waiting_cargo > 600) || - (rating += 10, ge->max_waiting_cargo > 300) || - (rating += 20, ge->max_waiting_cargo > 100) || - (rating += 10, true); + if (waittime <= 21) rating += 25; + if (waittime <= 12) rating += 25; + if (waittime <= 6) rating += 45; + if (waittime <= 3) rating += 35; + + rating -= 90; + if (ge->max_waiting_cargo <= 1500) rating += 55; + if (ge->max_waiting_cargo <= 1000) rating += 35; + if (ge->max_waiting_cargo <= 600) rating += 10; + if (ge->max_waiting_cargo <= 300) rating += 20; + if (ge->max_waiting_cargo <= 100) rating += 10; } if (Company::IsValidID(st->owner) && HasBit(st->town->statues, st->owner)) rating += 26; byte age = ge->last_age; - (age >= 3) || - (rating += 10, age >= 2) || - (rating += 10, age >= 1) || - (rating += 13, true); + if (age < 3) rating += 10; + if (age < 2) rating += 10; + if (age < 1) rating += 13; { int or_ = ge->rating; // old rating |