src/tools/clippy/tests/ui/iter_kv_map.fixed

   1 // run-rustfix
   2
   3 #![warn(clippy::iter_kv_map)]
   4 #![allow(clippy::redundant_clone)]
   5 #![allow(clippy::suspicious_map)]
   6 #![allow(clippy::map_identity)]
   7
   8 use std::collections::{BTreeMap, HashMap};
   9
  10 fn main() {
  11     let get_key = |(key, _val)| key;
  12
  13     let map: HashMap<u32, u32> = HashMap::new();
  14
  15     let _ = map.keys().collect::<Vec<_>>();
  16     let _ = map.values().collect::<Vec<_>>();
  17     let _ = map.values().map(|v| v + 2).collect::<Vec<_>>();
  18
  19     let _ = map.clone().into_keys().collect::<Vec<_>>();
  20     let _ = map.clone().into_keys().map(|key| key + 2).collect::<Vec<_>>();
  21
  22     let _ = map.clone().into_values().collect::<Vec<_>>();
  23     let _ = map.clone().into_values().map(|val| val + 2).collect::<Vec<_>>();
  24
  25     let _ = map.clone().values().collect::<Vec<_>>();
  26     let _ = map.keys().filter(|x| *x % 2 == 0).count();
  27
  28     // Don't lint
  29     let _ = map.iter().filter(|(_, val)| *val % 2 == 0).map(|(key, _)| key).count();
  30     let _ = map.iter().map(get_key).collect::<Vec<_>>();
  31
  32     // Linting the following could be an improvement to the lint
  33     // map.iter().filter_map(|(_, val)| (val % 2 == 0).then(val * 17)).count();
  34
  35     // Lint
  36     let _ = map.keys().map(|key| key * 9).count();
  37     let _ = map.values().map(|value| value * 17).count();
  38
  39     let map: BTreeMap<u32, u32> = BTreeMap::new();
  40
  41     let _ = map.keys().collect::<Vec<_>>();
  42     let _ = map.values().collect::<Vec<_>>();
  43     let _ = map.values().map(|v| v + 2).collect::<Vec<_>>();
  44
  45     let _ = map.clone().into_keys().collect::<Vec<_>>();
  46     let _ = map.clone().into_keys().map(|key| key + 2).collect::<Vec<_>>();
  47
  48     let _ = map.clone().into_values().collect::<Vec<_>>();
  49     let _ = map.clone().into_values().map(|val| val + 2).collect::<Vec<_>>();
  50
  51     let _ = map.clone().values().collect::<Vec<_>>();
  52     let _ = map.keys().filter(|x| *x % 2 == 0).count();
  53
  54     // Don't lint
  55     let _ = map.iter().filter(|(_, val)| *val % 2 == 0).map(|(key, _)| key).count();
  56     let _ = map.iter().map(get_key).collect::<Vec<_>>();
  57
  58     // Linting the following could be an improvement to the lint
  59     // map.iter().filter_map(|(_, val)| (val % 2 == 0).then(val * 17)).count();
  60
  61     // Lint
  62     let _ = map.keys().map(|key| key * 9).count();
  63     let _ = map.values().map(|value| value * 17).count();
  64 }