Возврат к предыдущему значению в HashMap

Question

Я обновляю значение ключа в HashMap, а затем сохраняю HashMap в файл.Я хочу убедиться, что в случае сбоя при сохранении в файл обновление будет отменено.Вот код, который я написал ( Rust Playground ):

use std::collections::HashMap;
use std::fs;
extern crate serde_json; // 1.0.37

fn save_map_to_file(map: &HashMap<String, String>) -> Result<(), ()> {
    // serialize map to json
    let map_as_string = match serde_json::to_string(map) {
        Ok(json_map) => json_map,
        Err(_) => return Err(()),
    };

    // write the json to a file
    match fs::write("map.bin", map_as_string) {
        Ok(_) => Ok(()),
        Err(_) => Err(()),
    }
}

fn change_map(map: &mut HashMap<String, String>) {
    // save current value in "key1" (if exists)
    let val = map.get("key1");

    // insert a new value to "key1"
    map.insert(String::from("key1"), String::from("value2"));

    // try to save the map to a file
    match save_map_to_file(map) {
        Ok(_) => (),
        Err(_) => {
            // if save fails, revert back to the original value
            match val {
                Some(value) => {
                    // if "key1" existed before the change, revert back to
                    // original value
                    map.insert(String::from("key1"), value.to_string());
                }

                None => {
                    // if "key1" didn't exist before the change, remove the
                    // new "key1"->"value2" record
                    map.remove("key1");
                }
            }
            ()
        }
    }
}

fn main() {
    let mut map: HashMap<String, String> = HashMap::new();

    map.insert(String::from("key1"), String::from("value1"));

    change_map(&mut map);

    println!("Map: {:?}", map);
}

Когда я компилирую этот код, я получаю ошибку:

error[E0502]: cannot borrow `*map` as mutable because it is also borrowed as immutable
  --> src/main.rs:24:5
   |
21 |     let val = map.get("key1");
   |               --- immutable borrow occurs here
...
24 |     map.insert(String::from("key1"), String::from("value2"));
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ mutable borrow occurs here
...
31 |             match val {
   |                   --- immutable borrow later used here

Я понимаю этоошибка, но я не могу понять, как правильно вернуть значение, если сохранение в файл не удалось.

Answer 1

insert() возвращает предыдущее значение, если оно есть, поэтому ваш код может быть значительно упрощен.Это также решает проблему заимствования:

fn change_map(map: &mut HashMap<&str, String>) {
    let previous = map.insert("key1", String::from("value2"));
    match save_map_to_file(map) {
        Ok(_) => (),
        Err(_) => {
            previous
                .and_then(|previous| map.insert("key1", previous))
                .or_else(|| map.remove("key1"));
        }
    }
}

Answer 2

Компилятор Rust не доволен этим заимствованием, просто избавьтесь от него

--- let val = map.get("key1");
+++ let val = map.get("key1").cloned();