4

В общем, есть switch длиной с Эверест, как бы вы избавились от него, кроме как записывать это в Dictionary?

        switch (currencyLeft)
        {
            case "BTC":
                return 0.001m;
            case "BCH":
            case "ETH":
            case "FCT":
            case "NEO":
            case "UBTC":
            case "XEM":
            case "ZEC":
                return 0.01m;
            case "MCO":
            case "QTUM":
            case "REP":
            case "STX":
            case "VET":
                return 0.1m;
            case "ETC":
            case "DASH":
            case "LTC":
            case "XLM":
            case "XRP":
            case "XMR":
                return 0.5m;
            case "GAT":
                return 8m;
            case "BMC":
            case "DENT":
            case "DRG":
            case "ETN":
            case "FDX":
            case "FLIXX":
            case "FSN":
            case "GZE":
            case "IND":
            case "IXT":
            case "MTN":
            case "OAX":
            case "ONG":
            case "QASH":
            case "RKT":
            case "SAL":
            case "SER":
            case "SNIP":
            case "SPHTX":
            case "STAC":
            case "STORJ":
            case "STU":
            case "TPT":
            case "TRX":
            case "VZT":
                return 1m;
            default:
                throw new Exception("Unsupported symbol");
        }

UPD: сам switch и так находится в приватном методе, который возвращает decimal и принимает string. Задача в том, чтобы этот код (логику, если так можно "это" назвать) как-то упростить (уменьшить), если это возможно, но не словарем.

13
  • 2
    Словарь это единственное что в голову приходит. Такое непотребство точно не стоит оставлять.
    – srvr4vr
    2 апр 2018 в 14:18
  • 3
    А кстати, со словарем это у вас хорошая идея.
    – Bulson
    2 апр 2018 в 14:20
  • 3
    @Bulson, я бы добавил пустые строки после return (чтобы зрительно выделить блоки), а саму конструкцию упаковал бы в приватный метод и оставил в покое.
    – free_ze
    2 апр 2018 в 14:34
  • 3
    Мне нравится и словарь и вынести в отдельную функцию, на самом деле сам код плох если он будет частью какой-то функции и внутри неё идти inline, если же он будет в отдельной своей функции, то почему бы и нет, кстати инициализатор словаря будет не более красиво выглядеть, а может даже и менее, там придётся дубликаты значений заводить, а если значение поменяется все дубликаты придётся поменять. Конечно можно их константами задать, перед определением словаря, но это ещё больше усложнит код, и сделает менее понятным. Так что я за отдельную функцию. Но мой лайк стоит и на словаре тоже. 2 апр 2018 в 14:59
  • 2
    И кстати с точки зрения инкапсуляции и красоты кода не особо будет важна реализация, главное что на поверхности будет просто функция принимающая строку а возвращающая число. 2 апр 2018 в 15:01

2 ответа 2

7

Хардкод значений и курсов в коде - это нехорошо.

Положите все это в CSV / JSON / базу данных. Т.е. в любое хранилище, кроме кода. Поднимайте из хранилища по первому обращению и десериализуйте в Dictionary<string, decimal>.

3

В switch – данные. Данные, по возможности, отделяют от логики программы. Раз уж данных много, можно хранить их в файлах, в настройках программы или в базе. Вот подходящий ответКак хранить данные в программе?

Ваш ответ

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

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