http://dev.clojure.org/jira This file is an XML representation of an issue en-us 4.4 649 25-07-2011 http://dev.clojure.org/jira/browse/CLJS-339 ClojureScript <p>(inc nil) =&gt; 1 in ClojureScript<br/> (inc nil) =&gt; raise NullPointerException in Clojure</p> <p>I think that Clojure's behavior (throwing) makes more sense in this context.</p> CLJS-339

(inc nil) returns 1 instead of throwing an exception

Defect Minor Open Unresolved Unassigned Evan Mezeske bug Mon, 23 Jul 2012 00:35:21 -0500 Mon, 23 Jul 2012 13:21:27 -0500 0 2 <p>I observe that in my JS console, "undefined + 1 =&gt; NaN" but "null + 1 =&gt; 1". I assume this has something to do with this issue.</p> <p>Do you have any suggestions on how to make this work without slowing arithmetic down? If not I'm inclined to close this as Won't Fix.</p> <p>I do not. I haven't had a chance to put a lot of thought into it, though.</p> <p>I consider this kind of problem to be rather insidious. Null is not a number, and treating it as zero <b>will</b> lead to nasty bugs creeping into people's code.</p> <p>A historical anecdote: the C function "int atoi(char* s)" takes a string and returns the number it represents. However, if the string can't be converted, it returns zero (which could also be a valid result for e.g. the string "0"). Technically, it's possible to distinguish "0 meaning error" and "0 meaning 0" by taking additional steps after the function call, but in practice people forget to do this. Thus, atoi is universally a disaster. I have <b>literally</b> seen the lights go off in a warehouse due to atoi's poor design (I used to work in building control).</p> <p>Treating null as zero will result in similar problems. Someone forgets to check the result of a computation, it's null, and the program silently continues as if nothing is wrong. Hello, data corruption.</p> <p>So I seriously urge you to not close this as Won't Fix, even if a performant solution is not yet obvious. IMHO it's worth more thought.</p> <p>Closing as Won't Fix doesn't mean we don't care, but this issue is simply a symptom of something much larger - punting on numerics. A ticket is the wrong place for this discussion as it has many nuances. It should be a part of a larger design for ClojureScript numerics. Without that larger discussion this ticket is likely to get stuck in limbo.</p> <p>Fair enough! I thought Won't Fix had stronger implications than that. Do you know if there's already a design doc that addresses numerics? Or should I start one?</p> <p>One place that this has burned me, that may warrant a separate card is <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>(update-in some-map [:foo :bar] inc)</pre> </div></div> where the key at that location did not exist. I would have preferred an exception rather than an broken 1.</p> <p>@Fogus: Huh, that is the <b>exact</b> same use-case that burned me &#8211; I was using update-in/inc on a nested map. The default value for my <span class="error">&#91;:foo :bar&#93;</span> was, in fact, supposed to be zero, so it silently worked for me. But when I moved that code from the client in my webapp (ClojureScript) to the server (Clojure), I found out that my original implementation was wonky.</p> <p>It doesn't seem to me that you could do that safely without fnil + 0. I too hate JS's silent failures around numerics. Been hoping someone would tackle this challenge with gusto.</p> <p>@David Nolen: Yeah, fnil would do the job. In my case, I was calling update-in on what I thought was an initialized map, but it was actually nil. update-in initialized each level of nesting with an empty map, and then inc was called on nil. My fix was simply to initialize the nested maps like I meant to. <img class="emoticon" src="http://dev.clojure.org/jira/images/icons/emoticons/smile.gif" height="20" width="20" align="absmiddle" alt="" border="0"/></p> <p>David,<br/> That was ultimately the solution, so I suppose we need to start advocating this pattern so that others might avoid the same fate. <img class="emoticon" src="http://dev.clojure.org/jira/images/icons/emoticons/wink.gif" height="20" width="20" align="absmiddle" alt="" border="0"/></p> Global Rank