C# 如何在启动新线程前检查指定名称的线程是否正在运行
现有代码问题分析
你的现有实现存在3个核心问题:
- 判断逻辑完全失效:你每次都新实例化一个Thread对象,未启动的线程
IsAlive属性默认就是false,该判断完全没有检查当前已经在运行的同名线程 - 存在循环闭包捕获陷阱:直接把循环迭代变量传入Thread的委托,会因为延迟捕获特性,导致所有线程最终拿到的都是最后一次循环的acct、pass、serv值
- 存在冗余逻辑:参数校验阶段重复写了两次
acct != ""判断
推荐修复方案
自行维护一个运行中账号的哈希集合做判断,性能高、兼容性好,实现代码如下:
首先在类中定义两个静态成员:
// 存储正在运行的账号名 private static readonly HashSet<string> _runningAccounts = new HashSet<string>(); // 锁对象保证并发判断的原子性 private static readonly object _lockObj = new object();
循环启动逻辑修改为:
for (int i = 0; i < result.Length; i++) { string[] data = result[i].Split(';'); string acct = data[0]; string pass = data[1]; string serv = data[2]; // 校验参数非空,用IsNullOrWhiteSpace兼容空字符串、空格、null三种情况 if (!string.IsNullOrWhiteSpace(acct) && !string.IsNullOrWhiteSpace(pass) && !string.IsNullOrWhiteSpace(serv)) { lock (_lockObj) { // 账号已经在运行则跳过 if (_runningAccounts.Contains(acct)) { continue; } // 先加入运行集合再启动线程,避免并发场景下的判断漏洞 _runningAccounts.Add(acct); } // 生成当前循环的变量副本,解决闭包捕获问题 var currAcct = acct; var currPass = pass; var currServ = serv; var t = new Thread(() => { try { StartAccountMonitoring(currAcct, currPass, currServ); } finally { // 线程执行结束(含异常退出)后移除账号,避免后续无法启动同账号线程 lock (_lockObj) { _runningAccounts.Remove(currAcct); } } }); t.Name = currAcct; t.Start(); } }
方案优势
- 不依赖.NET内部实现,所有版本都兼容,判断性能比遍历所有线程高一个数量级
- 加锁保证了并发场景下的判断原子性,不会出现两个线程同时判断同一个账号不存在、重复启动的问题
- 局部变量副本彻底解决循环闭包捕获的问题,每个线程拿到的参数都是对应循环次的正确值
- finally块保证无论线程正常执行还是抛出异常退出,都会释放对应账号的运行标记
内容的提问来源于stack exchange,提问作者M_Arad
相关产品推荐
相关产品推荐

