tomelm/oregon_trail — comment on commit 0e9861e87a02f54146f57063efcd9cf096ea55ee
File: src/models/River.java line 52
By LPessoa3:

Thanks for letting me know, I fixed it.

I am thinking when calling an event we'd do something like:
event.doEvent(1, player, party);

with 1 representing an int that would read the xml file and decide which event it corresponds with. The player holds the player, inventory and money so sending that in will allow changes to be made to it. The party allows changes to the other characters in the party.

Just a suggestion, since it's you and Al that will work on the Event stuff ya'll decide, just tell me if any changes need to be made to River class.

----- Original Message -----
From: "tomelm"
To: "LPessoa3"
Sent: Sunday, November 6, 2011 4:43:42 PM
Subject: Re: [oregon_trail] River.java: renamed riverChoice method to choiceRiver. Added payRiver, fordRiver, caulkRiver methods. (0e9861e)

Was scanning over your stuff quickly and noticed that this won't work. You're literally setting their money to -50, any money they have will be gone and they will be broke! What you might want to do is player.setMoney(player.getMoney - 50);

---
Reply to this email directly or view it on GitHub:
https://github.com/tomelm/oregon_trail/commit/0e9861e87a02f54146f57063efcd9cf096ea55ee#commitcomment-698748