JEP325: Switch expressions spec

Brian Goetz brian.goetz at oracle.com
Fri Apr 20 18:40:53 UTC 2018


One thing that is relevant to the short term is that now that we killed 
mixed labels, we'd have to have a way to say "case null or default" in 
arrow world.  The least stupid thing seems to be to allow default to be 
tacked on to a comma-separated case list as if it were a pattern:

     case A -> s1;
     case null, default -> s2;

since you can no longer say:

     case A -> s1;
     case null:
     default:
         s2;


On 4/20/2018 2:36 PM, Kevin Bourrillion wrote:
> On Fri, Apr 20, 2018 at 10:45 AM, Brian Goetz <brian.goetz at oracle.com 
> <mailto:brian.goetz at oracle.com>> wrote:
>
>>     So, all I'm asking is: can we make this particular change
>>     atomically with patterns itself, not before? I believe that the
>>     change has negative value until then because it is too easy to
>>     use it to write bugs. (If this means that long and booleanalso
>>     wait until then, I don't think anyone would really mind. If
>>     necessary I can look up how common
>>     simulated-switch-on-longhappens in our codebase, but we all know
>>     it won't be much.)
>     The extra primitive types are separable, so could be deferred. 
>     I'd be less sanguine about adding long now but not float.
>
>
> Agreed, it would seem weird to keep adding more piecemeal over and over.
>
>>     Separately but similarly, the merits of case null: have also been
>>     justified almost entirely in the context of patterns. Without
>>     patterns, I believe the benefits are far too slight. We studied
>>     six digits' worth of switch statements in the Google codebase,
>>     using a /liberal/ interpretation of whether they are simulating a
>>     null case, and came up with ... 2.4%.  (You will find that
>>     suspicious as hell, since it's the exact same percentage I cited
>>     for fall-through yesterday, but I swear it's a coincidence!)
>     More nervous about this.  Would rather start the education curve
>     on this earlier. And there are plenty of existing switches that
>     are wrapped with "if target != null" that would be clearer/have
>     less needless repetition by pushing the case null into the switch.
>
>
> Er - just clarifying that this is the /same/ 2.4% that I am referring 
> to. Of course, numbers will vary (and I concede that we are quite 
> toward the null-hostile end of the spectrum in our general dev 
> practices). Still, I'm sure we would not be making this change for 
> this reason alone, so it really is about this issue of "starting the 
> education curve earlier". Trying to figure out how much that matters.
>
> For what it's worth, Guava took the position at the start that, since 
> working with null is risky and problematic, it's /okay/ if code that 
> deals with null is uglier than code that doesn't. It's only natural, 
> so we don't bend over backwards to try to smooth it over. If that 
> decision has played /some/ /small/ part in helping shift the world 
> away from rampant overuse of null everywhere, we wouldn't regret it a 
> bit. I think JDK collections post-1.4 could say the same thing to a 
> larger degree. Okay, I guess this is just the "moral hazard" argument 
> stated a different way - sorry.
>
> (Full disclosure: if you accuse me of wanting more time before `case 
> null:` lands just so I have more time to try to talk us out of it 
> completely, I suppose have no defense to that. :-))
>
> --
> Kevin Bourrillion | Java Librarian | Google, Inc. |kevinb at google.com 
> <mailto:kevinb at google.com>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/amber-spec-experts/attachments/20180420/a2c5b77f/attachment.html>


More information about the amber-spec-experts mailing list