search

bru7us / HS-DeckTrack

A Terminal-based Deck Tracker for Hearthstone
MIT License
19 stars 2 forks source link

Adding card odds to deck display #8

Closed juliettepaul closed 1 week ago

juliettepaul commented 10 years ago

This adds Odds in 1 or 2 draws (to cover arcane intellect, etc) to the cards left in deck display.

juliettepaul commented 10 years ago

I found one bug in the odds fixing, will update shortly

Edit: Never mind it works. I'm just a bit crazy today it seems

bru7us commented 10 years ago

Given the amount of re-used code, I'd prefer to keep it all in the same sub rather than add the extra one.

There should be a command line option to enable displaying of the odds, this option could be passed into print_cards() as an optional parameter (would only be set to true when printing the %game_hash AND the user specified the option).

juliettepaul commented 10 years ago

@bru7us Updated per your request. Let me know if that looks good.

Edit: we could use boolean, but I kept it simpler so that isn't needed to run this. Let me know if you would like me to switch it use that.

Also please forgive my perl, it's not my main language. I know this could probably be a lot slicker than it is.