1、嵌套判断反例：var isValid = false;if (!string.IsNullOrEmpty(user.UserName)){ if 烫喇霰嘴(!string.IsNullOrEmpty(user.Password)) { if (!string.IsNullOrEmpty(user.Email)) { isValid = true; } }}return isValid;优化后：if (string.IsNullOrEmpty(user.UserName)) return false;if (string.IsNullOrEmpty(user.Password)) return false;if (string.IsNullOrEmpty(user.Email)) return false;return true;第一种代码是受到早期的某些思想：使用一个变量来存储返回结果。 事实上当你知道结果第一时间就应返回。
2、使用前置条件反麻质跹礼例：if (!string.IsNullOrEmpty(userName)){ if (!string.IsNullOrEmpty(password)) { //register } else { throw new ArgumentException("user password can not be empty"); }}else{ throw new ArgumentException("user name can not be empty");}优化后：if (string.IsNullOrEmpty(userName)) throw new ArgumentException("user name can not be empty");if (string.IsNullOrEmpty(password)) throw new ArgumentException("user password can not be empty");//register优化后的更符合编程的顺序，首先要满足前置条件，否则wrong！
3、参数过多，超过3个反例：public void RegisterUser(string userName, string password, string email, string phone){}优化后：public void RegisterUser(User user){}过多的参数让读者抓不住代码的意图，过多的参数会影响运行的稳定性。当参数过多你首先应该考虑把它们聚合为一个Model