[pspowner]

void DefineCPUShips() { Как я понимаю, то это функция расставляет кораблики

Это прототип, уже выглядит подругому.

if (!direction && px<=6 || direction && py<=6) isOK=1; <-- Это вообще сомнительно всё.  А если этому условию не будет соответствовать примерно n-ое кол-во раз? Как я понял ты старался сделать условия так, чтобы при генерации корабля, он не выпал за рамки самого поля.

А пофиг сколько не соответствует, будет кидать случайные числа пока не попадёт куда надо. Добавил ещё ИФов - , конечно, выглядит зловеще.

<-- Ясное дело заполняем поле, но почему 3-х палубным корабликом?

Ну что ты цепляешься к пустякам там >= для 4 палубного. Этож не дописано ещё

Я бы сделал это при помощи рекурсивной функции.

Э?