Как по коду понять, написал ли его начинающий разработчик или более опытный и что выдаёт "начинающий" уровень вне зависимости от количества лет опыта. Если у вас есть эти ошибки в портфолио — поправьте, чтобы улучшить впечатление :)
Сразу небольшой дисклеймер))
🔹У качества кода нет прямой зависимости от количества лет опыта;
🔹Код может работать, но показывать с не очень выгодной стороны;
🔹Пост основан на личной планке, поэтому для кого-то мои мерки могут быть "а что тут такого? Нормально же")) Моя политика такая: планку нужно постоянно повышать, хотя бы медленно, а не стоять на месте и тем более не понижать)
1️⃣Неаккуратные горизонтальные отступы
Горизонтальные отступы показывают вложенность кода. Если эти отступы не соблюдаются и не расставлены по общим правилам — код теряет визуальную структурность, его во много раз сложнее читать. Разобраться можно, но поддержка и скорость чтения затрудняется фактором "чёрт ногу сломит".
2️⃣Неаккуратные вертикальные отступы
Логически-цельные фрагменты кода принято разделять пустой строкой. В редком случае на 10 строк нет ни одного вертикального отступа. Если где-то отбивка в 1 строку, где-то в 2, где-то 3 — код выглядит неаккуратно, страдает структурность, сложнее читать. Отсутствие таких отступов — тоже плохо.
Аналогия с текстом: делать отступ между абзацами, в случайном месте в предложениях или же вообще писать монолитом без переноса строки.
3️⃣Стили по тегам
Крайне редкий случай, когда действительно стоит давать стили по тегам, а не по классам.
Когда давать стили по тегам ок:
🔹Первоначальный сброс браузерных стилей. Они в разных браузерах разные, нужно привести к единому виду перед добавлением стилей проекта.
🔹Если это, например, ссылка и 100% заранее уверены, что на сайте весь текст внутри ссылок должен быть чёрным, а при наведении — красным. Но даже это — спорный момент. В этом случае можно и так и эдак. Но давать стили div'у, опираясь на то, что он лежит внутри элемента с классом .something — нет, так делать не надо, надо дать div'у класс и дать стили через этот класс)
