Сучасна диджитал-освіта для дітей — безоплатне заняття в GoITeens ×
Mazda CX 30
×

Верстка по наследству и как с этим жить?

Підписуйтеся на Telegram-канал «DOU #tech», щоб не пропустити нові технічні статті

Доброго дня, уважаемые гуру Frontend-a.

В начале недели хорошие люди попросили подключился к верстке страниц на проекте, поскольку основной девелопер удаленный и не успевает со всеми задачами. работаю сейчас над версткой одной из страниц и пытаюсь переиспользовать уже готовые куски соблюдая стиль других страниц, поскольку времени очень ограниченно.

Сложность в том, что текущий код перегружен вызовом кучи библиотек и версткой, которая как на меня далека от семантической. Хотя приложение не самое навороченное по бизнес-логике и интерфейсу. Можете ли Вы подсказать какие-то эффективные стратегии/лайфхаки по работе с такой вот Legacy-версткой? Есть ли инструменты которые могут в автоматическом/полуавтоматическом режиме оптимизировать текущий код и облегчить его понимание? Показать какие из подключенных файлов вообще не используются?

Буду благодарен за любые подсказки.

👍ПодобаєтьсяСподобалось0
До обраногоВ обраному0
LinkedIn
Дозволені теги: blockquote, a, pre, code, ul, ol, li, b, i, del.
Ctrl + Enter
Дозволені теги: blockquote, a, pre, code, ul, ol, li, b, i, del.
Ctrl + Enter
версткой, которая как на меня далека от семантической
если хочешь нормальную верстку, здесь главный тул — это твои руки и ум. хотя менять чужой код в данной ситуации не совсем то правильно. да и зачем? если мешает, то да. но вряд ли. хочешь чтобы твой хтмл был семантическим — делай его таким. смакс и бем в помощь.
и не совсем понимаю, как это
соблюдая стиль других страниц
снова, зачем? техника кода неправильная, но ты ее соблюдаешь? если используешь селекторы, которые не хелеперы, не имеют семантики, лучше напиши хелперы или миксины (сасс). это вроде как уменьшит зависимость код и сделает его модульным, и что главное — читабельным.
кучу либов можна проверить на использование, может собрать в один.мин. но, снова, зачем? может я неправильно тебя понял. но если задача сверстать несколько страниц / компонентов — верстай соблюдая (свои) правильные техники. если задача — рефакторинг, тогда тебе предстоит пройти нелегкой путь

Ты в этом деле подчинённый исполнитель. Главное правило в этом случае не сломалось — не чини!

Пока тебе с этого никакого профита, не заморачивайся с рефакторингом. Делай свою часть, делай как считаешь правильным, и пусть себе работает. Перфекционизм в нашем деле вреден. Если со слишком большим количеством зависимостей код тем не менее работает — пусть себе работает. В конце концов, ты пишешь для компьютера. И пока компьютеру понятно чего ты хочешь, код удовлетворительный.

И только в тех редких случаях, если тебе выгодно качество выше оценки «удовлетворительно», то есть это репутационный проект для портфолио, твой собственный проект или с долей в нём, ну или хотя бы ты планируешь его достаточно долго сопровождать и дорабатывать — только тогда и улучшай. Ну или если платят щедро.

Все по делу.
Топикстартер очевидно еще и временно на проекте, так что тем более не стóит со своим уставом в чужой монастырь лезть.

Ну если ты работаешь только за деньги — то вполне логично. А если хочется чтобы доставляло удовольствие, и не было жалко прожитого времени — приходится рефакторить.

Вот этой программой www.getatomiq.com (возможно, есть и получше) можно попробовать обнаружить копипаст, а потом подумать, как от этого копипаста избавиться. Для неиспользуемого CSS есть инструменты типа такого: github.com/addyosmani/grunt-uncss

Підписатись на коментарі