C# Index was out of range - ошибка при чистке списка

[Astraport]

Обычная задача: есть список с фразами (result) и ключ (key).
Нужно удалить все строки со списка содержащие любое слово из ключа, если это слово длиннее трёх символов.
Решаю так:

var list = project.Lists["result"];
string str = "";
string keyword = "";
string text = project.Variables["key"].Value;
string[] words = text.Split(' ');
int value = list.Count;
for (int i = 0; i < list.Count; i++) {
    str = list[i];
    for (int j = 0; j < words.Length; j++) {
        keyword = words[j];
        if (keyword.Length > 3 && str.Contains(keyword)) {
            list.RemoveAt(i);
        }
    }
}

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

Выполнение действия CSharp OwnCode Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index

 

[ssXXXss]

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

 

[Astraport]

Так вроде list.Count же тоже уменьшается и в каждой итерации просчитывается.

 

[ssXXXss]

Так вроде list.Count же тоже уменьшается и в каждой итерации просчитывается.

list.Count это общее количество цикла, например count равен 10, ты удаляешь со списка строку там уже 9, а цикл у тебя 10, такого индекса нет в списке, как то так )

 

[Astraport]

а цикл у тебя 10,

Но почему? Во всех ЯП длина списка в каждой итерации цикла заново рассчитывается.

 

[ssXXXss]

Но почему? Во всех ЯП длина списка в каждой итерации цикла заново рассчитывается.

она заново рассчитывается если ты заново зайдёшь в него

 

[Lord_Alfred]

@ssXXXss правильно говорит, тут нужно идти от большего элемента (i = list.Count-1) к меньшему (i >= 0) с уменьшением (i--)

 

[ssXXXss]

попробуй так

var list = project.Lists["result"];
string text = project.Variables["key"].Value;
string[] words = text.Split(' ');

for(int i = list.Count-1; i >= 0; i--)
{
    var str = list[i];
    for (int j = 0; j < words.Length; j++)
    {
        var keyword = words[j];
        if (keyword.Length > 3 && str.Contains(keyword))
        {
            list.RemoveAt(i);
        }
    }
}

 

[Astraport]

Ну хорошо, обратный цикл (на уменьшение) тоже выдает такую же ошибку.

 

[ssXXXss]

Ну хорошо, обратный цикл (на уменьшение) тоже выдает такую же ошибку.

мой код ошибку выдаёт ?

 

[Astraport]

Да, я пробовал этот вариант.

 

[ssXXXss]

Да, я пробовал этот вариант.

значит массив пуст, ставь проверку выше

 

[Astraport]

if (words.Length == 0 || list.Count == 0) {
    return -1;
}

Ставлю, не выходит по ошибке.

 

[Astraport]

<p>wedding guest dresses</p><p>party dresses</p><p>long sleeve cocktail dresses</p><p>maxi dresses</p><p>nordstrom dresses</p><p>dress barn</p><p>women's dresses for weddings</p><p>women's special occasion dresses</p><p>dresses for juniors</p><p>jcpenney dresses</p><p>dresses for prom</p><p>women's casual dresses</p><p>dresses to wear to a wedding</p><p>midi cocktail dress</p><p>neiman marcus cocktail dresses</p><p>midi dresses with 3/4 length sleeves</p><p>cocktail dresses with sleeves</p><p>cocktail dresses for weddings</p><p>mothers day dresses</p><p>kohls dresses formal</p><p>women's summer dresses</p><p>women's cocktail dresses</p><p>spring dresses</p><p>spring 2015 wedding guest fashion</p><p>summer wedding guest fashion</p><p>cute summer wedding guest dresses</p><p>2015 wedding guest attire</p><p>wedding party dresses for mother of groom</p><p>mother of the groom dresses</p><p>macy's wedding party dresses</p><p>plus size wedding party dresses</p><p>inexpensive wedding party dresses</p><p>wedding party dresses for sale</p><p>wedding event dresses</p><p>cocktail dress</p><p>formal dresses</p><p>rent the runway</p><p>david's bridal</p><p>lulus</p><p>windsor</p><p>fall wedding guest dresses</p><p>cocktail dress wedding</p><p>wedding guest dresses for summer</p><p>wedding guest dresses plus size</p><p>wedding guest dresses for spring</p><p>10 best celebrity wedding guest dresses</p><p>wedding guest dresses 2016</p><p>wedding guest dresses canada</p><p>summer wedding guest dresses 2015</p><p>wedding guest dresses 2015</p><p>summer wedding guest dresses</p><p>wedding dresses 2015 summer</p><p>wedding party dresses</p><p>dresses wedding party</p><p>purple wedding party dresses</p><p>wedding party dresses for women</p><p>evening dresses wedding party</p><p>mother of the bride dresses</p><p>wedding suit with silver inserts and sequins</p><p>macys dresses</p>

Вот для конкретики данные для списка (распарсить (?<=<p>).*?(?=</p>))
И вот ключ: macys dresses

 

[ssXXXss]

пробуй

var list = project.Lists["result"];
string text = project.Variables["key"].Value;
string[] words = text.Split(' ');

for(int i = list.Count-1; i >= 0; i--)
{
    var str = list[i];
    for (int j = 0; j < words.Length; j++)
    {
        var keyword = words[j];
        if (keyword.Length > 3 || str.Contains(keyword))
        {
            list.RemoveAt(i);
            break;
        }
    }
}

return string.Join("...", list);

 

[Astraport]

Неа, пустой список.

 

[ssXXXss]

Неа, пустой список.

там надо заменить || на && или | , видимо он всё удалил что больше трёх сиволов

 

[Astraport]

Вот, теперь я понял! Спасибо.
Второй цикл удаляет строки несколько раз, точно - нужен был break.

 

[ssXXXss]

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

var list = project.Lists["result"];
string text = project.Variables["key"].Value;
string[] words = text.Split(' ');

for(int i = list.Count-1; i >= 0; i--)
{
    var str = list[i];
    for (int j = 0; j < words.Length; j++)
    {
        var keyword = words[j];
        if (str.Length > 3 && str.Contains(keyword))
        {
            list.RemoveAt(i);
            break;
        }
    }
}

return string.Join("...", list);

вот мой результат



удалило aaaa, правильно ?

 

[Astraport]

Да, я понял ошибку, спасибо. Волшебный break)