basic-practice/refactoring

[giscus[bot]]

basic-practice/refactoring

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

[lxsvn]

1

[liuyuanjun520]

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

[Lstmxx]

done

[systemX1]

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

[uyplayer]

题海战术？

[FrankKai]

赞

[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 ? 谁解惑一下

[Blyzes]

Vec 是一个动态数组，而 &[T] 是一个切片（slice），表示对数组的一部分的引用。Rust 提供了从 &Vec 到 &[T] 的自动解引用转换。这是因为 Vec 实现了 Deref 特性，其 deref 方法返回一个切片：

impl<T> Deref for Vec<T> {
    type Target = [T];

    fn deref(&self) -> &[T] {
        // 返回切片
    }
}

因此，当你传递 &Vec 给一个期望 &[String] 的函数时，Rust 会自动调用 deref 方法，将 &Vec 转换为 &[String]。

至于为什么要转换：parse_config 函数接受一个 &[String]，这使得它可以处理任何类型的字符串集合，而不仅仅是 Vec。这提高了代码的灵活性和可重用性。

&String同理

[ZzzzHeJ]

&[String]好像是第一次见到，是&Vec的语法糖吗，还是说是一个可以隐式转换的其他类型？

[Freedempire]

&[String] 是数组切片类型，参考：
切片
数组切片

[YbrStudio]

点赞