xoma99

Потому что значение функции test не возвращается при рекурсивном вызове. Должно быть так

<?php
function test($a) 
{
    if($a === 5) {
        return 'не возвращает значение';
    }
    
   return test($a + 1);    // Здесь нужен return 
}

echo test(1);

sandbox.onlinephpfunctions.com/code/3473fa114823e1...

Ну, на мой взгляд, с точки зрения ООП и масштабируемости, ваше решение правильнее (хотя не идеально), решение "из топа", прогнозирует победителя, исходя из характеристик здоровья и урона. Но, это будет работать лишь до тех пор, пока не изменятся условия боя, если добавить например блоки, криты, промахи, то решение из топа нужно будет тупо выбросить и переписать. А в вашем решении изменить только функцию attack.