Executing multiple statements in if-else without nullpointer exception

Question

I'm trying to dig a little deeper into clojure and functional programming.

At some point of my code I have a (def server (spawn-server)). Now I want a short function for the REPL to check the state of this socket.

This is what I have at the moment:

(defn status []
  (if server 
    (
      (println "server is up and running")
      (println "connections:" (connection-count server)) 
     )    
    (println "server is down")))

If server is nil, everything works fine, but this is the output on the REPL if the server is running:

=> (status)
server is up and running
connections: 0
#<CompilerException java.lang.NullPointerException (NO_SOURCE_FILE:0)>

I'm not really sure if I see the problem, but I can't figure out how this should work :-) What I have here is this:

((println "foo")(println "foo"))

which will be evaluated to (nil nil) which results in the NullPointerException ?

Usally I wouldn't use the outer parentheses but how can I create some kind of "block" statement for the if-condition. If I don't use them, the second println will be used as else.

What would work is the usage of let as some kind of "block"-statement:

(let [] 
  (println "server is up and running"),
  (println "connections:" (connection-count server)) )

But I'm not really sure if this the "right" solution?

danlei danlei · Accepted Answer · 2011-06-04T02:10:25

Use do:

(defn status []
  (if server 
    (do
      (println "server is up and running")
      (println "connections:" (connection-count server)))    
    (println "server is down")))

In Lisps, generally, you can't just add parens for grouping.

((println "foo") (println "foo"))

Here, the return value of the first (println "foo") will be tried to be called (as a function), with the return value of the second as an argument. Those are very basic evaluation rules, so I suggest that you hit some introductory books or documentation about Clojure or Lisps in general.

From the evaluation section of the Clojure homepage:

Non-empty Lists are considered calls to either special forms, macros, or functions. A call has the form (operator operands*).

Macros or special forms may "break" this rule.

Executing multiple statements in if-else without nullpointer exception

1 Answers