extraneous test that makes code less readable

[bennetyee]

battleship-demo/core/game/src/lib.rs

Line 120 in 3b44b0e

Tile::Ship(id) if id != ship_id => false,

the if test is unnecessary because the ship is not placed on the board until L130-L134.

also, instead of using the valid boolean, this match could just continue the random-placement loop (add a label at

battleship-demo/core/game/src/lib.rs

Line 97 in 3b44b0e

loop {

might be more readable).