Добрый день уважаемые! Есть справочники (Спр) Банки и Банковские счета (БС). В справочник Банки занесли список банков по городам (БИК банков совпадают, разнятся только названия и города). Теперь решили оставить по одному банку из всех списков, чтобы плагиата не было. А БС по всем то уже занесли, есть ссылки. Помечаем на удаление банки, оставляем по одному из каждого вида.
Написал внешнюю обработку. На форме обработки первый запрос получает из Спр Банки все записи не помеченные на удаление, которых оставили. Второй запрос получает все те записи из Спр БС ,у которых реквизит Банк помечен на удаление. Цель: заменить в спр БС реквизиты банк на те банки из СПр Банки не помеченные на удаление. После того как ссылки перебьются на новые, стандартной обработкой удалить лишние записи.
Пока удается получить из запросов записи, а найти по БИК и записать Объекту банковские счета новое значение не получается. Конечный цикл у меня неправильно перебирает и методом получить объект не выходит. Надеюсь на какие ни будь направления где что искать и что не правильно по сути

&НаКлиенте
Процедура ИзменитьСсылки(Команда)
    
ИзменитьСсылкиНаСервере();
    
КонецПроцедуры

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

                  
           ТаблицаЗапросаБанки = Запрос.Выполнить().Выбрать();
          
           Пока ТаблицаЗапросаБанки.Следующий() Цикл
              
           КонецЦикла;


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

          ТаблицаЗапросаБС = Запрос.Выполнить().Выбрать();


          Пока ТаблицаЗапросаБС.Следующий() Цикл
                          
          КонецЦикла;
          
          Если  ТаблицаЗапросаБС.БИКБанка = ТаблицаЗапросаБанки.БИК Тогда;
                  
                    БанковкийСчет = ТаблицаЗапросаБС.СсылкаНаСпр.ПолучитьОбъект();
                    БанковкийСчет.Банк = ТаблицаЗапросаБанки.Ссылка;
                    БанковкийСчет.Записать();      
          КонецЕсли;

КонецПроцедуры;

Может быть так:
Для Каждого  Строка Из ТаблицаЗапросаБанки Цикл

Если  ТаблицаЗапросаБС.БИКБанка = ТаблицаЗапросаБанки.БИК Тогда
                  
                    БанковкийСчет = ТаблицаЗапросаБС.СсылкаНаСпр.ПолучитьОбъект();
                    БанковкийСчет.Банк = Строка;
                    БанковкийСчет.Записать();      
          КонецЕсли;

КонецЦикла

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

(4) Я знаю то, что с логикой и вообще примитивными навыками программирования у тебя не то что плохо... И вообще нет.

*их

(0) Для начала: что за конфигурация? В типовых имеется встроенный механизм замены дублей, который позволяет сделать, то что вам надо без программирования.

(1) Код не соответствует описанной проблеме в (0) :)
Вам банки мешают или вы решили нагадить компании и удалить из счетов все банки ? :))))

(5) как вы это определили?

(9) по предложенному нашему вниманию коду.

(9) Код в (1) делает совсем другое, нежели в (0) :)

(7) Конфигурация не типовая. Дублей нет. Поэтому универсальной обработкой не стал

(11) нет почему же. Может я не так выразился когда пытался объяснить суть задачи

(10) что же вас там смутило? ЗАпросом получаем данные и после справочнику прописываем новое значение

(13) Вам нужно взять один банк по бику и заменить на него все что не соответствует этому банку, т.е. элементу справочника, но соответствует бику :)

Добавь проверку на пометку удаления
Добавь поиск нужного элемента банка в ТЗ по Бику :)

(14) Я ж говорю - логики в коде никакой.

(15) нет вы не поняли. Не нужно один банк записывать всем. Есть n количество банков. Казмкоммерцбанк, ИнвестБанк, Сбербанк и т.д. и все они занесли еще по городам. В справочник Банки они ввели один и тот же банк к примеру Сбербанк в Алмате, В Астане еще и по остальным городам. Теперь они хотят убрать запси по городам. Понятие города у Банка вообще убрать, т.к. он и не нужен. Оставить например просто Сбербанки все. И так по всем остальным банкам.

(15) и конечно же просто лишние стереть не получится или допустим перебивать ссылки на оставшиеся банки в справочниках в ручную тоже не вариант. Было решено пометить лишние записи , а в справочнике банковских счетов обработка изменит ссылки на актуальные

(16) какой содержательный ответ. Логика есть. Написал выше. И как это связано с мышлением программирования не понял))

(19) ну смотри. У тебя 2 цикла. Совершенно не связанных друг с другом.
И что ты от этого кода хочешь?

(20) Вы об этом ? (2)

(21) И (2)  тоже никак не связано с с.
Ты сначала перебираешь все банки. А уже потом, после перебора - перебираешь счета.
Намёк понял?

(15) проверка на пометку уже есть в запросах. Я уже получаю результат отсеивая в первом запросе не помеченные, во втором наоборот

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

(22) не понял

(17) >>> Понятие города у Банка вообще убрать, т.к. он и не нужен.

Это вам пользователи напели?
Лукавят ;)

(20) Он хочет "Напишите мне, я буду признателен" :)

(25) Печально. Я ж говорил в (5)...

(23) Ты не понял. Проверка на пометку по тем банкам, которые еще присутствуют в счетах и которые еще не заменены на нужные :)

+ В общем делай просто в цикле, забудь про запросы

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

Повторяем для каждого банка.

Для разовой задачи - приемлем запрос в цикле. Для регулярной - можно запрос написать один.

(27) нет я не хотел и просил)) (28) молодец

(32) не просил

(30) забудь про запросы (31) запрос в цикле))))

(33) Ладненько.

1. Выбираешь в запросе список Банков,  которые не помеченные на удаление с полями:
         - Ссылка на элемент банка
         - БИК
   Выгружаешь все в ТЗ

2. Формируешь запросом список счетов с отбором, все помеченные на удаление Банки

3. В цикле по счетам ищешь в ТЗ банков по бику нужный банк и как только ты его находишь, что подставляешь вместо прошлого банка и записываешь его.

(35)
4. Записываешь его - Записываешь Банк в нужном счете :)

+(34) Код сам надеюсь осилишь :)

(34) Я же  сказал - для разовой запрос в цикле терпимо.
А вообще можно и одним запросом.

(38) +100 :)

А внешняя обработка поиска дублей что, не работает?

(40) зачем изобретать велосипед, особенно, когда ясно, что велосипед не получается

(35) ну первым запросом я и так получил. Только перебирал еще его. Выходит лишнее. Но и не загрузил в ТЗ
2 . пункт я это и сделал.
3. У меня так и было. Но тупо не срабатывал. Отладчиком смотрел, он находил записи, сверял, результат ложь и зарвешал чикл и все. И как я знаю такой цикл завершается когнда получает значение ложь и тупо завершал

Пока ТаблицаЗапросаБС.Следующий() Цикл
                          
          
          
          Если  ТаблицаЗапросаБС.БИКБанка = ТаблицаЗапросаБанки.БИК Тогда;
                  
                    БанковкийСчет = ТаблицаЗапросаБС.СсылкаНаСпр.ПолучитьОбъект();
                    БанковкийСчет.Банк = ТаблицаЗапросаБанки.Ссылка;
                    БанковкийСчет.Записать();      
          КонецЕсли;

КонецЦикла;


ВЫБРАТЬ БС.Ссылка КАК БС, Б2.Ссылка КАК Банк
ИЗ Справочник.БанковскиеСчета КАК БС
СОЕДИНЕНИЕ Справочник.Банки КАК Б1 ПО БС.Банк = Б1.Ссылка
И Б1.ПометкаУдаления
СОЕДИНЕНИЕ Справочник.Банки КАК Б2 ПО Б1.БИК = Б2.БИК И Б1.Ссылка <> Б2.Ссылка И НЕ Б2.ПометкаУдаления


(40) Не работает, и возможно биков 1000, немного нерационально 1000 раз запускать обработку? :)

(42) Условие тут лишнее.
Тебе БИК известен и его надо лишь найти в ТЗ биков Банков
И подставить нужный элемент "Банк" вместо старого :)

(45) если не условием то НайтиПоРеквизиту("БИК") ?

(43) эт я так понял когда одним запросом получаем. Проблема не в этом была. Это лишь оптимизация так?

(46) Вот так

СтрокаТЗ = ТЗ_БИК.Найти(<ТутИскомыйБИК>,"БИК");

(47) Для вас это высший пилотаж. Сделай как проще :)

(45) вообще, поиск дублей хоть 1000 дублей обработает, она для этого и придумана. И запускать 1000 раз её не придётся. ..

(50) Тогда ТС так ничему не научится :)

(51) А может, оно и к лучшему?
А то снегу на улицах много.. Весна придёт - грязь.. Лето - грязь... Осень листья...
Убирать некому...

(52) остряк

(51) я вот не понимаю кем вы себя считаете? Я незнаю, поэтому обратился к форуму, будучи знающим нафига мне сюда писать. Если не получается и пытаюсь понять, то почему это плохо и лишний раз является предметом стеба таких как вы амебы?

(54) у тебя не то что не получается...
Просто ты даже не пытаешься понять, к сожалению.

(55) почему не пытаюсь, я максимально пытаюсь понять , а не тупо не сижу и не жду ответа. Но сложно сразу все уловить

(56) книжки читай.

(54) тебе расписали алгоритм. Садись и пиши

(58) слушаюсь

Ну можно как то так, не мудря особо -

Запрос = Новый Запрос;
    Запрос.Текст =
        "ВЫБРАТЬ
        |    БанковскиеСчета.Ссылка КАК Ссылка,
        |    Банки.Ссылка КАК Банк
        |ИЗ
        |    Справочник.БанковскиеСчета КАК БанковскиеСчета
        |        ЛЕВОЕ СОЕДИНЕНИЕ Справочник.Банки КАК Банки
        |        ПО БанковскиеСчета.Банк.Код = Банки.Код
        |            И (НЕ Банки.ПометкаУдаления)
        |ГДЕ
        |    БанковскиеСчета.Банк.ПометкаУдаления";

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

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

    Пока ВыборкаДетальныеЗаписи.Следующий() Цикл
        ОбъектСчет = ВыборкаДетальныеЗаписи.Ссылка.ПолучитьОбъект();
        ОбъектСчет.Банк = ВыборкаДетальныеЗаписи.Банк;
        ОбъектСчет.Записать();
    КонецЦикла;

(60) БИК

(61) Ну тут я хз, чо за конфа не написал наш ТС, у меня в БП БИК - это код :) А так да, конечно вместо "Банки.Код" - "Банки.БИК" и т.д.