Skip to main content
3 events
when toggle format what by license comment
Nov 30, 2013 at 18:10 comment added amon @SimonAndréForsberg You raise some good counter-points. You are right about where fight really belongs. Add it to the interface along edge? It would be really nice to have traits here. WRT pickAny – There Is More Than One Way To Do It, and my way does indeed seem unnecessarily complicated. In retrospect it does show that you “deferred” the design, but I wouldn't call that a “trap” (its agile, hah ha). Compared with what this site sees at other times, your's still is some beautiful and extremely well-factored code.
Nov 30, 2013 at 17:56 comment added Simon Forsberg You're right that fight doesn't belong in that class, I don't think it belongs in Items though, since it works on all IItems. You're also right that a challenge method would be useful in ItemPlayer. What I should have done is to create a Game class, but I got stuck in the get-it-done-fast trap. I don't think an ItemChoices class would be necessary here... I think the pickAny implementation belongs in the AI implementation and not in some utility class for all the options.
Nov 30, 2013 at 17:35 history answered amon CC BY-SA 3.0