1

подскажите как можно сделать код лучше? А именно мне не нравится вот этот кусок кода:

double balanceConvert = double.Parse(balanse.Replace("₽", "").Replace(".", ",").Trim());

Вот основной код:

//Получаю данные в таком формате
string balanse = "5000.00 ₽";

//Основной баланс
int baseBalance = 10000;


//конвертирую баланс, для дальнейшего сравнения
double balanceConvert = double.Parse(balanse.Replace("₽", "").Replace(".", ",").Trim());


if (baseBalance > balanceConvert)
{
    Console.WriteLine($"[{balanceConvert}] - все норм");
}
else Console.WriteLine($"[{balanceConvert}] - необходимо пополнить баланс");
6
  • 2
    double balanceConvert = double.Parse(balanse.Split(' ')[0], CultureInfo.InvariantCulture); ?
    – tym32167
    5 окт 2020 в 23:33
  • Уже лучше, спасибо)
    – srgo211
    6 окт 2020 в 1:06
  • double.Parse(currencyValue, NumberStyles.Currency, cultureInfo) 6 окт 2020 в 2:04
  • double balance - аж передёрнуло! Никогда не используйте вещественные типы: double, float для хранения денежных величин. Используйте decimal (или long в долях копеек/центов). 7 окт 2020 в 20:49
  • В воспитательных целях поставлю минус. Так вы лучше осознаете, что double для денег приводит к потере как денег, так и репутации. 7 окт 2020 в 20:50

2 ответа 2

5

Необходимо использовать NumberFormatInfo, в этом случае ваш код будет удобным для чтения, плюс закроет многие проблемы с валидацией входящих данных, так же рекомендую для манипуляций с деньгами использовать тип данных decimal. Тип double не подходит из-за его неточности.

Пример:

    var balanceString = "5000.00 ₽";
    var format = new NumberFormatInfo
    {
        CurrencySymbol = "₽",
        CurrencyDecimalDigits = 2,
        CurrencyDecimalSeparator = ".",
        NumberDecimalSeparator = ".",
        CurrencyGroupSeparator = " ",
        NumberGroupSeparator = " "
    };
    var balance = decimal.Parse(balanceString, NumberStyles.Currency, format);

Переменную format лучше вынести в приватный, статический член типа.

0

Возможно так:

string balance = "5000.00 ₽";
double balanceConvert = double.Parse(Regex.Replace(balance, "[ ₽]", ""), CultureInfo.InvariantCulture)

Ваш ответ

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

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