2

Хочу написать класс с различными сортировками. Первоначальная версия была такая:

public static T[] BubbleSort(this T[] array, string sortOrder = "ASC")
{
    if (array.Equals(null))
        return null;

    switch (sortOrder.ToLower())
    {
        case "asc":
            for (var i = 0; i < array.Length; i++)
            {
                for (var j = 1; j < array.Length - i; j++)
                {
                    if (array[j - 1].CompareTo(array[j]) > 0)
                        Swap(ref array[j - 1], ref array[j]);
                }
            }
        break;
    // etc.
}

Мне показалось, что использовать таким образом строку (2 параметр) не красиво и поэтому я решил применить enum:

public enum SortOrder
{
    ASC,
    DESC
};

И теперь, собственно, сам вопрос: "Куда лучше написать enum?"

  • В отдельный файл? Но тогда конечному пользователю придётся таскать с собой два файла (да, я хочу загрузить код на свой github репозиторий).
  • Внутри класса? Но тогда, придётся писать огромную строчку из кода:
Sorting<int>.SortOrder.ASC

вместо

SortOrder.ASC
  • В том же файле, где находится класс, но снаружи? Вроде бы да, но.. Красиво ли это смотрится? Помогите мне решить, как лучше поступить? :)

Исполняемый файл (если enum лежит в отдельном файле или в том же файле, где и класс, но вне класса):

int[] array = { 2, 1 };
Sorting<Int32> sortingInt32 = new Sorting<Int32>();
sortingInt32.BubbleSort(array, SortOrder.ASC);

Исполняемый файл (enum лежит внутри класса):

int[] array = { 2, 1 };
Sorting<Int32> sortingInt32 = new Sorting<Int32>();
sortingInt32.BubbleSort(array, Sorting<int>.SortOrder.ASC); // Вообще не красиво, согласны?

Так выглядит мой класс:

namespace Sorting
{
    public class Sorting<T>
        where T : IComparable<T>
    {
        // etc.
    }
}

Заранее спасибо! :) P.S. Если будут предложения не с enum, с большим интересом рассмотрю.

11
  • В том же неймспейсе, где находится класс, но снаружи.
    – aepot
    8 фев 2022 в 14:17
  • Это как-то регламентируется или просто негласное правило хорошего кода? :D
    – Danila
    8 фев 2022 в 14:19
  • Ну вам уже это и так почти очевидно. Публичные вложенные типы даже Microsoft не рекомендует использовать.
    – aepot
    8 фев 2022 в 14:20
  • Я одного не могу понять this T[] array выглядит как расширение, а потом вы показываете вполне обычный класс public class Sorting<T>, где и что я упустил? Чем вам метод расширения не зашел?
    – aepot
    8 фев 2022 в 14:22
  • 2
    Ого! У меня получилось даже без "public static ArrayExtensions { тут метод } ", убрав просто <T> у класса.. Изначально я так и хотел сделать, это так круто!
    – Danila
    8 фев 2022 в 14:48

1 ответ 1

2

Метод расширения вполне резонно смотрится, перечисление можно рядом положить

public static class ArrayExtensions
{
    public static void BubbleSort<T>(this T[] array, SortOrder sortOrder = SortOrder.ASC)
        where T : IComparable<T>
    {
        // ...
    }
}

public enum SortOrder
{
    Asc,
    Desc
}

Вызывать как-то так

int[] array = new int[] { 1, 3, 2 };
array.BubbleSort(SortOrder.Desc);
4
  • А не могли бы вы посоветовать, как элегантно заменить конструкцию switch-case в методе. Она используется для определения sortOrder. Если asc, то по возрастанию, если desc - по убыванию, default => goto asc. Может существует более элегантный способ, ведь.. Код в case`ах один и тот же, за исключением знаков > и <. Можно, конечно, обойтись и без этого и написать if внутри циклов, но производительность сильно упадет, ведь проверка будет осуществляться постоянно. Как лучше сделать?)
    – Danila
    9 фев 2022 в 21:10
  • Также есть мысль написать отдельные приватные функции: BubbleSortAsc и BubbleSortDesc. Выызвать их, если sortOrder.Asc и *.Desc соответственно. Почему-то идеи в моей голове на индийском языке :D Тогда уже, наверное, проще просто сделать вместо 3 функций 2 и пользователь сам будет явно выбирать, какую ему использовать. Я отталкиваюсь от удобства и чистоты кода. :)
    – Danila
    9 фев 2022 в 21:21
  • @Danila если у вас появился новый вопрос, его стоит задать отдельно. Если ответ был полезен, вы можете принять его, поставив зеленую галочку слева от ответа.
    – aepot
    9 фев 2022 в 23:40
  • 1
    Спасибо, отметил ваш ответ полезным. Написал новый вопрос: [ru.stackoverflow.com/questions/1379883/… Если вам не сложно, помогите с выбором :)
    – Danila
    10 фев 2022 в 12:12

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.