• Вывод отладочной информации. А как делаете вы?

    27cm
    @27cm
    TODO: Написать статус
    Ответ написан
    Комментировать
  • Попросили проверить код, на что смотреть нужно?

    index0h
    @index0h
    PHP, Golang. https://github.com/index0h
    Смотря зачем)). Я когда делаю Code Review критерии следующие:

    * Безопасность:
    - Каждый аргумент метода простого типа должен проверяться на тип в случае его проксирования и на граничные значения в случае обработки. Чуть что не так - бросается исключение. Если метод с кучкой аргументов на 80% состоит из поверки из аргументов - это вполне норм))
    - Никаких trigger_error, только исключения.
    - Исключения ДОЛЖНЫ быть человеко-понятны, всякие "Something went wrong" можно отдавать пользователю, но в лог должно попасть исключение со стектрейсом и человеко-понятным описанием, что же там пошло не так.
    - Каждый аргумент (объект) метода должен быть с тайпхинтингом на этот его класс, или интерфейс.
    - За eval как правило шлю на **й.
    - @ допускается только в безвыходных ситуациях, например проверка json_last_error.
    - Перед работой с БД - обязательная проверка данных.
    - Никаких == и !=. Со swtich - единственное исключение, по ситуации.
    - Если метод возвращает не только bool, а еще что-то - жесткая проверка с ===, или !== обязательна.
    - Никаких условий с присваиваниями внутри. while($row = ...) - тоже идет лесом.
    - Магические геттеры/сеттеры разрешаются только в безвыходных ситуациях, в остальном - запрещены.
    - Конкатенации в sql - только в безвыходных ситуациях.
    - Параметры в sql - ТОЛЬКО через плейсхолдеры.
    - Никаких глобальных переменных.
    - Даты в виде строки разрешаются только в шаблонах и в БД, в пхп коде сразу преобразуется в \DateTimeImmutable (в безвыходных ситуациях разрешено \DateTime)
    - Конечно зависит от проекта, но как приавло должно быть всего две точки входа: index.php для web и console(или как-то по другому назваться) - для консоли.

    * Кодстайл PSR-2 + PSR-5 как минимум, + еще куча более жестких требований (для начала все то что в PSR помечено как SHOULD - становится MUST)
    - В PhpStorm ни одна строчка не должна подсвечиваться (исключением является typo ошибки, например словарик не знает какой-то из аббревиатур, принятых в вашем проекте). При этом разрешается использовать /** @noinspection *** */ для безвыходных ситуаций.
    - Если кто-то говорит, что пишет в другом редакторе и у него не подсвечивается, на эти отговорки кладется ВОТ ТАКЕЕЕНЫЙ мужской половой **й и отправляется на доработку)).

    * Организация кода:
    - Никаких глобальных функций.
    - Классы без неймспейса разрешаются только в исключительно безвыходных ситуациях.

    * Тестируемость (в смысле простота тестирования) кода должна быть высокая.
    - Покрытие кода обязательно для всех возможных кейсов использования каждого публичного метода с моками зависимостей.

    * Принципы MVC:
    - Никаких обработок пользовательского ввода в моделях, от слова совсем.
    - Никаких ***ть запросов в БД из шаблонов.
    - Никаких верстки/js/css/sql-ин в контроллерах.
    - В моделях НИКАКОЙ МАГИИ, только приватные свойства + геттеры с сеттерами.
    - В моделях разрешено использовать метод save(при наличии такого разумеется) только в исключительных ситуациях. Во всех остальных - либо insert, либо update.

    * Принципы SOLD:
    - Никаких божественных объектов умеющих во все.
    - Если метод для внутреннего пользования - private, никаких public.
    - Статические методы разрешаются только в случае безвыходности.

    * Принцип DRY разрешено нарушать в случаях:
    - Явного разделения обязанностей
    - В тестах (каждый тест должен быть независимым, на сколько это возможно)

    * Работа с БД:
    - Запрос в цикле должен быть РЕАЛЬНО обоснован.
    - За ORDER BY RAND() - шлю на***й.
    - Поиск не по ключам (конечно если таблица НЕ на 5 строк) запрещен.
    - Поиск без LIMIT (опять же если таблица НЕ на 5 строк) запрещен.
    - SELECT * - запрещен.
    - Денормализация БД должна быть обоснована.
    - MyISAM не используется (так уж)) )
    - Множественные операции обязательно в транзакции, с откатом если чо пошло не так.
    - БД не должна содержать бизнес логики, только данные в целостном виде.
    - Не должно быть нецелесообразного дерганья БД там, где без этого можно обойтись.

    * Кэш должен очищаться по двум условиям (не по одному из, а именно по двум):
    - Время.
    - Протухание по бизнес логике.
    Разрешается по только времени в безвыходных ситуациях, но тогда время - короткий период.
    - При расчете ключей кэша должна использоваться переменная из конфигурации приложения (на случай обновлений кэш сбрасывается кодом, а не флашем кэш-сервера). В случае использования множества серверов - это очень удобный и гибкий инструмент при диплое.

    * О людях:
    - "Я привык писать так и буду дальше" - не вопрос, ревью пройдешь только когда поменяешь свое мнение.
    - "Я пишу в vim-е и мне так удобно" - здорово, код консолью я тоже в нем пишу)) но есть требования к коду, если в них не сможешь - не пройдешь ревью.
    - "Я скопировал этот страшный метод и поменял 2 строчки" - это конечно замечательно, но по блейму автор всего этого метода ты, так что давай без говняшек, хорошо?
    - "Оно же работает!" - вот эта фраза переводится примерно так: "да, я понимаю, что пишу полную хрень, но не могу писать нормально потому, что руки из жо", я правильно тебя понял?))
    - "У меня все работает!" - рад за тебя, а как на счет продакшна?
    - "Там все просто" - не используй слово "просто", от слова "совсем". Вот тебе кусок кода (первого попавшегося с сложной бизнес логикой), где там ошибка (не важно есть она, или нет)? Ты смотришь его уже 2 минуты, в чем проблема, там же все "просто"))

    * Всякое:
    ActiveRecord (это я вам как в прошлом фанат Yii говорю) - полное говно, примите за исходную. По факту у вас бесконтрольно по проекту гуляют модельки с подключением к БД. Не раз натыкался на то, что в тех же шаблонах вызывают save, или update (за такое надо сжигать).
    То, что используется Laravel - это печально((. Что бы выполнить требования приведенные выше, приходится "воевать" с фреймворком.

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

    UPD

    Формализировал данные критерии по ссылочке: https://github.com/index0h/php-conventions
    Ответ написан
    55 комментариев
  • Как в битрикс подключить Font-Awesome?

    @fenixol Автор вопроса
    Это подключение в хедере всех скриптов.
    <?
    # Если пользователь набрал домен с www
    if (preg_match( '|^www\..*|', $_SERVER [ 'HTTP_HOST' ]))
    { # переадресовываем на домен без www
    header ( 'HTTP/1.0 301 Moved Permanently' );
    $url = trim ($_SERVER [ 'REQUEST_URI' ], '/');
    if(trim($_SERVER [ 'REQUEST_URI' ], '/') != '')
    $url .= '/';
    header('Location: electrolopata.ru'. $url);
    die();
    }

    if(preg_match('#index\.php#siU',$_SERVER['REQUEST_URI'])){
    $url = preg_replace('#index\.php#siU','',$_SERVER['REQUEST_URI'],1);
    LocalRedirect($url);
    }

    if( $_SERVER['REQUEST_URI'] == '/index.html' ){
    LocalRedirect('/');
    }
    if( $_SERVER['REQUEST_URI'] == '/index.php/' ){
    LocalRedirect('/');
    }

    if (!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED !== true)
    die();
    IncludeTemplateLangFile($_SERVER["DOCUMENT_ROOT"] . "/bitrix/templates/" . SITE_TEMPLATE_ID . "/header.php");
    $wizTemplateId = COption::GetOptionString("main", "wizard_template_id", "eshop_adapt_horizontal", SITE_ID);
    CUtil::InitJSCore();
    CJSCore::Init(array("fx"));
    $curPage = $APPLICATION->GetCurPage(true);
    /*
    





    */
    ?>




    <? $APPLICATION->ShowHead(false);?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/font-awesome.css"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/colors.css"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/reset.css"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/jquery.bxslider.css"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/modal.css"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/mistakes.css"); ?>
    <?
    if(strpos($curPage, '/cart/order/make/') === false)
    $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/select.css");
    else
    $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/jQueryFormStyler/jquery.formstyler.css");
    ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/checkbox.css"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/fancybox/jquery.fancybox.css?v=2.1.5"); ?>
    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/css/main.css"); ?>

    <? $APPLICATION->SetAdditionalCSS(SITE_TEMPLATE_PATH . "/jquery.fancybox.css.css"); ?>


    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/modernizr-2.6.2.min.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery-1.8.3.min.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.bxslider.min.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/rating_stars.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/fancybox/jquery.fancybox.pack.js?v=2.1.5"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.countdown.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.slider.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/mistakes.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.maskedinput.min.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/jquery.fancybox.js"); ?>

    <? if(strpos($curPage, '/cart/order/make/') === false)
    {

    $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.select.js");
    ?>


    <?
    }
    else

    {
    $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/jQueryFormStyler/jquery.formstyler.min.js");
    ?>


    <?}

    ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.nice.radio.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.checkbox.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/js/jquery.main.js"); ?>
    <? $APPLICATION->AddHeadScript(SITE_TEMPLATE_PATH . "/script.js"); ?>


    <? $APPLICATION->ShowTitle() ?>

    <?if( (strpos($_SERVER['REQUEST_URI'], '?') !== false) || ($_SERVER['REQUEST_URI'] == '/cart/') || ($_SERVER['REQUEST_URI'] == '/search/') ):?>

    <?endif;?>

    Уже перезаливали заново и при внесении в файл main.css небольшого изменения (закоментировал строку ) - вылетали стили битрикса.
    .sub-menu div > a {
    display: block;
    padding: 8px 5px 9px 49px;
    font-size: 11px;
    line-height: 13px;
    font-family: "dinpro-medium";
    /* color: #fff;*/
    text-transform: uppercase;
    white-space: pre-line;
    position: relative;
    z-index: 2;
    text-decoration: none;
    }
    После того как снова убрал сжатие файлов css - сайт заработал но файл main.css не показывает мои изменения. Кеши чистил с админки не один раз. Режим ускорения и композитный сайт отключены. Файл main.css если смотреть через фаербаг имеет такой вид main.css?144983...
    Понятно что он где то закешированый... но где его убрать? Есть предположение что на стороне хоста он закешировался и висит...
    Ответ написан
    1 комментарий
  • Что такое Autowiring в DI-контейнерах?

    Fesor
    @Fesor
    Full-stack developer (Symfony, Angular)
    Autowiring - автосвязывание. То есть контейнер сам на основе типов пытается догадаться от кого кто зависит. Не нужно прописывать явно все зависимости.
    Ответ написан
    Комментировать
  • Что такое static в ООП php?

    Fesor
    @Fesor
    Full-stack developer (Symfony, Angular)
    Для этого вам надо понимать в чем разница между классами и объектами. Вот есть методы и свойства объектов а есть методы и свойства классов. Последние как раз статические свойства и методы. Отсюда и все особенности их работы и возможные варианты использования. Класс у нас в системе может быть только один, а экземпляров этого класса (объектов) много.

    Как правило статические методы используются в качестве порождающих. То есть вы вызываете статический метод класса а он тебе экземпляр этого класса.

    $foo = Singleton::instance();
    $bar = AbstractFactory::create('bar');
    $buz = Buz::fromArray([
        'many' => 'arguments', 'Buz' => 'has', 'private' => 'constructor'
    ]);


    В PHP статику еще любят применять как замену обычным функциям в силу того, что для классов автозагрузка у нас есть, а для функций нету. Не сказать что это сильно хорошо, я бы даже сказал что это плохо. Учитывая что сейчас есть composer а благодаря opcache оверхэда от подключения для каждого запроса файла особо и нет. В целом лучше стараться избегать использования статики или во всяком случае делать в статических методах хоть сколько нибудь сложные вещи. И лучше всегда ограничиваться только случаями для порождения объектов, тут статика выглядит логично.

    Если рассматривать с точки зрения пораждающиз статических методов, нам так же надо знать кого создавать. И тут появляются два ключевых слова - self и static. Причем self равносильно написанию имени класса в котором наш статический метод находится и просто позволяет уменьшить дублирование. static же намного интереснее, так как оно указывает непосдерственно на тот класс, из под которого был совершен вызов. Скажем если у вас есть наследование вы можете запихнуть порождающий метод в базовый класс, и тогда узнать кого создавать в принципе не проблема.

    class Foo {
        public static function createWithSelf() {
             // равносильно new Foo();
             return new self();
        }
        public static function createWithStatic() {
             // а тут мы пока не знаем кто такой этот static
             $foo = new static();
        }
    }
    
    class Bar extends Foo {}
    
    $foo = Bar::createWithSelf(); // тут будет экземпляр Foo
    $bar = Bar::createWithStatic(); // тут будет экземпляр Bar
    Ответ написан
    1 комментарий
  • Какие есть бесплатные домены без рекламы?

    webxaser
    @webxaser
    localhost
    Ответ написан
    Комментировать
  • Где можно найти хорошие уроки по CSS @media screen?

    zooks
    @zooks
    Frontend
    В общем изучать тут нечего. Изменение цвета фона при ширине документа меньше 300px:
    @media screen and (max-width: 300px) {
        body {
            background-color: lightblue;
        }
    }

    www.w3schools.com/cssref/css3_pr_mediaquery.asp - полный список значений.
    Ответ написан
    Комментировать
  • Зачем в PHP нужна рефлексия?

    @Nc_Soft
    <?php
    
    class Stp008 {
       /**
        * Вытащи этот текст проще, че это сделал я
        */
    	public function get() {
    
    	}
    }
    
    $reflection = new ReflectionClass('Stp008');
    $method = $reflection->getMethod('get');
    echo $method->getDocComment();
    Ответ написан
    Комментировать
  • Какой CMS лучше в данной ситуации DataLife Engine или 1С-Битрикс?

    Fesor
    @Fesor
    Full-stack developer (Symfony, Angular)
    я не знаю какой посоветовать, ибо не имею с ними дела... Но ради бога, только не битрикс и только не DLE.... Вам wordpress-а должно хватить.
    Ответ написан
    5 комментариев
  • Какой смысл в использовании шаблонизаторов?

    Fesor
    @Fesor
    Full-stack developer (Symfony, Angular)
    Шаблонизатор шаблонизатору рознь. Но в целом следует выделить общие задачи. которые должны решать за вас шаблонизаторы. С blade не работал и не вижу смысла есть есть twig.

    Безопасность. Это пожалуй можно поднять на верх. Типичная картина в шаблонах на php - <?= $someUserInput; ?>. Частенько это можно встретить в выводе инпутов, при формировании ошибок поиска (мол "по запросу $userInput ничего не найдено. То есть вставляем в инпут подключение наших js скриптиков, если это форма поиска - делимся с "другом" и забираем его сессию. Ну или еще какие забавные штуки можно делать. А ведь все очень просто решается. Ставим какую-то функцию, которая по умолчанию будет фильтровать XSS инъекции при выводе, и не будет этого делать только если мы попросим. Если писать просто на php - появляются отвратные функции, которые можно просто забыть вызвать. А с шаблонизаторами мы пишем красивые {{ someUserInput }} и можем спать спокойно.

    Помогают соблюдать принцип DRY. Современные средства шаблонизации (twig например), предоставляют вам возможность разделять шаблоны на блоки, переиспользовать их несколько раз, выделять макросы, наследовать шаблоны... словом все что угодно. лишь бы вы могли реюзать куски html а не копипастить их.

    Ограничивают полет фантазии разработчика. Далеко не новость что разработчики ленивые засранцы. Особенно молодые. Если им в шаблоне внезапно понадобились какие-то данные из БД, или данные связанные с запросом, большинство не будет париться и зафигачит нужный код прямо в темплейте. Так же некоторые грешат тем что часть бизнес логики размазывают по шаблонам. Так же встречал проекты отданные на суппорт, где чуваки в шаблонах разбирали через xpath ответы от сторонней апишки (которая использовалась вместо базы данных. То есть это дело было размазано по всему проекту). Рефакторинг в случае изменения апишки будет болью.

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

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

    Так как за все эти приятные вещи мы по сути ничего не платим (шаблонизатор должен компилировать все это в нативный php так что оверхэда просто не будет), почему бы не пользоваться?
    Ответ написан
    1 комментарий
  • Как в Yii2 поменять /web на www?

    SamDark
    @SamDark
    Yii2 core team
    Можно просто папку переименовать.
    Ответ написан
    Комментировать
  • Хороший пример реализации роутинга/чпу?

    FanKiLL
    @FanKiLL
    У меня правда для java.
    routes = new ArrayList<RouteData>();
    routes.add(new RouteData("GET", "home/index/{^[0-9]{1,15}$}/valid/{^[A-z]{3,15}$}"));
    routes.add(new RouteData("POST", "home/index/{^[0-9]{1,15}$}/valid/{^[A-z]{3,15}$}"));
    


    Я в своё время решил параметры держать в {} скобках, так парсер знает что это параметр + можно регулярку туда засунуть, чтоб роутинг чётко совпадал. Плюс держа в скобках, вы можете делать роутинг не по паттерну Controller/Action/id А что-нибуть более гибкое(особенно с регулярками описывающие парраметр)

    Controller/Action/{id}
    Controller/{id}/Action
    Controller/Action/{name}/delete/{id}
    


    Плюс в моём примере метод запроса (POST, GET DELETE...) тоже имеет значение.
    Парсер роутера, разбирает пришедший запрос и ищет есть ли такой роутинг у нас, при первом же совпадение, например
    Question/Read/{name}/{id} + метод запроса тоже должен совпадать.

    Запустится Controller = Question, Action = Read с параметрами name, id

    Конечно с помощью reflection проверяется есть ли такой контроллер и action метод с такими параметрами если нет — 404

    Вообщем как то так, трудно описать, если что спрашивайте. Но думаю принцип понятен.

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

    Http Method Delete: User/Profile/{name}/delete/{id} параметр delete в url не играет никакой роли, он просто для удобства и понимания что делает URL.
    Ответ написан
    Комментировать