Hacker News Reader

Comment by dfan | Hacker News Reader

Seasoned Clojurians: is

  (every? #(= % “success”) (map :status (:state task)))

actually considered crappy code? It looks perfectly natural to me, but my Lisp background is more generic.

No, not at all.

The threading macros are more useful when you have more nested code. For example:

    (->> coll
         (remove nil?)
         (filter :available?)
         (map :stock)
         (reduce +))

Is perhaps a little easier to read than:

    (reduce +
            (map :stock
                 (filter :available?
                         (remove nil? coll))))

I had the same reaction. I much prefer this to the threading macro.

crapiness comes more so on the readability/presentation aspect.