jaxonavena / Battleship

1 stars 1 forks source link

Finish adding the ability to orient ships. #6

Closed jaxonavena closed 1 week ago

jaxonavena commented 1 week ago

Fixes: #5

We need error handling done for this @peteraw03. I think it only checks if the initial point is a valid coordinate, and there's nothing checking afterwards if things are going out of bounds or hitting another ship. It's also currently implemented as you pick one end of the ship and have it propagate in a single direction, so like we talked about, no starting in the middle and filling in the edges. This will be tricky to change if we want to in the future. If you feel like we should definitely do that, then lmk.

Some methods that might be useful for this: