Как переделать скорость скачивания?

Question

[Сергей Анонимус]

Не получается переделать скорость скачивания в Class.
Приложение просто падает с ошибкой...
Использую данную махинацию:

static readonly string[] SizeSuffixes =
                  { "bytes", "KB", "MB", "GB", "TB", "PB", "EB", "ZB", "YB" };

        static string SizeSuffix(Int64 value, int decimalPlaces = 1)
        {
            if (value < 0) { return "-" + SizeSuffix(-value, decimalPlaces); }

            int i = 0;
            decimal dValue = (decimal)value;
            while (Math.Round(dValue, decimalPlaces) >= 1000)
            {
                dValue /= 1024;
                i++;
            }

            return string.Format("{0:n" + decimalPlaces + "} {1}//s", dValue, SizeSuffixes[i]);
        }

Дальше у меня есть вот такая штука:

private string speed;
private void Wc_DownloadProgressChanged(object sender, System.Net.DownloadProgressChangedEventArgs e)
        {
            if (prevDownloadedSize == 0)
            {
                prevDownloadedSize = e.BytesReceived / 1024;
            }

            if (stopwatch.Elapsed.TotalSeconds >= 1)
            {
                var dKBytes = (e.BytesReceived / 1024) - prevDownloadedSize;
                currentDownloadedSize += dKBytes;
                speed = String.Format("{0}", SizeSuffix(Convert.ToInt64(dKBytes / stopwatch.Elapsed.TotalSeconds)));
                prevDownloadedSize = 0L;
                OnProgressChanged();
                stopwatch.Restart();
            }
        }
private void OnProgressChanged()
        {
            Events.DownloadProgressChangedEventArgs eventInfo;
            eventInfo = new Events.DownloadProgressChangedEventArgs() { NewPercentage = GetPercentage(), 
                Speed = Math.Round(double.Parse(speed)), DownloadedSize = currentDownloadedSize, AllSize = downloadProvider.Size };
           
            ProgressChanged?.Invoke(this, eventInfo);
        }

Comments

[cicatrix]

А где ошибка-то возникает?

[Сергей Анонимус]

cicatrix, вот тут

private void OnProgressChanged()
        {
            Events.DownloadProgressChangedEventArgs eventInfo;
            eventInfo = new Events.DownloadProgressChangedEventArgs() { NewPercentage = GetPercentage(), 
                Speed = Math.Round(double.Parse(speed)), DownloadedSize = currentDownloadedSize, AllSize = downloadProvider.Size };
           
            ProgressChanged?.Invoke(this, eventInfo);
        }

[twobomb]

Что за ошибка, какое исключение, текст, при каких условиях нужна информация

[Сергей Анонимус]

twobomb, Вот, самое обычное условие, скачиваем файлы.
System.FormatException: "Входная строка имела неверный формат."

Изначально это исключение было создано в этом стеке вызовов: 
    [Внешний код]
    Launcher.Game.Downloader.OnProgressChanged() в Downloader.cs
    Launcher.Game.Downloader.Wc_DownloadProgressChanged(object, System.Net.DownloadProgressChangedEventArgs) в Downloader.cs
    [Внешний код]

[cicatrix]

Сергей Анонимус, Такая ошибка могла возникнуть в твоём коде только здесь:

double.Parse(speed)

Поставь на неё точку останова и посмотри значение speed.

Если я всё правильно понял, оно устанавливается здесь:

speed = String.Format("{0}", SizeSuffix(Convert.ToInt64(dKBytes / stopwatch.Elapsed.TotalSeconds)));

И сильно подозреваю, что у тебя там запятая вместо точки. Локаль русская в системе - используется разделитель целой и дробной части - запятая, а надо точку или для конверсии указывай кульутру.

Answer 1

[Владимир Коротенко]

Это GUI программа?

Comments

[Сергей Анонимус]

da

[Владимир Коротенко]

Сергей Анонимус, проверяйте это свойство https://docs.microsoft.com/ru-ru/dotnet/api/system... и присваивайте значение в гуи потоке

[Сергей Анонимус]

Владимир Коротенко, Это и есть решение?

[Владимир Коротенко]

Сергей Анонимус, ну да. Нельзя дергать интерфейс из другого потока. Это не только windows

[Сергей Анонимус]

Владимир Коротенко, И как тогда быть?

[Владимир Коротенко]

Сергей Анонимус, почитайте статью, в подвале ссылка как делать правильно.

[Сергей Анонимус]

Владимир Коротенко, Не получается сделать правильно, по этому я и задал вопрос.

[Владимир Коротенко]

Сергей Анонимус, пробуйте как описано в этой статье https://docs.microsoft.com/ru-ru/dotnet/desktop/wi...

[Сергей Анонимус]

Владимир Коротенко, Так, с этим разобрался, но, теперь не отображается

static readonly string[] SizeSuffixes =
                  { "bytes", "KB", "MB", "GB", "TB", "PB", "EB", "ZB", "YB" };

Т.е всё показано в таком вот виде: 5.1
А должно быть: 5.1 MB/s

[Владимир Коротенко]

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