How does "flat_map" affect my code?

Question

I am working on the following code all day, ( here is the playpen)

/// The rule that moves state from one to another.
///
/// `S` - the type parameter of state.
///
/// `T` - the type parameter of input symbol.
#[deriving(PartialEq, Eq, Hash)]
pub struct Rule<S, T> {
  pub state: S,
  pub symbol: Option<T>,
  pub next_state: S
}

impl<S: PartialEq, T: PartialEq> Rule<S, T> {
  /// determine whether the rule applies to the given state and symbol
  pub fn apply_to(&self, state: &S, symbol: &Option<T>) -> bool {
    self.state == *state && self.symbol == *symbol
  }
}

/// The transition relation in NFA,
/// containing all the rules needed by the NFA.
pub struct NFATransitions<S, T> {
  pub rules: HashSet<Rule<S, T>>
}

impl<S: Eq + Hash + Clone, T: Eq + Hash> NFATransitions<S, T> {

  pub fn next_states(&self, states: &HashSet<S>, symbol: &Option<T>) -> HashSet<S> {
    states.iter().flat_map(|state| {
      // error goes here: borrowed value does not live long enough
      self.next_states_for(state, symbol).iter().map(|s| s.clone())
    }).collect()

    // Howover, the following code which have the same behavior can compile

    // let mut result = HashSet::new();
    // for state in states.iter() {
    //   result.extend(self.next_states_for(state, symbol).iter().map(|s| s.clone()));
    // }
    //
    // result
  }

  /// get the next state for the given state and symbol
  fn next_states_for(&self, state: &S, symbol: &Option<T>) -> HashSet<S> {
    self.rules.iter().filter_map(|rule| {
      if rule.apply_to(state, symbol) { Some(rule.next_state.clone()) } else { None }
    }).collect()
  }
}

Code is just a hash shell used for nfa transition rules. (This is not what concerns me)

flat_map- this is where I got the compilation error. It seems strange to me, since commented out lines, which I think have the same behavior as well flat_map, can succeed.

I can not understand how the error occurs error: borrowed value does not live long enough.

Any ideas?

Answer 1

The problem here is iter(), it is tied to the lifetime of the result next_states_for()and is an iterator of &-pointers.

next_states_for() , into_iter() - , , .

  pub fn next_states(&self, states: &HashSet<S>, symbol: &Option<T>) -> HashSet<S> {
    states.iter().flat_map(|state| {
      // error goes here: borrowed value does not live long enough
      self.next_states_for(state, symbol).into_iter()
    }).collect()
  }

, for.