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

&НаКлиенте
Процедура ТабличнаяЧасть1НоменклатураПриИзменении(Элемент)
   ТекСтрока = Элементы.ТабличнаяЧасть1.ТекущиеДанные;
   ТекСтрока.Цена = ПолучитьЦенуНаСервере(ТекСтрока.Номенклатура);
   //ТекСтрока.Сумма = ТекСтрока.Цена * ТекСтрока.Количество;
КонецПроцедуры

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

{Документ.Инвентаризация.Форма.ФормаДокумента.Форма(42)}: Значение не является значением объектного типа (Цена)
    Возврат Данные.Цена;

А с чего Вы решили, что в Данные что-то есть?

Регистр пуст или по твоей ном-ре нет данных

(3)регистр проверил, с записями

(4) типы проверяй. В любом случае получить последнее как я понимаю возвращает пустую структуру и это нужно проверять

(2)а как правильно?


&НаКлиенте
Процедура ТабличнаяЧасть1НоменклатураПриИзменении(Элемент)
ТекДанные = ЭтаФорма. Элементы. ТабличнаяЧасть1. ТекущиеДанные;
ТекДанные.Цена = ПолучитьЦенуНоменклатуры (ТекДанные. Номенклатура);
КонецПроцедуры

&НаСервере
Функция ПолучитьЦенуНоменклатуры (НоменклатураСсылка)
Если НоменклатураСсылка <> Справочники. Номенклатура. ПустаяСсылка () Тогда
Запрос = Новый Запрос;
Запрос. Текст =
"ВЫБРАТЬ
|ЦеныНоменклатурыСрезПоследних. Цена
|ИЗ
|РегистрСведений. ЦеныНоменклатуры. СрезПоследних (&Дата, Номенклатура = &Номенклатура) КАК ЦеныНоменклатурыСрезПоследних";

Запрос. УстановитьПараметр ("Дата", ?(ЗначениеЗаполнено (Объект. Дата), Объект. Дата, ТекущаяДата ()));
Запрос. УстановитьПараметр ("Номенклатура", НоменклатураСсылка);

РезультатЗапроса = Запрос. Выполнить ();

ВыборкаДетальныеЗаписи = РезультатЗапроса. Выбрать ();

Если ВыборкаДетальныеЗаписи. Количество () > 0 Тогда
ВыборкаДетальныеЗаписи. Следующий ();
Возврат ВыборкаДетальныеЗаписи. Цена
КонецЕсли;
КонецЕсли;

Возврат 0;
КонецФункции

так тоже не работает((

такая ошибка:
{Документ.Инвентаризация.Форма.ФормаДокумента.Форма(64)}: Ошибка при вызове метода контекста (Выполнить)
РезультатЗапроса = Запрос. Выполнить ();
по причине:
{(2, 32)}: Ожидается выражение "("
ЦеныНоменклатурыСрезПоследних. <<?>>Цена

(7) правильно запросом сделать

(6) ЦеныНоменклатурыСрезПоследних. Цена а нахер тут пробел после точки?

+ конструктором запроса а не вручную

(7) а пробел нафига?

Вообще нафига ты в коде и запросе после точек пробелы рисуешь?

(12) + И перед скобками...вообще мрак.

А так тоже не работает((

&НаКлиенте
Процедура ТабличнаяЧасть1НоменклатураПриИзменении(Элемент)
ТекДанные = ЭтаФорма. Элементы. ТабличнаяЧасть1. ТекущиеДанные;
ТекДанные.Цена = ПолучитьЦенуНоменклатуры (ТекДанные. Номенклатура);
КонецПроцедуры


&НаСервере
Функция ПолучитьЦенуНоменклатуры (НоменклатураСсылка)
    
    Запрос = Новый Запрос;
    Запрос.Текст =
        "ВЫБРАТЬ
        |    ЦеныНоменклатуры.Номенклатура КАК Номенклатура,
        |    ЦеныНоменклатуры.ТипЦен КАК ТипЦен,
        |    ЦеныНоменклатуры.Цена КАК Цена
        |ИЗ
        |    РегистрСведений.ЦеныНоменклатуры КАК ЦеныНоменклатуры";
    
    РезультатЗапроса = Запрос.Выполнить();
    ВыборкаДетальныеЗаписи = РезультатЗапроса.Выбрать();
    
    Пока ВыборкаДетальныеЗаписи.Следующий() Цикл
        // Вставить обработку выборки ВыборкаДетальныеЗаписи
    КонецЦикла;

КонецФункции

Отбор в запросе где?

(14) Функция, ничего не возвращающая? :)

Точно!) Спасибо!

&НаСервере
Функция ПолучитьЦенуНоменклатуры (НоменклатураСсылка)
    
    Запрос = Новый Запрос;
    Запрос.Текст =
        "ВЫБРАТЬ
        |    ЦеныНоменклатуры.Номенклатура КАК Номенклатура,
        |    ЦеныНоменклатуры.ТипЦен КАК ТипЦен,
        |    ЦеныНоменклатуры.Цена КАК Цена
        |ИЗ
        |    РегистрСведений.ЦеныНоменклатуры КАК ЦеныНоменклатуры";
    
    РезультатЗапроса = Запрос.Выполнить();
    ВыборкаДетальныеЗаписи = РезультатЗапроса.Выбрать();
    
    Пока ВыборкаДетальныеЗаписи.Следующий() Цикл
        // Вставить обработку выборки ВыборкаДетальныеЗаписи
    КонецЦикла;
    Возврат ВыборкаДетальныеЗаписи.Цена
КонецФункции

ОТБОР.ОТБОР.ОТБОР.

(17) весь справочник Номенклатура возвращаешь? А если там 1000000 номенклатур?

(17) К отбору еще и проверять нужно выборку данных, иначе в ВыборкаДетальныеЗаписи. Может она пустая.

(17) А тебя вот это не наводит на мысли?
// Вставить обработку выборки ВыборкаДетальныеЗаписи

(21) заметил :-))

&НаСервере
Функция ПолучитьЦенуНоменклатуры (НоменклатураСсылка)
    
    Запрос = Новый Запрос;
    Запрос.Текст =
        "ВЫБРАТЬ
        |    ЦеныНоменклатуры.Номенклатура КАК Номенклатура,
        |    ЦеныНоменклатуры.ТипЦен КАК ТипЦен,
        |    ЦеныНоменклатуры.Цена КАК Цена
        |ИЗ
        |    РегистрСведений.ЦеныНоменклатуры КАК ЦеныНоменклатуры";
    
    РезультатЗапроса = Запрос.Выполнить();
    ВыборкаДетальныеЗаписи = РезультатЗапроса.Выбрать();
    
    Если ВыборкаДетальныеЗаписи.Количество() > 0 Тогда

    Пока ВыборкаДетальныеЗаписи.Следующий() Цикл
        // Вставить обработку выборки ВыборкаДетальныеЗаписи
        Возврат ВыборкаДетальныеЗаписи.Цена
      КонецЦикла;
    КонецЕсли;    
КонецФункции

(22) А если ВыборкаДетальныеЗаписи.Количество() = 0, то что какое значение вернет функция?

(23)пустоту наверное

(24) И это некорректное поведение.
К тому же.
Зачем цикл, если у тебя всегда будет возвращаться только первый элемент выборки.
Делай просто

Если ВыборкаДетальныеЗаписи.Следующий() Тогда
    Возврат ВыборкаДетальныеЗаписи.Цена;
Иначе
    Возврат 0;
КонецЕсли;

Кошернее проверять по Выборка.Количество()

Ну или РезультатЗапроса.Пустой()

(26) Чем кошернее? Всё равно придётся делать Выборка.Следующий(), если не пустой или кол-во больше нуля.

все, сделал максимально кошерно, а как сделать чтоб цены то подставлялись?))

(28) в зависимости от количества записей что вернет запрос наверно РезультатЗапроса.Пустой(), я правильно думаю?

Хотя в данном случает, пофик, тут одна или ноль

(28) А чтобы не лезть в выборку, которая и так пустая. Для красоты.

Помогите сделать отбор по типу цен (нужна закупочная цена). ТипЦен справочник, а не перечисление

&НаСервере
Функция ПолучитьЦенуНоменклатуры (НоменклатураСсылка)
    
    //{{КОНСТРУКТОР_ЗАПРОСА_С_ОБРАБОТКОЙ_РЕЗУЛЬТАТА
    // Данный фрагмент построен конструктором.
    // При повторном использовании конструктора, внесенные вручную изменения будут утеряны!!!
    
    Запрос = Новый Запрос;
    Запрос.Текст =
        "ВЫБРАТЬ
        |    ЦеныНоменклатурыСрезПоследних.Номенклатура КАК Номенклатура,
        |    ЦеныНоменклатурыСрезПоследних.ТипЦен КАК ТипЦен,
        |    ЦеныНоменклатурыСрезПоследних.Цена КАК Цена
        |ИЗ
        |    РегистрСведений.ЦеныНоменклатуры.СрезПоследних КАК ЦеныНоменклатурыСрезПоследних
        |ГДЕ
        |    ЦеныНоменклатурыСрезПоследних.Номенклатура = &Номенклатура";
        //|    И ЦеныНоменклатурыСрезПоследних.ТипЦен.Наименование = &Закупочная";

    
    Запрос.УстановитьПараметр("Номенклатура", НоменклатураСсылка);
    //Запрос.УстановитьПараметр("ТипЦен", ТипЦен.Закупочная);    
    
    РезультатЗапроса = Запрос.Выполнить();
    ВыборкаДетальныеЗаписи = РезультатЗапроса.Выбрать();
    
    Если ВыборкаДетальныеЗаписи.Следующий() Тогда
         Возврат ВыборкаДетальныеЗаписи.Цена;
    Иначе
         Возврат 0;
    КонецЕсли;
      
КонецФункции

никто не знает?

Вместо
//|    И ЦеныНоменклатурыСрезПоследних.ТипЦен.Наименование = &Закупочная";
Сделать
|    И ЦеныНоменклатурыСрезПоследних.ТипЦен = &ТипЦен";

Уже пробовал?

А вот тут перед
//Запрос.УстановитьПараметр("ТипЦен", ТипЦен.Закупочная);
Уже ищи по наименованию например так
ТипЦенЗакупочная = Справочники.ТипЦен.НайтиПоНаименованию("Закупочная");
И
Запрос.УстановитьПараметр("ТипЦен", ТипЦенЗакупочная);

Да,  пробовал, выходит такое:
{Документ.Инвентаризация.Форма.ФормаДокумента.Форма(61)}: Ошибка при вызове метода контекста (Выполнить)
    РезультатЗапроса = Запрос.Выполнить();
по причине:
{(9, 43)}: Не задано значение параметра "ТипЦен"
И ЦеныНоменклатурыСрезПоследних.ТипЦен = <<?>>&ТипЦен

И так пробовал:

//Запрос.УстановитьПараметр("ТипЦен", ТипЦен.Закупочная);    
//Запрос.УстановитьПараметр(Справочники.ТипЦен.найтипонаименованию("закупочная"));
//Запрос.УстановитьПараметр("ТипЦен", ТипЦен.Наименование = ""Закупочная"");

(39) Ты издеваешься?

Запрос.УстановитьПараметр("ТипЦен", Справочники.ТипЦен.найтипонаименованию("закупочная"));

Но сначала лучше получить в переменную и ТипЦенЗакупочная и проверить что ссылка не пустая, что нашло то что надо.
Если не нашло то пишем ошибку "Не нашли ТипЦен по наименованию Закупочная!" и прерываем

(40) конечно нет,
я просто не понимаю, почему не работает

(41) Потому что книжки читать надо. И не просто читать а понимать что там написано и зачем.

Ты прав! работает.

Если бы стать программистом было легко и просто после небольшого обучения в пару месяцев - давно уже все таксисты в 1Сники бы подались.

ОФФ: А мне понравился заголовок сабжа. Заманивает. Жаль, что внутри совсем другое.