39

我有这样的代码：

let things = vec![/* ...*/]; // e.g. Vec<String>
things
    .map(|thing| {
        let a = try!(do_stuff(thing));
        Ok(other_stuff(a))
    })
    .filter(|thing_result| match *thing_result {
        Err(e) => true,
        Ok(a) => check(a),
    })
    .map(|thing_result| {
        let a = try!(thing_result);
        // do stuff
        b
    })
    .collect::<Result<Vec<_>, _>>()

在语义方面，我想在第一个错误之后停止处理。

上面的代码可以，但是感觉挺麻烦的。有没有更好的办法？我已经查看了类似的文档filter_if_ok，但我没有找到任何东西。

我知道collect::<Result<Vec<_>, _>>，而且效果很好。我特别想消除以下样板：

  • 在过滤器的关闭中，我必须使用matchon thing_result。我觉得这应该只是一个单行，例如.filter_if_ok(|thing| check(a))。
  • 每次我使用map时，我都必须包含一个额外的语句let a = try!(thing_result);以处理Err. 同样，我觉得这可以抽象为.map_if_ok(|thing| ...).

我可以使用另一种方法来获得这种简洁程度，还是我只需要坚持下去？

4

4 回答 4

65

有很多方法可以表达这个意思。

如果您只想恐慌，请使用.map(|x| x.unwrap()).

如果您想要所有结果或单个错误，请collect进入Result<X<T>>：

let results: Result<Vec<i32>, _> = result_i32_iter.collect();

如果您想要除错误之外的所有内容，请使用.filter_map(|x| x.ok())or .flat_map(|x| x)。

如果您想要直到第一个错误的所有内容，请使用.scan((), |_, x| x.ok()).

let results: Vec<i32> = result_i32_iter.scan((), |_, x| x.ok());

请注意，在许多情况下，这些操作可以与之前的操作结合使用。

于 2016-04-02T10:08:50.513 回答
17

从 Rust 1.27 开始，Iterator::try_for_each可能会感兴趣：

一个迭代器方法，它对迭代器中的每个项目应用一个错误函数，在第一个错误处停止并返回该错误。

这也可以被认为是 的易错形式for_each()或无状态版本try_fold()。

于 2018-07-20T09:41:59.610 回答
13

您可以自己实现这些迭代器。查看标准库中的实现方式filter和实现方式。map

map_ok执行：

#[derive(Clone)]
pub struct MapOkIterator<I, F> {
    iter: I,
    f: F,
}

impl<A, B, E, I, F> Iterator for MapOkIterator<I, F>
where
    F: FnMut(A) -> B,
    I: Iterator<Item = Result<A, E>>,
{
    type Item = Result<B, E>;

    #[inline]
    fn next(&mut self) -> Option<Self::Item> {
        self.iter.next().map(|x| x.map(&mut self.f))
    }
}

pub trait MapOkTrait {
    fn map_ok<F, A, B, E>(self, func: F) -> MapOkIterator<Self, F>
    where
        Self: Sized + Iterator<Item = Result<A, E>>,
        F: FnMut(A) -> B,
    {
        MapOkIterator {
            iter: self,
            f: func,
        }
    }
}

impl<I, T, E> MapOkTrait for I
where
    I: Sized + Iterator<Item = Result<T, E>>,
{
}

filter_ok几乎一样：

#[derive(Clone)]
pub struct FilterOkIterator<I, P> {
    iter: I,
    predicate: P,
}

impl<I, P, A, E> Iterator for FilterOkIterator<I, P>
where
    P: FnMut(&A) -> bool,
    I: Iterator<Item = Result<A, E>>,
{
    type Item = Result<A, E>;

    #[inline]
    fn next(&mut self) -> Option<Result<A, E>> {
        for x in self.iter.by_ref() {
            match x {
                Ok(xx) => if (self.predicate)(&xx) {
                    return Some(Ok(xx));
                },
                Err(_) => return Some(x),
            }
        }
        None
    }
}

pub trait FilterOkTrait {
    fn filter_ok<P, A, E>(self, predicate: P) -> FilterOkIterator<Self, P>
    where
        Self: Sized + Iterator<Item = Result<A, E>>,
        P: FnMut(&A) -> bool,
    {
        FilterOkIterator {
            iter: self,
            predicate: predicate,
        }
    }
}

impl<I, T, E> FilterOkTrait for I
where
    I: Sized + Iterator<Item = Result<T, E>>,
{
}

您的代码可能如下所示：

["1", "2", "3", "4"]
    .iter()
    .map(|x| x.parse::<u16>().map(|a| a + 10))
    .filter_ok(|x| x % 2 == 0)
    .map_ok(|x| x + 100)
    .collect::<Result<Vec<_>, std::num::ParseIntError>>()

操场

于 2016-04-02T07:01:08.223 回答
3

filter_map可用于减少映射然后过滤的简单情况。在您的示例中，过滤器有一些逻辑，所以我认为它不会简化事情。不幸的是，我在文档中没有看到任何有用的功能Result。我认为您的示例尽可能地惯用，但这里有一些小的改进：

let things = vec![...]; // e.g. Vec<String>
things.iter().map(|thing| {
     // The ? operator can be used in place of try! in the nightly version of Rust
    let a = do_stuff(thing)?;
    Ok(other_stuff(a))
// The closure braces can be removed if the code is a single expression
}).filter(|thing_result| match *thing_result {
        Err(e) => true,
        Ok(a) => check(a),
    }
).map(|thing_result| {
    let a = thing_result?;
    // do stuff
    b
})

在?某些情况下，运算符的可读性可能会降低，因此您可能不想使用它。

如果您能够将check函数更改为返回Some(x)而不是 true，而None不是 false，则可以使用filter_map：

let bar = things.iter().filter_map(|thing| {
    match do_stuff(thing) {
        Err(e) => Some(Err(e)),
        Ok(a) => {
            let x = other_stuff(a);
            if check_2(x) {
                Some(Ok(x))
            } else {
                None
            }
        }
    }
}).map(|thing_result| {
    let a = try!(thing_result);
    // do stuff
    b
}).collect::<Result<Vec<_>, _>>();

let a = try!(thing);在某些情况下，您也可以通过使用匹配来摆脱。但是，filter_map在这里使用似乎没有帮助。

于 2016-04-02T05:10:46.857 回答