Rust处理Result时，是否应替换map_or_else为其他函数？

我写了一段Rust代码，定义了CommandError枚举来处理ExecutionError和NonZeroExit两类错误，实现了run_command函数返回Result<Output, CommandError>，并在center_window函数里用map_or_else处理错误或成功结果。但成功分支只需要写|_| ()空逻辑，这让我疑惑：是否应该用map_or_else？有没有更合适的函数？

代码示例 #

use std::process::{Command, Output};
use std::io;

#[derive(Debug)]
enum CommandError {
    ExecutionError { source: io::Error },
    NonZeroExit {
        status: std::process::ExitStatus,
        stderr: Vec<u8>,
        stdout: Vec<u8>,
    },
}

impl std::fmt::Display for CommandError {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            CommandError::ExecutionError { source } => write!(f, "执行命令失败: {}", source),
            CommandError::NonZeroExit { status, stderr, stdout } => {
                writeln!(f, "命令退出码非零: {}", status)?;
                writeln!(f, "标准输出: {}", String::from_utf8_lossy(stdout))?;
                writeln!(f, "标准错误: {}", String::from_utf8_lossy(stderr))
            }
        }
    }
}

fn create_execution_error(err: io::Error) -> CommandError {
    CommandError::ExecutionError { source: err }
}

fn handle_output(output: Output) -> Result<Output, CommandError> {
    if output.status.success() {
        Ok(output)
    } else {
        Err(CommandError::NonZeroExit {
            status: output.status,
            stderr: output.stderr,
            stdout: output.stdout,
        })
    }
}

fn run_command(command: &str) -> Result<Output, CommandError> {
    Command::new("sh")
        .arg("-c")
        .arg(command)
        .output()
        .map_err(|err| create_execution_error(err))
        .and_then(|output| handle_output(output))
}

pub fn center_window(window_id: &str) {
    let command = format!("xdotool windowmove {} 780 400", window_id);

    run_command(&command).map_or_else(
        |err| {
            eprintln!("Command: {}", command);
            eprintln!("Error: \n{}", err);
        },
        |_| (),
    )
}

你当前的写法虽然能工作，但因为只需要处理错误分支，用map_or_else会显得冗余——毕竟成功分支是空逻辑。这里有两种更合适的替代方案：

方案1：if let Err匹配（最直观） #

直接匹配错误情况，成功时什么都不做，代码可读性更强：

pub fn center_window(window_id: &str) {
    let command = format!("xdotool windowmove {} 780 400", window_id);

    if let Err(err) = run_command(&command) {
        eprintln!("Command: {}", command);
        eprintln!("Error: \n{}", err);
    }
}

方案2：inspect_err + ok() #

如果喜欢链式调用风格，可以用inspect_err来处理错误分支，再用ok()将Result转为Option后丢弃：

pub fn center_window(window_id: &str) {
    let command = format!("xdotool windowmove {} 780 400", window_id);

    run_command(&command)
        .inspect_err(|err| {
            eprintln!("Command: {}", command);
            eprintln!("Error: \n{}", err);
        })
        .ok();
}

为什么这两种更好？ #

• if let Err直接表达了“只关心错误情况”的意图，代码逻辑一目了然，比map_or_else的空成功分支更清晰。
• inspect_err专门用于在不改变Result值的前提下执行副作用（比如打印错误），配合ok()可以优雅地忽略成功结果，符合链式调用的习惯。

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