Ça trolle…
/
troll.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
FIX: IT belongs in a MODEL!
[troll.git]
/
src
/
main
/
java
/
com
/
codingame
/
game
/
Referee.java
diff --git
a/src/main/java/com/codingame/game/Referee.java
b/src/main/java/com/codingame/game/Referee.java
index
7a718e9
..
8f5fc97
100644
(file)
--- a/
src/main/java/com/codingame/game/Referee.java
+++ b/
src/main/java/com/codingame/game/Referee.java
@@
-81,17
+81,7
@@
public class Referee extends AbstractReferee {
case Throw:
try { player.model.consumeStones(player.stoneThrow); }
catch (Model.Player.ThrewMoreStonesThanHad e) {
case Throw:
try { player.model.consumeStones(player.stoneThrow); }
catch (Model.Player.ThrewMoreStonesThanHad e) {
- boolean FIX_IT;
- switch (league.fixLevel) {
- case SOMETIMES:
- FIX_IT = model.random.nextInt(10) > 0;
- break;
- case NEVER:
- FIX_IT = false;
- break;
- default: throw new JavaLimitationError();
- }
- if (FIX_IT) {
+ if (model.FIX_IT()) {
player.view.threwMoreStonesThanHad();
player.stoneThrow = player.model.consumeMaxStones();
}
player.view.threwMoreStonesThanHad();
player.stoneThrow = player.model.consumeMaxStones();
}
@@
-102,17
+92,7
@@
public class Referee extends AbstractReferee {
}
}
catch (Model.Player.FailedToThrowStonesAndShouldHave e) {
}
}
catch (Model.Player.FailedToThrowStonesAndShouldHave e) {
- boolean FIX_IT;
- switch (league.fixLevel) {
- case SOMETIMES:
- FIX_IT = model.random.nextInt(10) > 0;
- break;
- case NEVER:
- FIX_IT = false;
- break;
- default: throw new JavaLimitationError();
- }
- if (FIX_IT) {
+ if (model.FIX_IT()) {
player.view.failedToThrowStonesAndShouldHave();
player.stoneThrow = player.model.consumeMinStones();
}
player.view.failedToThrowStonesAndShouldHave();
player.stoneThrow = player.model.consumeMinStones();
}