Given that your functions expect a list of exactly 6 inputs, I'd be
inclined to write these as functions that take 6 inputs, rather than a
list.  That way you get a meaningful error if they pass the wrong number of
inputs.

If you do prefer to keep them as-is, you can also shorten the code by a
line by destructuring directly in the input:
(defn f [[a b c d e f :as edges]] ...)

On Thu, Jul 30, 2015 at 3:21 PM, Mark Engelberg <mark.engelb...@gmail.com>
wrote:

> (println (format ...)) can be rewritten as (printf ...) if you add a \n to
> your string.
>
> A large chunk of your computations after the definitions appear to be
> global definitions and print messages to achieve some sort of unit
> testing.  I encourage you to refactor this using clojure.test so you can
> get a feel for how unit testing is generally done in Clojure.  Those
> definitions could then be made local, and the printing is probably
> unnecessary.
>
>

-- 
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to