From a70f72603f3886cd8cbbdd0a78dcf348674d11aa Mon Sep 17 00:00:00 2001 From: Paul-Christian Volkmer Date: Wed, 1 Jan 2025 19:47:08 +0100 Subject: [PATCH] refactor: use filter_map() --- src/model/data_catalogue.rs | 4 +--- src/model/requirements.rs | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/src/model/data_catalogue.rs b/src/model/data_catalogue.rs index 2e03a4d..ca34846 100644 --- a/src/model/data_catalogue.rs +++ b/src/model/data_catalogue.rs @@ -127,15 +127,13 @@ impl Requires for DataCatalogue { self.to_listed_string(), self.get_required_entries(all) .iter() - .map(|entry| match entry { + .filter_map(|entry| match entry { Requirement::PropertyCatalogue(_) | Requirement::ExternalPropertyCatalogue(_) => { Some(format!(" - {entry}\n")) } _ => None, }) - .filter(Option::is_some) - .flatten() .collect::() ) } diff --git a/src/model/requirements.rs b/src/model/requirements.rs index 57837cc..ab18681 100644 --- a/src/model/requirements.rs +++ b/src/model/requirements.rs @@ -111,7 +111,7 @@ where self.to_listed_string(), self.get_required_entries(all) .iter() - .map(|entry| match entry { + .filter_map(|entry| match entry { Requirement::DataCatalogue(x) => { let inner = x .get_required_entries(all) @@ -148,8 +148,6 @@ where } _ => None, }) - .filter(Option::is_some) - .flatten() .collect::() ) }