ここ、実は  `@length` 分 `each` する理由が既に失われてしまっていそう…!!
(中で戦艦の長さ分の配置場所を一気に決定しているので)
`@positions`が決定できるまで処理を繰り返す、ということを表現する必要がありそうです :turtle:

ひとつのアイデアとしては、

- 繰り返し処理に入る前に @positions を空の配列に初期化しておいて
- ここを「@positions が空の配列である限り繰り返す」という繰り返し文に変更する
- やりなおしにあたった場合は`redo` じゃなくて、再度 @positions を空に戻す

みたいな感じがあるかなあと思います :crystal_ball: