[swift-evolution] ternary operator ?: suggestion
al_skipp at fastmail.fm
Sun Dec 13 05:59:59 CST 2015
> On 13 Dec 2015, at 05:54, Paul Ossenbruggen via swift-evolution <swift-evolution at swift.org> wrote:
> Hello All,
> Been sick in bed all day, but decided to try to be productive…
> I did a rough draft of a proposal for implementing if expressions and switch expressions based upon the discussions we had here. I have tried to keep the scope of the changes as small as possible, only added one keyword and kept things as similar to the existing language constructs as possible. If anyone wants to help me with this, or has feedback, please let me know,
> https://github.com/possen/swift-evolution/blob/master/0020.md <https://github.com/possen/swift-evolution/blob/master/0020.md>
> - Paul
Thanks for taking the time to put this detailed proposal together.
Regarding the if…then…else expression, it all looks good to me.
I have a few comments on the proposed switch expression:
Is it worth reusing the ‘then’ keyword, instead of colon? If a new keyword is to be added, it’d be good if it wasn’t restricted to only one use. The advantage of using ‘then’ in switch would be to further distinguish the expression from the statement and would fit with the if…then…else expression. The disadvantage is that it adds a little more verbosity (ideally, expressions should be syntactically light-weight).
Having the ‘case’ keyword appear only once, looks a bit odd to me, it gives the impression that it is only associated with the first case.
My preference would be to omit the ‘case’ keyword entirely. In the switch statement it can be helpful, as the cases can run over several lines, therefore the keyword is a handy visual marker for the different cases. The expression for each case should only be one line, so this visual marker shouldn’t be required for switch expressions. Having said that, I understand if people think this is too much of a change. Therefore my suggestion is, either the ‘case’ keyword is omitted from the expression, or it is associated with every case as it currently is with the switch statement.
It might be worth showing an example of binding to associated values in a enum using a switch expression? However, the behaviour should be identical to the switch statement, so this might not be required.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the swift-evolution