WinForm闪记卡应用按钮需点击两次才切换单词问题及代码优化咨询

现有代码存在的问题 #

• 跨线程操作控件违规：Winform控件属性只能在创建它的UI线程修改，单独开线程直接操作englishWord、answerBox等控件属性属于非法操作，即使当前没有报错，也随时可能触发跨线程访问异常，稳定性极差。
• 业务逻辑顺序颠倒：当前代码先加载下标对应单词到界面，再校验用户输入的答案，本质上校验的是上一个单词的答案，这也是之前需要点击两次才跳词、提示滞后的核心原因。
• 无管控的多线程创建：每次点击按钮都新建一个Thread，多个线程同时运行时会争抢element全局变量，出现下标错乱、提示异常的竞态问题。
• 数据存储不合理：单词和释义分开存在两个独立数组，一旦后续维护时两个数组长度不一致，会直接触发数组越界崩溃。
• 缺少基础交互优化：没有输入空校验，答对后不会自动清空输入框，用户体验差。

优化方案 #

• 移除不必要的多线程逻辑：当前业务逻辑没有耗时操作，完全不需要开新线程，所有操作直接放在按钮点击事件的UI线程执行即可。
• 调整逻辑执行顺序：先校验当前单词的用户输入 → 展示校验结果 → 下标自增 → 加载下一个单词，完全匹配用户操作路径。
• 优化数据存储结构：用Dictionary<string, string>存储单词和对应释义，保证两者一一对应，避免数组越界风险。
• 补充基础交互逻辑：新增空输入校验，答对后自动清空输入框，优化下标循环逻辑。

优化后参考代码 #

public partial class EnglishPolishScreen : Form
{
    // 用键值对存储单词和对应释义，避免两个数组不同步的问题
    private readonly Dictionary<string, string> _wordPairs = new Dictionary<string, string>()
    {
        {"word1", "slowo1"},
        {"word2", "slowo2"},
        {"word3", "slowo3"},
        {"word4", "slowo4"}
    };
    // 存储单词列表方便遍历
    private readonly List<string> _words = new List<string>();
    private int _currentIndex = 0;

    public EnglishPolishScreen()
    {
        InitializeComponent();
        // 初始化单词列表
        _words = _wordPairs.Keys.ToList();
    }

    private void CloseAppEvent(object sender, FormClosingEventArgs e)
    {
        Application.Exit();
    }

    private void button1_Click(object sender, EventArgs e)
    {
        // 先做输入空校验
        if (string.IsNullOrWhiteSpace(answerBox.Text))
        {
            resultLabel.Text = "请输入翻译内容";
            return;
        }

        // 校验当前单词的答案
        var currentWord = _words[_currentIndex];
        if (answerBox.Text.Trim().Equals(_wordPairs[currentWord]))
        {
            resultLabel.Text = "Good";
            // 答对后下标+1，取模实现循环
            _currentIndex = (_currentIndex + 1) % _words.Count;
            // 清空输入框
            answerBox.Clear();
            // 加载下一个单词
            englishWord.Text = _words[_currentIndex];
            counterLabel.Text = $"{_currentIndex + 1} of {_words.Count}";
        }
        else
        {
            resultLabel.Text = "Bad";
        }
    }

    private void EnglishPolishScreen_Load(object sender, EventArgs e)
    {
        // 初始化加载第一个单词
        englishWord.Text = _words[_currentIndex];
        counterLabel.Text = $"{_currentIndex + 1} of {_words.Count}";
    }
}

内容的提问来源于stack exchange，提问作者Rop 3K