public override void Execute(Player player)
{
// These four lines of code here are why Board.Move() exists and putting this
// logic here feels bad. However, Board.Move() ends by called the
// Space.LandedOnBy() method, which just does too much FOR THIS ONE INSTANCE.
// So do I ruin some really nice, elegant code for this one divergence, or
// do I live with shit like this?
List <Property> properties = BoardReference.GetPropertiesInGroup("Railroads");
int distance = BoardReference.FindDistanceToNearestProperty(player.Position, properties);
player.Position += distance;
player.Position = player.Position % BoardReference.NumSpaces;
if (player.Position - distance < 0)
{
player.Bank += 200;
}
Railroad railroad = (Railroad)BoardReference.GetSpace(player.Position);
if (!railroad.IsOwned)
{
railroad.SellTo(player);
}
else if (railroad.Owner != player)
{
int rent = railroad.CalculateRent();
player.Bank -= (rent * 2);
railroad.Owner.Bank += (rent * 2);
}
}
public override void Execute(Player player)
{
// These four lines of code here are why Board.Move() exists and putting this
// logic here feels bad. However, Board.Move() ends by called the
// Space.LandedOnBy() method, which just does too much FOR THIS ONE INSTANCE.
// So do I ruin some really nice, elegant code for this one divergence, or
// do I live with shit like this?
List <Property> properties = BoardReference.GetPropertiesInGroup("Utilities");
int distance = BoardReference.FindDistanceToNearestProperty(player.Position, properties);
player.Position += distance;
player.Position = player.Position % BoardReference.NumSpaces;
if (player.Position - distance < 0)
{
player.Bank += 200;
}
Utility util = (Utility)BoardReference.GetSpace(player.Position);
if (!util.IsOwned)
{
util.SellTo(player);
}
else if (util.Owner != player)
{
Dice dice = new Dice();
int roll1;
int roll2;
(roll1, roll2) = dice.Roll();
int rent = (roll1 + roll2) * 10;
player.Bank -= rent;
util.Owner.Bank += rent;
}
}