ここを含め、`ship.positions` さんが何回か登場しているので、
そうした箇所はもう1ステップ良いコードに育てられそう匂いを感じました :seedling:
たとえば、ここの処理は、
```
@ships.all?(&:sunk?)
```
みたいにして、`Ship`側に
```
def sunk?
  @positions.empty?
end
```
という風なメソッドを用意した方が、何をしているかの見通しが良くなりそうなあと思います！
(`#miss?`とか他の `ship.positions` さんが登場しているところも同じような作戦で見通しをよくできそうです :surfer: )