diff options
| author | Nathan West <Lucretiel@gmail.com> | 2020-01-06 17:06:19 -0500 |
|---|---|---|
| committer | dylan_DPC <dylan.dpc@gmail.com> | 2020-03-26 20:41:57 +0100 |
| commit | 9b429fd1219bfd15ec3fd9ce3cb073cdae6ed03c (patch) | |
| tree | d64258354118e4d5bb17d16e84658a02c7432713 | |
| parent | 2fbb07525e2f07a815e780a4268b11916248b5a9 (diff) | |
| download | rust-9b429fd1219bfd15ec3fd9ce3cb073cdae6ed03c.tar.gz rust-9b429fd1219bfd15ec3fd9ce3cb073cdae6ed03c.zip | |
Add fold_self
- Added `Iterator::fold_first`, which is like `fold`, but uses the first element in the iterator as the initial accumulator - Includes doc and doctest - Rebase commit; see #65222 for details Co-Authored-By: Tim Vermeulen <tvermeulen@me.com>
| -rw-r--r-- | src/libcore/iter/traits/iterator.rs | 56 |
1 files changed, 40 insertions, 16 deletions
diff --git a/src/libcore/iter/traits/iterator.rs b/src/libcore/iter/traits/iterator.rs index 39bd270da86..c066d535f24 100644 --- a/src/libcore/iter/traits/iterator.rs +++ b/src/libcore/iter/traits/iterator.rs @@ -2005,6 +2005,44 @@ pub trait Iterator { self.try_fold(init, ok(f)).unwrap() } + /// The same as [`fold()`](#method.fold), but uses the first element in the + /// iterator as the initial value, folding every subsequent element into it. + /// If the iterator is empty, return `None`; otherwise, return the result + /// of the fold. + /// + /// # Example + /// + /// Find the maximum value: + /// + /// ``` + /// fn find_max<I>(iter: I) -> Option<I::Item> + /// where I: Iterator, + /// I::Item: Ord, + /// { + /// iter.fold_first(|a, b| { + /// a.partial_cmp(b).map(move |cmp| match cmp { + /// Ordering::Greater | Ordering::Equal => a, + /// Ordering::Less => b, + /// }) + /// }) + /// } + /// let a = [10, 20, 5, -23, 0]; + /// let b = []; + /// + /// assert_eq!(find_max(a.iter()), Some(20)); + /// assert_eq!(find_max(b.iter()), None); + /// ``` + #[inline] + #[unstable(feature = "iterator_fold_self", issue = "68125")] + fn fold_first<F>(mut self, f: F) -> Option<Self::Item> + where + Self: Sized, + F: FnMut(Self::Item, Self::Item) -> Self::Item, + { + let first = self.next()?; + Some(self.fold(first, f)) + } + /// Tests if every element of the iterator matches a predicate. /// /// `all()` takes a closure that returns `true` or `false`. It applies @@ -2497,7 +2535,7 @@ pub trait Iterator { move |x, y| cmp::max_by(x, y, &mut compare) } - fold1(self, fold(compare)) + self.fold_first(fold(compare)) } /// Returns the element that gives the minimum value from the @@ -2561,7 +2599,7 @@ pub trait Iterator { move |x, y| cmp::min_by(x, y, &mut compare) } - fold1(self, fold(compare)) + self.fold_first(fold(compare)) } /// Reverses an iterator's direction. @@ -3214,20 +3252,6 @@ pub trait Iterator { } } -/// Fold an iterator without having to provide an initial value. -#[inline] -fn fold1<I, F>(mut it: I, f: F) -> Option<I::Item> -where - I: Iterator, - F: FnMut(I::Item, I::Item) -> I::Item, -{ - // start with the first element as our selection. This avoids - // having to use `Option`s inside the loop, translating to a - // sizeable performance gain (6x in one case). - let first = it.next()?; - Some(it.fold(first, f)) -} - #[stable(feature = "rust1", since = "1.0.0")] impl<I: Iterator + ?Sized> Iterator for &mut I { type Item = I::Item; |
