C# 窗体在启动时加载 ComboBox 项

Question

public Form1()
{
    InitializeComponent();
    loadComboEmail();
}

private void loadComboEmail()
{
    string path = Directory.GetCurrentDirectory();
    string build = (path + "\\" + "email.txt");
    string[] lines = System.IO.File.ReadAllLines(build);

    comboEmail.Items.AddRange(lines);
    comboEmail.SelectedIndex=0;
}

我有一个组合框，我想加载存储在文本文件中的客户电子邮件地址。使用loadComboEmail()，我从文本文件中读取并加载Combobox。通过在表单启动时调用loadComboEmail，我是否在做一些被认为是不好的做法？如果是这样，我如何正确读取文本文件，然后将其加载到组合中，以便在加载表单时，它具有必要的数据？

Answer 1

不，这似乎很合法。您不必担心，因为这是在WinForms 中加载所有内容的方式...这将在短时间内阻塞UI 线程，但是由于您不会加载大量内容，你甚至不会注意到它。

当您开始执行大量操作和大文件时，您应该考虑使用BackgroundWorker 或Task！

无论如何，您还应该考虑使用以下代码而不是您的代码：

private void loadComboEmail()
{
    string path = System.IO.Path.GetDirectoryName(Application.ExecutablePath); //seems to be the same to me, but is safer
    string build = System.IO.Path.Combine(path, "email.txt"); //much more safer then simple string addition
    string[] lines = System.IO.File.ReadAllLines(build);

    comboEmail.Items.AddRange(lines);
    comboEmail.SelectedIndex = 0;
}

Answer 2

你可以使用：

 Task task = Task.Factory.StartNew(() =>
 {
    // Background work
    loadComboEmail();
 }).ContinueWith((t) => 
 {
    //If you need to execute something after the loading process.
 });

为了更新 UI 线程，您可以在另一个线程上进行阅读，当电子邮件列表加载时，只需更新它。

Task.Factory.StartNew(() =>
        {
            // Background work - read the file
        }).ContinueWith((t) => {
            // Update UI thread here

        }, TaskScheduler.FromCurrentSynchronizationContext());

Answer 3

使用构造函数加载不被认为是一种不好的做法。阅读 Hans Passant 关于何时应该使用窗口的加载事件的答案。 What setup code should go in Form Constructors versus Form Load event?

尽管如 cmets 中所说，您正在阻塞 UI 线程。在构造函数中不能使用关键字 await。所以你必须“开火然后忘记”。使用Load 事件时，您可以使用await，但事件处理程序是async void。所以他们没有等待，你仍然有'Fire and forget'。

public Form1()
{
    InitializeComponent();
    loadComboEmail();
}

private async Task loadComboEmail()
{
    string path = Directory.GetCurrentDirectory();
    string build = (path + "\\" + "email.txt");
    string[] lines = await Task.Run(() => File.ReadAllLines(build));

    comboEmail.Items.AddRange(lines);
    comboEmail.SelectedIndex=0;
}