我正在研究 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();
}));
}
}
最佳答案
您不需要使用 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++;
就这些。
关于c# - 阻塞 GUI 的网络线程,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/49939633/