Как реверсировать строку с помощью метода расширения?

Question

[Nonpacie]

Нужно решить задачу на реверсирование массива целых чисел методом расширения, собственно почитал про эти методы и написал метод, который не выдаёт ошибок и варнингов, но и не работает, подскажите, в может быть проблема?

Program.cs

using System;

namespace Lab10
{
    internal static class Program
    {
        
        public static void Main(string[] args)
        {
            int[] array = new[] {1, 4, 5, 1, 3, 235, 634, 234, 12, 4, 3, 77, 141};
            array.PrintArray();

            array.ReverseArray();

            Console.WriteLine();
            foreach (var elem in array)
            {
                Console.Write($"{elem} _ ");
            }
        }
    }
}

Extension.cs

...
public static int[] ReverseArray(this int[] arr)
        {
            List<int> buffer = arr.ToList();
            buffer.Reverse();
            arr = buffer.ToArray();
            return arr;
        }
...

Answer 1

[Владислав]

for (int i = 0; i < arr.Length / 2; i++)
{
   int tmp = arr[i];
   arr[i] = arr[arr.Length - i - 1];
   arr[arr.Length - i - 1] = tmp;
}

Comments

[Nonpacie]

Боюсь вы не правильно поняли суть проблемы. Алгоритм реверса вроде в порядке, суть в том что он не работает при вызове метода расширения, хотя ошибок никаких нет.

[Владислав]

Nonpacie, а, то есть я должен за вас этот код отдебажить и сказать причину?
Я вам скинул рабочий код

у тебя метод

public static int[] ReverseArray(this int[] arr)

возвращает массив int, но ты никуда не присваиваешь этот результат а просто идешь дальше

Мой же код без проблем ваш массив обработает и не придется никуда его присваивать.

Вот готовое решение и проверенное:

namespace ConsoleApp1
{
    static class Program
    {
        static void Main(string[] args)
        {
            int[] array = new[] {1, 4, 5, 1, 3, 235, 634, 234, 12, 4, 3, 77, 141};

            array.ReverseArray();

            Console.WriteLine();
            foreach (var elem in array)
            {
                Console.Write($"{elem} _ ");
            }
        }
        
        public static void ReverseArray(this int[] arr)
        {
            for (int i = 0; i < arr.Length / 2; i++)
            {
                var tmp = arr[i];
                arr[i] = arr[arr.Length - i - 1];
                arr[arr.Length - i - 1] = tmp;
            }
        }
    }
}

[Nonpacie]

Владислав, Я просто не понял в чем я ошибся, и не говорил что кто-то что-то должен, увидев ваш код, понял свою ошибку. Спасибо.

Answer 2

[Sergey В.]

Я так понимаю у Вас именно задание по учебе?

Если нет, то можно:
1. Перевести array в List через .ToList
2. Выполнить для List команду .Reverse
3. Перевести List обратно в array командой .ToArray
Три строчки кода.

Comments

[Nonpacie]

Задание по учёбе, верно. Реверс сам по себе сделать не проблема, проблема в том что есть массив чисел, вызывается метод расширения который я описал выше, а с массивом ничего не происходит, изначальное положение остается...

[Nonpacie]

P.S. Попробовал реверсировать массив так как вы описали, проблема не исчезла

[Sergey В.]

Nonpacie, Реверс по моему способу работает, только что проверил:

int[] array = new[] { 1, 4, 5, 1, 3, 235, 634, 234, 12, 4, 3, 77, 141 };

            Console.WriteLine("First Before: " + array[0]);

            List<int> buffer = array.ToList();
            buffer.Reverse();
            array = buffer.ToArray();

            Console.WriteLine("First After: " + array[0]);

Выводит:
First Before: 1
First After: 141

Остаётся только правильно оформить решение. Чему сейчас учат и что требуют - не знаю.

[Nonpacie]

BasiC2k, Увидел свою ошибку, сам алгоритм верен, просто было неверное возращаемое значение метода, спасибо!