核心内容摘要
撸撸馆：不只是放松，更是身心的深度疗愈之旅
C# 是一门成熟而强大的语言语法友好、工具链完善按理说“很难写出垃圾的代码”。
但现实恰恰相反越是熟练的开发者越容易在惯性和时间压力下踩坑。
这些错误往往不会立刻导致程序崩溃而是以更隐蔽的方式出现性能缓慢下降、代码越来越难维护、线上问题难以定位最终在某个关键时刻集中爆发。
下面这 10 个错误很多人都“写过、用过、踩过”。
滥用 async/await却忽视它的真实成本在没有任何异步操作的方法中使用async/await是一个非常常见的误区public async Taskint GetCountAsync() { return 10; // 没有任何 await }这种写法会让编译器生成状态机带来额外的内存分配和调用开销更重要的是它会误导后续维护者让人以为这里存在 I/O 或异步行为。
正确做法是只有在方法内部真的存在异步操作时才使用 async/await。
对于纯同步逻辑直接返回一个已完成的任务即可public Taskint GetCountAsync() { return Task.FromResult(
; }
忽视 IDisposable错误相信“GC 会帮我处理”文件流、数据库连接、Socket 等资源如果不及时释放问题往往不会立刻显现但迟早会出事文件被锁死、连接池耗尽、内存压力异常。
var stream new FileStream(path, FileMode.Open); // 使用完未释放“垃圾回收器会处理”的想法非常危险。
GC 只负责托管内存并不保证非托管资源能被及时释放。
正确做法很简单也很重要using var stream new FileStream(path, FileMode.Open); // 使用 stream
在性能敏感路径中滥用 LINQLINQ 让代码非常优雅但在高频调用或大数据量场景下它并不总是免费的var emails users .Where(u u.IsActive) .Select(u u.Email) .ToList();链式 LINQ 会创建多个枚举器和闭包对象在热点路径中这些“看不见的分配”会持续侵蚀性能。
在性能敏感位置使用显式循环反而更清晰、也更可控var emails new Liststring(); foreach (var user in users) { if (user.IsActive) emails.Add(user.Email); }LINQ 不是不能用而是要用在合适的地方。
过早抽象陷入过度工程很多项目一开始就把简单问题设计得极其复杂IUserService → IUserRepository → IUserDataProvider → IUserStorage结果是每次调试都要跨好几层修改一个逻辑要改三四个接口开发效率直线下降。
判断是否需要抽象可以问自己三个问题 这个组件未来真的可能被替换吗 当前是否已经存在多个实现 抽象是否解决了真实问题如果答案是否定的保持简单往往才是最优解。
忽略 DateTime 的时区与夏令时陷阱随手一个DateTime.Now在分布式系统中可能埋下巨雷var expiresAt DateTime.Now.AddDays(
;不同服务器的本地时间、时区配置、夏令时切换都可能导致逻辑不一致。
更安全的选择是统一使用 UTC或者直接使用DateTimeOffsetvar expiresAt DateTimeOffset.UtcNow.AddDays(
;时间相关的 Bug往往最难复现也最难排查。
过度宽泛地捕获 Exception下面这种代码看起来“很稳”实际上风险极高try { DoSomething(); } catch (Exception) { // 什么都不做 }它会吞掉所有异常包括你根本处理不了的异常类型系统可能在错误状态下继续运行造成更严重的后果。
正确做法是只捕获你能处理的异常并在必要时重新抛出try { DoSomething(); } catch (IOException ex) { _logger.LogError(ex, 文件处理失败); throw; }一句话原则如果你不知道怎么处理这个异常那就不要捕获它。
混淆值类型与引用类型的语义很多人对struct和class的差异理解不够深入public struct Counter { public int Value; } var c1 new Counter(); var c2 c1; c
Value 10; // c
Value 仍然是 0结构体是值类型赋值时会完整复制。
这种行为在复杂场景中非常容易引发隐蔽 Bug。
一般建议是默认使用 class只有在明确需要性能优化、并且数据结构足够简单时才考虑 struct。
日志没有上下文等于没打日志下面这样的日志在真实排查问题时几乎毫无价值_logger.LogError(出错了);你不知道是谁出的错也不知道出在哪一步。
真正有用的日志必须包含上下文信息_logger.LogError(ex, 订单处理失败OrderId{OrderId}, orderId);请记住一句话日志不是给现在的你看的而是给未来凌晨三点值班的你看的。
在 ASP.NET Core 中错误使用 Task.Run不少人会在控制器里写出这样的代码await Task.Run(() SaveToDatabase());这通常是一个设计错误。
Web 应用的主要瓶颈是 I/O而不是 CPU。
数据库、HTTP、文件操作本身就有异步 API用Task.Run只会白白浪费线程。
如果你确实有 CPU 密集型任务应该考虑后台服务或独立计算服务而不是塞进请求管道。
忽视 CancellationToken让任务“无法被取消”很多异步方法签名中都有CancellationToken但实际却被完全忽略public async Task ProcessAsync() { await Task.Delay(
; }当请求已被客户端取消服务器仍然继续执行这些任务纯属资源浪费。
更合理的做法是public async Task ProcessAsync(CancellationToken token) { await Task.Delay(5000, token); }能取消的任务才是对系统友好的任务。
结语这些错误之所以常见并不是因为它们“太低级”而是因为它们大多能正常运行问题却在系统演进中被不断放大。
写 C# 写到最后拼的从来不是“会不会用”而是“知不知道什么时候不该用”。
大家遇到过什么错误欢迎留言讨论!