Pull to refresh

Comments 74

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

Очень рекомендую вам почитать про стандарты PSR. Даже тот факт, что вы работаете в среде Wordpress и на PHP 5.6 не может быть оправданием того, что вы, к примеру, не используете пространства имён или не пишете обязательные модификаторы видимости у методов в классах…

И это я еще не придираюсь к бесконечным «array», выжигающим глаза ))
«не используете пространства имён» серьёзно не использую?
"+namespace Migesco;" — такой строчки в исходниках нет?

И вот это: "// это свойство должно быть private (задел на будущий рефакторинг)" в коментах к коду не встречается?

чем плох «array»?

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

>> "+namespace Migesco;" — такой строчки в исходниках нет?
Такая строчка, безусловно, есть. И даже есть определение класса непосредственно после нее. Однако следующий же класс «class Registrar» у вас следует без предварительного объявления пространства имён.
Из этого следует, что либо вы часть классов размещаете в неймспейсах, а часть — нет, либо то, что у вас несколько классов находятся в одном и том же файле. Оба варианта плохи, оба являются серьезной стилистической ошибкой и нарушением стандартов.

Далее.
У вас многократно встречаются определения методов без ключевого слова public:
static function attachConfigurationJS()

function registerScript()

это прямое нарушение PSR-2

>> чем плох «array»?
Тем, что вот уже 6 лет как у нас есть замечательный лаконичный синтаксис с квадратными скобками.
«что у вас несколько классов находятся в одном и том же файле»
проект на WP написан (на процедурах, ООП нет ни в каком виде), мне надо было заморочиться за написание своего автозагрузчика? или надо было подключить Композер которого там отродясь не было?
На весь проект, это все три его класса, подключаются одной строчкой (require_once), у вас боль вызывает что я 200 строчек насочинял, а подключить Композер ради трёх классов которые используются в паре мест это норм?

Вам нравиться объявлять массив как [], мне нравиться — array(), это преступление?

«без ключевого слова public»
согласен, можно добавить, это всё что нарушается из PSR-1 / 2? это повод сказать что мой код не соответствует PSR? мне тут перечислить в чём соответствует?

Граммар-наци, казалось бы взрослые люди…

Если не соответсвует хотя бы по 1 параметру, то результат: не соответствует.
Или я что-то упустил? Иначе кто определит тот порог количества "соответствий", чтобы можно было проигнорировать остальные пункты? А может какие-то пункты более/менее важны, чем остальные?

это вопрос качества, допустим 39 требований из 40 выполнены, качество = 39/40*100% = 97,5%

кто то видит мир в чёрных и белых цветах, а кто то в 50 оттенках :)
К сожалению, в реальной жизни всё не так, как вам кажется.

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

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

Извините, не заметил, что исхдиники написаны на go с ассемблерными вставками.


Я ошибочно посчитал, что речь идёт он php и php-comunity. Даже не знаю, как такое могло случиться

Array () это код стайл от WP для совместимости. Но я призываю всех его нарушать, потому что невозможно тащить дохлую лошадь и любить юзера за его лень переключения веток php в панели брпузера.

Слишком сложное использование у вас получилось, слишком много строчек. Проведём ещё немного рефакторинга: автогенерация идентификаторов, последовательное по порядку назначение зависимостей.
Migesco\Configurator::attachFiles('/e-cryptex.js', '/wp-content/themes/tol-child/js/child_realforex.js');

Вуаля! Получился практически исходный код. Что принесла нового ваша работа? Ничего. Вот так сюрприз)
По поводу автогенерации не понял в чём юмор, избавились от аргумента в вызове — это не плюс?
Про назначение зависимостей по порядку не очень согласен, если файлы между собой не зависимы, то пусть грузятся все десять в параллель.
У меня не было цели поменять функционал, была цель сделать его более наглядным, и попутно получилось сделать более гибким. Нет ну не знаю, пишите стены кода, зачем нам модульность, правда?

Теперь, такой вариант, что нам надо прицепить 10-ть файлов, это по версии WP 20 строчек, по моей версии это 10-ть строчек в генерации массива и одна строчка на добавление к странице.
Было 20, стало 11, это ничего?
Ок.

Теперь в 4-х разных шаблонах повторим этот фокус — раньше код добавлении файлов на страницу занимал пол экрана (20 строк), теперь только четверть (11), писать меньше, читать меньше, это ничего?
Ок.

У меня привычка думать шире поставленной задачи, на короткой дистанции — проигрышно, но я на короткие не бегаю, только марафоны :)
Вы молодец, мыслите как программист. Только вот, по сути, вы всего лишь перенесли тег script на бэкенд. Все ваши достижения уже по-умолчанию реализованы всеми браузерами. Тот же base или async загрузка скриптов.

Ирония в том, что вы, судя по статье, этого и не поняли) Я тоже люблю писать велосипеды. Естественно, велосипед работает хуже. Что можно доработать? Кэширование, подключение в 1 строку без массивов, балансировка нагрузки, подстановка шаблона script в клиентский код, какая-нибудь версионность…

Не хотелось бы вас расстраивать, но держите патч:


-<script src="e-cryptex.js"></script>
-<script src="wp-content/themes/tol-child/js/child_realforex.js"></script>
+<script src="/e-cryptex.js"></script>
+<script src="/wp-content/themes/tol-child/js/child_realforex.js"></script>
И как потом в отчет написать два дня на исследование проблемы и два на реализацию?
PS. Не хочу никого обидеть, просто юмор.
спасибо, на момент написания кода, такой вариант в голову не пришёл, сейчас когда статью писал тоже об этом подумал, но суть то статьи не в этом, это был только повод.
Посмотрел я на этот код, и моё чувство прекрасного взбунтовалось:
человек не знакомый с WordPress ни чего не поймёт — придётся лезть в справочник что бы понять назначение функций wp_register_script() и wp_enqueue_script()
явно делается одно и то же два раза подряд, аргументы только разные — нарушается DRY

  1. Теперь не поймет в том числе и человек знакомый с Wordpress, в любом случае придется лезть в код и разбираться
  2. DRY вовсе не про то, чтобы каждые две повторяющиеся строчки оборачивать в отдельный класс. Тем более что по факту результирующий код практически не уменьшился (потому что он и изначально в основном состоял из аргументов)

По итогу — обе поставленные цели не достигнуты


Решение было протестировано для главной страницы и там оно работало, позже выяснилось что для страниц с адресом example.com/about, решение не работало, потому что браузер пытался подгрузить example.com/about/e-cryptex.js и браузер получал — ошибку 404, потому что файл реально лежал в корне — example.com/e-cryptex.js .

Насколько я знаю HTML, для этого достаточно немного изменить путь


+<script src="/e-cryptex.js"></script>
+<script src="/wp-content/themes/tol-child/js/child_realforex.js"></script>
на мой вкус набор аргументов это хуже чем «именованные» аргументы при «создании» экземпляра.
wp_register_script(FILE-ID,FILE-PATH);
wp_enqueue_script(FILE-ID);

wp_register_script — функция называется регистрировать скрипт, но может регать в глобальном списке любой файл
wp_enqueue_script — функция называется зауникалить скрипт, по сути прикрепляет файл к странице и может не уникалить
Функции вызываются от какого то перечисления констант (в исходниках платформы), какая константа какой имеет смысл? не понятно.
Сравним с:
$traderoomScriptFiles = array(
    (new Migesco\WebResource('child_realforex'))
        ->setSource('/wp-content/themes/tol-child/js/child_realforex.js')
        ->setDependency(array(ECRYPTEX_JS))
);
Migesco\Configurator::attachFiles($traderoomScriptFiles);

И тут:
1)создать ресурс, установить источник, установить зависимость;
2)добавить файлы;

всё как называется такой смысл и имеет, назначение каждой константы понятно из названия сеттера, можно не быть гуру вордПресса и всё понять.
Функции вызываются от какого то перечисления констант (в исходниках платформы), какая константа какой имеет смысл? не понятно.

Вы могли бы просто добавить комментарий


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

Тут, как говорится, дело вкуса. По моему личному мнению вы просто нарушили KISS и скорее всего YAGNI, а DRY от этого ничего не выиграл

Мне было бы ближе встретить стандартные функции wp, которые должны быть документированы и известны, и которые можно загуглить, чем разбираться в чужих классах.

Тем более их имена говорят сами за себя.

Ps enqueue это не про уникальность (unique), а про очередь (queue), т.е. добавить в очередь.
«enqueue» — спасибо, тогда, да, это хорошее название для функции.
В можно было бы сделать так:
+<script src="/e-cryptex.js"></script>
+<script src="/wp-content/themes/tol-child/js/child_realforex.js"></script>

конечно, святыни PHP, вордпресса в частности и ынтырпрайза в целом были бы поруганы, зато лично я спал бы спокойнее.

PS а потом мы удивляемся, почему веб такой жирный и нужны все ресурсы как клиента, так и сервера.
«Почему я не знал про корневой слэш и поэтому сделал 200 строчек из 2»
Поиск по «html absolute path» даёт ответ в 1-й ссылке.
Или это пост из 2к29, в котором гугл забанен?
UPD: по логике надо было б использовать «html root path», но в таком случае более надёжный источник становится 2-м.
Черт даже с этими корневыми слэшами о которых тут уже написали, однако я не вижу никакой адекватной причины не макнуть разраба в документацию инструмента, с которым он работает. Если вас прямо хлебом не корми, но дай коллегам помочь, просто воткните ссылку на документацию в комментарий перед таким местом, а не плодите кучу паршивого кода, который в результате ничуть не очевиднее изначально работавшего способа.
явно делается одно и то же два раза подряд, аргументы только разные — нарушается DRY

У вас фундаментальный пробел — функции придуманы как-раз для того чтобы не дублировать код, т.е. Don’t repeat yourself
И поэтому вы придумали еще 200 строк чтобы продублировать существующие функции…
а массивы придумали для хранения и доступа к однотипным данным, а циклы придуманы для того что бы в коде был один вызов и данные обрабатывались по одному алгоритму, а не каждый раз по новому.
Расскажите мне про программирование ещё.
Я вас понимаю, тоже в своё время был очарован объектным подходом и обычные функции нарушающие эстетику кода беспощадно переписывались на объекты.
Но, почему-то на указание явной нестыковки в вашей аргументации, вы уводите обсуждение в другую область, я в недоумении…
Вы свою мысль как то чётче выразите.
Я придумал 200 строк кода, что бы унифицировать, то с какими аргументами вызываются функции WP для управления ассетами.
Для того что бы скрыть реализацию функционала через WP функции, тем самым сделать код шаблона независимым от WP
Что делает мой код? берёт массив с аргументами и делает вызовы функций, вызов функции в коде это одна строчка, а не *цать строчек как было бы на WP.
Что с чем не стыкуется?
Они уже существуют и унифицированы, вы просто нарастили поверх функций дополнительную обертку, чтобы вам было удобно не изучать справочник функций WP.
Или спрошу по другому:
Зачем скрывать функции WP в шаблоне WP и делать код независимым от WP?
Ощущаете абсурдность?
может быть потому что из этого шаблона WP надо сделать шаблон независимый от WP? html он есть html, что там делает WP? ассеты цепляет? ссылку на en локализацию добавляет? header footer body склеивает? в основе всё равно куски html и эти куски от смены фреймворка не поменяются.

" дополнительную обертку, чтобы вам было удобно не изучать справочник функций WP"
а что бы обёртку накрутить, оборачиваемые функции WP изучить не надо было?
Ощущаете абсурдность?
Ну вообще то пресс позволяет еще перекрывать родительские темы дочерними, дочки перекрывают одименные функции у родителя. Темы вордпресса это не просто склейка html, это еще и шаблоны по именам которых пресс понимает когда их нужно задействовать.

Говорить что можно сделать тему для WP раздельно от WP это как-то через не похоже на создание темы для WP. Мне это напоминает скучную стратегию, когда помимо основного сюжета игры человек сам себе придумывает мини игры. Вам тогда не нужен WP или работайте через REST. Уж через него куда проще сделать независимую от движка тему, только нафига.
Жесть какая.
человек не знакомый с WordPress ни чего не поймёт — придётся лезть в справочник что бы понять назначение функций wp_register_script() и wp_enqueue_script()
явно делается одно и то же два раза подряд, аргументы только разные — нарушается DRY

Человек, который не вы, что должен здесь понять?
+$traderoomScriptFiles = array(
+    (new Migesco\WebResource(ECRYPTEX_JS))
+        ->setSource('/e-cryptex.js'),
+    (new Migesco\WebResource('child_realforex'))
+        ->setSource('/wp-content/themes/tol-child/js/child_realforex.js')
+        ->setDependency(array(ECRYPTEX_JS))
+);
+Migesco\Configurator::attachFiles($traderoomScriptFiles);
 ?>

И в какой справочник ему смотреть? В «документацию» вида
/**
+     * @param string $name
+     * @return WebResource
+     */

что ли?
А как же DRY? два раза же повторяете одно и то же, аргументы только меняются!%)
А как же DRY? два раза же повторяете одно и то же, аргументы только меняются!%)

Уберите это! А то следующая статья будет «как из 200 строчек сделать 20 000...» :)

А так… действительно — «жесть какая». Вроде и красиво написано. Всякие injection, классики, но читать — сложно. От создания кучи страшных обьектов просто чтоб передать строку, да еще и без видимого смысла сейчас и призрачного выигрыша в будущем — просто коробит Что оно такое и как работает — не понятно. Наверное, единственное оправдание этому всему — разные платформы. Но если их нет и не предвидится — просто потеря времени.

З.Ы.: ТС, спрячьте название скриптов — уже от одного названия realforex хочется нажать «минус»)
«уже от одного названия realforex хочется нажать «минус»»
не мой скрипт, не мне менять.
вот кому призрачные шансы, а мне потом везде копипастить генерацию версии? спасибо, один класс, один метод поправлю и молодец.
Кому то не нужна единая стратегия подгрузки файлов? ок, мне нужна, не хочу по всему коду править вызовы wp_*, хочу просто поправить один метод, а если стратегий будет две, то добавить ещё один метод и не ловить баги от того что где то поправил, а где то нет.
Сейчас на WordPress? наша цель свою платформу написать, что бы вендору помахать рукой, и в качестве фреймворка будет точно не процедурный WP и точно не JQuery.
+wp_register_script(
+ CHILD_REALFOREX_JS,
+ '/wp-content/themes/tol-child/js/child_realforex.js',
+ array(E_CRYPTEX_JS),
+ null);
+wp_enqueue_script(CHILD_REALFOREX_JS);

Простите вы код в консоли 25 на 80 пишите, почему тут нужен силь Маяковского?

я не спец в js но неуежели нельзя вытащить '/wp-content/themes/tol-child/js/child_realforex.js' в константы, потом переименовали файл, и ищи его full text search, а что еще хуже, непонтен весь скоуп файлов, которые так передаются. Именование отдельная тема.
wp_register_script(CHILD_REALFOREX_JS,Somefiles.ChildRealForex, array(E_CRYPTEX_JS),null);
// итого 91 символ, даже на 1024*768 вполне можно смотреть.
Это слишком трэшово что бы воспринимать это искренне.
Автор, признайтесь, эта статья высокоуровневый сарказм или пародия на что-нибудь?
Понятнее код не стал, меньше код не стал.
Если была цель написать класс-оболочку — это хоть как-то можно было понять, но у Вас даже не заявлена такая цель.
обратите внимание на главу — «Что мы получили ?», пункт 2.
Жесть.
Решение было протестировано для главной страницы и там оно работало, позже выяснилось что для страниц с адресом example.com/about, решение не работало, потому что браузер пытался подгрузить example.com/about/e-cryptex.js и браузер получал — ошибку 404, потому что файл реально лежал в корне — example.com/e-cryptex.js .

про относительный и абсолютный адрес не слышали?

человек не знакомый с WordPress ни чего не поймёт — придётся лезть в справочник что бы понять назначение функций wp_register_script() и wp_enqueue_script()

судя по всему, такой человек именно вы.

явно делается одно и то же два раза подряд, аргументы только разные — нарушается DRY

вытекает из предыдущего. Нет, делается не одно и то же.

'/wp-content/themes/tol-child/js/child_realforex.js',

Еще, неплохо бы формировать абсолютный адрес для подключаемых ассетов. Сделать это можно через
get_template_directory_uri()
для тем, или
plugin_dir_url()
для плагинов.

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

Если посмотреть на главу «Второй вариант», то кажется как подключать — нашёл способ, но этот способ не удовлетворяет моих требований к гибкости.
Кто то пишет на языке программирования, кто то пишет на фреймворке, а мне удобно писать с использованием языка программирования и с использованием фреймворка.
Нравиться WP прибитый гвоздями к коду? Нравятся вызовы WP, не нравиться лишняя абстракция? о вкусах не спорят.

«вытекает из предыдущего. Нет, делается не одно и то же.»
Смысл в том что бы аргументы засунуть в массив и в цикле сделать вызовы с этими аргументами. Количество строк с данными останется прежним, а количество строк кода сократиться в разы. Это ни чего не меняет? в результате работы кода ни чего не меняет, в количестве работы для поддержки кода меняет — количество правок сокращается, нет?
Самое смешное когда ты видишь в и-нете примеры реализации, видишь документацию, которой эта реализация соответствует, но у тебя это не работает


Скорее всего вы что-то недопоняли. Подключить скрипт — нет ничего проще. Начиная с того, что можно просто прописать тег script в файле с хедером/футером темы (если речь про плагин, то так не получится), но при этом задать правильный адрес к файлу, что бы он не отваливался.

Или же, если нужно сделать все по-правильному, что бы можно было управлять версиями, зависимостями, порядком, положением (в head или перед /body), локализовать скрипты, кешировать и минифицировать — то нужно использовать register и enqueue script/style. А вам достаточно было написать два вызова
wp_enqueue_script()
БЕЗ использования
wp_register_script()


Если пишите тему, то идете в functions.php и добавляете свой экшн в хук 'wp_enqueue_scripts'

Выглядит это, примерно, так

/**
 * Enqueue scripts and styles.
 */
function my_wp_scripts() {
    /*
     * Styles
     */
    //вызовы wp_enqueue_style()

    /*
     * Scripts
     */
//примеры подключения скриптов
    wp_enqueue_script( 'jquery-countdown-js', get_template_directory_uri() . '/js/plugins/jquery.countdown.min.js', array('jquery'), '20151215', true );
    wp_enqueue_script( 'jquery-flexslider-js', get_template_directory_uri() . '/js/plugins/jquery.flexslider-min.js', array('jquery'), '20151215', true );
    wp_enqueue_script( 'main-js', get_template_directory_uri() . '/js/js.js', array(), '1.2', true );
}
add_action( 'wp_enqueue_scripts', 'my_wp_scripts' );


и все.

не нравиться лишняя абстракция?

Одно дело если бы вы написали свою абстракцию, которая бы не зависела от CMS/фреймворка. Это было бы оправданно.
Но вы же написали тонну кода поверх существующих абстракций WP. Ваше решение же нельзя перенести на другую CMS/фреймворк без правок вашего класса.

Теперь мы можем сменить WordPress на что угодно и переписать придётся только класс
Registrar, не надо заменять все вызовы wp_register_script и wp_enqueue_script и следовать логике WordPress`а.

И следующий разработчик вместо того, что бы подключить два скрипта характерным для данного фреймворка способом (я уверен, почти везде это две строчки кода) вынужден будет ковыряться в вашем классе и править методы подключения… да он выкинет ваши классы и просто подключит так, как это принято на данной конкретной платформе.

Я бы, если бы был консультантом на стороне заказчика, точно бы не заплатил за время, которое вы потратили на свою абстракцию.
проблема была не с тем что бы подключить скрипт, а с тем что бы он не уникалился. В документации написано передать false (или null не хочу вспоминать), и версия не будет добавлена к ссылке на файл, но ссылка добавлялась.
Смысл абстракции в том что бы шаблон не переписывать (переписывать поменьше), когда смениться фреймворк.

Откуда вдруг появилась проблема с 'не уникалился'? Вы её только что придумали?
Вы действительно думаете, что с Вашей реализацией будет проще сменить фреймворк?

Когда платят не за результат, а за каждую строчку кода…
Не понимаю зачем вы это сделали вообще. Но к комментариям выше относительно кода, PSR и прочего есть вопрос к вот такой конструкции:
class WebResource
{
    public $dependency = array();
	
    public function setDependency($dependency)
    {
        $isArray = is_array($dependency);
        if ($isArray) {
            $this->dependency = $dependency;
        }
        return $this;
    }
}

даже в php уровня 5.5 и выше она выглядит как то излишне избыточной, зачем вы объявляете переменную сразу как пустой массив (ведь на null куда проще проверять чем на count() пустой массив) и объявляете ее публичной, уже тогда делали вот так:
class WebResource
{
    private $dependency;
	
    public function setDependency($dependency)
    {
	$this->dependency = (is_array($dependency) ? $dependency : null)
	return $this;
    }
}

ну а в эпоху php 7.x можно запросто делать вот так:
class WebResource
{
    private $dependency;
	
    public function setDependency(?array $dependency): self
    {
	$this->dependency = $dependency;
	return $this;
    }
}


Похоже вам действительно платят либо за время, либо за количество строк кода. Не делайте так, пожалуйста, такие труды явно не заслуживают публикации в 2019ом на хабре.
по версии для PHP 5.5 ваше предложение это сделать свойство приватным и использовать тернарный оператор?
про сделать приватным — можно конечно, если коменты в коде почитать то даже можно найти:
"// это свойство должно быть private (задел на будущий рефакторинг)"
и сделать вывод что автор в курсе, что некоторые свойства и методы стоит делать приватными. А можно сделать другой вывод, на ваш выбор.
Стать рассказывает идею, а не демонстрирует идеальную реализацию, вы ожидаете идеальную реализацию, извините, вы ошиблись статьёй.

Про тернарный оператор.
Если весь код это такие цепочки:
$isSuccess = someMethod();
if($isSuccess ){
    $isSuccess = otherMethod();
}
if($isSuccess ){
    $isSuccess = differentMethod();
}

, то почему я должен использовать тернарный оператор?
Я пишу так:
if($isSuccess ){
    whenSuccess();
}
if(!$isSuccess ){
    whenFail();
}

и по необходимости могу или удалить первый условный оператор, или второй, в результате правок может остаться только негативный вариант.
if(!$isSuccess ){
    whenFail();
}

При таком подходе я могу тусовать и переносить кусочки кода с меньшими усилиями, для того что бы оставить только одну ветку мне не надо из тернарного оператора делать условный, мне так проще.
Никто не заставляет вас его использовать, но как минимум объявлять переменную лишь для того, чтобы ее использовать в последующем условии — звучит бредово, при условии что метод возвращает булевый результат:
$result = someMethod($args)
if ($result) {
    // do something
}
// а почему не вот так?
if (someMethod($args)) {
    // do something
}
// при однородности условия тернарник выглядит вполне себе вменяемо:
$doAnything = (someMethod() ? action1() : action2()); 

Впереди вас еще ждут «магические» методы __isset(), __get(), __set() и тогда вы еще больше сократите этот «макаронный» код — ваш WebResource превратиться в 8-10 строчек с 3мя методами…
Я это к чему — код выше совершенно не выдерживает критики ни с какой стороны: ни с точки зрения целесообразности такой реализации, ни с точки зрения его качества и принципов проектирования. Не следует воспринимать комментарии как личную критику, вам скорей дают советы «как не нужно делать в 2019» на php.
вы не понимаете почему я не пользуюсь тернарником.
// при однородности условия тернарник выглядит вполне себе вменяемо:
$doAnything = (someMethod() ? action1() : action2()); 

по моим понятиям должно быть одна строка одно действие, тут в одной строке три действия, мой код это разделяй и властвуй, дроби и разжёвывай.
Вы предлагаете наворачивать экспрешены, привет тому кто будет это отлаживать, сколько раз придётся нажать F11 что бы понять в какой метод/экшен пойдёт выполнение? а если меня интересует только один из этих трёх вариантов и я ставлю брекпоинт на эту строку, то мне все попадания в этот код придётся просматривать, вместо того что бы поставить один брейкпоинт на том варианте развития событий которое меня интересует и только его дебажить, нет придётся дебажить все три варианта. Спасибо.
Что такое код тусовать, вы не понимаете, давайте каждый будет работать как ему понятней, как ему удобней?
Вот тут кстати вас поддержу. Конструкция? *: * -это рассадник для генерации малочитаемого кода.
по моим понятиям должно быть одна строка одно действие

Пишите так тогда.
$doAnything = someMethod() 
    ? action1() 
    : action2(); 
Автор просто «бриллиант» для проектов, где платят за часы или строки кода. Немного поработать с аргументацией и вообще бомба. Надо будет запомнить ваш ник, может подвергнётся такой проект.
немного не в тему, но и в тему:

Сейчас вышел новый редактор Gutenberg в WordPress.

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

Теперь нельзя разрабатывать под WP одним notepad++. ВСЕ.

Разрабов так достало, что их тыкают в говноархикетуру, им и сама она не очень… и надо же что-то делать и они сделали финт ушами!

Gutenberg теперь на реакте. Общение по REST API. Примеры кода на NEXTJS прямо в докуметации, никакого ES5 (редко). Без npm, правильного сборщика, теперь ничего не сделаешь!
Куча функционала теперь будет на JS. И сейчас крайне бедна документашка шо писец.

Вот так надо выкручиваться от скучной работы) То есть пыха теперь не достаточно явно для разработки на WP.

От части подытожу всё выше сказанное:

  1. Мотивация. Если я правильно понял, то вы пытались сделать код более «универсальным и гибким». Но в итоге только усложнили ситуацию.

    человек не знакомый с WordPress ни чего не поймёт

    Ваш проект — плагин для WordPress. Какова вероятность, что его поддержку доверят человеку который не знаком с этой CMS? Это ж как доверить доработку сайта на php человеку, который не знаком с php.

    Кроме того, открыть справку по функции WP, куда проще, чем лезть в чужой код и пытаться розобраться в его работе.

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

  2. Результат, даже менее гибок, чем изначальный.

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

    Мне нужно подключить файл в подвал страницы. И как мне это сделать? Судя по вашему коду, это не возможно, так как ваш код не предусматривает передачу параметра $in_footer в wp_enqueue_script.

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

    В итоге, при здаче, я смогу отметить 20 часов работы как «Отрефакторил код предыдущего разработчика, и привёл его к виду в соответствии с стандартами разработки под WordPress». Хоят по факту — я просто удалю ваше решение и заменю его на два вызова wp_enqueue_script

«Какова вероятность, что его поддержку доверят человеку который не знаком с этой CMS? Это ж как доверить доработку сайта на php человеку, который не знаком с php»
Мне доверили разработку SPA и вёрстки, при нулевом опыте JS и CSS (моё резюме), что теперь скажите?
И да, в моей жизни был опыт доработки сайта на php человеком не знаком с php.
Расскажите мне об этой жизни ещё, я так мало знаю, так мало видел.

«Мне нужно подключить файл в подвал страницы» + «обязательно необходимо указывать его версию»
Моё решение позволяет какую угодно логику накрутить не переписывая все подряд вызовы wp_enqueue_script, это называется гибкость, я не продаю библиотеку, я продаю код который можно дальше без болезненно развивать.

«я смогу отметить 20 часов работы»
у меня на все три рефакторинга ушло 6 часов. Если вам на 150 строк банального кода надо столько времени, то таково качество вашей работы.
При чём из этих 6-ти часов, два минимум ушло на чтение доки и поиск адекватного примера.
Но бумага всё стерпит, вы пишите.
Мне доверили разработку SPA и вёрстки, при нулевом опыте JS и CSS (моё резюме), что теперь скажите?

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

у меня на все три рефакторинга ушло 6 часов

А изначально нужно было лишь добавить слеш и base
И время на поддержку будет лишь расти

Накручивание любой логики требует времени
Накручивание любой логики требует времени

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

который можно найти в гугл на раз два

вы меня не слышите, я два часа от Гугла получал примеры использования, которые в моём случаи не работали. Вообще вся логика событий изложена в посте, это не была революция (от html
<script>
к неймспейсу Migesco), это была эволюция в 4 этапа.

А изначально нужно было лишь добавить

Знал бы прикуп, жил бы в Сочи, статья не про реализацию, а про подход, про логику событий, 8+1 человек проголосовали за вариант «да» — вот для этих людей эта статья. Не для вас.
А 130+ человек, пытаются до вас донести, что идея, заложенная в материал — в корне не правильная. И что не нужно так делать.
я продаю код который можно дальше без болезненно развивать.

Так в том то и дело, что болезненно. Я смотрю на ситуацию, с точки зрения WordPress разработчика, который придёт на проект после вас и не видел этой статьи. Пытаюсь представить себя на его месте. И описываю вам те мысли и проблемы возникающие у меня.

Предположим я открыл ваши исходники. Изучил их. Досконально понимаю что за что отвичает.

И у меня в такой ситуации остаётся один вопрос — а зачем? Я как сторонний человек, без вашего объяснения, смотря только на ваше решение, не пойму в чем принципиальное отличие между

$traderoomScriptFiles = array(
    (new Migesco\WebResource(ECRYPTEX_JS))
        ->setSource('/e-cryptex.js'),
    (new Migesco\WebResource('child_realforex'))
        ->setSource('/child_realforex.js')
        ->setDependency(array(ECRYPTEX_JS))

и

wp_enqueue_script(ECRYPTEX_JS, '/e-cryptex.js');
wp_enqueue_script('child_realforex', '/child_realforex.js', array(ECRYPTEX_JS));


И что ж мне тогда делать? Оставить ваше решение? Я не могу его просто так удалить, вы ж его для чего-то написали. Связаться с вами для пояснения? Подключать часть файлов по вашему а часть по моему? Или использовать ваше решение? А что если в процессе появится какия-то проблема? Дебадить? Допиливать его?

Даже если игнорировать все замечания по коду, все люди, написавшие комментарии пытаются донести до вас мысль — что вы таким образом, усложнили жизнь людям, которые будут поддерживать проект дальше. Увеличили время на выполнение простых задач. Увеличили стоимость проекта для заказчика.
в чём разница?
разница в том что писать две строчки с вызовом wp_enqueue_script или не писать ни одной вообще, потому что эта строчка в единственно верном виде была написано до вас.
Чем меньше ручного ввода тем меньше ошибок. Если вдруг потребовалось поменять логику подключения скриптов, то менять придётся в одном методе, а не по всему коду.

Это вещи которые к WP не имеют отношение, это философия программирования.
Меньше думать, больше использовать готовые проверенные решения, написать один раз — использовать везде. Любой кусок кода который повторяется больше одного раза вынести в функцию, любой литерал который с один смыслом используется больше одного раза — в константы. Все настройки из кода вынести наружу, или в файлы или в БД и так далее. Банальные вещи. Есть у меня два идентичных вызова которые могут поменяться идентичным образом? Значит делаем обёртку. Не хотите не делайте.

Я опытом поделился — написал статью, вы её прочитали, спасибо. На этом наши отношения заканчиваются. Хотите пользуйтесь, не хотите не пользуйтесь, ваше личное дело.

А 130+ человек, пытаются до вас донести, что идея, заложенная в материал — в корне не правильная. И что не нужно так делать.

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

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

Ваш код не является скажем так плохим. Но и не является хорошим. Он просто бесполезен в рамках задач которые выполняет. Ваш код не абстрагирует в достаточной мере поведение, да и смысла абстрагировать там нет, писать обёртку над фрэймворком? Я конечно не веб программист, но как мне кажется это всё равно что писать обёртку над unreal engine, то есть столь же бессмысленно, т.к. при смене движка всё равно практически весь код будет переписан. По мне вам пока просто не достаёт опыта, это видно и по коду и потому как строится ваша аргументация, вы не готовы принять то что более опытные программисты оценивают вашу наработку как бессмысленную и потому бунтуете. Выражения по типу миллионы мух не могут ошибаться не делает вам чести, в глазах тех кто прочитал вашу статью и ваши комментарии к ней, вы выглядите как очередной студент с «гениальной» идеей, сомневаюсь что вы собирались добиться подобного.
И маленькое дополнение: «Имена пространств имён и имена классов ДОЛЖНЫ следовать стандарту PSR-0. В конечном итоге это означает, что каждый класс должен располагаться в отдельном файле и в пространстве имён с хотя бы одним верхним уровнем (именем производителя).»
про отдельный файл уже поговорили, про имя производителя — Migesco, неймспейс так и именуется, или надо Migesco\Attachments?
Да, каждый класс в отдельный файл со своим нейспейсом Migesco\Класс
в описанном решении три класса, это:
Migesco\Configurator
Migesco\Registrar
Migesco\WebResource

В шаблон «Migesco\Класс» вроде укладывается. И в чём замечание?
Вот почему нельзя изучать объектно-ориентированное программирование сразу. Потому, что для задачи типа «Hello word» тот, кто так изучал, будет наворачивать огромные куски кода в котором чёрт ногу сломит, пытаясь понять.
Тогда как
+wp_register_script(E_CRYPTEX_JS,'/e-cryptex.js',array(),null);
+wp_enqueue_script(E_CRYPTEX_JS);
— это примерно поймёт даже человек не знающий и не работающий с JS, что он делает.
Единственное, что оправдывает автора — иногда жизнь требует, чтобы вместо двух строчек на этом месте была достаточно сложная логика, которая уже требует и классов и больших асбтракций. А разработчику не дают время, чтобы рефакторить код. В результате он сходит с ума и начинает любой функциона писать как потенциально расширяемый -и всё равно не помогает, потому, что переделка и расширение потом оказываются нужными совсем в другом месте.
Теги статьи «JavaScript, Анализ и проектирование систем»
это не теги, это Хабы (тематики), я писал в Анализ, в JS я в принципе не мог писать, это редактура или движка Хабра или модераторов.
Sign up to leave a comment.

Articles