Reviewer1C · ревью кода 1С

Узнайте, что вам сдали,
до того как это сломается

Независимая приёмка доработок 1С. Загрузите доработку и задание — получите отчёт: соответствует ли код тому, что вы просили, что сломается при обновлении типовой, где риски потери данных и прав.

Без доступа к вашей базе. Первая проверка одной обработки — бесплатно, всегда.

Знакомо?

Вы приняли работу, а проверить её было некому

Три ситуации, ради которых к нам приходят. Во всех отчёт стоит дешевле одного дня разбирательств.

«Через полгода всё сломалось при обновлении»

Подрядчик правил типовые объекты вместо расширения. Обновление затёрло правки, доработку пришлось делать заново — за ваш счёт.

«Работает, но не то, что просили»

Половина пунктов задания в коде отсутствует. Обнаруживается через месяц, когда подрядчик уже закрыл акт.

«К вечеру база встаёт»

Запрос внутри цикла на тысяче строк. В приёмке на тестовых данных не видно — видно в пятницу в конце месяца.

Что вы получаете

Отчёт, который читается за пять минут

Ниже — настоящие фрагменты отчёта по реальной доработке. Вердикт одной строкой, дальше подробности для вашего разработчика.

Выгрузка задолженности контрагентов · epf · УТ 11.5 108 строк · 5 методов
Вердикт Переделать

Соответствие заданию

Что просилиСостояние
Отобрать просроченную задолженностьне сделано
Вывести число дней просрочкине сделано
Файл в кодировке UTF-8ANSI
Исключить помеченных на удалениене сделано
Нет долгов — не создавать файлсоздаётся пустой
Разделитель «точка с запятой»сделано

Замечания

Критично Обработка не запускалась ни разу
Переменная объявлена на клиенте, а читается в серверной процедуре. Серверный контекст клиентских переменных не видит: при первом же нажатии кнопки — отказ «Переменная не определена». Это значит, что переданный вам код не проверяли даже запуском.
&НаКлиенте
Перем ПутьКФайлу;
...
&НаСервере
Процедура ЗаписатьФайл(Строки)
    Текст = Новый ЗаписьТекста(ПутьКФайлу, КодировкаТекста.ANSI);
Критично Ошибка записи файла подавляется молча
Блок обработки исключения пуст. Нет прав на каталог, файл открыт в Excel — пользователь увидит «Готово, строк: 812» и уйдёт, а файла не будет. Это хуже падения: ошибка всплывёт в бухгалтерии через неделю.
Критично Выполнение произвольного кода на сервере
В экспортной процедуре значение параметра подставляется в исполняемый код. Любой, кто может её вызвать, выполнит на сервере что угодно с правами сеанса.

В полном отчёте ещё пять замечаний уровня «важно», раздел «что сделано хорошо», границы проверки и чек-лист для повторной сверки после исправлений.

Что проверяем

Не пятьдесят придирок к стилю, а то, что ломается

Главное — первый пункт. Соответствие заданию не проверит ни один анализатор кода: он не знает, о чём вы договаривались.

Соответствие заданию

  • каждый пункт задания — есть ли в коде
  • что сделано сверх того, о чём договаривались
  • граничные случаи: пусто, ноль, много строк

Обновляемость

  • правки типовых объектов вместо расширения
  • переопределённые процедуры без пометок
  • что затрёт следующее обновление типовой

Данные и безопасность

  • запись без транзакции там, где нужна атомарность
  • проглоченные ошибки без записи в журнал
  • привилегированный режим, обход прав
  • выполнение внешнего кода

Производительность

  • запросы в цикле, обращения к базе через точку
  • лишние поля, соединения с подзапросами
  • загрузка объекта там, где хватит ссылки

Цены

Цена на сайте. Без «по запросу»

Разовая приёмка — для тех, кто принимает работу подрядчика. Повторная проверка после исправлений — бесплатно, одна.

Первая проверка

Бесплатно

Одна внешняя обработка или отчёт. Отчёт в полном формате — чтобы вы решали по результату, а не по обещаниям.

Обработка или отчёт

4–6 тыс. ₽

Внешняя обработка, отчёт. Срок — один рабочий день.

Расширение или поставка

10–15 тыс. ₽

Расширение конфигурации либо cf-поставка до 20 объектов метаданных.

Крупная доработка

от 25 тыс. ₽

По объёму. Оценку называем до начала работы, после просмотра состава.

Подписка для команд разработки

Когда доработки идут потоком, а тимлида нет или ему некогда.

Старт

15 тыс. ₽ / мес

До 10 проверок в месяц.

Команда

40 тыс. ₽ / мес

До 40 проверок, история проверок, регрессионные тесты по найденным дефектам.

Безлимит

от 80 тыс. ₽ / мес

Встраивание в процесс: Git, EDT, чат команды.

Как это устроено

Три шага, без личного кабинета

Присылаете файл и задание Письмом: доработка (epf, erf, cfe или cf) и текст того, что просили сделать. Задание — хоть перепиской, лишь бы было понятно, о чём договаривались.
Получаете отчёт Один рабочий день на обработку или отчёт. PDF письмом: вердикт, соответствие заданию, замечания с фрагментами кода и объяснением последствий.
Отдаёте отчёт подрядчику Каждое замечание указывает место и способ исправления. Когда исправят — проверим повторно бесплатно и сверим по чек-листу.

Что мы не обещаем

Отчёт — экспертная оценка переданных файлов, а не гарантия отсутствия ошибок. Мы не заменяем тестирование на ваших данных и не берём на себя ответственность за работу подрядчика.

Проверка идёт без доступа к вашей базе. Это осознанное ограничение: вы не передаёте посторонним ни данные, ни доступы. Обратная сторона — поведение на боевых данных мы проверить не можем, и в каждом отчёте есть раздел с перечнем того, что осталось за границей проверки.

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

Проверим первую доработку бесплатно

Пришлите файл и задание — вернём отчёт в том же формате, что выше. Дальше решите сами.

mail@reviewer1c.ru