Найдено замечаний: 1 критических, 2 серьёзных, 3 мелких.
| Уровень | Где | Что не так |
|---|---|---|
| Критично | МодульОбъекта строка 45 |
НачатьТранзакцию() без Попытки — при ошибке транзакция не откатится
Оберните в Попытка/Исключение с ОтменитьТранзакцию() в блоке Исключение.
|
| Серьёзно | Форма строка 102 |
ПолучитьОбъект() вызывается внутри цикла — обращение к БД на каждой итерации
Рассмотрите пакетное получение данных запросом перед циклом.
|
| Серьёзно | Форма строка 139 |
Записать() вызывается внутри цикла — медленно при большом объёме данных
Рассмотрите пакетную запись или вынос транзакции за цикл.
|
| Мелкое | Форма строка 44 |
Запрос без РАЗРЕШЕННЫЕ — может упасть у пользователей с ограниченными правами
Добавьте ВЫБРАТЬ РАЗРЕШЕННЫЕ.
|
| Мелкое | Форма строка 145 |
Опечатка: «заполненны» → «заполнены» |
| Мелкое | МодульОбъекта строка 210 |
Процедура ОбработкаЗаполнения() — 247 строк (рекомендуется до 200)
Разбейте на несколько процедур для читаемости.
|
Зарегистрируйтесь и загрузите свой .cf, .epf или .erf — первые 5 ревью бесплатно.
Попробовать бесплатно