Метод заставляет компьютер обожраться памятью?

Question

[Дмитрий Королев]

public int[] SearchLexem(string findText)
        {
            lenghtText = 1;
            findText = findText.ToLower();
            first = new List<int>();
            i = -2;
            firstCharacter = 0;
            startIndex = 0;
            while (firstCharacter != -1)
            {
                i++;
                if (i > -1)
                {
                    first.Add(firstCharacter);
                }
                firstCharacter = allText.IndexOf(findText, startIndex);
                secondCharacter = (firstCharacter + lenghtText) - 1;
                startIndex = secondCharacter;
            }
            return first.ToArray();
        }

Комп обжирается памятью
ищу все совпадения символа в строке

Comments

[Довольный Айтишникъ]

И что значит "обжирается памятью"? Тут не все экстрасенсы.

[Дмитрий Королев]

зависает

[Александр Юдаков]

Отладчиком не пробовали пользоваться?

Answer 1

[eRKa]

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

И мне кажется у Вас там ошибка. Возможно lenghtText должен быть равен findText.Lenght. Т.е. после того как Вы нашли позицию вхождения, дальше надо искать с позиции

secondCharacter = (firstCharacter + findText.Lenght) - 1;

нет смысла сдвигать только на один символ. И не проще ли было найти все вхождения регуляркой? Это было бы две строки текста.

Comments

[Дмитрий Королев]

да это длинна текста который я ищу. весь текст 15 символов
что такое регулярка?

[marssx]

Дмитрий Королев, https://tproger.ru/articles/regexp-for-beginners/

[Дмитрий Королев]

marssx, а с методом что?

[eRKa]

Дмитрий Королев, у Вас проблема в этой строке

secondCharacter = (firstCharacter + lenghtText) - 1;

Т.к. по условию у Вас lenghtText = 1, то secondCharacter всегда будет равен firstCharacter, индекс для поиска вхождения сдвигаться никогда не будет.
Оставьте просто firstCharacter + lenghtText

[Дмитрий Королев]

eRKa, спасибо