Время жизни Rust, данные перетекают в другие ссылки - PullRequest
Новая
       60

Время жизни Rust, данные перетекают в другие ссылки

0 голосов
/ 03 мая 2019

Я написал следующий код, который фильтрует поток данных, который работал нормально, пока я не перешел от анализа простых чисел к типам, которые привязаны к временам жизни, таким как &str и &[u8]. 

use wirefilter::{ExecutionContext, Filter, Scheme};

lazy_static::lazy_static! {
    static ref SCHEME: Scheme = Scheme! {
        port: Int,
        name: Bytes,
    };
}

#[derive(Debug)]
struct MyStruct {
    port: i32,
    name: String,
}

impl MyStruct {
    fn scheme() -> &'static Scheme {
        &SCHEME
    }

    fn filter_matches<'s>(&self, filter: &Filter<'s>) -> bool {
        let mut ctx = ExecutionContext::new(Self::scheme());
        ctx.set_field_value("port", self.port).unwrap();
        ctx.set_field_value("name", self.name.as_str()).unwrap();

        filter.execute(&ctx).unwrap()
    }
}

fn main() -> Result<(), failure::Error> {
    let data = expensive_data_iterator();
    let scheme = MyStruct::scheme();
    let filter = scheme
        .parse("port in {2 5} && name matches \"http.*\"")?
        .compile();

    for my_struct in data
        .filter(|my_struct| my_struct.filter_matches(&filter))
        .take(2)
    {
        println!("{:?}", my_struct);
    }

    Ok(())
}

fn expensive_data_iterator() -> impl Iterator<Item = MyStruct> {
    (0..).map(|port| MyStruct {
        port,
        name: format!("http {}", port % 2),
    })
}

Если я попытаюсь скомпилировать его, компилятор потерпит неудачу с этим: 

error[E0623]: lifetime mismatch
  --> src/main.rs:26:16
   |
21 |     fn filter_matches<'s>(&self, filter: &Filter<'s>) -> bool {
   |                           -----           ----------
   |                           |
   |                           these two types are declared with different lifetimes...
...
26 |         filter.execute(&ctx).unwrap()
   |                ^^^^^^^ ...but data from `self` flows into `filter` here

error: aborting due to previous error

error: Could not compile `wirefilter_playground`.

To learn more, run the command again with --verbose.

Process finished with exit code 101

Моей первой мыслью было, что self и filter должны иметь одинаковое время жизни в fn filter_matches<'s>(&self, filter: &Filter<'s>) -> bool, но если я изменю подпись на fn filter_matches<'s>(&'s self, filter: &Filter<'s>) -> boolЯ начну получать эту ошибку: 

error: borrowed data cannot be stored outside of its closure
  --> src/main.rs:38:29
   |
33 |     let filter = scheme
   |         ------ ...so that variable is valid at time of its declaration
...
38 |         .filter(|my_struct| my_struct.filter_matches(&filter))
   |                 ----------- ^^^^^^^^^ -------------- cannot infer an appropriate lifetime...
   |                 |           |
   |                 |           cannot be stored outside of its closure
   |                 borrowed data cannot outlive this closure

error: aborting due to previous error

error: Could not compile `wirefilter_playground`.

To learn more, run the command again with --verbose.

Process finished with exit code 101

Я не могу понять причину, Filter<'s> связан с SCHEME, который генерируется лениво и связан с 'static, что имеет смысл не разрешать фильтрацию.execute, чтобы взять ссылку на &self.name.as_str(), потому что он был бы изжит, но разве filter.execute(&ctx) подпись не должна pub fn execute(&self, ctx: &ExecutionContext<'s>) -> Result<bool, SchemeMismatchError> отбрасывать ссылки, как только она заканчивается, так как в результате у нее нет других времен жизни?

Чтобы попытаться скомпилировать приведенный выше код, вы можете использовать это Cargo.toml: 

[package]
name = "wirefilter_playground"
version = "0.1.0"
edition = "2018"

[dependencies]
wirefilter-engine = "0.6.1"
failure = "0.1.5"
lazy_static = "1.3.0"

PS: Это можно решить путем компиляции как внутри filter_matches метода, ноэто было бы плохо, потому что пользователь мог получить ошибку синтаксического анализа только при попытке фильтрации, и это могло бы быть медленнее.

1 Ответ

1 голос
/ 03 мая 2019

Я вижу 2 способа решения этой проблемы:
1) продлить срок службы self.name.Это может быть достигнуто путем сбора expensive_data_iterator, скажем, в Vec.

--- let data = expensive_data_iterator();
+++ let data: Vec<_> = expensive_data_iterator().collect();

2) сократить время жизни filter. 

--- let filter = scheme.parse("...")?.compile();
+++ let filter = scheme.parse("...")?;

--- .filter(|my_struct| my_struct.filter_matches(&filter))
+++ .filter(|my_struct| my_struct.filter_matches(&filter.clone().compile()))

Я пропустил некоторые другие незначительные изменения.И да, filter_matches<'s>(&'s self, ...) является обязательным в любом случае.

PS да, второй вариант работает, потому что my_struct переживает filter.Что ж, если оба подхода несколько плохие, вы можете их объединить!Обработайте data кусками, собирая каждый в вектор.

const N: usize = 10; // or any other size
loop {
    let cur_chunk: Vec<_> = data.by_ref().take(N).collect();
    if cur_chunk.is_empty() {
        break;
    }
    let cur_filter = filter.clone().compile();
    // etc
}

он использует только O (N) памяти и компилирует фильтр в N раз меньше

...