c# - 阻塞 GUI 的网络线程

Question

我正在研究 C# 和 WPF。

我正在开发一个应用程序来处理 LAN 上的数据传输。

特别是，对于每个传输显示进度条，如果是下载则为绿色，如果是上传则为红色。

最初我做了一些模拟传输的虚构尝试(网络线程被停用)。因此，条形图以编程方式填充，我检查了 GUI 的性能，一切正常。特别是我可以选择栏并显示上下文菜单。

一段时间后，我们开发了网络的一部分，主要执行 TCP 读写。对于每个 block (或 block 组)执行一个步骤，以便进度条前进。

block 的数量因文件而异，但如果有超过 100 个 block ，则执行的步骤数始终为 100，这样每次传输总是执行 100 个或更少的执行步骤。

问题是，当真正的传输开始时，一切都变慢了，当我四处拖动窗口并且 GUI 不再响应时，我什至无法点击栏来显示上下文菜单。但是，条形图的进度有效。

我们尝试禁用栏并进行传输，GUI 响应迅速，没有问题。

可能我们从网络上破坏了 GUI 刷新。

网络有自己的线程，GUI 有自己的线程。

通过网络刷新条形图的最佳方法是什么？

我们目前使用这个，但我们不认为这是一个好方法，肯定它效率不高。

例如，我们以网络客户端代码为例:

...
// Object representing a transfer (shared between GUI and Network)
Transfer t = new Transfer(...);

int bytesRead;
var buffer = new byte[chunkSize];
while ((bytesRead = file.Read(buffer, 0, buffer.Length)) > 0)
{
    if (t.Stop)
    {
        break;
    }

    nwStream.Write(buffer, 0, bytesRead);
    PBcount++;
    if (PBcount == PBchuks)
    {
        t.PerformStep(); // Make the progress bar to advance, works, but laggy
        PBcount = 0;
    }
}
...

与PerformStep()相关的代码是Transfer共享对象的公共(public)方法:

public void PerformStep()
{
    CurrentStep = CurrentStep + 1;
    Application.Current.Dispatcher.Invoke(delegate { 
        MainWindow wnd = (MainWindow)Application.Current.MainWindow;
        wnd?.PerformStepProgressBarRefresh();
    });
}

以下是委托(delegate)刷新 ListBox 项目的 MainWindow GUI 方法的代码:

public void PerformStepProgressBarRefresh()
{
    Application.Current.Dispatcher.Invoke(delegate
    {
        TransfersXAML.Items.Refresh();
    });
}

我不知道它是否有用，但我还是把它放上来，它是与进度条相关的XAML代码:

<ListBox x:Name="TransfersXAML"
         HorizontalContentAlignment="Stretch"
         ItemsSource="{Binding Transfers}">
    <ListBox.ItemTemplate>
        <DataTemplate>
            <ProgressBar Height="30" Minimum="0"
                         Maximum="{Binding NSteps}"
                         Value="{Binding CurrentStep}"
                         Foreground="{Binding Color}" />
        </DataTemplate>
    </ListBox.ItemTemplate>
    <ListBox.ContextMenu>
        <ContextMenu>
            <MenuItem Header="Get info" Click="GetTransferInfoClick" />
            <MenuItem Header="Cancel" Click="CancelTransferClick" />
        </ContextMenu>
    </ListBox.ContextMenu>
</ListBox>

您是否知道更好、更高效的系统，以便网络线程可以某种方式更新进度条？

毕竟，我们不知道自己是否做对了事情。

---

我们可能没有提供所有可能有助于更好地理解问题的信息:发出信号以便我们可以更新问题。

Update

public class Transfer
{
    public enum Type_t
    {
        upload,
        download
    }

    public int NSteps { get; set; }
    public int CurrentStep { get; set; }
    public Type_t Type { get; set; }
    public string Color { get; set; }
    public string ID { get; set; }
    public string SenderID { get; set; }
    public string ReceiverID { get; set; }

    public Transfer(string senderID, string receiverID, Type_t _type, string _color)
    {
        SenderID = senderID;
        ReceiverID = receiverID;
        NSteps = 100;
        CurrentStep = 0;
        Type = _type;
        Color = _color;
        ID = Utils.GetHashString(senderID + receiverID + Utils.GetCurrentTimestamp());
    }

    public void PerformStep()
    {
        CurrentStep = CurrentStep + 1;
   Application.Current.Dispatcher.BeginInvoke(DispatcherPriority.Background, new Action(() =>
        {
            MainWindow wnd = (MainWindow)Application.Current.MainWindow;
            wnd?.PerformStepProgressBarRefresh();
        }));
    }
}

Answer 1

您不需要使用 Refresh 或类似技术手动刷新 WPF 控件。你当然不需要在这里做。

您正确地将进度条 Value 属性绑定(bind)到 Transfer.CurrentStep，但您没有实现更改通知以自动通知绑定(bind)有关更改的信息。为此，您需要在 Transfer 类上实现 INotifyPropertyChanged 接口(interface)，例如:

public class Transfer : INotifyPropertyChanged {
    private int _currentStep;
    public int CurrentStep
    {
        get { return _currentStep; }
        set
        {
            if (_currentStep != value) {
                _currentStep = value;
                OnPropertyChanged();
            }
        }
    }

    public event PropertyChangedEventHandler PropertyChanged;

    protected virtual void OnPropertyChanged([CallerMemberName] string propertyName = null) {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
}

然后，删除所有 PerformStep() 和 PerformStepProgressBarRefresh - 您不再需要它们，也不需要手动执行 Dispatcher.Invoke 或 BeginInvoke。在您的网络代码中执行:

t.CurrentStep++;

就这些。