你写的 return null 正确吗？
上次《 你写的try…catch真的有必要吗 》引起了很多朋友的讨论。本次我在code review又发现了一个问题，那就是有人有意无意的写出了return null这样的代码，例如：
publicUser GetUser(Guid userId) { if(/*for some reason*/) returnnull; returnDB.GetByUserId(userId); } 这样的写法有木有问题？
在我看来没有充分的理由不应该返回null，因为方法的使用者并不知道在何种条件下会得到null，从而导致臭名昭著的NullReferenceException异常。
如果由于其他原因导致无法得到一个User，我们应该抛出一个明确的异常供上层服务决定如何处理：
publicUser GetUser(stringuserName,stringpassword) { if(/*for some reason*/) returnnewSpecificException("can't get this user because ...."); returnDB.GetByUserId(userId); } 在我读过的开源项目中我几乎没有见到过return null的写法。能让我一时想到的两个linq方法FirstOrDefault()和LastOrDefault()，这两个方法通过命名的方式提醒使用者该方法会返回null。
说到FirstOrDefault()方法让我想起了很多人容易犯的另一个错误：
publicUser GetUserById(Guid userId) { returnlist.FirstOrDefault(x=>x.UserId==userId); } 在确认数据库中该userId必须对应一个User的情况下使用FirstOrDefault()方法， 此种场景我会建议你果断使用Single()方法。因为此时使用FirstOrDefault()会隐藏bug。 你期望该userId必须得到一个User，如果Single()方法抛出异常则说明有bug出现，并且让你在第一时间发现该bug。
F#为了减少null类型的使用引入了option类型，在将option用作函数的返回类型时，如果没有对未定义的类型做处理，编译器会报错。
let invalidInt = None match invalidInt with | Some x -> printfn "the valid value is %A"x | None -> printfn "the value is None"
如果此处的模式匹配忘记编写None->分支，编译器将会报错，从而提醒你必须处理invalidInt值为None时的逻辑。但是在C#中使用null类型，编译器给予不了我们帮助，所以我们应该避免return null这样的代码，你觉得呢？
刚才搜了一下stackoverflow，发现一篇很有意思的讨论 Should a retrieval method return ‘null’ or throw an exception when it can’t produce the return value? 我觉得里面的回答比较准确。
正文到此结束
热门推荐
相关文章
近期评论
-
文章和留言都翻到11页了 没有OOM
-
-
朋友，翻页到11页，及以后，会出现OOM，无法访问
-
-
搞个gitee的项目
-
-
版本号是多少，你可以下载哪个代码仓库，jdk选1.8 直接跑就行
-
-
-
Loading...
![[HBLOG]公众号](https://www.liuhaihua.cn/img/qrcode_gzh.jpg)
