Question

我正在尝试从给定的String路径中提取文件的扩展名。

以下代码有效，但我想知道是否有更清洁，更强大的惯用方法来实现这一目标：

use std::path::Path;

fn main() {

    fn get_extension_from_filename(filename: String) -> String {

        //Change it to a canonical file path.
        let path = Path::new(&filename).canonicalize().expect(
            "Expecting an existing filename",
        );

        let filepath = path.to_str();
        let name = filepath.unwrap().split('/');
        let names: Vec<&str> = name.collect();
        let extension = names.last().expect("File extension can not be read.");
        let extens: Vec<&str> = extension.split(".").collect();

        extens[1..(extens.len())].join(".").to_string()
    }

    assert_eq!(get_extension_from_filename("abc.tar.gz".to_string()) ,"tar.gz" );
    assert_eq!(get_extension_from_filename("abc..gz".to_string()) ,".gz" );
    assert_eq!(get_extension_from_filename("abc.gz".to_string()) , "gz");

}

Answer 1

在惯用Rust中，可能失败的函数的返回类型应为Option或Result。通常，函数也应该接受切片而不是String s，并且只在必要时创建新的String。这减少了过多的复制和堆分配。

您可以使用提供的extension()方法，然后将结果OsStr转换为&str：

use std::path::Path;
use std::ffi::OsStr;

fn get_extension_from_filename(filename: &str) -> Option<&str> {
    Path::new(filename)
        .extension()
        .and_then(OsStr::to_str)
}

assert_eq!(get_extension_from_filename("abc.gz"), Some("gz"));

使用and_then在这里很方便，因为这意味着您无需打开Option<&OsStr>返回的extension()并在调用之前处理它None的可能性to_str。我也可以使用lambda |s| s.to_str()而不是OsStr::to_str - 这可能是一个偏好或意见问题，哪个更惯用。

请注意，参数&str和返回值都是对为断言创建的原始字符串切片的引用。返回的切片不能超过它引用的原始切片，因此如果您需要更长的时间，可能需要从此结果中创建一个拥有的String。

Answer 2

什么比使用Rust的builtin method更具惯用性呢？

Path::new(&filename).extension()

用于从Rust中的给定路径中提取文件扩展名

2 个答案: