Pull to refresh

Comments 6

1.

Попытка

ЗаписьПациента.Записать();

Исключение

КонецПопытки;

Вы не пишите в журнал регистрации ничего при ошибке записи? в ветке Исключение. Почему?

  1. Вы не слышали про КоннекторHttp? Опенсорсный проект, просто добавляете общий модуль и работа с http запросами становится лаконичнее и проще

  2. Справочники.СтатусДокумента.НайтиПоНаименованию("Бронь не подтвержденная");

    НайтиПоНаименованию? Это ваш же справочник, может предопределенные элементы создать

Спасибо за комментарий.

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

  2. Используем функционал 1С без дополнительных модулей. В данной задаче наличие внешней компоненты для передачи-приема пары строк запроса не является необходимостью.

  3. Справочник не мой, а находится в не редактируемой в данном случае части конфигурации. Предопределенные значения в нем имеются, но не все необходимые. На вопрос: "Есть ли требование создавать новые предопределенные значения?" заказчик задачи просил не менять ничего в конфигурации, так как ведется "сложный" процесс ее обновления на несколько релизов и у заказчика нет желания возвращаться к доработке приведенного кода в дальнейшем.

Ну, хотя бы "НайтиПоКоду()", и то несколько надёжнее будет.

Вопрос привычки, надежность привязки к коду может быть сомнительна. Разработчик меняет тип кода с Числа на Строку и приплыли.

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

  1. Только админ

  2. Будет наказан

Я помечу сейчас в статье рекомендации этой ветки, спасибо за комментарий

Sign up to leave a comment.