Как проверить код?

Question

[Lrest]

Заказал фрилансеру сделать очень сложный проект на "symfony 2", вот теперь не знаю как проверить код на ошибки и чтобы все было написано по уму. Я не обладаю большими техничесткими знаниями для конкретной и точной проверки (знаю так, поверхностно). Кто что посоветует? Нанимать аудит для проверки?

Comments

[Lrest]

Ребята тогда подскажите еще вот что,
1) - Когда нанимаешь аудит, то сколько это примерно стоит и что туда входит (ну и вообще механизм проверки)?
2) - И есть ли договоренности такого рода: что к примеру цена аудита 50$ а за каждую найденную ошибку идет бонус ну к примеру 10$ итд, то в таком раскладе как мне кажется будет реальный аудит и поиск ошибок (т.к человек заинтересован) а не просто посидел 2 минуты глянул код а через неделю скажет "там все четко!"

Answer 1

[Александр Губарев]

Кроме как нанять профессионального программиста для ревью кода и тестировщика (протестировать я думаю вы можете сами) никак. Всякие lint'ы укажут на проблемы, только если там реально говнокод.

Answer 2

[Сергей Протько]

Есть сервисы типа scrutinizer-ci, которые имеют неплохие средства для анализа кода (различные метрики, учет best practice и т.д.), но без соответсвующего технического бэкграунда пользы вам от него не будет. Вы просто получите оценку кода по 10-ти бальной шкале.

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

Ну а если хотите ревью - нужно заказывать. И тут сложности, ибо нужно реально опытного разработчика искать. Так же немного не понятно что по итогу вы хотите, что бы нашли недостатки архитектуры? Оценили "поддерживаемость" кода? Сказали "говнокод" или "не говнокод"?

Comments

[index0h]

+1 Дополню:
scrutinizer не всегда правильно оценивает код. Например существует практика для защищенного кода: выполнять проверки входных данных на уровне функции, а не на уровне модуля, как правило - это будет оцениваться phpcpd как копипаст, но для данного кейса это необходимо.

[Сергей Протько]

index0h: а можно пример привести? Как-то может не правильно понимаю...

[index0h]

Сергей Протько Ну вот например pastebin.com/J8hXFNmY с эстетической точки зрения оно конечно не очень, но помогает довольно быстро искать ошибки + взломать такой код будет сложнее.

[index0h]

В примере только одна функция, но предполагается такой подход для всех.

[Сергей Протько]

index0h: а вы од этом... считаю подобное допустимым только внутри отдельных библиотек. Для клиентского кода такие загоны излишни. Исключение только в эмуляции enum.

Еще есть вариант запилить на том же goAOP обертку, которая будет на основе phpdoc верифаить типы/правила, тогда и код чистый будет и в продакшене можно отключить.

Если же у вас такие требования к аргументам, так же можно замутить ValueObject и валидировать каким-нибудь валидатором (symfony/validator например). Тогда код тоже будет чистеньким.

В целом же, ждем https://github.com/php/php-src/pull/998 к принятию.

[Lrest]

Сергей Протько: да я именно об этом и говорю чтобы нашли недостатки архитектуры? Оценили "поддерживаемость" кода? Сказали "говнокод" или "не говнокод"? Де искать такого спеца теперь > ?

[Сергей Протько]

Lrest: ну там же где вы разработчика искали - фриланс биржи, знакомые HR-ы, как угодно.

[Lrest]

Сергей Протько: это то понятно, но вот в чем прикол, как я пойму или узнаю что этот человек обладает реально большим пакетом знаний для того чтобы найти чужие ошибки :) получается тупняк :) мне нужен человек чтобы проверить другого человека, а следующий чтобы проверил другого :) Есть знакомые или компаний где сидят знатоки symfony, чтобы можно было к ним обратится, посоветуйте плиз

[Александр Губарев]

index0h: такое можно отрефакторить, написав кастомную валидацию на входящие параметры, где в ф-ии валидаторы будут засунуты проверки наподобии тех, что у вас в if`ах. Хотя допускаю что есть предметные области, где нужен именно такой подход.

[index0h]

Александр Верно, но в этой ситуации узким горлышком системы будет как раз этот валидатор (с точки зрения безопасности). Есть например https://github.com/Respect/Validation довольно неплохое решение.

Answer 3

[OlegSivokobov]

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

Answer 4

[keltanas]

Думаю, вам нужен технически подкованный партнер :)
Для Symfony есть сервис https://insight.sensiolabs.com Но он по большей части анализирует правильность оформления кода.
Для ошибок обычно используются системы автоматического тестирования PhpUnit, Behat. Тесты для Behat может прочесть даже не обладающий техническими знаниями человек.