嗨，我来帮你理清这个问题～首先得澄清一个小误区：你代码里的fileTypes其实是类的私有成员变量，并不是严格意义上的全局变量（全局变量是那种整个程序所有模块都能直接访问的变量），这两者的区别很关键，很多人会把这两个概念混在一起，才会有“全局变量不好”的困惑。

先说说你的具体场景：
你需要在按钮多次点击时往同一个列表里添加内容，如果把List<string> fileTypes放在btnManualBackup_Click方法里，每次点击都会新建一个空列表，之前添加的内容都会丢失，这显然不符合你的需求。这种情况下，把列表作为类的成员变量是完全合理的选择，不属于“坏实践”。

那什么时候适合用这类成员变量（而非局部变量）？

• 当类的多个方法/事件处理程序需要共享同一个状态时（比如你的Form1里可能还有其他按钮需要读取或修改这个文件类型列表）
• 当这个状态是属于当前类实例的“固有属性”时（比如这个Form1窗口需要维护自己的备份文件类型集合）

而大家常说的“全局变量不好”，指的是那种无限制的全局变量（比如public static的全局变量）——这类变量会让代码耦合度变高，任何地方都能修改它，后期排查问题时很难追踪谁改了这个变量的值，维护起来非常麻烦。但你的fileTypes是private的，只有Form1内部能访问，这种封装是安全的，完全没问题。

最后说代码摆放的规范：

• 最佳实践是把类的成员变量放在类的顶部，靠近类的声明位置。这样其他阅读代码的人打开文件就能一眼看到这个类维护了哪些状态，逻辑更清晰。
• 如果你遇到非常庞大的类，成员变量特别多，也可以按功能分组摆放，但对于普通的WinForm窗体类来说，放在顶部是最直观的选择。

给你修正后的代码示例（注意C#里List的添加方法是大写的Add）：

public partial class Form1 : Form
{
    // 类成员变量放在顶部，清晰展示类的状态
    private List<string> fileTypes = new List<string>();

    private void btnManualBackup_Click(object sender, EventArgs e)
    {
        fileTypes.Add("add stuff");
    }
}

备注：内容来源于stack exchange，提问作者jacob7140