我刚刚在pull请求中看到了以下更改:
- .ok_or(Error::new(ErrorKind::Other, "Decode error"));
+ .ok_or_else(|| Error::new(ErrorKind::Other, "Decode error"));
我所知道的唯一区别是:
ok_or中，我们已经创建了Error×Error::new，并将其传递到适配器中.ok_or_else中，我们通过了一个闭包，该闭包将产生这样一个值，但如果Option中有Some个数据，则可能不会调用该闭包.我错过什么了吗？
使用..._or_else个方法中的ok_or_else个或any个的主要原因是避免在不需要的时候执行函数.在Option::ok_or_else或Option::unwrap_or_else的情况下，当Option为Some时，不需要运行额外的代码.这可以使代码更快，这取决于错误情况下发生的情况
在本例中，Error::new可能执行分配，但它也可以写入标准输出、发出网络请求，或者任何一段 rust 迹代码都可以执行的操作；从外面很难分辨.通常，将这样的代码放在闭包中更安全，这样在成功 case 发生时就不必担心额外的副作用.
Clippy lints也为你准备了这个:
fn main() {
let foo = None;
foo.unwrap_or("hello".to_string());
}
warning: use of `unwrap_or` followed by a function call
--> src/main.rs:3:9
|
3 | foo.unwrap_or("hello".to_string());
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|| "hello".to_string())`
|
= note: `#[warn(clippy::or_fun_call)]` on by default
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#or_fun_call