Необходимо оценить умение программиста

А на какие виды работ берете человека?

Если подпорки писать в виде скриптов на 3 строки, то пойдет.

А более сложные системы, разве кто-то еще пишет на чистом PHP ?

CMS средсвами PHP+MySQL+Java по моему вполне нормальное решение, собственно программист для этого и нужен.

Скрипты писались в разное время, может поэтому стиль разный.

Помешать может только незнание PHP и лень. Кавычки вобще-то на ура экранируются. HTML код хоть как-то претендующий на валидность ИМХО стоит того, чтобы 1 раз в жизни потратить 10 мин. на поиск (в гугле, конечно 😂) "php echo кавычки" ☝

Strom, уровень программиста оценивается не годами, а его опытом. Примеры скриптов хорошие.

Экранирование кавычек это как раз к PHP относится, а не к HTML, кавычки ведь и в обычном тексте могут быть, а не только в HTML.

Смотря для чего берете человека. Небольшие скрипты для нужд SEO (и прочих служебных) он делать умеет, это видно. Думаю, делает довольно быстро. А справится ли с чем-то более сложным - не могу знать.

1. Небрежный стиль - фигурные скобки ставятся, когда захочется, переносы вокруг управляющих конструкций - тоже.

2. Довольно бестолковые имена переменных и методов.

3. Процедурный стиль, плохая декомпозиция, весь скрипт в одном методе.

4. Мешанина из HTML и PHP.

Разумеется, если человек сознательно не вылизывал скрипты, понимая, что это нечто одноразовое, и что нужно "сделать и выбросить" - мои претензии бессмысленны. Но в чуть более сложной системе (эдак от 3К строк) все эти мелочи очень усложняют жизнь.

Попросите его прислать пример кода из большого проекта, желательно с применением ООП - на таком примере будет лучше видно, есть ли у человека порядок в голове.

Для этого есть испытательный срок, устраивает-работаете, нет-расстаетесь.

Но испытательный срок он не только для работника )), он и для работодателя.

😂

А ведь достаточно почитать об экранировании символов, обратным слэшем:

echo "<form method=\"post\">";

Абсолютно согласен. По нескольким мелким функциям уровень не определишь, даже при всем желании.

Или юзать хоть какое-то подобие разделения дизайна и данных:

<?

// данные и логика

$form = new Form();

$form->loadData($_POST);

if ($form->isSent()) { 

   // какая-то обработка

}

//а теперь дизайн

?>

<form action="script.php">

<label for="f_username">Ваше имя</label>

<input id="f_username" type="text" value="<?=$form->value('username');?>" name="username" /> 

etc.

</form>

<?

Т.е. мы не "выводим HTML из PHP", а "вставляем данные в HTML, используя PHP", и в этом случае ничего не нужно экранировать - на мой вкус, такой синтаксис чуть более понятен. Хотя лучше, конечно, юзать полноценные шаблоны.

Дык PHP сам по себе полноценный шаблон.