Modifica di un mutex <Option<>>

Question

Ho una rapida domanda stilistica o idiomatica su Mutex.

Esiste un modo più elegante per modificare i dati in fn upper() rispetto all'utilizzo di *string = match *string. Sembra strano il dereferenziamento da entrambe le parti, ma se non lo faccio, non so come gestire la parte di stringa <MutexGuard>.

Link playground

use std::sync::{Mutex}; 

#[derive(Debug)] 
struct SharedFile{ 
    file: Mutex<Option<String>> 
} 

impl SharedFile{ 
    fn new()-> SharedFile{ 
     SharedFile{ 
      file: Mutex::new(Some("test".to_owned())), 
      //file: Mutex::new(None), 
     } 
    } 

    fn upper(&self){ 
     let mut string = self.file.lock().unwrap(); 

     *string= match *string{ 
     Some(ref mut x) => Some(x.to_uppercase()), 
     None => Some("Empty".to_owned()), 
     }; 

     println!("{:?}", *string); 
    } 
} 

fn main() { 
     let shared = SharedFile::new(); 
     shared.upper(); 
     println!("{:?}", shared); 
} 

Answer 1

Certo, c'è:

*string = string.as_ref() 
    .map(|x| x.to_uppercase()) 
    .unwrap_or_else(|| "Empty".to_owned()) 

Questo, infatti, non è specifico per Mutex; lo stesso problema e soluzione si applicano a &mut Option<String>, ad esempio. Detto questo, la soluzione con *string = match *string { ... } è assolutamente soddisfacente. A proposito, non è necessario mut in Some(ref mut x), solo Some(ref x) va bene - to_uppercase() richiede solo un riferimento condiviso alla stringa.