Почему не работает метод с большими записями?

Question

[Анна Панова]

private List<string> convertInList(string a)
        {
            List<string> simvols = new List<string> { "+", "-", "*", "/", "^" };
            List<string> primer = new List<string>();
            string cifra = "";

            int i = 0;

            while (i < a.Length)
            {
                if (simvols.Contains(a[i].ToString()))
                {
                    primer.Add(a[i].ToString());
                    i++;
                }
                else
                {
                    while (i < a.Length && !simvols.Contains(a[i].ToString()) && a[i].ToString() != ")" && a[i].ToString() != "(")
                    {
                        cifra += a[i];
                        i++;
                    }

                    primer.Add(cifra);
                    cifra = "";
                }
            }

Данный метод преобразует строку в лист. Пример: string "30+4" в List {"30", "+", "4"}. Проблема возникает когда вводятся длинные выражения.

Answer 1

[soloveid]

.ToString()

Из-за этого

Comments

[Анна Панова]

soloveid, где именно?

[Анна Панова]

Каким образом это влияет на память?

[soloveid]

Анна Панова, Каждый символ переводится в строку, а это выделение памяти,
плюс сравнение строк значительно сложнее чем сравнение пары символов

[Виталий Качан]

Анна Панова, Почитай про особенности типа string. Например https://metanit.com/sharp/tutorial/7.3.php
рекомендую использовать метод indexof

[soloveid]

Анна Панова,

List<string> ConvertInList(string a)
        {
            List<char> simvols = new List<char> {'+', '-', '*', '/', '^'};
            List<string> primer = new List<string>();
            StringBuilder cifra = new StringBuilder();

            int i = 0;

            while (i < a.Length)
            {
                if (simvols.Contains(a[i]))
                {
                    primer.Add(a[i].ToString());
                    i++;
                }
                else
                {
                    while (i < a.Length && !simvols.Contains(a[i]) && a[i] != ')' && a[i] != '(')
                    {
                        cifra.Append(a[i]);
                        i++;
                    }

                    primer.Add(cifra.ToString());
                    cifra.Clear();
                }
            }

            return primer;
        }

[Василий Банников]

Анна Панова ,
1. а зачем вам List simvols? Вы же не добавляете туда новые элементы => Лист не нужен
2. из-за того что вы часто делаете simvols.Contains, выгоднее будет сделать HashSet<char>

[soloveid]

Анна Панова,

Каким образом это влияет на память?

У Вас ошибка ещё в коде есть, поэтому и память всю выедает когда встречаются скобки.