exercises/18_iterators/iterators4.rs

@@ -1,4 +1,4 @@
-fn factorial(num: u64) -> u64 {
+fn factorial(num: u8) -> u64 {
     // TODO: Complete this function to return the factorial of `num`.
     // Do not use:
     // - early returns (using the `return` keyword explicitly)

solutions/18_iterators/iterators5.rs

@@ -47,23 +47,6 @@ fn count_collection_iterator(collection: &[HashMap<String, Progress>], value: Pr
         .sum()
 }
 
-// Equivalent to `count_collection_iterator`+`count_iterator`, iterating as if
-// the collection was a single container instead of a container of containers
-// (and more accurately, a single iterator instead of an iterator of iterators).
-fn count_collection_iterator_flat(
-    collection: &[HashMap<String, Progress>],
-    value: Progress,
-) -> usize {
-    // `collection` is a slice of hash maps.
-    // collection = [{ "variables1": Complete, "from_str": None, … },
-    //               { "variables2": Complete, … }, … ]
-    collection
-        .iter()
-        .flat_map(HashMap::values) // or just `.flatten()` when wanting the default iterator (`HashMap::iter`)
-        .filter(|val| **val == value)
-        .count()
-}
-
 fn main() {
     // You can optionally experiment here.
 }
@@ -138,30 +121,18 @@ mod tests {
             count_collection_iterator(&collection, Progress::Complete),
             6,
         );
-        assert_eq!(
-            count_collection_iterator_flat(&collection, Progress::Complete),
-            6,
-        );
     }
 
     #[test]
     fn count_collection_some() {
         let collection = get_vec_map();
         assert_eq!(count_collection_iterator(&collection, Progress::Some), 1);
-        assert_eq!(
-            count_collection_iterator_flat(&collection, Progress::Some),
-            1
-        );
     }
 
     #[test]
     fn count_collection_none() {
         let collection = get_vec_map();
         assert_eq!(count_collection_iterator(&collection, Progress::None), 4);
-        assert_eq!(
-            count_collection_iterator_flat(&collection, Progress::None),
-            4
-        );
     }
 
     #[test]
@@ -174,10 +145,6 @@ mod tests {
                 count_collection_for(&collection, progress_state),
                 count_collection_iterator(&collection, progress_state),
             );
-            assert_eq!(
-                count_collection_for(&collection, progress_state),
-                count_collection_iterator_flat(&collection, progress_state),
-            );
         }
     }
 }

src/cargo_toml.rs

@@ -111,7 +111,6 @@ mod tests {
                 test: true,
                 strict_clippy: true,
                 hint: String::new(),
-                skip_check_unsolved: false,
             },
             ExerciseInfo {
                 name: String::from("2"),
@@ -119,7 +118,6 @@ mod tests {
                 test: false,
                 strict_clippy: false,
                 hint: String::new(),
-                skip_check_unsolved: false,
             },
         ];