Merge #470

470: refactor: use iterator instead for-loop r=bidoubiwa a=bestgopher

# Pull Request

## Related issue
Fixes #<issue_number>

## What does this PR do?
- ...

## PR checklist
Please check if your PR fulfills the following requirements:
- [ ] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [ ] Have you read the contributing guidelines?
- [ ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: bestgopher <[email protected]>

Author: meili-bors[bot]

src/search.rs

@@ -103,16 +103,19 @@ fn serialize_attributes_to_crop_with_wildcard<S: Serializer>(
     match data {
         Some(Selectors::All) => ["*"].serialize(s),
         Some(Selectors::Some(data)) => {
-            let mut results = Vec::new();
-            for (name, value) in data.iter() {
-                let mut result = String::new();
-                result.push_str(name);
-                if let Some(value) = value {
-                    result.push(':');
-                    result.push_str(value.to_string().as_str());
-                }
-                results.push(result)
-            }
+            let results = data
+                .iter()
+                .map(|(name, value)| {
+                    let mut result = String::new();
+                    result.push_str(name);
+                    if let Some(value) = value {
+                        result.push(':');
+                        result.push_str(value.to_string().as_str());
+                    }
+                    result
+                })
+                .collect::<Vec<_>>();
+
             results.serialize(s)
         }
         None => s.serialize_none(),