Можно ли отдать небольшой промышленный проект PHP на оценку правильности архитектуры?

Question

[Денис Сепетов]

На некоем промышленном предприятии единственным программистом было разработано ПО и теперь сам разработчик хочет проверить его "качественность". Хочется взгляда со стороны более опытных коллег.

Претензий к работе ПО нет, все довольны и счастливы, но делалась в спешке, а потому только некоторые части реализованы красиво. Весь объем написанного кода около 5 тысяч строк включая html-вёрстку форм. Разумеется, развёрнутое разъяснение и возможность вникнуть в проект - прилагаются.

Стоял ли когда-нибудь перед вами такой вопрос? Стоит ли игра свеч? Куда/кому бы вы отдали такой проект на проверку? Цена вопроса? Не попахивает ли "студенчеством"?

Система решает задачу маркировки товаров и предоставляет рабочим уже готовый графический интерфейс для управления станками и другим оборудованием в цехах. Система умеет:
- получать из уже существующей ERP штрихкоды/товары и их реквизиты
- получать задания на производство и маркировку
- управлять принтерами/маркираторами/перемотчиками/сканерами/камерами технического зрения и другим специфическим оборудованием
- сохранять в БД все ошибки и результаты своей работы
- отдавать обратно в ERP всё, что требуется

Comments

[Денис Сепетов]

Да, я знаю, что PHP для программирования станков ЧПУ и тому подобных не применяют. На это нашлись причины, теперь успокаиваю себя тем, что в общих чертах PHP - это язык общего назначения, а потому можно :-)

[Владислав Лысков]

То есть ты хочешь показать код коммерческого продукта человеку, которого ты даже лично не знаешь? Смело

[Денис Сепетов]

Владислав Лысков, без паники. Всё предусмотрено, никаких рисков для предприятия.

Answer 1

[rPman]

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

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

Если второе - то все будет зависеть от требований этой отчетности, и там способ проведения аудита будет определяться наличием и доступностью компаний под эти требования.

Comments

[Денис Сепетов]

Да, первое. Второе не интересует. Спасибо.

Уже сильней склоняюсь к тому, чтобы всё-таки заморочиться и выделить на это время и деньги :-)

Answer 2

[ThunderCat]

Претензий к его работе нет, все довольны и счастливы, но делалась в спешке,

...

и теперь хочется проверить его "качественность".

"Вы или крестик снимите, или трусы оденьте...", это не к вам конкретно, а в целом всем таким заказчикам. Я ведь верно понимаю что заказчик был в курсе что сроки не соответствуют необходимым для данной задачи?

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

Если есть сомнения в качестве наскоро сляпанного продукта - достаточно обратиться к разработчикам, попросить написать что можно улучшить/переписать нормально и сколько это будет стоить. Скорее всего будет быстрее и дешевле какой-либо экспертизы. Тем более что основная задача решена, и можно уже не гнать по срокам.

Comments

[Денис Сепетов]

Я отредактировал и переформулировал вопрос, так как он и правда оказался двусмысленным и был неверно понят.

Я - не заказчик. Я и есть разработчик этого ПО. Движет мной желание саморазвития, ибо коллег у меня, имеющих возможность оценить, нет. Я тут один и за архитектора, и за аналитика, и программиста и т.п. Работодателя этот вопрос вообще не интересует.

[ThunderCat]

1) Если код не содержит технологических нюансов, которое производство могло бы охранять как интеллектуальную собственность и иное неразглашение не прописано в контракте/соглашении, то почему бы и нет, хотя я не вижу смысла тратить на это время, силы и средства.
2) При дальнейшей работе в отрасли, если не перестанете расти (а перестать расти в принципе очень близко к вылететь из профессии), то через какое-то сравнительно небольшое время вы и сами достаточно хорошо будете понимать что было не так. Стоит ли ускорять этот процесс за деньги - дело ваше, за это время можно пару проектов сделать, узнать и научиться чему то новому...

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

[Денис Сепетов]

ThunderCat, спасибо, учту.
В том-то и проблема, что:
- "если не перестанете расти"
- "то через какое-то сравнительно небольшое время"
не всегда работает. Многие разработчики не могут самостоятельно перепрыгнуть определённую планку ввиду отсутствия критики от других разработчиков. Я как раз из таких.

По цене - да, уже оценил и понял, что не зря решил оценить только архитектуру. :-)

[ThunderCat]

Денис Сепетов,

Многие разработчики не могут самостоятельно перепрыгнуть определённую планку ввиду отсутствия критики от других разработчиков.

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

[Денис Сепетов]

ThunderCat, вы безусловно правы :-) Команда - это незаменимо!