Сколько стоит codereview и что дают на выходе?

Question

[pup_pupets]

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

Answer 1

[Ярослав]

За "потыкать носом", я думаю, нужно заплатить одну сумму.
За "исправить" необходимо заплатить отдельно еще сумму.
Цена верьируется от уровня и опыта проверяющего. Может потянуть от пары баксов в час (индус-консультант-эксперт) до 50 баксов и выше.
По сути ведь специалист продает свое время независимо от того, пишет код, проверяет или разрабатывает архитектуру.

Comments

[pup_pupets]

Да я как бы не спорю, что это стоит денег и понимаю, что нужно ещё разобраться в проекте, но любопытны цены на подобное

[spotifi]

pup_pupets: цена зависит от масштаба проекта, глубины проверки, используемых тобой технологий, порядка в твоем проекте....

если серьезная оценка/совет/разбор полета - считай как ставка рядового программиста проекта умножить н 3 или на 5

если просто code review - то примерно как твоя ставка, то есть это такой же человек как и ты, но смотрит свежим взглядом.

Answer 2

[Максим Кудрявцев]

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

Answer 3

[atis //]

Средняя цена разработки от 1000p день ( 8 часов ). Считайте.

Comments

[pup_pupets]

Что-то мало

[atis //]

От 1000p!

[atis //]

Ключевое слово ОТ

[JihadTV]

1000р в день? Стажеры больше получают.

[thenno]

JihadTV: интересные стажеры, которые зарабатывают на руки 160 000 в месяц. Это крепкий мидл, даже не джуниор и тем более не стажер.

1000 рублей/час * 40 часов/неделю * 4 недели в месяце.

[JihadTV]

thenno: прочитайте, пожалуйста, внимательнее.

[atis //]

1000p * 30 Дней = 30к. минимальный уровень джуна.

[thenno]

JihadTV: да, день, а не час, прошу прощения. 125 в час - нормальная ставка для стажера.

[JihadTV]

atis //: Несчастные джуны. Вы в курсе что по ТК им положен отдых?%)

Answer 4

[Пума Тайланд]

дорого )
обычно человек с левелом чтобы ревьюить код довольно хорошо в программировании.

Answer 5

[spotifi]

Code review - это нечто иное.
Это когда прежде чем выпустить твою программу в production ее просматривает еще один программист.
Но это делается небольшими кусками.

Небольшое изменение внесли.
Просмотрели.
Выпустили в production.

А то что ты хочешь - это не code review, строго говоря.
Это общая оценка архитектуры проекта - это дорого.
Возьми свою почасовую ставку и умножь на 3-5.

Answer 6

[Алексей Козлов]

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

Если же нужно просто отловить глупые косяки, которые не видно "замыленым" глазом именно в своём коде, то проще всего объединиться с коллегой-программистом и просматривать друг у друга код.

Answer 7

[dmitriy]

прогони через https://scrutinizer-ci.com
выловит часть ошибок

Answer 8

[Запорожченко Олег]

Из Вашего вопроса не совсем понятно зачем Вам это нужно.

Если проект разросся так, что его дальнейшее развитие невозможно, то тогда Вам как раз и нужен ревьювер и он будет стоить кучу денег. От 1500р в час, если Вам повезет.

Если перед выпуском продукта хотите быть уверены. что все будет работать нормально, лучше покройте код тестами.

Если вы хотите просто подтянуть качество кода и получить опыт, то есть gem https://github.com/whitesmith/rubycritic который укажет на много ошибок. Также у меня в блоге есть цикл статей, который описывает самые распространенные ошибки начинающего рейлс разработчика c3gdlk.ru/blog/rails/ruby-on-rails-samye-rasprostr... Некоторые из них просто рекомендации, что-то вроде style guide. Некоторые делают код и подход к работе намного лучше.

В самой архитектуре придерживайтесь простого правила - Совсем тонкий контроллер, в нем нет бизнес логики. Чуть потолще модель, она служит только для доступа к данным и работе с ними,в ней тоже нет бизнес логики.

Все логика в так называемых "Сервисах" - https://en.wikipedia.org/wiki/Service-oriented_arc...
Хороший пример реализации, в который можно подсматривать - https://github.com/gitlabhq/gitlabhq