Как исправить код с неверным выводом кол-ва строк?

Question

[Максим К]

Есть 2 textBox, в левый вставляю строки построчно, во второй по шаблону заполняю текст.

Код кнопки преобразования:

private void button1_Click(object sender, EventArgs e)
        {
            string s;
            s = "";
            b = textBox1.Lines.Length / 4;

            if (a % 2 == 0)
            {
                for (int i = 0; i <= a; i++)
                {
                    s = s + string.Format(str + Environment.NewLine, textBox1.Lines[0], textBox1.Lines[1], textBox1.Lines[2], textBox1.Lines[3]);
                    var lines = textBox1.Lines.ToList();
                    lines.RemoveAt(3);
                    lines.RemoveAt(2);
                    lines.RemoveAt(1);
                    lines.RemoveAt(0);
                    textBox1.Lines = lines.ToArray();
                }
            }
            else
            {
                MessageBox.Show("Нечетное количество строк", "Ошибка");
                textBox2.Clear();
            }

        }

Если вставить 60 строк- то перенесется только 52 строки. Почему так?

Comments

[Sergey В.]

А чему у Вас "а" равно?

[Максим К]

BasiC2k, ой, извиняю,

a = textBox1.Lines.Length;

[Максим К]

BasiC2k, убрал условие (a % 2 == 0) и все равно только 52 строки переносится, 8 остаются на месте (( Из 120 строк, переносится только 20 .

[Sergey В.]

Максим К, судя по коду и исходным данным - код должен проходить if (a % 2 == 0) к циклу. Цикл должен выполниться ровно столько раз, сколько было изначально строк в textBox1. А вот уже в цикле идён непонятная для меня логика. Строки textBox1 преобразуются в список, список корректируется и потом переносится обратно в textBox1. И так 60 раз.

[Максим К]

BasiC2k, убрал проверки, вставил принудительно вместо а - 60:

for (int i = 0; i <= 60; i++)
                {
                    s = s + string.Format(str, textBox1.Lines[0], textBox1.Lines[1], textBox1.Lines[2], textBox1.Lines[3]);
                    var lines = textBox1.Lines.ToList();
                    lines.RemoveAt(3);
                    lines.RemoveAt(2);
                    lines.RemoveAt(1);
                    lines.RemoveAt(0);
                    textBox1.Lines = lines.ToArray();
                }

Ошибка вот в этой строке s = s + string.Format(str, textBox1.Lines[0], textBox1.Lines[1], textBox1.Lines[2], textBox1.Lines[3]); - System.IndexOutOfRangeException: Индекс находился вне границ массива.

[Sergey В.]

Максим К, переменная "str" у Вас также - не объявлена.
А вы можете показать, что у Вас в textBox1?

[Максим К]

BasiC2k, тут 2 файла - один исходник полностью рабочий на delphi, 2-й мой переписанный на C# https://cloud.mail.ru/public/Y5d6/gsrmo6fhA

[Sergey В.]

Максим К, первая ошибка: textBox1.Lines = lines.ToArray(); нужно вынести за границы цикла.

[Sergey В.]

Максим К, Второе, после цикла: textBox1.Lines = s;

[Максим К]

BasiC2k, это удаление строк и заполнение следующих 4-х. Вынес, код перестал должным образом работать )

[Sergey В.]

Максим К,

private void button1_Click(object sender, EventArgs e)
        {
            string s = "";

            var lines = textBox1.Lines.ToList();

            if (lines.Count % 2 == 0) 
            {
                for (int i = 0; i < lines.Count; i+=4)
                    s = s + string.Format(str, lines[i], lines[i + 1], lines[i + 2], lines[i+ 3]);
            }
            else
            {
                MessageBox.Show("Нечетное количество строк", "Ошибка");
            }

            textBox2.Text = s;
        }

[Максим К]

BasiC2k, работает )))) вот вроде простая задачка, а не все так просто оказалось. СПАСИБО. Буду разбираться в вашем коде.

[Максим К]

BasiC2k, я не могу отметить решением, т.к. это видимо в переписке, сможете еще раз вынести под вопрос и я отмечу решением. СПАСИБО