是否使用to_owned()来更新结构的惯用方法?

时间:2017-05-16 03:10:09

标签: struct rust method-chaining borrow-checker

我正在使用链式方法更新Rust结构。我找到了一种方法来做到这一点,但我不确定下面的代码是否是惯用的Rust而不仅仅是一种解决方法。

特别是,我在链式方法的末尾使用.to_owned()来返回借用的struct。代码编译并正常工作。这是最小的例子。

//struct.rs
#[derive(Debug, Default, Clone, PartialEq)]
pub struct ModelDataCapture {
    run: i32,
    year: i32,
}
impl ModelDataCapture {
    pub fn new() -> Self {
        ModelDataCapture::default()
    }
    pub fn set_run(&mut self, run: i32) -> &mut ModelDataCapture {
        self.run = run;
        self
    }
    pub fn set_year(&mut self, year: i32) -> &mut ModelDataCapture {
        self.year = year;
        self
    }
}

//main.rs
let data_capture = ModelDataCapture::new()
    .set_run(0)
    .set_year(1)
    .to_owned(); // <<< QUESTION

println!("here is the data capture {:?}", data_capture);

这是编写结构的这种就地修改的正确方法吗?如果我没有在链的末尾包含.to_owned()方法,则编译失败,并显示临时变量的生存时间不够长的消息。

1 个答案:

答案 0 :(得分:3)

您的代码“有效”但对我没有意义。它:

  • 创建值
  • 突变值
  • 克隆值
  • 丢掉原始值

看到效率低下?此外,所有“就地突变”都被完全丢弃,因此没有任何好处。

我通常会引入一个mutate绑定:

let mut data_capture = ModelDataCapture::new();
data_capture.set_run(0).set_year(1);

或者一路走来,创建一个具有相当于finishbuild

的构建器
#[derive(Debug)]
struct ModelDataCapture {
    run: i32,
    year: i32,
}

#[derive(Debug, Default)]
struct ModelDataCaptureBuilder {
    run: i32,
    year: i32,
}

impl ModelDataCaptureBuilder {
    fn set_run(self, run: i32) -> Self {
        ModelDataCaptureBuilder { run, ..self }
    }

    fn set_year(self, year: i32) -> Self {
        ModelDataCaptureBuilder { year, ..self }
    }

    fn build(self) -> ModelDataCapture {
        let ModelDataCaptureBuilder { run, year } = self;
        ModelDataCapture { run, year }
    }
}

fn main() {
    let data_capture = ModelDataCaptureBuilder::default().set_run(0).set_year(1).build();

    println!("here is the data capture {:?}", data_capture);
}

有关镜像构建项目的构建器的更多示例,请参阅Do Rust builder patterns have to use redundant struct code?

您可以在第一个示例中使用self by-value,但在大多数情况下这很烦人,因为您始终必须记住绑定结果。