重构是指在不改变代码行为的情况下，提高代码的可读性、可维护性和效率的过程。本文介绍8种重构C#代码的基本方法。
1. 删除冗余代码
重构前
List<int> userIds = new List<int>();
userIds.AddRange(output.Select(s => s.UserId).Distinct().ToList());
userIds = userIds.Distinct().ToList();
重构后
var userIds = output.Select(s => s.UserId).Distinct().ToList();
2. 使用LINQ替代循环
重构前
List<int> evenNumbers = new List<int>();
foreach (var num in numbers) {
if (num % 2 == 0) {
evenNumbers.Add(num);
}
}
重构后
var evenNumbers = numbers.Where(n => n % 2 == 0).ToList();
3. 避免重复代码(封装方法)
重构前
var segmentIDs = output.Select(s => s.SegmentID).Distinct().ToList();
if (segmentIDs.Any()) {
segments = db.CheckCode
.Where(c => segmentIDs.Contains(c.CheckCodeID))
.ToList();
}
重构后
List<int> GetDistinctIDs<T>(IEnumerable<T> source, Func<T, int?> selector) =>
source.Select(selector).Where(id => id.HasValue).Select(id => id.Value).Distinct().ToList();
4. 使用空合并和三元运算符
重构前
string name;
if (user != null && user.Name != null) {
name = user.Name;
} else {
name = "Unknown";
}
重构后
string name = user?.Name ?? "Unknown";
5. 使用var提升可读性
重构前
List<int> numbers = new List<int> { 1, 2, 3, 4 };
重构后
var numbers = new List<int> { 1, 2, 3, 4 };
6. 避免嵌套的if语句
重构前
if (user != null) {
if (user.Age > 18) {
Console.WriteLine("Adult");
}
}
重构后
if (user == null) return;
if (user.Age <= 18) return;
Console.WriteLine("Adult");
7. 使用字符串插值
重构前
string message = string.Format("Hello, {0}! You have {1} messages.", name, count);
重构后
string message = $"Hello, {name}! You have {count} messages.";
8. 使用async/await提升性能
重构前
public List<User> GetUsers() {
return db.Users.ToList();
}
重构后
public async Task<List<User>> GetUsersAsync() {
return await db.Users.ToListAsync();
}
总结
在代码重构时，应该始终遵循以下原则：
- 可读性：代码应当易于理解。
- 可维护性：让未来的开发者能够轻松修改代码。
- 性能：避免不必要的计算和冗余操作。
- DRY(不要重复自己)：提取可复用的逻辑到方法中，提高代码复用性。
译文：csharp.com/article/learn-c-sharp-refactor-code
