Как понимаю я, в разрезе субконто открываем выборку, а затем открываем выборку по счетам. Сначала идет группировка в разрезе субконто, а внутри каждого субконто уже смотрим на каких счетах оно числится.

(0) ну да

ок, понял, это так как напишешь, так лучше укладывается.

(0) На каких счетах были обороты по субконто за указанный период.

а зачем вставлять пустой цикл, между выборками, типа такого -
Пока Ит.ПолучитьСубконто(...) = 1 Цикл
КонецЦикла;

(4) Может там планировали что-то дописать ?

(0) Пропуск субконто

Ит.ВыбратьСубконто();
Пока Ит.ПолучитьСубконто() = 1 Цикл
// Проверка цехов
  Ит.ВыбратьСубконто(2);
  Если Ит.ПолучитьСубконто(2) = 1 Тогда
       Расшифровка.Установить("Счет",Сч);
  КонецЕсли;

  // Не понятно зачем это делать ???????
  Пока Ит.ПолучитьСубконто(2) = 1 Цикл
  КонецЦикла;

  // Проверка субсчетов
  Ит.ВыбратьСчета();
  Пока Ит.ПолучитьСчет() = 1 Цикл

(7) видимо какой то мусор

Если не прогнать пустой цикл, в выборку счетов попадут только счета имеющие итоги для первого значения субконто2

(9) блин, вот были подозрения на что то. Что именно на это, не знал.

Вместо этого можно так сделать.

  //Пока Ит.ПолучитьСубконто(2) = 1 Цикл
  //КонецЦикла;

  // Проверка субсчетов

  Ит.ВыбратьСчета(1);
  Пока Ит.ПолучитьСчет() = 1 Цикл

(11) в этом случае вы пропускаете группировку по субконто, в моем случае это ОС - сначала, а потом 10, 13 счета.

(12) Я пропускаю вторую группировку, которая ненужна. И пишу

Ит.ВыбратьСчета(1);

Вот эта единичка и заменит пустой цикл.

Параметры:
<ФлагВсе> - число: 0 - отбирать те счета, которые имели итоги на этом уровне обхода итогов запроса; 1 - включить в выборку все счета, которые имели итоги в данном запросе; -1, -2 : включить в выборку счета, которые имели итоги в группировке n-го вышестоящего уровня. По умолчанию - 0.

(12) В (7) бред какой-то

Хотя че-то я тоже не совсем вкурил что там надо вывести.

И тогда уж так заменить.

 Если Ит.ВыбратьСубконто(2) =1 Тогда
//   Если Ит.ПолучитьСубконто(2) = 1 Тогда
       Расшифровка.Установить("Счет",Сч);
  КонецЕсли;

Ну по идее вот так должно заменить тот код.

Ит.ВыбратьСубконто();
Пока Ит.ПолучитьСубконто() = 1 Цикл
// Проверка цехов

  Если Ит.ВыбратьСубконто(2) = 1 Тогда
//   Если Ит.ПолучитьСубконто(2) = 1 Тогда
       Расшифровка.Установить("Счет",Сч);
  КонецЕсли;

  // Не понятно зачем это делать ???????

 // Пока Ит.ПолучитьСубконто(2) = 1 Цикл
//   КонецЦикла;

  // Проверка субсчетов

  Ит.ВыбратьСчета(-1); // или -2
  Пока Ит.ПолучитьСчет() = 1 Цикл

Перебираем все ОС и смотрим весит ли ОС более чем на одном 10 или 13 счетах.

(18) А второе субконто, это что ?

(19) подразделения

Если мен не изменяет память, то вместо (17) можно и так поменять местами.

Ит.ВыбратьСубконто();
Пока Ит.ПолучитьСубконто() = 1 Цикл
     // Проверка субсчетов
  Ит.ВыбратьСчета();

  Пока Ит.ПолучитьСчет() = 1 Цикл

  КонецЦикла ;

// Проверка цехов
  Если Ит.ВыбратьСубконто(2) = 1 Тогда
//   Если Ит.ПолучитьСубконто(2) = 1 Тогда

       Расшифровка.Установить("Счет",Сч);
  КонецЕсли;

  // Не понятно зачем это делать ???????


 // Пока Ит.ПолучитьСубконто(2) = 1 Цикл

//   КонецЦикла;

То есть счета можно выбирать на любом уровне обхода субконто.

(20) И по подразделениям надо узнать только, есть там хоть что-то или нет для конкретного ОС ?

получается что выбрали сначала по 1-ому субконто, потом по 2-ому и только потом по счету

(9) как правильно подмечено.

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

(25) совершенно верно, надо что бы в выборку попали все подразделение(субконто2).

(26) ну так и начинай выборку со второго субконто. Или я не догоняю что тебе нужно