如果Result返回Err(_),我希望整个函数返回HTTP请求错误

时间:2016-04-03 09:46:31

标签: rust iron

我正在尝试使用Iron框架在Rust中构建一个简单的后端。这个处理程序只是返回某个文件的内容,我可以使它与unwrap()一起正常工作,但我想尝试正确的错误处理。这就是我想象的样子:

fn get_content(res: &mut Request) -> IronResult<Response> {
    let mut id = String::new();
    res.body.read_to_string(&mut id).unwrap();

    let file_path_string = &("../content/".to_string() + &id + ".rdt");

    // TODO: Error handling
    match File::open(file_path_string) {
        Ok(f) => {
            let mut s = String::new();
            f.read_to_string(&mut s);
            Ok(Response::with(((status::Ok), s)))
        }
        Err(err) => Err(Response::with(((status::InternalServerError), "File not found")))
    };
}

这会引发错误not all control paths return a value [E0269],这很好。但是如果我在匹配部分之后添加一个响应:

match File::open(file_path_string) {
    Ok(f) => {
        let mut s = String::new();
        f.read_to_string(&mut s);
        Ok(Response::with(((status::Ok), s)))
    }
    Err(err) => Err(Response::with(((status::InternalServerError), "File not found")))
};

Err(Response::with(((status::InternalServerError), "File not found")))

我改为收到错误消息:

expected `iron::error::IronError`,
    found `iron::response::Response`
(expected struct `iron::error::IronError`,
    found struct `iron::response::Response`) [E0308]
src/main.rs:95        
Err(Response::with(((status::InternalServerError), "File not found")))

我认为问题是Rust Err和Iron Err之间的碰撞?我不确定。我过去没有做太多的网页开发(或Rust),所以对代码的反馈也很感激!

更新:我认为这更像是“The Rust Way”吗?但我不确定

fn get_content(res: &mut Request) -> IronResult<Response> {
    let mut id = String::new();
    res.body.read_to_string(&mut id).unwrap();

    let file_path_string = &("../content/".to_string() + &id + ".rdt");

    // TODO: Error handling
    let f;
    match File::open(file_path_string) {
        Ok(file) => f = file,
        Err(err) => Err(HttpError::Io(err))
    };
    let mut s = String::new();
    f.read_to_string(&mut s);
    Ok(Response::with(((status::Ok), s)))
}

将代码置于错误处理中似乎很奇怪,因为read_to_string也需要处理,这会产生嵌套的错误处理混乱?但是,这些匹配的武器显然是不兼容的类型,所以它不起作用......任何建议?

1 个答案:

答案 0 :(得分:2)

Ok()需要Response,而Err()需要IronError

因此,当Err(...)...时,您的通话Response无效!

如何纠正?那么第一步是,你必须创建一个IronError发回。我相信(不熟悉Iron)Iron会自动提供相应的错误代码,并且这不是你的工作。在文档中,我们找到了一个实现IronError的关键类型:

pub enum HttpError {
    Method,
    Uri(ParseError),
    Version,
    Header,
    TooLarge,
    Status,
    Io(Error),
    Ssl(Box<Error + 'static + Send + Sync>),
    Http2(HttpError),
    Utf8(Utf8Error),
    // some variants omitted
}

我看不到允许像“找不到文件”这样的任意字符串。但是,您的用例是IO故障之一,对吧?因此,将HttpError::Iostd::IoError取回的File::open()一起使用是有意义的:

match File::open(file_path_string) {
    Ok(f) => {
        let mut s = String::new();
        f.read_to_string(&mut s);
        Ok(Response::with(((status::Ok), s)))
    }
    Err(err) => Err(HttpError::Io(err))
};

顺便说一句,它还修复了你的“TODO:错误处理”!真漂亮!

(代码未经测试,如果编译失败,请随时编辑)