Умножая фракции безопасным способом C#

Question

Я пытаюсь разработать, как вычислить общую стоимость ремонта автомобиля, численно безопасным способом, избегая ошибок округления. Я получаю время, потраченное на работу, а затем умножаю его на постоянную стоимость рабочей силы, чтобы получить правильную сумму затрат на рабочую силу, однако она не выясняет, как это должно быть. Вот мой пример, когда на работу было потрачено 20 минут.

Это явно работает неправильно, как треть 30 £ составляет £ 10, так как я избежать ошибки округления я получаю?

Вот как я получаю общее время.

TimeSpan totalTime = TimeSpan.Zero; 

    foreach (DataRow timeEntry in dhJob.DataStore.Tables[jobTimeCollectionName].Rows) 
    { 
     DateTime start = Convert.ToDateTime(timeEntry["jobtimestart"]); 
     DateTime end = Convert.ToDateTime(timeEntry["jobtimeend"]); 
     totalTime += (end - start); 
    } 

    tb_labourtime.Text = Convert.ToString(Math.Round(totalTime.TotalHours, 2)); 

    tb_labourtotal.Text = (Convert.ToDouble(tb_labourtime.Text) * Convert.ToInt32(tb_labourrate.Text)).ToString(); 

Любая помощь/совет приветствуются.

Answer 1

Во-первых, вы преобразовываете totalTime в строковое представление, которое имеет только две цифры точности, что не будет очень точным.

Во-вторых, при проведении финансовых расчетов вы должны использовать тип decimal, а не тип double, что даст вам большую точность (хотя оно все еще не полностью точно).

Первое, что нужно сделать, это использовать totalTime для расчета общей суммы заработной платы, а не использовать преобразованное значение строки:

tb_labourtotal.Text = (totalTime.TotalHours * Convert.ToInt32(tb_labourrate.Text)).ToString();