Один code-review работника

Подрядил работника под доработку загрузки из файла Excel. У клиента уже есть загрузка из двух вариантов файлов.

Вот мои замечания.

Добавил информирование пользователя подробнее, каких колонок нет:

Есть функция Моечисло для этих целей:

Вместо этого соответствия надо использовать таблицу значений и свертку ее:

Это сообщение выскочит в 1 из 1000 случаев, нет смысла тратить на эту проверку:

Вот это тоже чересчур усложнено, в жизни не требуется. Просто получили контрагента очередного, проверили, есть или нет. Если нет, то сообщили что нет. итоги и статистика лишнее:

Смысл этой проверки? ведь мы же гарантированно все поля передаем:

Надо было делать по аналогии, а не писать самому. поправил:

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

Это конечно красиво, но им всегда выводилось в окно сообщений и нет смысла делать по иному, непривычно:

Табличная часть расшифровки платежа так не заполнится:

Есть функция по записи документа:

Есть функция по поиску договора для документа. В принципе можно было посмотреть, как договор подставляется при выборе контрагента (отладить), если бы функции не было:

Главная рекомендация — если указано делать по аналогии, то делать по аналогии. остальное мелкие замечания.