Что лучше изменить и как?

Question

[Mister_krid]

Всем хэлоу. Есть такая задачка:
-Пользователь должен ввести исходные данные с консоли — три числа через пробел: исходную сумму, процентную ставку (в процентах) и срок вклада в месяцах.
-Программа должна вывести накопившуюся сумму на момент окончания вклада.
-Вот порядок действий:
Создайте новый проект с типом Console Application.
В методе Main напишите ввод с помощью Console.ReadLine() и вывод с помощью Console.WriteLine().
Все вычисление вынесите во вспомогательный метод Calculate. Код этого метода и нужно сдать в этой задаче.
и все это без циклов!

static void Main(string[] args)
        {
            var input = Console.ReadLine();        
            Console.WriteLine(Calculate(input));
        }

       
        static double Calculate(string userInput)
        {
            string[] arrayUserInput = userInput.Split(' ');                    
            if (int.Parse(arrayUserInput[2]) == 0)
            {               
                return double.Parse(arrayUserInput[0]);
            }
            else
            {
                var outputAmount = (Convert.ToDouble(arrayUserInput[0], CultureInfo.InvariantCulture) / 100) * (Convert.ToDouble(arrayUserInput[1]) / 12) + Convert.ToDouble(arrayUserInput[0], CultureInfo.InvariantCulture);
                arrayUserInput[0] = Convert.ToString(outputAmount);
                arrayUserInput[2] = Convert.ToString(Convert.ToInt32(arrayUserInput[2]) - 1);
                return Calculate(string.Join(" ", arrayUserInput));            
            }

        }

Можете посоветовать что лучше не делать и как правильно поступать при такой задаче. Крч на критику код кидаю

Answer 1

[Дмитрий Павлов]

Здесь
var outputAmount = (Convert.ToDouble(arrayUserInput[0], CultureInfo.InvariantCulture) / 100) * (Convert.ToDouble(arrayUserInput[1]) / 12) + Convert.ToDouble(arrayUserInput[0], CultureInfo.InvariantCulture);

посоветую создать три переменные double в них соответственно сохранить:

исходную сумму, процентную ставку и срок вклада в месяцах.

это может улучшить понимание кода.

Comments

[Mister_krid]

Дмитрий Павлов Изначально я так и делал, но я делал рекурсией и каждый раз заходя в метод он бы обновлял данные переменных на изначальные поэтому вышло так.

Answer 2

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

Я бы ещё заменил double на decimal - всё-таки деньги считаем.
А Convert заменил на TryParse.
Парсинг лучше сделать в Main, а в метод Calculate передавать три числа.
Тогда не придётся делать постоянно Split и Join
В остальном всё хорошо

Comments

[Дмитрий Павлов]

А Convert заменил на TryParse.

Может, просто Parse.

[Mister_krid]

Василий Банников в задании требовалось double вернуть и сделать все в методе. В Main только ввод и вывод данных.

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

Mister_krid, парсинг данных - это тоже часть ввода. А засовывать эту финю в Calculate - нарушение SRP и лишний оверхед при рекурсивных вызовах, тк нужно постоянно собирать новую строку. Если уж сильно не хотите парсить ввод в мейне - вынесите это в ещё 1 метод.

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

Дмитрий Павлов, я считаю, что если можно обойтись без исключений - лучше обойтись без них.

[Mister_krid]

Василий Банников, говорю же это просто задание для обучения, может специально так сделано для усложнения, но я вас понял, в реальных проектах так делать не буду(если бы не задание я бы так и сделал ибо тоже считал что каждый раз создавать и разбивать строку некорректно + приходилось мудрить с вычислениями т.к. не хотел чтобы с каждый рекурсивным вызовом создавались переменные ).

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

Mister_krid, может быть, вы неправильно поняли задание?

[Mister_krid]

@vabka

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

Mister_krid, ок, тогда ладно. Но никто не запрещает добавить ещё 1 метод :)

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

Mister_krid,

spoiler

public static double Calculate(string userInput)
{
    var tokens = userInput.Split(' ');
    var sum = decimal.Parse(tokens[0]);
    var percents = decimal.Parse(tokens[1]) / 100m / 12m;
    var months = int.Parse(tokens[2]);
    return (double) CalculateInternal(sum, percents, months);
}

private static decimal CalculateInternal(decimal sum, decimal percents, int months) =>
    months == 0 
    ? sum
    : CalculateInternal(sum * percents + sum, percents, months - 1);

Думаю, можно как-нибудь решить без итерирования.