Все вы видели недавние посты о найме на работу в команду пикабу. Но так ли хорошо это рабочее место, как нам описывают. Давайте разберемся вместе. Вот один из скриптов, недавно написанных для новогоднего функционала.
Что же на пишет разработчик: "ПОМОГИТЕ! Позвоните в полицию!!! Я в подвале, работаю за еду и воду"
Как видим пикабу проставляет отличный соц. пакет. Бесплатное проживание! Полный рацион питания за счет работодателя! Добро пожаловать в команду!
комментарий "конструктор" над конструктором, да еще и из трех строчек. Ну и правильно его в клетке держат
Можно по юзерагенту отдавать транспаленый код.
Я бы даже сказал, что это очень хорошо, так как транспайленный код раздутый как не знаю что.
Двойной зашквар: браузер это не доверенная среда. Верить тому что пришло из браузера\запрошено браузером принципиально нельзя.
так надо определиться, чего ты хочешь.
смотреть, какой браузер, и отдавать современный или заполифиленный код, в зависимости от того, сможет браузер его использовать или нет. При чем тут доверенная среда? Путаете теплое с мягким
// render component
render() { ...
Неудачная шутка вышла, согласен
Хех, читаю комменты ваши и забавляет реакция: es6 не транспайленный, нет минифакации, обфускации, комменты к методам даже смущают) Ребята, это декорация для сайта, написанная на коленках, на две новогодние недели :) Она даже не включена в общий js сайта, который как раз все как вы любите: минифицированный es5 )
Поддержка es6 достаточная (больше 98% наших пользователей) для такой временной
функции.
Минификация? Скрипт игры состоит из трех классов весит 64 КБ. Я даже не заморачивался сборщик gulp или webpack подключать для такой задачи.
Комменты с описанием к публичным членам классов - это культура программирования. Я не нахожу ничего в этом плохого, лишь хорошее :) Для этой игры я спешил, поэтому не всё на 100% покрыл полноценными jsdoc.
Вся js-статика упаковывается gzip (64КБ превращаются в 14Кб), затем кешируется, и отдается с cdn с TTL кеша 7 дней. Т.е. 14КБ раз в 7 дней это много? Для справки одна страница свежего или горячего весит 7 МБ.
Признаюсь, что для этой игры я даже отдельного проекта не делал, просто в рамках одного общего Backend проекта написал js, соединил в один файл и пустил на прод.
> Как бы не была мала задача, Ее все-равно нужно решать правильно
Красиво звучит.Согласен) В этом смысле я поступил без культуры :)
а в чем профит покрывать jsdoc? тем более, комментировать, что конструктор - это конструктор. только читать код мешает имхо
если нужна типизация, то тогда flow/ts - уже другое дело
Профит есть, когда ты не один дома под пивасик пишешь код, а в компании разработчиков, где от культуры и правил чистоты кода зависит насколько код будет понятен, единообразен и обезличен.
Писать, что конструктор - это конструктор, конечно, избыточно. Но это часть правила - каждый публичный метод должен иметь jsdoc, с описанием, перечнем параметров, типов возврата. Если даже void тип, или описание кажется глупым - все равно правило есть правило. От компании к компании правила меняются, кто-то принимает чьи-то стандарты, кто-то решает ввести свои стандарты. Я привык к таким правилам, что в том коде, и нахожу их оправданными для тех задач, где их применяю :)
