Замечания по работе #9

[volgatech-web-development]

• [x] Нужно снимать состояние ошибки с поля, когда оно в фокусе
• [x] Подсветку border'а надо делать не инлайновыми стилями, а добавлением класса (например, .error)
• [x] Все html-конструкции должны быть в html.

  errorEmail.innerHTML = "<p id=\"email_reg\">Пользователь уже зарегистрирован!</p>";

• [x] Есть способы автоматически собрать данные с формы. Возможно этот: https://learn.javascript.ru/formdata
• [x] Это тоже делать необязательно

  const params = 'first_name=' + encodeURIComponent(dataForm.first_name) + '&email=' + encodeURIComponent(dataForm.email) + '&country=' + encodeURIComponent(dataForm.country) + '&gender=' + encodeURIComponent(dataForm.gender) + '&message=' + encodeURIComponent(dataForm.message);

• [x] Это искусственное ограничение, может привести к ошибке если ответ будет идти дольше 1 секунды. И задерживает обработку ответа, если он пришел раньше.

  setTimeout(answer, 1000);

• [x] Не стоит пользоваться boolean-переменными как строками, можно опечататься

  if (first_name == 'false') {

• [x] $my_array - не лучшее название переменной

[volgatech-web-development]

• [x] сейчас функция answer неявно использует переменную out. Лучше передать out в функцию answer как аргумент.