ЗначениеВыбора.Свойство("ВыборИзСправочника", ВыборИзСправочника);
   ВыборИзСправочника = ВыборИзСправочника = Истина;

> ВыборИзСправочника = ВыборИзСправочника = Истина;
явный г-код

(1) Типовой из ут 10.3, процедура ОбработкаПодбора()
=)

в РТиУ

(2) вот не аргумент, хотя УТ10 должна быть отлажена вся кругом ;)
(1) это я о свойстве не читал
вторая строка делает нам явную булеву ложь, если в структуре нет значения ВыборИзСправочника. нету - значит, ложь, есть - значит, истина или ложь;

(4)хм... прикольно...

(1) Я тоже соглашусь, что это авнокод.

(6) это приведение к булеву.
аналог: ВыборИзСправочника=?(ВыборИзСправочника=Неопределено, Ложь, ВыборИзСправочника);
или: ВыборИзСправочника=НЕ(ВыборИзСправочника=Неопределено) И ВыборИзСправочника;

(7) Ну ведь тогда и не авнокод совсем? Просто новичкам типа меня не сразу понятен смысл, а так он короче, и даже более читаем, чем варианты из (7).

(7) Я знаю, что это приведение типа. Но я бы сделал так:
ЗначениеВыбора.Свойство("ВыборИзСправочника", ВыборИзСправочника);
ВыборИзСправочника = ?(ТипЗнч(ВыборИзСправочника) = Тип("Булево"), ВыборИзСправочника, Ложь);

Это как-то более читабельно...

так читабельней

ЗначениеВыбора.Свойство("ВыборИзСправочника", ВыборИзСправочника);

ВыборИзСправочника = НЕ ВыборИзСправочника = Неопределено;

(10) представть, что в ВыборИзСправочника у тебя ложь

(11)тогда так :-)


ВыборИзСправочника = ?(ВыборИзСправочника=Ложь,ВыборИзСправочника,НЕ ВыборИзСправочника = Неопределено );

ну и для полноты картины ещё один вариант ;)
ВыборИзСправочника = ?(ВыборИзСправочника, Истина, Ложь);

(13)Сдуется если Неопределено :-)

(14) ага, думал, оно мне неявно преобразует

Нормальный код в сабже

Мужики, чтоб не плодить новых тем:
Делаю свой документ, организую подбор в таб.часть. Дернул функционал с РТиУ, подбор открывается, все в поряде, но в док-те не срабатывает ОбработкаВыбора()
Вызываю подбор так: РаботаСДиалогами.ОткрытьПодборНоменклатуры(ЭтаФорма, СтруктураПараметровПодбора, Метаданные());
В подборе после выбора номенклатуры срабатывает: ОповеститьОВыборе(), но в документе в процедуру ОбработкаВыбора() не заходит. Что я упустил?

(17)Может ты ОбработкаВыбора() ручакми написал и забыл прописать в свойствах формы?

(18) точно! Спасибо!

(19) методический совет. не пиши руками, выбирай из имеющихся. и писанины меньше, и привязка сама встаёт

Нормальный код. Я правда для читабельности в скобки включаю:
ВыборИзСправочника = (ВыборИзСправочника = Истина);

(20) когда что-то копипастишь, легче скопировать, а потом привязать... главное не забыть :)

в (9),(10),(12) и (13) настоящий овнокод. а в (0) все понятно и кратко.

если бы все непонятное новичкам разжевывали и заменяли 1 на 5 строк кода - пришлось бы сервера в 2 раза дороже покупать

Продолжу с непонятными примерами из типовых. Не вижу смысла вот в этом:


Если НЕ ВходитВХолдинг Тогда
       ТекущаяМодифицированность = Модифицированность;
       ГоловнойКонтрагент        = Ссылка;
       Модифицированность        = ТекущаяМодифицированность;
   КонецЕсли;


А если быть точнее:


ТекущаяМодифицированность = Модифицированность;
Модифицированность        = ТекущаяМодифицированность;

Грубо говоря А=Б, Б=А. Для чего?

(24) это чтоб реквизит поменять, а модифицированность оставить прежней

(25) Точно! =) я и не додумался )

(26) >А=Б, Б=А
навеяло. поменяй местами значения двух переменных без использования третьей

(27) *числовых переменных

(27) Классика! =)

(27)
http://www.fermi.mn.it/linux/quarta/x86/xchg.htm

(1) И чем это гкод?
Истина остается истиной, все остальное - Ложь.
Компактней и не напишешь.

(30) а что, было такое? а я ксорил, как дурак

Нормально читаемый компактный код. Конечно для более читабельности можно было в скобки выражение поместить, но по крайней мере лучше, чем (12)

А в (12) получается:
Если СтрДлина(Строка(ВыборИзСправочника))>6 тогда
ВыборИзСправочника = Ложь;
КонецЕсли;

(32) Пока x86 не умерло такое даже есть :)

(32) более того, xchg ax,ax используется для так называемого выравнивания (alignment).

(0) значит, что иногда в свойстве ВыборИзСправочника может прийти не булево

Код в сабже неплох, а вот г*код находится где-то выше его, если пришлось так вы*ться.

(33) лучше так

Если НЕ ЗначениеВыбора.Свойство("ВыборИзСправочника", ВыборИзСправочника) Тогда
  ВыборИзСправочника = Ложь;
КонецЕсли;

просто авторы УТ не догадываются, что метод Свойство() возвращает значение.

может просто (38) не догадывается, что метод Свойство() возвращает наличие соответствующего ключа? :)

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

(0) ИМХО, код сам по себе нормальный, но подход не правильный. Лучше сделать универсальную функцию, возвращающую либо значение из структуры, либо значение по-умолчанию, если этого значения в структуре нет.

(39) нет это 39 не догадывается, что метод не может возвращать наличие и безналичие, а метод возвражает значение типа булево.

(42) Как бы (39) намекает, что в структуре будет ключ, но значение может быть вовсе не булево.