Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ShipHistory object idea-- decide how many moves are needed to get back to shipyard? Structuring ideas in general? #60

Open
dgets opened this issue Dec 29, 2018 · 1 comment
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Comments

@dgets
Copy link
Owner

dgets commented Dec 29, 2018

I was just thinking that, for a few purposes in the game (including the scuttle processing and perhaps for some purposes in calculating what moves to begin working with early_blockade at), it may be a good idea to toss some processing into the current_assignments object, or (probably a better idea) somewhere in the MapChunk object to determine exactly how many moves it will take from whatever position to return to the shipyard in an ideal scenario.

I'm just not sure how that should fit in with things exactly; I hate to put it out there as a static method, knowing that it should be handled in a different way if I'm going to be processing things the 'right' OO way... We'll see how impatient I get, I guess. I'm totally open to ideas on the placement of something like this, or better ideas for structuring.

@dgets dgets added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Dec 29, 2018
@dgets dgets added this to the just some gravy milestone Dec 29, 2018
@dgets dgets self-assigned this Dec 29, 2018
@dgets
Copy link
Owner Author

dgets commented Jan 5, 2019

Okay, so it looks like this really isn't necessary; I didn't check the existing hlt.* libraries, and it looks like hlt.game_map.GameMap has an existing method called calculate_distance which will, unsurprisingly, do exactly what we need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant