добрый вечер,подскажите как правильно удалить строку в ТЗ?
    Для каждого СтрокаТЧ Из ТаблицаТоваров Цикл
Если  СтрокаТЧ.Склад=Перечисления.Склад.Основной Тогда
            ТаблицаТоваров.Удалить()    
            КонецЕсли;    
        КонецЦикла;

Удалять надо с конца.

(0) правильно обратным циклом, гугли, на твой вопрос есть не один ответ

(2)это как

я циклом сначала иду.как с конца идти?

(4)  программирование совсем непонятная для тебя вещь?

в очередной раз повторю
правильное удаление строки из ТЗ, это:

Перенос в новую ТЗ только тех строк которые не нужно удалять

можно НайтиСтроки(<Отбор>) и потом удалить все из получившегося массива строк

правильнее не удалять а не помещать их туда

(8) +5

(8)Хотеть не вредно. Ну вот например Тз как результат запроса. Если добавить нужное условие, то не попадаешь в индекс.

(10) сложно сделать второй запрос который вернет только нужное?

(10) Вот ты знаешь, что сейчас сказал?
Ты сказал "я вообще ни разу не умею писать запросы, зато знаю умное слово индекс"...

(12) ха
может чтобы узнать "индекс" сначала нужно получить ТЗ из запроса и для каждой строчки индивидуально функциями обработать...

(12)Самомнение штука хорошая
Ну или вариант 2 - загрузка из Excel

Да ладно, может он ТЗ пользователю показывает, а тот уже решает что там удалять.

(2) Я удаляю выбрав строки в массив. Можно ссылку на то, что я это делаю неправильно?


Для каждого т Из ТаблицаТоваров.НайтиСтроки(новый Структура("Сумма", 0)) Цикл
    т.Удалить();    
КонецЦикла;


(17) чуть не так


Для каждого т Из ТаблицаТоваров.НайтиСтроки(новый Структура("Сумма", 0)) Цикл
    ТаблицаТоваров.Удалить(т);    
КонецЦикла;


Если обход строк по индексу - удалять с конца, если по ссылкам (Для Каждого Из) - без разницы.
Еще можно через ПостроительЗапроса/ИсточникДанных/Отбор )

(10) напиши запрос так, чтобы ничего не удалять

Варианта два:
1. Удалять с конца
2. НайтиСтроки, которые не нужны, и сложить их в массив (или циклом собрать эти строки), потом удалить мороки в цикле для каждого по этому массиву

Вариант типа (18) - хрень, ибо производительность его будет уменьшаться экспоненциально с ростом размера таблицы

Отбор, НайтиСтроки() и обходи полученный массив. Удалять не обязательно

(21) Есть замеры, подтверждающие "хреновость" варианта?

(21) Гм. А чем твой вариант №2 отличается от (18). Фатальный недостаток?

(24) фатальный недостаток в том что он ищет строки для каждой итерации

(24) у меня поиск в таблице или перебор один раз происходит, в 18 каждую строку приходится искать заново.
Хотя, при учете, что при построчном удалении таблица скорее всего перстраивается целиком, все варианты с последовательным удалением одинаково хреновые и оптимальнее всего, как уже было отмечено выше, сформировать новую ТЗ из строк, которые должны остаться.
Лет пять назад вроде был тут целый холивар на тему, как правильнее всего удалять строки. Сотрясли кубометры воздуха, но универсального варианта на все времена не еашли

а, в (18) найтиСтрокии(). Пардон - тогда он идентичен моему второму варианту. Если платформа не вызывает этот найтиСтроки на каждой итерации.

- Удалять надо с конца.
- Это как?

Для Индекс = -Таблица.Количество()+1 По 0 Цикл
   Строка = Таблица.Получить(-Индекс);
   Если УсловиеУдаления Тогда
      Таблица.Удалить(Строка);
   КонецЕсли;
КонецЦикла;

(27) не слишком вглядываясь в ранее написанное.
НайтиСтроки() возвращает массив. Его нужно задать в виде переменной перед циклом МассивДляУдаления. В условии цикла поставить Из МассивДляУдаления.

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

Так что все равно в каком направлении обходить массив, но не все равно в каком направлении использовать индексы. в значениях ссылок индексов нет.

Хотел бы я посмотреть, как вы будете найтистроки использовать при удалении по сложному условию.

(28)
Индекс = ТЗ.Количество();
Пока Индекс > 0 Цикл
    Индекс = Индекс - 1;
    // это если нужно для операций над строкой
    СтрокаТЗ = ТЗ.Получить(Индекс);
    // а здесь просто удаляем по индексу
    ТЗ.Удалить(Индекс);
КонецЦикла;

(30) у кого посмотреть хочешь?

(31) Автор в (0) обращается к строке для условия удаления - и потому (28)

(33) условие можно вставить и в том виде, как в 31 сделано.
А я всего лишь дал упрощенную форму для условия цикла.

(33) просто он отрицательного значения индекса глаза выворачиваются :) - для субботы это слишком тяжко

Классика жанра (из типовых)

Отбор = Новый Структура();
Отбор.Вставить(...);
Строки = Таблица.НайтиСтроки(Отбор);
Для Каждой Строка Из Строки Цикл
...

(35) Ты прав, что-то я не подумал про субботу :)

(29) Можно и без отдельного массива. Читаться будет сложнее, работать точно также.
(31) В пока >= 0 надо проверять

Суббота - не твой день =)

(38) Хе, и не мой =)

(39) :)

Строки к удалению в отдельный массив неважно как, а дальше стандартно.

Для (0) подробно, что то он пропал куда то

МассивСтрокТЗ = ТаблицаТоваров.НайтиСтроки(Новый Структура("Склад", Перечисления.Склад.Основной));

Для каждого СтрокаТЗ Из МассивСтрок Цикл
    ТаблицаТоваров.Удалить(СтрокаТЗ);    
КонецЦикла;

поправил
МассивСтрокТЗ = ТаблицаТоваров.НайтиСтроки(Новый Структура("Склад", Перечисления.Склад.Основной));

Для каждого СтрокаТЗ Из МассивСтрокТЗ Цикл
    ТаблицаТоваров.Удалить(СтрокаТЗ);    
КонецЦикла;

Ветку не читай комментарии пиши, я думаю мой способ будет самый правильный и никто еще не разу про него не написал. В общем так:
1. Сначала получаем массив строк с помощью "Найти строки" или еще каким либо путем, неважно
2. С помощью цикла "Для каждого Из" удаляем строки из ТЗ, те которые есть в этом массиве
3. Я гениален!
Наверняка из предыдущих 40 постов этого никто не предложил :)

(0)
Шутку поняли. Смешно.
Ты офигенный юморист!

(45) -> (44)

(44) Все конечно читать лень, но хотя бы пару последних постов прочитал бы...

(21) > Вариант типа (18) - хрень, ибо производительность его будет уменьшаться экспоненциально с ростом размера таблицы

Я возможно и не прав, но вот сколько раз будет вызвана функция?


Для каждого СтрокаТЗ Из Фун(ТаблицаТоваров.НайтиСтроки(Новый Структура("Склад", Перечисления.Склад.Основной))) Цикл
    ТаблицаТоваров.Удалить(СтрокаТЗ);    
КонецЦикла;

Функция Фун(МассивСтрокТЗ )
    Сообщить("Тест");
    Возврат МассивСтрокТЗ;
КонецФункции


(25) неправда, ищет один раз
Конструкция из (18) имеет право на существование и в скорости не уступает, есть замеры

Так с конца идешь:

Для нс = -ТЗ.Количество() По -1 Цикл


КонецЦикла;


Можешь сначала в массив закинуть нужные строки, а потом разом их и грохнуть.

если не умеешь, закинь ТЗ в запрос, отфильтруй и выгрузи ТЗ :)

можешь отбором по строкам и грохнуть разом

Жаль, что в языке нету удаления массива строк, это был бы самый быстрый вариант

(53) напиши свою удобную процедуру :)

(49) если в цикле добавить новую строку удовлетворяющую условиям отбора, она будет удалена?

(55) У цикла будет столько итераций, сколько было изначально элементов

(54) скорости это не прибавит

(16) а я разве сказал, что все остальные способы неправильные?

(56) как-то ты ушел от ответа. Строка будет удалена?

(59) Он же ответил неявно - нет.

ТаблицаЗначений.Скопировать (ValueTable.Copy)
ТаблицаЗначений (ValueTable)
Скопировать (Copy)
Вариант синтаксиса: Скопировать строки

Синтаксис:

Скопировать(<Строки>, <Колонки>)
Параметры:

<Строки> (необязательный)

Тип: Массив.
Массив строк для копирования. Если не указан, копируются все строки таблицы значений.
<Колонки> (необязательный)

Тип: Строка.
Список колонок для копирования в формате: "Колонка1, Колонка2...".
Описание варианта метода:

Если указаны строки и колонки, то только они будут скопированы. Если не указаны, то будет создана полная копия таблицы значений.
Вариант синтаксиса: Скопировать по отбору

Синтаксис:

Скопировать(<ПараметрыОтбора>, <Колонки>)
Параметры:

<ПараметрыОтбора> (необязательный)

Тип: Структура.
Ключ структуры - идентификатор колонки, а значение структуры - значение отбора.
<Колонки> (необязательный)

Тип: Строка.
Список колонок для копирования в формате: "Колонка1, Колонка2...".
Описание варианта метода:

Если указан отбор, то только строки из отбора будут скопированы. Если отбор не указан, то будут скопированы все строки таблицы значений. Если указаны колонки, то только эти колонки будут скопированы. Иначе, будут скопированы все колонки таблицы значений.
Возвращаемое значение:

Тип: ТаблицаЗначений.

Описание:

Создает копию исходной таблицы значений.

Доступность:

Сервер, толстый клиент, внешнее соединение, мобильное приложение(сервер).
Пример:

НовТаблицаЦен = ТаблицаЦен.Скопировать();


--------------------------------------------------------------------------------

     Методическая информация

(61) если нет четкого признака в отдельной колонке "для удаления" то простейше это добавляем новую колонку, в нужные "для удаления" строки пишем нечто
затем копируем только те строки через "ПараметрыОтбора" куда ничего не проставили

(62) И в процессе "написания нечто" перебираем все строки? =)

(63) 1 раз
и еще 1 раз они "переберутся" при копировании

а если удалять по одной строке, то каждое удаление вызывает "перебор" части строк
т.е. сколько строк удалили - столько раз и "перебрали"

(64) Ну если тебя производительность выделения память так беспокоит и хочется именно копию таблицы получить, то делай наоборот - при проверке условия запоминай строки, которые надо оставить =)

(65) очень рекомендую попробовать создать ТЗ в несколько сотен тысяч строк

а потом попробовать удалить из нее половину строк через ТЗ.Удалить(Строка|Индекс)

(66) Очень рекомендую сильно думать перед созданием "ТЗ в несколько сотен тысяч строк"
Говорят, иногда помогает быстрее работать программе...

(66) Несколько сотен тысяч из которых половина не нужна? Герой!

(67) так эту "ТЗ" не я создавал а другие "программисты"
предлагается лезти в их "код" и переписывать?

(68) ну да все как обычно: "Сначала создаем себе трудности, а потом героически их преодолеваем"

(59) В тз она останется

Просто собрать строки к удалению, например, в массив, а потом удалить.

Я в шоке от тупизма.

(71) + и читайте про коллекции.
А если впадлу- то просто работайте, слушайтесь и не выеж..)