Задать вопрос
  • Попросили проверить код, на что смотреть нужно?

    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 комментариев
  • Почему в bootstrap реальный размер и размер который указан в стилях класса .container отличаются?

    zona7o
    @zona7o
    Веб-разработчик
    Из-за парамерта box-sizing: border-box;
    Синтаксис
    box-sizing: content-box | border-box | padding-box | inherit

    Значения
    content-box
    Основывается на стандартах CSS, при этом свойства width и height задают ширину и высоту контента и не включают в себя значения отступов, полей и границ.
    border-box
    Свойства width и height включают в себя значения полей и границ, но не отступов (margin). Эта модель используется браузером Internet Exporer в режиме несовместимости.
    padding-box
    Свойства width и height включают в себя значения полей, но не отступов (margin) и границ (border).
    inherit
    Наследует значение родителя.
    Ответ написан
    Комментировать
  • На какой бесплатной CMS можно реализовать сайт с информацией о населенных пунктах области?

    tlito
    @tlito
    drupal, c++, seo
    на D7 вы также сможете отмечать пункты на карте и выводить на общей карте все ваши пункты с ссылками на страницы этих городов.
    вам потребуются модули drupal 7:
    views,
    geofield,
    google maps.
    Ответ написан
    Комментировать
  • Как вытащить нужный параметр из JSON в PHP?

    27cm
    @27cm
    TODO: Написать статус
    Может всё-таки json_decode()?
    Ответ написан
    4 комментария
  • Как решить проблему с адаптивным дизайном?

    Punkie
    @Punkie
    Но, заходя через телефон, например Nexus 5 (у которого разрешение экрана 1920px на 1080px)


    Вы забываете учитывать плотность пикселей (в css именуемый как device pixel aspect ratio). У Некcуса вашего он равен 3:

    4d0988400ec84ad6b38626d464ed0327.png

    То есть, на вашем девайсе страница рендерится в разрешении 360х567. От сюда возможны ваши проблемы с таргетированием медиа-запросов.

    А вот как на седьмой нексус таргетироваться: https://gist.github.com/afj176/6357322
    Ответ написан
    4 комментария
  • Как решить проблему с адаптивным дизайном?

    zorro76
    @zorro76
    max-device-width укажите

    и почитайте вот это
    Ответ написан
    Комментировать
  • Как правильно создать репозиторий Github?

    zorro76
    @zorro76
    1.Нужно зарегистрироваться на сервисе.
    2.Создайте форк (fork) интересующего вас репозитория.
    3.Теперь у вас есть собственная копия удаленного репозитория, связанная с изначальной. Клонируйте эту копию.
    Теперь вы можете сохранять коммиты в свой локальный репозиторий, а потом пушить их в свой удаленный. Если вы захотите предложить свои изменения в изначальный репозиторий, для этого есть специальный инструмент под названием pull request.
    Ответ написан
    Комментировать
  • Можете посоветовать CMS для корпоративного портала, для дилеров?

    @mr_ko
    Javascript, Node.js. React.js, Vue.js, Wordpress
    Из CMS можно Drupal 7 брать. Там уровни доступа гибко можно настроить, да и с разными типами контента никаких проблем. То что вам нужно есть "из коробки" :) Но вот с IP скорее всего готового не найдете. Хотя в Drupal это тоже в пару хуков решить можно.
    Ответ написан
    Комментировать
  • Как можно сделать бонус за вход на сайт?

    MetaDone
    @MetaDone
    Хорошо сформулированный вопрос - 50% решения
    $query = mysql_query
    2015 год на дворе, mysql_query в php7 уже не будет.
    по вопросу - обновляйте последнюю дату входа или положите в куки, чтоб базу данных не мучать. есть в куках вход - даете бонусы и т.д.
    Ответ написан
    3 комментария
  • В каком виде сделать CMS для конкретного сайта?

    @FOGstudio
    Лучше выбрать CMS, под такой сайт подойдут почти все. Выберите просто на уровне удобства админки для себя. Я предпочитаю Drupal
    Ответ написан
    Комментировать
  • Какой лучший datepicker?

    nazarpc
    @nazarpc
    Open Source enthusiast
    Я когда-то написал PickMeUp, многие пользуются, статья на хабре была.
    Нужен только jQuery, настраивается внешний вид как угодно, есть SCSS переменные для удобства.
    Как его можно было не найти мне не понятно, поиск на Bower/GitHub быстро выдает
    Ответ написан
    2 комментария
  • На чем делают такой frontend?

    mmmaaak
    @mmmaaak
    А может просто открыть код этого сайта и посмотреть?
    Ответ написан
    1 комментарий
  • Как на бордер засунуть слово?

    DevMan
    @DevMan
    можно так jsfiddle.net/v148uo28
    но насколько это верно, я не в курсе.
    Ответ написан
    Комментировать
  • Требуется ли знание DOM для фронтендеров?

    DOM нужно знать как "отче наш".
    Ответ написан
    Комментировать
  • Существует ли обширная книга по js?

    IonDen
    @IonDen
    JavaScript developer. IonDen.com
    Эй, не забываете про MDN! https://developer.mozilla.org/ru/
    Ответ написан
    Комментировать
  • Как сделать так что бы он автоматом загружал из папки те страницы (если они есть), которые есть в папке?

    Denormalization
    @Denormalization
    Привет нулевые. Давненько я такого не видел.
    Используйте шаблонизатор + нормальный роутинг
    Ответ написан
    Комментировать
  • Есть ли жинь за пределами CMS?

    Fesor
    @Fesor
    Full-stack developer (Symfony, Angular)
    Определитесь, вы хотите во фронтэнд перебраться или все же фул-стэк?

    но как насчет бекэнда/бд ?

    Между фронтэндом и бэкэндом лежит обычно REST/JSON RPC апишки (особое внимание стоит обратить на стандарт jsonapi.org). Важно сначала с этим разобраться хорошенько, ибо масса толковых бэкэндеров делают апишки через одно место вне зависимости от стэка технологий.

    Что до выбора БД (если вы сами будете апишки писать) - рекомендую сразу забыть о MySQL и перебираться на PostgreSQL. Проблем сразу станет чуточку меньше.

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

    Да и в целом с каким бэк эндом работают люди использующие технологии вроде angular, backbone , BEM нынче ?

    Использую Angular на ES2015 (babel.js) (от Backbone отказались пару лет назад), крайне упрощенный вариант BEM с минимальным использованием каскада, хотя это особо не интересно. Бэкэнды крутятся на PHP, на Symfony2 + Doctrine2, всем доволен. Можно угорать по ООП и DDD и инструменты этому способствуют.

    Для документации API еще рекомендую посмотреть в сторону RAML или Api-blueprint (для последнего сейчас пишу парсер документации, ибо пока с этим все хорошо только для node.js), на основе которых хорошо продумывать апишки, а так же чуть автоматизировать рутину. Например генерировать стабы/моки для апишки пока пишите клиент а апишка еще не готова. Ну и т.д.
    Ответ написан
  • Эпоха монетизации простых сайтов ушла в прошлое?

    angrySCV
    @angrySCV
    machine learning, programming, startuping
    строить свой бизнес на основе чужих сервисов (поисковых систем с их капризами).
    очень не далёкая стратегия, она была дефектная ещё задолго до 2009года, хотя согласен что она давала выхлоп.
    какие перспективы у сео? -> уже давным давно никаких.
    Более, уверен, что через несколько лет не будет никакого СЕО -> изза падающей доходности и потери доли рынка, яндекс сам начнёт капитализировать СЕО (и что там можно выжать) , объявив исключительно платную выдачу -> типа кто сколько заплатит, такое и будет ранжирование, мотивируя тем что так в топе будут НАИБОЛЕЕ качественные сайты за которыми следят пользователи.
    вобще эпоха простых сайтов уже прошла, очнитесь. Глобализация. . .
    крупные площадки всё в себя пылесосят -> вы или создаёте такуюже, либо идите лучше на стройку работать -> там в разы больше чем 1200 в день платят.
    Ответ написан
    3 комментария