• Какую выбрать CMS для онлайн учебника?

    Kozack
    @Kozack
    Thinking about a11y
    Предположу, что учебник подразумевает множество статических страниц. В этом случае, я бы выбрал nuxt generateВ чем суть: вы создаёте сайт на VUE. ЧПУ, обратная связь, SEO, возможность сделать тестирование, всё тут. Обычно nuxt работает как сервер и генерирует страницы в рантайме, но с параметром generate, на этапе сборки создаётся пачка статических файлов, которые потом размещаются на сервере. Быстродействие такого сайта выше всяких похвал.

    Едиственный динамический елемент по вашуму ТЗ: это комментарии. При таком подходе их не реализовать обычным способом. Но никто не мешает подгружать их асинхронно.

    Таким образом мы имеем:
    Весь сайт — это статические html страницы. (ЧПУ, ссылки, СЕО всё остаётся рабочим)
    А вся работа с сервером и БД сводится к созданию и выдаче комментариев в JSON. А если использовать сторонний сервис комментариев, типа Disqus, то даже этого не требуется
    Ответ написан
    4 комментария
  • Как показать кнопку корзины в Joomla VirtueMart 3?

    pavelcarcass
    @pavelcarcass
    ИТ-менеджер из г. Иркутска
    В админке Joomla зайди в менеджер модулей, найди модуль корзины в позиции cart и проверь настройки доступа (в правой колонке). Если хочешь, чтобы доступ к модулю имели все пользователи, выставь значение Public. См. скриншот.
    5b4176dd682c4662257871.jpeg
    Ответ написан
    1 комментарий
  • Как делать fade эффект для bootstrap carousel?

    igorbelikov
    @igorbelikov
    Fullstack Developer
    Ответ написан
    Комментировать
  • Передача POST параметров в AltoRouter?

    $router = new AltoRouter();
    
    $router->map( 'GET', '/form', function() {
        # Вывод формы
    });
    
    $router->map( 'POST', '/form', function() {
       # Обработка формы и вывод результата или редирект на /form в случае неудачи
    });
    Ответ написан
    1 комментарий
  • Конфликт JS скрипта скроллинга по странице и Bootstrap?

    @BLVST
    Frontend. Открыт к предложениям.
    href="#carousel-example-generic"
    и
    $("a[href^=#]").click(function(e){
    нужно уточнить выборку ссылок, так как кнопки карусели попадают под a[href^=#]
    Ответ написан
    1 комментарий
  • Конфликт JS скрипта скроллинга по странице и Bootstrap?

    CRYPTOMUSK
    @CRYPTOMUSK
    Как вариант (да, он не идеален) можно после # поставить символ, типа - $("a[href^=#i]").click(function(e){
    id'шники называть как id="iclass"
    Ответ написан
    1 комментарий
  • Что стоит изменить в классе PHP для работы с файлами?

    index0h
    @index0h
    PHP, Golang. https://github.com/index0h
    Конкретно по коду отписал ниже. Как хелпер - ну может быть, почитайте на досуге php.net/manual/ru/spl.files.php
    // Я надеюсь вы знаете про PSR4 и символ подчеркивания тут случайно
    class MV_Files
    {
        // PSR2: методы пишутся в lowerCamelCase
        // Нет проверки аргументов. что произойдет, если $name будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        public function CreateFile($path, $name, $right = 0777)
        {
            if (!file_exists($path . '/' . $name)) {
                // Посмотрите http://php.net/manual/ru/function.touch.php
                fclose(fopen($path . '/' . $name, 'w'));
                chmod($path . '/' . $name, $right);
                // PSR2: Перед return перевод строки
                return true;
                // Не ясно, зачем вам тут else?
            } else {
                return false;
            }
        }
    
        // PSR2: методы пишутся в lowerCamelCase
        // Нет проверки аргументов. что произойдет, если $name будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        public function CreateFileContent($path, $name, $text, $right = 0777)
        {
            if (!file_exists($path . '/' . $name)) {
                // file_put_contents, не?
                $file = fopen($path . '/' . $name, 'w');
                fwrite($file, $text);
                fclose($file);
                chmod($path . '/' . $name, $right);
                // PSR2: Перед return перевод строки
                return true;
                // Не ясно, зачем вам тут else?
            } else {
                return false;
            }
        }
    
        // PSR2: методы пишутся в lowerCamelCase
        // Нет проверки аргументов. что произойдет, если $path будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        public function AddFileContent($path, $text)
        {
            if (file_exists($path)) {
                // file_put_contents($path, $content, FILE_APPEND)
                $file = fopen($path, 'a');
                fwrite($file, $text);
                fclose($file);
                // PSR2: Перед return перевод строки
                return true;
                // Не ясно, зачем вам тут else?
            } else {
                return false;
            }
        }
    
        // PSR2: методы пишутся в lowerCamelCase
        // Нет проверки аргументов. что произойдет, если $path будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        public function OverwriteFileContent($path, $text)
        {
            if (file_exists($path)) {
                // file_put_contents($path, $content)
                $file = fopen($path, 'w');
                fwrite($file, $text);
                fclose($file);
                // PSR2: Перед return перевод строки
                return true;
                // Не ясно, зачем вам тут else?
            } else {
                return false;
            }
        }
    
        // PSR2: методы пишутся в lowerCamelCase
        // Нет проверки аргументов. что произойдет, если $oldname будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        // Чего вдруг метод статический?
        public static function RenameFile($oldname, $newname) {
            if (file_exists($oldname)) {
                return rename($oldname, $newname);
            } else {
                return false;
            }
        }
    
        // PSR2: методы пишутся в lowerCamelCase
        // Нет проверки аргументов. что произойдет, если $path будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        public function DeleteFile($path)
        {
            if (file_exists($path)) {
                return unlink($path);
            } else {
                return false;
            }
        }
    
        // Нет проверки аргументов. что произойдет, если $path будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        public function getFileContent($path)
        {
            if (file_exists($path)) {
                return file_get_contents($path);
            } else {
                return false;
            }
        }
    
        // Нет проверки аргументов. что произойдет, если $path будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        // "size" обычно трактуется как число, а не форматированная строка.
        public function getFileSize($path)
        {
            if (file_exists($path)) {
                // Конструкция - гавно)). Если хотите интовый $size - то сохраняйте в него инт
                (int)$size = filesize($path);
                switch (true) {
                    case $size >= 1073741824:
                        $size = round($size / 1073741824 * 100) / 100 . ' GB';
                        break;
                    case $size >= 1048576:
                        $size = round($size / 1048576 * 100) / 100 . ' MB';
                        break;
                    case $size >= 1024:
                        $size = round($size / 1024 * 100) / 100 . ' KB';
                        break;
                    default:
                        $size .= ' Bytes';
                        break;
                }
                return $size;
            } else {
                return false;
            }
        }
    
        // Нет проверки аргументов. что произойдет, если $path будет массивом?
        // Не ясно, на кой вам тут возврат bool? Если что-то не так - бросайте исключение.
        // Прочитайте http://php.net/manual/ru/class.splfileinfo.php
        public function getFileInfo($path)
        {
            if (file_exists($path)) {
                return pathinfo($path);
            } else {
                return false;
            }
        }
    
        // Нет проверки аргументов. что произойдет, если $dirname будет массивом?
        // Прочитайте http://php.net/manual/ru/class.directoryiterator.php
        public function getFileList($dirname)
        {
            $dir = array_diff(scandir($dirname), ['.', '..']);
            $files = [];
            // Рекомендую перед управляющими конструкциями делать перевод строки, так читать проще
            foreach ($dir as $value) {
                if (is_file($value)) {
                    $files[] = $value;
                }
            }
            // PSR2: перед return перевод строки
            return $files;
        }
    // Лишний перевод строки
    }
    Ответ написан
    2 комментария
  • Как правильно использовать прозрачность(opacity в background-image)?

    webirus
    @webirus
    Тыжверстальщик! Наверстай мне упущенное...
    https://jsfiddle.net/webirus/e1mtr4sq/1/
    Еще как вариант, через псевдоэлементы)
    Ответ написан
    Комментировать
  • Switch с нулевым условием в php?

    index0h
    @index0h
    PHP, Golang. https://github.com/index0h
    Вы сравниваете красное и мягкое (int с bool), понятное дело, что результат будет не очевидным)). По сути вы сравниваете примерно так:echo 0 == (0 >= 1073741824);
    Ответ написан
    3 комментария
  • Попросили проверить код, на что смотреть нужно?

    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 комментариев