basic-practice/tests

[giscus[bot]]

basic-practice/tests

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

[Lengdrich]

接着，改进型测试驱动的第二步了：

这个地方地方是一个typo吧，是不是应该是：

接着，该进行测试驱动的第二步了：

[sunface]

是的，谢谢反馈

[wying111]

在 run 函数中调用 search 函数是指把测试模块写好的search函数copy到上面调用吧？

[wying111]

忽略我的问题吧，😂，我还以为写函数得写到测试模块呢

[CamaroW]

好头像

[iscas-zac]

显而易见测试用例将成功通过，真是一个居心叵测的测试用例！

为什么会“成功通过”，又为什么“居心叵测”啊

[yunxpeng]

作者魔怔了 。。。

[wrt97vedm2bpm5cefa2po4t]

这三个步骤将在我们的开发过程中不断循环，知道所有的代码都开发完成并成功通过所有测试。

捉虫：知道->直到

[ghost]

太棒了！它失败了...

太棒了 ~

[38602629]

let contents = "
Rust:
safe, fast, productive.
Pick three.";

注意，这行代码需要复制，而不是手写，我手写就无妨测试成功。

[XavierZXY]

你忘记填上转义符“\”

let contents = "\
Rust:
safe, fast, productive.
Pick three.";

[yizhixiaowutu]

测试失败后我改成这样了 hh（换行后不对齐真是难受死强迫症doge）

let contents = "\nRust:\nsafe, fast, productive.\nPick three.";

[weinberggithub]

// in lib.rs
pub fn search<'a>(query: &str, contents: &'a str) -> Vec<&'a str> {
let mut results = Vec::new();

for line in contents.lines() {
    if line.contains(query) {
        results.push(line);
    }
}

results

}

这个函数从函数内部返回一个值，怎么从生命周期角度理解？而且这个results地址和外部接收的一摸一样

[mongooooo]

个人理解：返回了Vec的所有权，而不是引用

[Seann0824]

告诉编译器，返回的Vec内部的字符串引用至少和contents的生命周期一样久

[Capri0110]

pub fn search2<'a>(query: &'a str, contents: &'a str) -> Vec<&'a str> {
    let mut ret = Vec::new();
    contents.lines()
        .filter(|line| line.contains(query))
        .for_each(|line| ret.push(line.trim()));
    ret
}

习惯函数式这样写了- -

[liwuhou]

可以直接返回 contents.lines().filter(|line| line.contains(query).collect()

[wuxiangchao]

这是比比谁更"短"吗？

[Icemans007]

contents.lines().filter(|line|line.contains(query)).map(|line| line.trim()).collect()

[pplmx]

contents.lines().filter(|line|line.contains(query)).map(|line| line.trim()).collect()
@lcemans007 不应该处理空白, 因为空白也可能是用户search的内容

[leo-davisf]

这样的思路很好啊，学到了

[luckyele]

// in src/lib.rs
pub fn run(config: Config) -> Result<(), Box> {
let contents = fs::read_to_string(config.file_path)?;

for line in search(&config.query, &contents) {
    println!("{line}");
}

Ok(())

}

fs::read_to_string(config.file_path) 与 search(&config.query, &contents) 相比，前面的config.file_path 是不是应该加 引用 &。

[EthanOK]

可以加也可以不加。如果是read_to_string(config.file_path)，config.file_path所有权转移给read_to_string函数，后续不能再访问config.file_path；如果read_to_string(&config.file_path)，后续可继续访问config.file_path。

[TechSavantDev]

config.file_path should be written by type of borrow, I think, because you need to use config.query in search function. If config.file_path not written with &, the compiler will tell you that config value partially moved in fs::read_to_string function.