DEV: Рефакторинг (уберем различные условия, упростим)

Evg Evg 1 Декабря 2022 (ред)

Надо убрать разные условия, упростить сценарий.

Например, Stemmer для поиска. Всё бы хорошо, но начинает: Stemmer работает только для русского и английского. А для других языков нет.

Вот зачем это тут в таком исполнении? Скрипт сообщества поддерживает многие языки, а Stemmer нет? Условия надо держать в уме, что для арабского он вообще собственно не работает, например. Да и качество его, вызывает вопросы…

Далее, ещё пример. Ввели «Команды», т.е. появились ещё условия, которые надо учитывать при доступе и выяснилось, что всунуть (именно всунуть, как заплатку) эти условия конечно можно, но такое исполнение ерунда же.

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

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

Плюс сейчас идет работа над локализацией (добавляем языки, правим существующие).

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

5 Ответов

  1. OleStep OleStep 1 Декабря 2022 (ред.)

    И codacy.com можно посмотреть, дня три назад смотрел, там есть рекомендации.

    1. Evg Evg 1 Декабря 2022 (ред.)

      Кстати, да. Давно не смотрел, гляну сегодня.

  1. Evg Evg 3 Декабря 2022

    Есть кто-то тестирует последние файлы с GitHub (без релиза, master) и не хочет использовать Composer, то папка vendor — тут: libarea.ru/dev/vendor.zip.

    Она старается поддерживаться в актуальном состоянии.


    Например, 5 минут назад изменил в скрипте название папки с /app/Libraries на /app/Core. Личные предпочтение, т.к. не смог себя заставить привыкнуть к названию Libraries, там всегда подразумевал слегка другое. Название Core ближе.

    В итоге, если не собрать (через Composer) или не обновить папку vendor, то скрипт работать не будет.

  1. German German 7 Декабря 2022 (ред.)

    К пятнице постараюсь посмотреть новую версию. Есть несколько предложений, но хочу сперва всё проверить.

  1. Evg Evg 8 Декабря 2022 (ред.)

    Значит ошибки все исправляем, они есть и делаем релиз. Далее идем дальше. Много чего делать надо. Есть множество вещей начатых, но отложенных из-за маштабности, например (плохо делать не хочется)…