一、.NET 官方 + 企业通用代码规范
1. 命名规范（最核心）
遵循 PascalCase / camelCase 严格区分，禁止拼音、无意义命名。
表格
| 类型 | 规范 | 示例 |
|---|---|---|
| 类、接口、方法、属性 | PascalCase | UserService、GetUserById() |
| 私有字段 | _camelCase | _userRepository |
| 方法参数、局部变量 | camelCase | userId、userName |
| 常量 | PascalCase | const int MaxRetryCount = 3; |
| 接口 | I + PascalCase | IUserRepository |
| 泛型类型 | T + 描述 | TResult、TEntity |
2. 代码格式规范
- 缩进：4 空格
- 大括号：独占一行（.NET 官方风格）
- 一行一个语句，不堆叠代码
- 文件长度：建议 < 300 行，类 < 50 行方法
- 方法长度：建议 < 20 行（复杂业务最多 50 行）
3. 语法最佳实践
- 优先使用
var（类型明确时） - 优先使用
string.IsNullOrEmpty()/string.IsNullOrWhiteSpace() - 集合优先
IEnumerable<T>/IList<T>接口声明 - 禁止硬编码字符串、魔法数字，抽成常量 / 配置
- 异步方法必须以
Async结尾：GetUserAsync() - 异常不捕获
catch(Exception)盲目捕获，只捕获预期异常 - 日志不记录敏感数据（密码、手机号、身份证）
4. 架构与分层规范
- 控制器只做：参数校验 → 调用服务 → 返回结果
- 业务逻辑全部放在 Service / Manager
- 数据访问放在 Repository / EF Core
- 禁止在控制器写 SQL、写业务逻辑
- 依赖注入优先构造函数注入，禁止属性注入
- 领域模型与 DTO 严格分离（不直接返回 Entity）
二、.NET CodeReview 检查清单
1. 基础代码质量
- 命名清晰，能看懂业务含义，无缩写、无拼音
- 无死代码、无注释掉的代码
- 无重复代码（Copy-Paste）
- 方法职责单一，一个方法只做一件事
- 代码格式统一，无杂乱缩进
- 无编译警告（Warn 必须处理）
2. 功能与逻辑正确性
- 边界值处理（null、空字符串、0、负数、空集合）
- 异常处理合理，不吞异常、不乱抛异常
- 异步代码正确使用 await，无 async void
- 循环无性能陷阱（如循环内查库）
- 业务规则是否完整实现
3. 性能与安全
- 无 N+1 查询（循环查询）
- 大数据量分页、限流，不一次性查全表
- 输入参数校验（防注入、非法参数）
- 密码、密钥不硬编码，使用配置 / 密钥服务
- 日志不输出敏感信息
4. 可维护性
- 关键业务逻辑有注释，复杂算法必须写注释
- No过度设计、No无用抽象
- 依赖关系清晰，符合 SOLID 原则
- 单元测试覆盖核心业务逻辑
5. .NET 专项检查（ASP.NET Core / EF Core）
- DTO 与 Entity 分离，不直接暴露数据库表
- 事务正确使用，不滥用、不遗漏
- 服务生命周期正确（Scoped/Transient/Singleton）
- API 返回统一格式结构
- 路由规范、Action 命名规范
三、.NET 重构技巧
1. 快速识别坏味道（需要重构的信号）
- 方法太长（>50）
- 类太大（>500）
- 重复代码
- 嵌套过深（if 嵌套 ≥3 层）
- 长参数列表（≥5 个参数）
- 一个类做太多事
2. 常用重构技巧（C#）
1）提取方法（Extract Method）
把一段代码变成独立方法，提高可读性。
坏
if (user.Age >= 18 && user.IsVerified && !user.IsBlacklisted)
{// 一堆逻辑
}
好
if (CanAccessService(user))
{// 逻辑
}private bool CanAccessService(User user)
{return user.Age >= 18 && user.IsVerified && !user.IsBlacklisted;
}
2）提取变量（Extract Variable）
把复杂表达式命名，可读性暴增。
3）合并重复代码
相同逻辑 >2 次出现 → 抽成公共方法 / 扩展方法。
4）减少嵌套（Guard Clauses 卫语句）
坏
if (user != null)
{if (user.Id > 0){// ...}
}
好
if (user == null) return;
if (user.Id <= 0) return;
// 正常逻辑
5）长参数 → 封装成对象（DTO/Model）
// 坏
CreateUser(string name, int age, string phone, string email, string address)// 好
CreateUser(CreateUserDto dto)
6）用 DTO 隔离领域对象
不把 EF Entity 直接返回给 API，避免字段泄露、序列化异常。
7）使用扩展方法简化重复逻辑
如字符串、集合、验证逻辑。
8）拆分大类（按职责拆分）
- UserService → UserAuthService、UserInfoService、UserOrderService
9）消除魔法字符串 / 魔法数字
// 坏
if (status == 1)// 好
const int ActiveStatus = 1;
if (status == ActiveStatus)
10）用 LINQ 简化循环
让集合操作更简洁、更易读。