JustPlay-Max / PluginYG

Creative Commons Zero v1.0 Universal
59 stars 10 forks source link

Метод TimeTypeConvertStatic #3

Open TheSsora opened 1 year ago

TheSsora commented 1 year ago

Данная функция не правильно округляет десятичное значение времени. Например возьмем число: 123321 -> 02:03.321 Функция конвертации же при округлении до 2, сделает так: 123321 -> 02:03.21 А при округлении до 1, так: 123321 -> 02:03.1 При округлении до 4, вообще появляется новое число: 123321 -> 02:03.3321 Как будто округление идет с конца. Возможно я не прав)

Предлагаю вообще упростить эту функцию, и конвертировать время как-то так: TimeSpan.FromMilliseconds(score).ToString("mm':'ss'.'ff");

JustPlay-Max commented 1 year ago

Это старый код. Чтобы мне убедиться, что с ним есть проблемы, мне нужно время на доработку и тестирование. Не хочу что то нарушить, если уделю этому недостаточно внимания. Сейчас я сфокусирован на других задачах. Если вы переделали метод и уверены, что он работает лучше, можете выслать мне код. Тогда я смогу быстрее исправить. Спасибо!)  

Вторник, 12 сентября 2023, 20:44 +07:00 от Artem @.***>:     Данная функция не правильно округляет десятичное значение времени. Например возьмем число: 123321 -> 02:03.321 Функция конвертации же при округлении до 2, сделает так: 123321 -> 02:03.21 А при округлении до 1, так: 123321 -> 02:03.1 При округлении до 4, вообще появляется новое число: 123321 -> 02:03.3321 Как будто округление идет с конца. Возможно я не прав) Предлагаю вообще упростить эту функцию, и конвертировать время как-то так: TimeSpan.FromMilliseconds(score).ToString("mm':'ss'.'ff"); — Reply to this email directly, view it on GitHub , or unsubscribe . You are receiving this because you are subscribed to this thread. Message ID: <JustPlay-Max/PluginYG/issues/3 @ github . com>     С уважением, Максим Борнысов