On Wed, Jan 5, 2011 at 9:21 PM, Timothy Pratley timothyprat...@gmail.comwrote:
Neat, looks pretty nice.
I love invitations to nit pick!
database.clj
(defn complete-todo [id]
(dosync (ref-set *todo* (vec (remove #(= (get % :id) id) @*todo*)
1) ref-set is unnecessary you could re-factor
Neat, looks pretty nice.
I love invitations to nit pick!
database.clj
(defn complete-todo [id]
(dosync (ref-set *todo* (vec (remove #(= (get % :id) id) @*todo*)
1) ref-set is unnecessary you could re-factor this to use alter. The
result is the same, but semantically set only applies when
I finally moved on from messing around with stuff in the repl and trying to
get a firm grasp on all things clojure and dove into doing a little web
development with it. I hadn't used ring, compojure or enlive before so I
kept that functionality in the app really minimal. I'd appreciate feedback