Не работает код. Ошибки CS0161 и CS1513. Что делать?

Question

[Ошибка 404]

using System;
using System.Linq;

namespace project {
    class Program {
        public static void Main() {
            Console.WriteLine(Reverse("This is an example!"));
        }
        public static string Reverse(string text) {
            string[] arr = text.Split();
            for(int i = 0; i != arr.Length; i++) {
                char[] array = arr[i].ToCharArray();
                Array.Reverse(array);
                arr[i] = new String(array);
            string result = string.Join(" ", arr);
            return result;
        }
    }
}

Не работает код. Проверил несколько раз, все правильно.

Answer 1

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

1. Что должна делать эта строчка?
string[] arr = text.Split();

2. Покажи, где начинается и заканчивается цикл

Спойлер, если лень играть в угадайку

public static string Reverse(string text) {
      string[] arr = text.Split();
      for(int i = 0; i != arr.Length; i++) {
          char[] array = arr[i].ToCharArray();
          Array.Reverse(array);
          arr[i] = new String(array);
      } // Забыл вот эту фигурную скобку
      string result = string.Join(" ", arr);
      return result;
  }

Comments

[Ошибка 404]

"This is an example!" ==> "sihT si na !elpmaxe"
"double spaces" ==> "elbuod secaps"

сплитом я разбиваю строку на массив. в массиве делаю реверс каждого объекта и потом назад соединяю назад в строку

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

Ошибка 404, дай ссылку на документацию, где будет написано, как себя ведёт сплит при вызове без параметров

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

Ошибка 404, я про конкретную строчку спросил

[Ошибка 404]

Василий Банников, https://docs.microsoft.com/en-us/dotnet/api/system...

https://stackoverflow.com/questions/21047886/diffe...

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

Ошибка 404, ок, тогда остаётся только цикл )

Answer 2

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

Проверьте правильность расстановки скобок {}

Answer 3

[Владимир Коротенко]

итак не было скобки,
если длина строки 0 то ничего не возвращалось на что компилятор и ругался, ну и смотрите на вывод иде, она показывает где ошибка

using System;

namespace project
{
    class Program
    {
        public static void Main()
        {
            Console.WriteLine(Reverse("This is an example!"));
        }

        public static string Reverse(string text)
        {
            var arr = text.Split();
            for (var i = 0; i != arr.Length; i++)
            {
                var array = arr[i].ToCharArray();
                Array.Reverse(array);
                arr[i] = new String(array);
                var result = string.Join(" ", arr);
                return result;
            }

            throw new ArgumentException("String must have size > 0");
        }
    }
}

Comments

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

1. Фигурную скобку не там поставил)
2. Этот метод вполне нормально работает даже с пустыми строками.

[Владимир Коротенко]

Василий Банников, кстати да, вот за что не люблю чужой код

public static string Reverse( string s )
{
    char[] charArray = s.ToCharArray();
    Array.Reverse( charArray );
    return new string( charArray );
}

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

Владимир Коротенко, а теперь оно делает не то)

[Владимир Коротенко]

Василий Банников, Пробежись кстати по коду.
Сейчас логика такая:
1. строка на входе разбивается по пробелам
2. первое слово инвертируется
3. потом склейка по пробелам
4. возврат строки

Если это верно, то название функции ну никак не Reverse, скорее ReverseFirstWord

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

Владимир Коротенко,

2. первое слово инвертируется

Не первое, а все по отступам же можно понять, что автор хотел в цикле выполнять этот код:

var array = arr[i].ToCharArray();
Array.Reverse(array);
arr[i] = new String(array);

Да и по комментариям с ожидаемым результатам то же самое.
А название да, всё равно не очень.