Добрый день всем.
Подскажите, пожалуйста.

Есть 2 команды на управляемой форме: "Установка цен" и "Список цен". Перейти по ним нельзя, пока элемент справочника не записан.

Можно ли как-то оптимизировать или переписать проще приведенный ниже код?


&НаКлиенте
Процедура УстановкаЦен(Команда)
    Номенклатура = ПроверкаНаНовыйЭлемент();                                      

    ФормаЗаписи = ПолучитьФорму("РегистрСведений.ЦеныПродажи.Форма.ФормаЗаписи");
    ФормаЗаписи.Запись.Период = ТекущаяДата();
    ФормаЗаписи.Запись.Номенклатура = Номенклатура;
    ФормаЗаписи.Открыть();
КонецПроцедуры

&НаКлиенте
Процедура СписокЦен(Команда)
    Номенклатура = ПроверкаНаНовыйЭлемент();
    ФормаСписка = ПолучитьФорму("РегистрСведений.ЦеныПродажи.Форма.ФормаСписка");
    ФормаСписка.ФильтрПоНоменклатуре = Номенклатура;
    ФормаСписка.Открыть();      
КонецПроцедуры

&НаСервере
Функция ПроверкаНаНовыйЭлемент()
    Если Объект.Ссылка.Пустая() Тогда
        Номенклатура = Справочники.Номенклатура.СоздатьЭлемент();
        Номенклатура.Наименование = Объект.Наименование;
        Номенклатура.Записать();
        ЭтаФорма.Модифицированность = Ложь;
        Возврат Номенклатура.Ссылка;
    Иначе
        Возврат Объект.Ссылка;
    КонецЕсли;
КонецФункции

У тебя при вызове каких-то процедур записывается элемент справочника?

Какая-то лютая дичь

(0) Надеюсь вы это делаете не в форме создания номенклатуры?)

Проверка на то, новый элемент или нет, в случае, если новый, создает какой-то новый элемент?!

(3) Не будь так скоропалителен со своими выводами, ты же не знаешь бизнес-процессов компании, может им так нужно :))

(0) У вас ЦеныПродажи не документ фиксирует?

(5) я потому и уточняю)

(7) Это была попытка доброго троллинга :))
На самом деле написанный в (0) код не имеет права на существование ни при каких обстоятельствах

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

Вернуть ссылку на другую запись справочника если Объект новый? О_О Да, согласен с (2)

(0) хочешь текущий элемент записать? или новый создать

(8) Тут как раз бы подошел метод ПоказатьВопрос. Мол для работы с ценами нужно записать номенклатуру, продолжить?. И все дела.

То есть записать религия не позволяет?

(13) Вот же есть Номенклатура.Записать();  =)

Где находится функция ПроверкаНаНовыйЭлемент() и что такое Объект?

(15) Автор от наших вопросов впал в ступор :)

Функция ПроверкаНаНовыйЭлемент()
    Если Объект.Ссылка.Пустая() Тогда
        Объект.Записать();
        ЭтаФорма.Модифицированность = Ложь;
    КонецЕсли;
    Возврат Объект.Ссылка;
КонецФункции

Почитал Ваши комментариями, огорчился своей тупости, сократил код до следующего (норм или всё равно говно?):

&НаКлиенте
Процедура УстановкаЦен(Команда)
    Если Не Объект.Ссылка.Пустая() Тогда            
        ПараметрыФормы = Новый Структура("Номенклатура", Объект.Ссылка);
        ОткрытьФорму("РегистрСведений.ЦеныПродажи.Форма.ФормаЗаписи", ПараметрыФормы);
    КонецЕсли;
КонецПроцедуры

&НаКлиенте
Процедура СписокЦен(Команда)
    Если Не Объект.Ссылка.Пустая() Тогда
        ФормаСписка = ПолучитьФорму("РегистрСведений.ЦеныПродажи.Форма.ФормаСписка");
        ФормаСписка.ФильтрПоНоменклатуре = Объект.Ссылка;
        ФормаСписка.Открыть();      
    КонецЕсли;
КонецПроцедуры

(18) Перечитай еще раз и объясни нахрена ты пустой ссылкой кидаешься туда обратно?

(18) Вместо Не Объект.Ссылка.Пустая() напиши ЗначениеЗаполнено(Объект.Ссылка)

(20) (18) ИТС говорит использовать "Параметры" https://its.1c.ru/db/metod8dev/content/3545/hdoc
Если Не Параметры.Ключ.Пустая() Тогда - как вариант. Имхо вкусовщина конечно)

(21) Хм, ну может быть, да

(21) Угу. Параметры.Ключ.Пустая() вместо старого доброго ЭтоНовый().

(21) этим я пользовался, тоже работает.
В чем разница между Объект.Ссылка.Пустая(), Параметры.Ключ.Пустая() и ЗначениеЗаполнено(Объект.Ссылка) в скорости выполнения?

(23) Для этого, вроде как, нужно получить объект. Но для чего нам это на клиенте?

(25) "Старый и добрый" - это из эры Толстых Клиентов до эпохи Управляемых Форм :)

(24) "в скорости выполнения" - ага. Объект.Ссылка - вроде как заставит весь объект прочитать, могу ошибаться.

(27) Мы что через Объект.Ссылка вызываем метод проверки ссылки Пустая(), что через Параметры.Ключ тот же метод для проверки ссылки Пустая(). Так что тут я не вижу разницы.

Решил проверить.
Данные по отладке.

1. ЗначениеЗаполнено
До: Тек вызовы 2, Накопленные 30
После : Тек вызовы 6, Накопленные 36

2. Объект.Ссылка.Пустая()
До: Тек вызовы 2, Накопленные 29
После : Тек вызовы 6, Накопленные 35

3. Параметры.Ключ.Пустая()
До: Тек вызовы 2, Накопленные 31
После : Тек вызовы 7, Накопленные 38

Менее требовательные ЗначениеЗаполнено и Объект.Ссылка.Пустая().

Тесты проводились на команде "УстановкаЦен"

(28) Хех :) про кэширование забыли.

(30) Вообще забыл про него)

(27) Ошибаешься