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

а где итерация выборки Следующий()?

О ужасс.

три раза Выполить() и не разу Следующий()

+(1) И зачем 3 раза Запрос.Выполнить() ?

(3) сервер должен быть нагружен!

(0) А если документов больше чем один?

(3) чем больше Выполнить тем точнее результат :)

жесть. такое ощущение, что добрая половина сегодняшних веток после жёстких корпоративов написана.

(4) ну он не сильно нагрузится , так как кеш сработает

(8) чорт, все пропало!

Нормальные запросы возвращают NULL

(0) возьми свой запрос, вставь в к конструктор с обходом результата - получи код, вырежи его и возьми к себе.

(10) Все молчу молчу.