С наступившим новым годом, хабровчане!

2015 год был очень насыщенным для мира PHP: это и долгожданный релиз 7-ой версии, и мажорное обновление многих фреймворков, и множество многообещающих RFC.

Для Php Inspections (EA Extended) прошедший год был тоже очень насыщенным: проект увидел свет, очень быстро повзрослел и смог сделать мир PHP чуточку лучше (symfony2, symfony 1.5, PHP CS Fixer).

Напомню, что Php Inspections (EA Extended) — это отдельный плагин, расширяющий возможности штатного анализа в PhpStorm и Idea Ultimate. Ранние анонсы на хабре: раз, два, три.

Далее (под катом): несколько опросов и возможность повлиять на выбор стратегии на 2016 год.


С самого начала проект решал одну-единственную задачу: сделать рецензирование кода и обучение людей не столь затратным по времени (и не столь болезненым). Получилось очень удачно и удалось помочь многим командам.

Тем не менее, не хочется двигаться вслепую, а получить обратную связь и иметь возможность хоть как-то управлять развитием проекта.

Как результат, на данный момент сформировались следующие сценарии:
  • фокус на создание сообщества (редкие релизы);
  • фокус на коммерческую версию (возможность автоматической замены кода);
  • специализация на high-load проектах.


Как вы думаете, на чём следует сосредоточиться?

PS: Для того чтобы быть в курсе жизни проекта, присоединяйтесь: твиттер. Или LinkedIn для профессионального нетворкинга.

UPD: поддержать проект можно здесь (PayPal).
А вы пользуетесь анализатором?

Проголосовало 200 человек. Воздержался 81 человек.

Помог ли анализатор?

Проголосовало 124 человека. Воздержалось 100 человек.

Какой сценарий важен для вашей команды?

Проголосовало 106 человек. Воздержалось 105 человек.

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

Комментарии (19)


  1. AlexGx
    02.01.2016 14:06

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


  1. kalessil
    02.01.2016 14:30

    Пожалуйста =) Для isset/empty можно даже поведение настраивать (часто жаловались).

    Фокус на создание сообщества (редкие релизы)

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

    фокус на коммерческую версию (возможность автоматической замены кода)

    Смена лицензии (исходники всё еще в опенсорсе), добавление авто-замены кода. Возможно создание заказных проверок. Сейчас у меня нет времени на добавление авто-замены, и, похоже, что это не критичная функциональность.

    Специализация на high-load проектах

    Работа с соответсвующими командами (если смогу набрать достаточно контактов) и новые проверки именно для high-load. Уровни репортинга анализатора тоже надо будет пере-определить.


    1. skobkin
      02.01.2016 19:36
      +1

      По-моему, автозамена действительно не критична. А вот какие-нибудь подсказки по оптимизации кода в сторону быстродействия — это было бы хорошо.


      1. kalessil
        03.01.2016 02:38

        Это как раз сценарий high-load.


        1. skobkin
          03.01.2016 11:09

          Я его и имел в виду.


          1. kalessil
            03.01.2016 13:30

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

            Простые случаи вроде запросов в цикле вместо булк-запроса довольно просто реализовать. Хочется случаев посложнее =)


    1. AlexGx
      02.01.2016 23:02

      тоже считаю что автозамена не нужна, я вообще не люблю автозамены, даже автоформатирование.


      1. kalessil
        03.01.2016 02:43

        Есть проекты где вычищать код без автозамены — это очень долго.

        Например нашей системе около восьми лет, сменились несколько команд и приводить в чувство это счастье приходится поэтапно.


      1. kalessil
        03.01.2016 15:35

        Часть коммента почему-то потерялась.

        Часть автозамены я законтрибутил в PHP CS Fixer: они идут в правильном (на мой взгляд) направлении и надежнее для проекта будет развивать утилиту для коммандной строки (хуки, CI и т.д.).


  1. artyfarty
    02.01.2016 21:31
    +1

    Не совсем понимаю, речь идёт о штатном анализаторе PHPStorm, или это какой-то отдельгный плагин? Если речь о втором, то более явное название и соответствующие пояснения очень помогли бы.


    1. kovalevsky
      03.01.2016 01:08

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


      1. artyfarty
        03.01.2016 01:16

        Очень неудачно называть плагин так же как встроенная функция PHPStorm. Из названия и описания не очень понятно, что это дополнительные испекшены.


        1. kovalevsky
          03.01.2016 01:19

          Вы придираетесь или Вам действительно лень загуглить / поставить и посмотреть? Мне кажется о нем не знает и не использует только ленивый. С момента моего переезда на фронтэнд я мечтаю о чем-нибудь подобном для JS. Так что советую погуглить ;)


          1. artyfarty
            03.01.2016 01:24
            +1

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

            Я не придираюсь, зачем мне это. Я лишь говорю, что популярности плагина очень бы поспособствовало более внятное описание (название-то менять не так просто), ну и в статье тоже не так уж трудно родить один абзац о том, о чем вообще речь. Потому что статик анализ и так есть в PHPStorm. И инспекции.


            1. kalessil
              03.01.2016 02:32

              Добавил параграф с пояснением:

              Напомню, что Php Inspections (EA Extended) — это отдельный плагин, расширяющий возможности штатного анализа в PhpStorm и Idea Ultimate. Ранние анонсы на хабре: раз, два, три.


          1. kalessil
            03.01.2016 15:31

            Это отличный отзыв, спасибо =)


  1. kalessil
    03.01.2016 15:29

    15% проголосовавших указало Другое в опросе со сценариями. А какие это сценарии?

    Обучение, аудит, оценка брать проект или нет или ещё что-то?


    1. greabock
      03.01.2016 18:38

      Не хотите взять пример с pvs-studio? У них, здесь же на хабре, интересные обзоры получаются. Хотя возможно есть более приоритетные задачи. Так что расценивайте это просто как «вброс» для обсуждения )
      И да, в голосовании не мешало бы добавить пункт «впервые слышу об этом анализаторе».


      1. kalessil
        03.01.2016 19:46

        Добавил.

        PVS-studio меня и вдохновили, но у них это бизнес и очень круто, что они готовы инвестировать ресурсы в статьи.