ch12-03-improving-error-handling-and-modularity.md
commit 4f2dc564851dc04b271a2260c834643dfd86c724
为了完善我们程序有四个问题需要修复,而他们都与潜在的错误和程序结构有关。第一个问题是在哪打开文件:我们使用了expect
来在打开文件失败时指定一个错误信息,不过这个错误信息只是说“文件不存在”。还有很多打开文件失败的方式,不过我们总是假设是由于缺少文件导致的。例如,文件存在但是没有打开它的权限:这时,我们就打印出了错误不符合事实的错误信息!
第二,我们不停的使用expect
,这就有点类似我们之前在不传递任何命令行参数时索引会panic!
时注意到的问题:这虽然时_可以工作_的,不过这有点没有原则性,而且整个程序中都需要他们,将错误处理都置于一处则会显得好很多。
第三个问题是main
函数现在处理两个工作:解析参数,并打开文件。对于一个小的函数来说,这不是什么大问题。然而随着程序中的main
函数不断增长,main
函数中独立的任务也会越来越多。因为一个函数拥有很多职责,它将难以理解、难以测试并难以在不破坏其他部分的情况下做出修改。
这也关系到我们的第四个问题:search
和filename
是程序中配置性的变量,而像f
和contents
则用来执行程序逻辑。随着main
函数增长,将引入更多的变量到作用域中,而当作用域中有更多的变量,将更难以追踪哪个变量用于什么目的。如果能够将配置型变量组织进一个结构就能使他们的目的更明确了。
让我们重新组成程序来解决这些问题。
这类项目组织上的问题在很多相似类型的项目中很常见,所以 Rust 社区开发出一种关注分离的组织模式。这种模式可以用来组织任何用 Rust 构建的二进制项目,所以可以证明应该更早的开始这项重构,以为我们的项目符合这个模式。这个模式看起来像这样:
将程序拆分成 main.rs 和 lib.rs 。
将命令行参数解析逻辑放入 main.rs 。
将程序逻辑放入 lib.rs 。
main
函数的工作是:
解析参数
设置所有配置性变量
调用 lib.rs 中的run
函数
如果run
返回错误则处理这个错误
好的!老实说这个模式好像还很复杂。这就是关注分离的所有内容:main.rs 负责实际的程序运行,而 lib.rs 处理所有真正的任务逻辑。让我们将程序重构成这种模式。首先,提取出一个目的只在于解析参数的函数。列表 12-4 中展示了一个新的开始,main
函数调用了一个新函数parse_config
,它仍然定义于 src/main.rs 中:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
#
fn main() {
let args: Vec<String> = env::args().collect();
let (search, filename) = parse_config(&args);
println!("Searching for {}", search);
println!("In file {}", filename);
// ...snip...
#
# let mut f = File::open(filename).expect("file not found");
#
# let mut contents = String::new();
# f.read_to_string(&mut contents).expect("something went wrong reading the file");
#
# println!("With text:\n{}", contents);
}
fn parse_config(args: &[String]) -> (&str, &str) {
let search = &args[1];
let filename = &args[2];
(search, filename)
}
Listing 12-4: Extract a parse_config
function from main
这看起来好像有点复杂,不过我们将一点一点的开展重构。在做出这些改变之后,再次运行程序并验证参数解析是否仍然正常。经常验证你的进展是一个好习惯,这样在遇到问题时就能更好地理解什么修改造成了错误。
现在我们有了一个函数了,让我们接着完善它。我们代码还能设计的更好一些:函数返回了一个元组,不过接着立刻就解构成了单独的部分。这些代码本身没有问题,不过有一个地方表明仍有改善的余地:我们调用了parse_config
方法。函数名中的config
部分也表明了返回的两个值应该是组合在一起的,因为他们都是某个配置值的一部分。
注意:一些同学将当使用符合类型更为合适的时候使用基本类型当作一种称为基本类型偏执 (primitive obsession )的反模式。
让我们引入一个结构体来存放所有的配置。列表 12-5 中展示了新增的Config
结构体定义、重构后的parse_config
和main
函数中的相关更新:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
#
fn main() {
let args: Vec<String> = env::args().collect();
let config = parse_config(&args);
println!("Searching for {}", config.search);
println!("In file {}", config.filename);
let mut f = File::open(config.filename).expect("file not found");
// ...snip...
# let mut contents = String::new();
# f.read_to_string(&mut contents).expect("something went wrong reading the file");
#
# println!("With text:\n{}", contents);
}
struct Config {
search: String,
filename: String,
}
fn parse_config(args: &[String]) -> Config {
let search = args[1].clone();
let filename = args[2].clone();
Config {
search: search,
filename: filename,
}
}
Listing 12-5: Refactoring parse_config
to return an instance of a Config
struct
parse_config
的签名现在表明它返回一个Config
值。在parse_config
的函数体中,我们之前返回了args
中String
值引用的字符串 slice,不过Config
定义为拥有两个有所有权的String
值。因为parse_config
的参数是一个String
值的 slice,Config
实例不能获取String
值的所有权:这违反了 Rust 的借用规则,因为main
函数中的args
变量拥有这些String
值并只允许parse_config
函数借用他们。
还有许多不同的方式可以处理String
的数据;现在我们使用简单但低效率的方式,在字符串 slice 上调用clone
方法。clone
调用会生成一个字符串数据的完整拷贝,而且Config
实例可以拥有它,不过这会消耗更多时间和内存来储存拷贝字符串数据的引用,不过拷贝数据让我们使我们的代码显得更加直白。
由于其运行时消耗,许多 Rustacean 之间有一个趋势是倾向于不使用clone
来解决所有权问题。在关于迭代器的第XX章中,我们将会学习如何更有效率的处理这种情况。现在,为了编写我们的程序拷贝一些字符串是没有问题。我们只进行了一次拷贝,而且文件名和要搜索的字符串都比较短。随着你对 Rust 更加熟练,将更轻松的省略这个权衡的步骤,不过现在调用clone
是完全可以接受的。
main
函数更新为将parse_config
返回的Config
实例放入变量config
中,并将分别使用search
和filename
变量的代码更新为使用Config
结构体的字段。
现在让我们考虑一下parse_config
的目的:这是一个创建Config
示例的函数。我们已经见过了一个创建实例函数的规范:像String::new
这样的new
函数。列表 12-6 中展示了将parse_config
转换为一个Config
结构体关联函数new
的代码:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
#
fn main() {
let args: Vec<String> = env::args().collect();
let config = Config::new(&args);
println!("Searching for {}", config.search);
println!("In file {}", config.filename);
// ...snip...
# let mut f = File::open(config.filename).expect("file not found");
#
# let mut contents = String::new();
# f.read_to_string(&mut contents).expect("something went wrong reading the file");
#
# println!("With text:\n{}", contents);
}
# struct Config {
# search: String,
# filename: String,
# }
#
// ...snip...
impl Config {
fn new(args: &[String]) -> Config {
let search = args[1].clone();
let filename = args[2].clone();
Config {
search: search,
filename: filename,
}
}
}
Listing 12-6: Changing parse_config
into Config::new
我们将parse_config
的名字改为new
并将其移动到impl
块中。我们也更新了main
中的调用代码。再次尝试编译并确保程序可以运行。
这是我们对这个方法最后的重构:还记得当 vector 含有少于三个项时访问索引 1 和 2 会 panic 并给出一个糟糕的错误信息的代码吗?让我们来修改它!列表 12-7 展示了如何在访问这些位置之前检查 slice 是否足够长,并使用一个更好的 panic 信息:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
#
# fn main() {
# let args: Vec<String> = env::args().collect();
#
# let config = Config::new(&args);
#
# println!("Searching for {}", config.search);
# println!("In file {}", config.filename);
#
# let mut f = File::open(config.filename).expect("file not found");
#
# let mut contents = String::new();
# f.read_to_string(&mut contents).expect("something went wrong reading the file");
#
# println!("With text:\n{}", contents);
# }
#
# struct Config {
# search: String,
# filename: String,
# }
#
# impl Config {
// ...snip...
fn new(args: &[String]) -> Config {
if args.len() < 3 {
panic!("not enough arguments");
}
let search = args[1].clone();
// ...snip...
# let filename = args[2].clone();
#
# Config {
# search: search,
# filename: filename,
# }
}
# }
Listing 12-7: Adding a check for the number of arguments
通过在new
中添加这额外的几行代码,再次尝试不带参数运行程序:
$ cargo run
Finished debug [unoptimized + debuginfo] target(s) in 0.0 secs
Running `target\debug\greprs.exe`
thread 'main' panicked at 'not enough arguments', src\main.rs:29
note: Run with `RUST_BACKTRACE=1` for a backtrace.
这样就好多了!至少有个一个符合常理的错误信息。然而,还有一堆额外的信息我们并不希望提供给用户。可以通过改变new
的签名来完善它。现在它只返回了一个Config
,所有没有办法表示创建Config
失败的情况。相反,可以如列表 12-8 所示返回一个Result
:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
# use std::process;
#
# fn main() {
# let args: Vec<String> = env::args().collect();
#
# let config = Config::new(&args).unwrap_or_else(|err| {
# println!("Problem parsing arguments: {}", err);
# process::exit(1);
# });
#
# println!("Searching for {}", config.search);
# println!("In file {}", config.filename);
#
# let mut f = File::open(config.filename).expect("file not found");
#
# let mut contents = String::new();
# f.read_to_string(&mut contents).expect("something went wrong reading the file");
#
# println!("With text:\n{}", contents);
# }
# struct Config {
# search: String,
# filename: String,
# }
#
impl Config {
fn new(args: &[String]) -> Result<Config, &'static str> {
if args.len() < 3 {
return Err("not enough arguments");
}
let search = args[1].clone();
let filename = args[2].clone();
Ok(Config {
search: search,
filename: filename,
})
}
}
Listing 12-8: Return a Result
from Config::new
现在new
函数返回一个Result
,在成功时带有一个Config
实例而在出现错误时带有一个&'static str
。回忆一下第十章“静态声明周期”中讲到&'static str
是一个字符串字面值,他也是现在我们的错误信息。
new
函数体中有两处修改:当没有足够参数时不再调用panic!
,而是返回Err
值。同时我们将Config
返回值包装进Ok
成员中。这些修改使得函数符合其新的类型签名。
现在我们需要对main
做一些修改,如列表 12-9 所示:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
// ...snip...
use std::process;
fn main() {
let args: Vec<String> = env::args().collect();
let config = Config::new(&args).unwrap_or_else(|err| {
println!("Problem parsing arguments: {}", err);
process::exit(1);
});
println!("Searching for {}", config.search);
println!("In file {}", config.filename);
// ...snip...
#
# let mut f = File::open(config.filename).expect("file not found");
#
# let mut contents = String::new();
# f.read_to_string(&mut contents).expect("something went wrong reading the file");
#
# println!("With text:\n{}", contents);
# }
#
# struct Config {
# search: String,
# filename: String,
# }
#
# impl Config {
# fn new(args: &[String]) -> Result<Config, &'static str> {
# if args.len() < 3 {
# return Err("not enough arguments");
# }
#
# let search = args[1].clone();
# let filename = args[2].clone();
#
# Ok(Config {
# search: search,
# filename: filename,
# })
# }
# }
Listing 12-9: Exiting with an error code if creating a new Config
fails
新增了一个use
行来从标准库中导入process
。在main
函数中我们将处理new
函数返回的Result
值,并在其返回Config::new
时以一种更加清楚的方式结束进程。
这里使用了一个之前没有讲到的标准库中定义的Result<T, E>
的方法:unwrap_or_else
。当Result
是Ok
时其行为类似于unwrap
:它返回Ok
内部封装的值。与unwrap
不同的是,当Result
是Err
时,它调用一个闭包 (closure ),也就是一个我们定义的作为参数传递给unwrap_or_else
的匿名函数。第XX章会更详细的介绍闭包;这里需要理解的重要部分是unwrap_or_else
会将Err
的内部值传递给闭包中位于两道竖线间的参数err
。使用unwrap_or_else
允许我们进行一些自定义的非panic!
的错误处理。
上述的错误处理其实只有两行:我们打印出了错误,接着调用了std::process::exit
。这个函数立刻停止程序的执行并将传递给它的数组作为返回码。依照惯例,零代表成功而任何其他数字表示失败。就结果来说这依然类似于列表 12-7 中的基于panic!
的错误处理,但是不再会有额外的输出了,让我们试一试:
$ cargo run
Compiling greprs v0.1.0 (file:///projects/greprs)
Finished debug [unoptimized + debuginfo] target(s) in 0.48 secs
Running `target\debug\greprs.exe`
Problem parsing arguments: not enough arguments
非常好!现在输出就友好多了。
现在重构完了参数解析部分,让我们再改进一下程序的逻辑。列表 12-10 中展示了在main
函数中调用提取出函数run
之后的代码。run
函数包含之前位于main
中的部分代码:
Filename: src/main.rs
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
# use std::process;
#
fn main() {
# let args: Vec<String> = env::args().collect();
#
# let config = Config::new(&args).unwrap_or_else(|err| {
# println!("Problem parsing arguments: {}", err);
# process::exit(1);
# });
// ...snip...
println!("Searching for {}", config.search);
println!("In file {}", config.filename);
run(config);
}
fn run(config: Config) {
let mut f = File::open(config.filename).expect("file not found");
let mut contents = String::new();
f.read_to_string(&mut contents).expect("something went wrong reading the file");
println!("With text:\n{}", contents);
}
// ...snip...
#
# struct Config {
# search: String,
# filename: String,
# }
#
# impl Config {
# fn new(args: &[String]) -> Result<Config, &'static str> {
# if args.len() < 3 {
# return Err("not enough arguments");
# }
#
# let search = args[1].clone();
# let filename = args[2].clone();
#
# Ok(Config {
# search: search,
# filename: filename,
# })
# }
# }
Listing 12-10: Extracting a run
functionality for the rest of the program logic
run
函数的内容是之前位于main
中的几行,而且run
函数获取一个Config
作为参数。现在有了一个单独的函数了,我们就可以像列表 12-8 中的Config::new
那样进行类似的改进了。列表 12-11 展示了另一个use
语句将std::error::Error
结构引入了作用域,还有使run
函数返回Result
的修改:
Filename: src/main.rs
use std::error::Error;
# use std::env;
# use std::fs::File;
# use std::io::prelude::*;
# use std::process;
// ...snip...
# fn main() {
# let args: Vec<String> = env::args().collect();
#
# let config = Config::new(&args).unwrap_or_else(|err| {
# println!("Problem parsing arguments: {}", err);
# process::exit(1);
# });
#
# println!("Searching for {}", config.search);
# println!("In file {}", config.filename);
#
# run(config);
#
# }
fn run(config: Config) -> Result<(), Box<Error>> {
let mut f = File::open(config.filename)?;
let mut contents = String::new();
f.read_to_string(&mut contents)?;
println!("With text:\n{}", contents);
Ok(())
}
#
# struct Config {
# search: String,
# filename: String,
# }
#
# impl Config {
# fn new(args: &[String]) -> Result<Config, &'static str> {
# if args.len() < 3 {
# return Err("not enough arguments");
# }
#
# let search = args[1].clone();
# let filename = args[2].clone();
#
# Ok(Config {
# search: search,
# filename: filename,
# })
# }
# }
Listing 12-11: Changing the run
function to return Result
这里有三个大的修改。第一个是现在run
函数的返回值是Result<(), Box<Error>>
类型的。之前,函数返回 unit 类型()
,现在它仍然是Ok
时的返回值。对于错误类型,我们将使用Box<Error>
。这是一个trait 对象 (trait object ),第XX章会讲到。现在可以这样理解它:Box<Error>
意味着函数返回了某个实现了Error
trait 的类型,不过并没有指定具体的返回值类型。这样就比较灵活,因为在不同的错误场景可能有不同类型的错误返回值。Box
是一个堆数据的智能指针,第YY章将会详细介绍Box
。
第二个改变是我们去掉了expect
调用并替换为第9章讲到的?
。不同于遇到错误就panic!
,这会从函数中返回错误值并让调用者来处理它。
第三个修改是现在成功时这个函数会返回一个Ok
值。因为run
函数签名中声明成功类型返回值是()
,所以需要将 unit 类型值包装进Ok
值中。Ok(())
一开始看起来有点奇怪,不过这样使用()
是表明我们调用run
只是为了它的副作用的惯用方式;它并没有返回什么有意义的值。
上述代码能够编译,不过会有一个警告:
warning: unused result which must be used, #[warn(unused_must_use)] on by default
--> src\main.rs:39:5
|
39 | run(config);
| ^^^^^^^^^^^^
Rust 尝试告诉我们忽略Result
,它有可能是一个错误值。让我们现在来处理它。我们将采用类似于列表 12-9 中处理Config::new
错误的技巧,不过还有少许不同:
Filename: src/main.rs
fn main() {
// ...snip...
println!("Searching for {}", config.search);
println!("In file {}", config.filename);
if let Err(e) = run(config) {
println!("Application error: {}", e);
process::exit(1);
}
}
fn run(config: Config) -> Result<(), Box<Error>> {
let mut f = File::open(config.filename)?;
let mut contents = String::new();
f.read_to_string(&mut contents)?;
println!("With text:\n{}", contents);
Ok(())
}
不同于unwrap_or_else
,我们使用if let
来检查run
是否返回Err
,如果是则调用process::exit(1)
。为什么呢?这个例子和Config::new
的区别有些微妙。对于Config::new
我们关心两件事:
检测出任何可能发生的错误
如果没有出现错误创建一个Config
而在这个情况下,因为run
在成功的时候返回一个()
,唯一需要担心的就是第一件事:检测错误。如果我们使用了unwrap_or_else
,则会得到()
的返回值。它并没有什么用处。
虽然两种情况下if let
和unwrap_or_else
的内容都是一样的:打印出错误并退出。
现在项目看起来好多了!还有一件我们尚未开始的工作:拆分 src/main.rs 并将一些代码放入 src/lib.rs 中。让我们现在就开始吧:将 src/main.rs 中的run
函数移动到新建的 src/lib.rs 中。还需要移动相关的use
语句和Config
的定义,以及其new
方法。现在 src/lib.rs 应该如列表 12-12 所示:
Filename: src/lib.rs
use std::error::Error;
use std::fs::File;
use std::io::prelude::*;
pub struct Config {
pub search: String,
pub filename: String,
}
impl Config {
pub fn new(args: &[String]) -> Result<Config, &'static str> {
if args.len() < 3 {
return Err("not enough arguments");
}
let search = args[1].clone();
let filename = args[2].clone();
Ok(Config {
search: search,
filename: filename,
})
}
}
pub fn run(config: Config) -> Result<(), Box<Error>>{
let mut f = File::open(config.filename)?;
let mut contents = String::new();
f.read_to_string(&mut contents)?;
println!("With text:\n{}", contents);
Ok(())
}
Listing 12-12: Moving Config
and run
into src/lib.rs
注意我们还需要使用公有的pub
:在Config
和其字段、它的new
方法和run
函数上。
现在在 src/main.rs 中,我们需要通过extern crate greprs
来引入现在位于 src/lib.rs 的代码。接着需要增加一行use greprs::Config
来引入Config
到作用域,并对run
函数加上 crate 名称前缀,如列表 12-13 所示:
Filename: src/main.rs
extern crate greprs;
use std::env;
use std::process;
use greprs::Config;
fn main() {
let args: Vec<String> = env::args().collect();
let config = Config::new(&args).unwrap_or_else(|err| {
println!("Problem parsing arguments: {}", err);
process::exit(1);
});
println!("Searching for {}", config.search);
println!("In file {}", config.filename);
if let Err(e) = greprs::run(config) {
println!("Application error: {}", e);
process::exit(1);
}
}
Listing 12-13: Bringing the greprs
crate into the scope of src/main.rs
通过这些重构,所有代码应该都能运行了。运行几次cargo run
来确保你没有破坏什么内容。好的!确实有很多的内容,不过已经为将来的成功奠定了基础。我们采用了一种更加优秀的方式来处理错误,并使得代码更模块化了一些。从现在开始几乎所有的工作都将在 src/lib.rs 中进行。
让我们利用这新创建的模块的优势来进行一些在旧代码中难以开开展的工作,他们在新代码中却很简单:编写测试!