Clojure: Idiomatic way to call contains? on a lazy sequence

前端 未结 2 449
小蘑菇
小蘑菇 2020-12-20 13:15

Is there an idiomatic way of determining if a LazySeq contains an element? As of Clojure 1.5 calling contains? throws an IllegalArgumentException:



        
相关标签:
2条回答
  • 2020-12-20 13:26

    If you use some instead of filter as in your example, you'll get an immediate return as soon as a value is found instead of forcing evaluation of the entire sequence.

    (defn lazy-contains? [coll key]
      (boolean (some #(= % key) coll)))
    

    Edit: If you don't coerce the result to a boolean, note that you'll get nil instead of false if the key isn't found.

    0 讨论(0)
  • 2020-12-20 13:27

    First, lazy seqs are not efficient for checking membership. Consider using a set instead of a lazy seq.

    If a set is impractical, your solution isn't bad. A couple of possible improvements:

    1. "Not empty" is a bit awkward. Just using seq is enough to get a nil-or-truthy value that your users can use in an if.You can wrap that in boolean if you want true or false.

    2. Since you only care about the first match, you can use some instead of filter and seq.

    3. A convenient way to write an equality predicate is with a literal set, like #{key}, though if key is nil this will always return nil whether nil is found our not.

    All together that gives you:

    (defn lazy-contains? [col key]
      (some #{key} col))
    
    0 讨论(0)
提交回复
热议问题