basic-practice/refactoring

[giscus[bot]]

basic-practice/refactoring

https://course.rs/basic-practice/refactoring.html

[lxsvn]

1

[liuyuanjun520]

喜(令)闻(人)乐(讨)见(厌)

[Lstmxx]

done

[systemX1]

"像很多国内的大学教材一样，只要列出定理和解题方法，然后留下足够的习题，就万事大吉了",hh

[uyplayer]

题海战术？

[FrankKai]

赞

[LeeeSe]

impl Config {
fn build(args: Vec) -> Result<Config, &'static str> {
if args.len() < 3 {
Err("Not enough arguments!")
}
dbg!(&args);
Ok(Config {
query: args[1].clone(),
file_path: args[2].clone(),
})
}
}
为什么这里的Err必须加return？不加会报错

[LeeeSe]

error[E0308]: mismatched types
--> src/main.rs:25:13
|
24 | / if args.len() < 3 {
25 | | Err("Not enough arguments!")
| | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected (), found enum Result
26 | | }
| |_________- expected this to be ()
|
= note: expected unit type ()
found enum Result<_, &str>
help: you might have meant to return this value
|
25 | return Err("Not enough arguments!");
| ++++++ +

[SlowSlicing]

"Err("Not enough arguments!")" 想要做的是在这一行直接返回错误，由于是在方法中间，所以你需要使用 return 明确"要返回了，下边的代码就别走了"。它不像最后的表达式 "Ok(Config {...})"，因为这里已经是方法中最后一行了，所以可以不用加 return 编译器也是可以知道的。

[SlowSlicing]

你如果不加 return，那就仅仅是创建了 Err 这个枚举，并不具有返回意义。

[duanyuluo]

你的Err这行代码没有；，所以是一个expression，而不是statement，因此Err对象将会是block的返回值。
举例：

Let e = if args.len() < 3 { Err(“….”) }

那么，e就得到了block的返回值的赋值，但你没有赋值，也就丢弃了这个Err对象，然后代码执行了dbg！代码，当然也就不会返回Err对象了。

[etscript]

rust 1.67.1

// 对 build 返回的 Result 进行处理
let config = Config::build(&args).unwrap_or_else(|err| {
println!("Problem parsing arguments: {err}");
process::exit(1);
});

这里提示：
let config = Config::build(&args).unwrap_or_else(|err| {
| _______________________________________________________^
10 | | println!("Problem parsing arguments: {err}");
11 | | process::exit(1);
12 | | });
| |^ expected struct Config, found ()

[etscript]

已解决，没有导入 use std::process;

[ghost]

用 Config::from(&args) 会更好。参考 String::new() 和 String::from("hello world") 以及 Vec::new() 和 Vec::from([1, 2])

[duanyuluo]

说的对，这确实是rust的惯用法。

[ghost]

重构后的代码就是优雅

[duanyuluo]

好的程序员，就是要懒。最懒的程序员，就必须热爱重构。否则，失控的代码会逼迫懒惰的程序员没日没夜的焦头烂额起来的：D

[duanyuluo]

其实，是一种强迫症

[NekoTarou]

impl Config {
    fn build(args: &[String]) -> Result<Config, &'static str> {
        if args.len() < 3 {
            return Err("not enough arguments");
        }

        let query = args[1].clone();
        let file_path = args[2].clone();

        Ok(Config { query, file_path })
    }
}

这里的'static是必须加的么，我不添加也不会报错呢

[ttsy1995]

有默认的声明周期，如果不加，返回值和参数的生命周期编译器认为一致的。不加任然是符合编译规则的

[zhuiye]

优雅

[kyoybs]

main.rs引用lib.rs里的东西，可以通过ns::fn_name实现，如果引用其他文件里的东西，需要使用 include!

[Endermanbugzjfc]

include 是嵌入档案，应该跟 Go 的 //go:embed 差不多

[kyoybs]

异常处理好晕，没有try.catch好理解.

[Endermanbugzjfc]

个人感觉像 Promise 的形式

[Pledoge]

太棒了，这一节把之前所学的东西基本上都串起来了。之前一直没弄懂Result类型的作用，现在联想到枚举类型一下子思路全畅通了😊

[microft11]

问个问题，println!("In file {}\n", &config.file_path); 这个宏结束后应该会释放资源，config.file_path的所有权是转移到宏上去的，然后这个变量应该就不能用了。但是可以看到run(config);中又调用了它，并且String也没有实现copy trait，为什么还能正常运行啊

[microft11]

我明白了，我把宏和函数对于所有权的影响混淆了。
宏通过模式匹配和替换代码的，在宏展开时，它通常是通过引用或复制参数的值，而不是移动参数的所有权。

[jack45j]

把run 函式放進impl裡面是否有可能會更優雅一些?
比如

pub fn run(self) -> Result<Config, Box<dyn Error>> {
    let contents = fs::read_to_string(&self.path)?; 
    println!("With text: \n{contents}");    
    
    Ok(self)
}

也可以實現鏈式調用

let config = Config::from(&args)
    .unwrap_or_else(|err| {
        println!("Problem parsing arguments: {}", err);
        process::exit(1);
    })
    .run()
    .unwrap_or_else(|err| { 
        println!("Application error: {}", err);
        process::exit(1);
    });

這兩種哪一種會是比較常規的做法呀@@"?

[Scott169]

实际项目里面似乎链式调用更多。

[sxfreesky123]

实际项目里不会为了Config实现run. 你自己读一遍这样的实现, 为一个配置实现了一个可执行的run方法, 配置完毕后,将这个配置跑起来...
有没有觉得缺了些什么? 缺了主体. 也就是面向对象里的Object.
所以run是不会放在Config里实现的. 这个工具是一个文件搜索工具. run是工具的执行器,Config是对工具做配置. 主体是工具,不是Config.

[CAESIUS-TIM]

写测试是喜(令)闻(人)乐(讨)见(厌)的，但是看着 coverage 慢慢涨起来还是蛮快乐的。
有什么比较主流的 code coverage？比较简单好用的知道一个 tarpaulin。llvm/clang 的太复杂了😥。

[daoyee]

main还可以简化，代码如下：

fn main() -> Result<(), Box<dyn Error>>{
    let args: Vec<String> = env::args().collect();
    
    let config= Config::build(&args)?; 

    let _ = minigrep::run(config)?;

    Ok(())
    
}

[zengleyi]

你这个简化是不可取的，根本没有处理错误，让错误冒到最外层了。

[Endermanbugzjfc]

我觉得还行
反正 minigrep::run() 四自己定义的，把错误信息写得够人性，冒到外层也不会有太大影响

[binchen0802]

"处理返回的 Result" 卡住我半天。。。
当直接输入process按回车的时候，系统会优先引入std::os::unix::process...
然后build config的地方就会出现编译错误：
mismatched types
expected Config, found ()rustcClick for full compiler diagnostic
Problem parsing arguments: {err}

这提示看不出来跟错误的use有任何关系。。。

[sixqa]

心累😭😭😭😭😭😭

[clarkzhang56]

深入浅出，讲的好

[syNack0]

// in main.rs
fn main() {
    let args: Vec<String> = env::args().collect();

    let (query, file_path) = parse_config(&args);

    // --省略--
}

fn parse_config(args: &[String]) -> (&str, &str) {
    let query = &args[1];
    let file_path = &args[2];

    (query, file_path)
}

为啥函数调用&Vec<String>能自动转成&[String]，返回值里&String又能直接转成&str ? 谁解惑一下