如何避免Rust中match分支与ok_or参数的不必要String克隆

核心思路是延迟错误消息的创建，仅在触发错误时才分配内存，从根源上消除不必要的克隆操作。以下是两种可行方案：

方案一：用函数封装错误生成逻辑 #

把错误消息的创建逻辑抽成一个小函数，只在出错时调用，完全避免提前分配和克隆：

fn make_invalid_height_error() -> RPCError {
    RPCError::CustomError(1, "Invalid height".to_string())
}

fn json_to_block_height(height: Value) -> Result<i32, RPCError> {
    let height = match height {
        Value::Number(h) => Ok(h),
        _ => Err(make_invalid_height_error()),
    }?;
    let height = height
        .as_i64()
        .ok_or_else(make_invalid_height_error)?;
    if height < 0 {
        return Err(make_invalid_height_error());
    }
    i32::try_from(height).map_err(|_| make_invalid_height_error())
}

这种方式既消除了代码重复，又保证只有在错误发生时才会创建String，正常执行路径没有任何额外开销。

方案二：用Cow彻底避免内存分配 #

如果想进一步优化，可将错误消息的类型改为Cow<str>，这样在不需要修改时直接借用字符串字面量，连to_string()的内存分配都能省掉：

use std::borrow::Cow;

#[derive(Debug)]
enum RPCError {
    CustomError(u32, Cow<'static, str>),
}

fn json_to_block_height(height: Value) -> Result<i32, RPCError> {
    let err_msg = "Invalid height";
    let height = match height {
        Value::Number(h) => Ok(h),
        _ => Err(RPCError::CustomError(1, Cow::Borrowed(err_msg))),
    }?;
    let height = height
        .as_i64()
        .ok_or_else(|| RPCError::CustomError(1, Cow::Borrowed(err_msg)))?;
    if height < 0 {
        return Err(RPCError::CustomError(1, Cow::Borrowed(err_msg)));
    }
    i32::try_from(height)
        .map_err(|_| RPCError::CustomError(1, Cow::Borrowed(err_msg)))
}

该方案下正常路径零额外内存开销，错误路径也仅在必要时才会转为Owned类型。

针对性能与代码简洁性的权衡：

• 对于这个场景，显式分支避免拷贝带来的性能提升可以忽略不计——错误路径本身就是异常情况，正常执行时克隆操作根本不会触发。而"地道的Rust"实现（比如上述延迟创建错误的方式）代码更简洁、可读性更强，维护成本更低。
• 只有当错误路径被高频触发时，避免克隆才会产生可感知的收益。但绝大多数业务场景中错误是少数情况，此时代码的简洁性优先级远高于这点微乎其微的性能优化。
• 另外，Rust的字符串克隆开销本身很小（短字符串会被优化到栈上），除非错误消息特别长，否则没必要为了这点性能牺牲代码的简洁性。

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