match => Option::map (#42)

Use `Option::<usize>::map` instead of a `match` expression in the `parse_month` function.
hello-rust · Jul 3, 2018 · 5e4c0a3 · 5e4c0a3
1 parent b49908c
commit 5e4c0a3
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/episode/7/humandate/src/lib.rs b/episode/7/humandate/src/lib.rs
@@ -35,10 +35,7 @@ impl From<std::num::ParseIntError> for Error {
 }
 
 fn parse_month(month: &str) -> Option<usize> {
-    match MONTH_NAMES.iter().position(|&elem| elem == month) {
-        Some(index) => Some(index + 1),
-        None => None,
-    }
+    MONTH_NAMES.iter().position(|&elem| elem == month).map(|day| day + 1)
 }
 
 fn parse_day(day_with_ordinal: &str) -> Result<u32, Error> {