Доброй всем ночи, мне необходимо отобрать все документы перемещение ТМЦ у которых склад получатель совпадает со складом который я выбираю в обработке. Делаю так

Если (Док.Вид() <>"ПеремещениеТМЦ")   И (Склад <> СкладПеремещение) Тогда

По ни чего не выходит, упорно выбираются все документы с выбранным складом в обработке, хотя чего проще поставить условие, но не как не пойму как это сделать
Делала так

Если (Док.Вид() <>"ПеремещениеТМЦ")   И (Док.Склад.Получить() <> "СкладПолучатель")

Тоже не получается, может кто подскажет?

опечатался( */делал

это что за бред? Склад <> СкладПеремещение)

Хорошо, а как правильно?

Зачем на вид проверять, почему нельзя сразу выбирать определенного вида документы?

СкладПеремещение это реквизит из документа перемещения

(5) Ну так и проверяй
ВыбСклад и Док.Склад

* ВыбСклад и Док.СкладПеремещение

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

прости, не сосем понял (7)

(8) Т.е.
Док=СоздатьОбъект("Документ.ПеремещениеТМЦ")
уже не кошерно и надо все доки шерстит?

СписДок.УдалитьВсе();
   Док=СоздатьОбъект("Документ");    

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

                                                         
       Если (Док.Вид() <>"ПеремещениеТМЦ")   И (Склад <> СкладПеремещение) Тогда
           Продолжить;
       КонецЕсли;

(9) Я тоже не совсем понял твой замысел. такое ощущения что выдранно из контекста, т.е. на входе какой то запрос или выборка

(9) Думаешь, тебя можно понять по ОДНОЙ строчке кода?

Док.ВыбратьПоЗначению(НачДат,КонДат,"Склад",ВыбКонтрагент); - это очепятка? или у тебя ВыбКонтрагент = справочник.склады?

неопечатка, это идентификатор, так обозвали =(

ВыбКонтрагент = справочник.склады?

угу

Док=СоздатьОбъект("Документ.ПеремещениеТМЦ");    
Док.УстановитьФильтр(1,0);
Док.ВыбратьДокументы();
Пока Док.ПолучитьДокумент()=1 Цикл
  Если (ВыбКонтрагент <> Док.СкладПеремещение) Тогда
     Продолжить;
  КонецЕсли;

(11) Ага, уже лучше.

Во-первых, см. (10)

Во-вторых, если интересно, в чем твоя ошибка, то правильно вот так:

Если (Док.Вид() <>"ПеремещениеТМЦ") ИЛИ (Склад <> СкладПеремещение) Тогда
           Продолжить;
КонецЕсли;

Хотя и так коряво, поэтому (10) и

Если Склад <> СкладПеремещение Тогда
           Продолжить;
КонецЕсли;

Не склад а выбКонтрагент (у нас же так реквизит называется)
и не СкладПеремещение, а Док.СкладПеремещение  - это же реквизит документы

Скажите а что нужно делать чтобы так программировать хорошо?

(31) Сначала думать, потом - кодить.

Но теперь опять ошибка

сли (ВыбКонтрагент <> Док.СкладПеремещение) Тогда

Поле агрегатного объекта не обнаружено (СкладПеремещение)

(23) Правильно, ты не поменял
Док=СоздатьОбъект("Документ");
на
Док=СоздатьОбъект("Документ.ПеремещениеТМЦ");

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

ну просто в обработке отбирается еще отрезок времени

Док.ВыбратьПоЗначению(НачДат,КонДат,,);  удали
   Док.ВыбратьДокументы(НачДат,КонДат); - вот так надо

Док.УстановитьФильтр(1,0);  - зря удалил. это фильтр отбирать проведенные и не помеченные на удаления. сейчас к тебе всё подряд будет валиться, ну или добавь

Если Док.Проведен()=0 Тогда
  Продолжить;
КонецЕсли;

(28) правильнее конечно фильтр наложить, но если некритично - я предпочитаю юзать проверку ввиду ея большей читабельности