Не меняется местами при четных значениях n 1-мерного массива

@Boba_Fet · Регистрация: 05.01.2013

Author24 — интернет-сервис помощи студентам

Всё вроде работает, при нечетных значениях всё меняется, последнее число меняется на ноль, что логично, а при четных значения не происходит ничего, подскажите, в чем проблема? Заранее спасибо...P.S Измененный массив выводится по 2-й кнопочке.

C#

        private void Form1_Load(object sender, EventArgs e)
        {
 
        }
        int n;
        int[] A;
        void Generate(int N, int[] A)
        {
            Random rnd = new Random();
            for (int i = 0; i < N; i++)
                    A[i] = rnd.Next(100);
                }
 
        private void button1_Click(object sender, EventArgs e)
        {
            n = Convert.ToInt32(numericUpDown1.Value.ToString());
            A = new int[n];
            Generate(n, A);
            dataGridView1.RowCount=n;
            for (int i = 0; i < n; i++)
            dataGridView1.Rows[i].Cells[0].Value=A[i];
        }
 
        void Perestanovka(int[] A)
        {
            int c=0;
            for (int i = 0; i < n; i+=2)
            {
                 c = A[i];
                 A[i] = A[i+1];
                 A[i+1] = c;
            }
        }
 
        
 
        private void button2_Click(object sender, EventArgs e)
        {
            int[] A = new int[10];
            for (int i = 0; i < dataGridView1.RowCount; i ++)
                A[i] = Convert.ToInt32(dataGridView1.Rows[i].Cells[0].Value);
            for (int i = 0; i < dataGridView1.RowCount; i++)
                Perestanovka(A);
            dataGridView2.RowCount = Convert.ToInt32(numericUpDown1.Value.ToString());
            for (int i = 0; i < dataGridView1.RowCount; i ++)
                dataGridView2.Rows[i].Cells[0].Value = A[i];
 
 
 
 
 
 
        }
 
        
 
    }
}

@kodv · 06.01.2013, 19:09

Ух, ны вы и объясняете. Как я правильно понял, когда массив содержит нечетное количество элементов, то все работает как вы хотите, а когда четное, то работает неправильно. Обратите внимание на данную строчку

C#

1 2	for (int i = 0; i < dataGridView1.RowCount; i++) Perestanovka(A);

Здесь вы делаете перестановку столько раз, сколько у вас элементов в массиве, при том, что каждый раз перестановка делается для всего массива. Зачем?
Вот смотрите, что получается. В результате перестановки все соседние элементы массива меняются местами. Если сделать данные манипуляции над уже "переставленным" массивом, то у вас элементы вернутся на исходные позиции (кроме последнего, в случае нечетного количества элементов). То есть, каждой второй вызов метода Perestnovka фактически отменяет каждый первый вызов этого метода. В случае, когда у вас четное количество элементов массива, последний запуск вышеупонятого метода возвращает все элементы массива на исходные позиции, а вы не видите результата.

@Boba_Fet · 07.01.2013, 12:00 **[ТС]**

kodv, но а что мне нужно, сделать то?...отдельный цикл ввести для это функции?

Добавлено через 1 минуту
kodv, всё, получилось всё, я убрал ...

C#

1	for (int i = 0; i < dataGridView1.RowCount; i++)

и всё получилось)))

	07.01.2013, 12:00