遍历Vec<&str>累加符合条件值时遇结果异常及未使用变量警告求助

核心问题点 #

• 迭代器未执行：Rust迭代器是惰性的，map仅定义了遍历逻辑但不会主动触发执行，导致匹配代码完全没运行，n始终保持初始值0。
• 累加操作错误：误用|=（按位或运算符）而非+=（加法赋值），即便逻辑执行也无法得到正确的累加结果。
• 未使用变量警告：因迭代器未被消费，编译器判定fruits_hashmap未被实际使用，因此抛出警告。

修正后的代码（基础版） #

use std::collections::HashMap;

fn calculate_total() -> i32 {
    let fruits: Vec<&str> = vec!["Banana", "Peach"];
    let fruits_hashmap: HashMap<_, _> = HashMap::from_iter([("Apple", 2), ("Banana", 4), ("Peach", 8)]);
    let mut n = 0;

    // 使用for_each消费迭代器，触发遍历逻辑
    fruits.iter().for_each(|s| {
        match fruits_hashmap.get(s) {
            Some(&value) => n += value, // 用+=实现累加，解引用获取哈希表中的值
            None => println!("{} does not exist", s),
        }
    });

    n
}

更简洁的函数式写法 #

use std::collections::HashMap;

fn calculate_total() -> i32 {
    let fruits: Vec<&str> = vec!["Banana", "Peach"];
    let fruits_hashmap: HashMap<_, _> = HashMap::from_iter([("Apple", 2), ("Banana", 4), ("Peach", 8)]);

    // 过滤存在的键并累加对应值
    fruits.iter()
        .filter_map(|s| fruits_hashmap.get(s).copied())
        .sum()
}

修正说明 #

1. 用for_each或sum消费迭代器，确保遍历逻辑真正执行，同时消除fruits_hashmap未使用的警告。
2. 替换|=为+=实现数值累加，注意HashMap::get返回Option<&V>，需通过解引用或copied()获取值的副本。
3. 函数式写法通过filter_map过滤不存在的键，直接用sum完成累加，代码更简洁且符合Rust风格。

内容的提问来源于stack exchange，提问作者Antonin GAVREL