<div dir="ltr">Hi!<br><br>I find this style convenient for debugging.<br>Sometimes you need some binding to be logged, but only under some rare conditions. After the problem is fixed the added code should be removed or commented-out.<div><br>Adding a simple if (or case) costs you 4 lines and an extra tabulation.</div><div>Removing/disabling that requires selecting several lines, then deleting/commenting them.<br><br></div><div>Adding a wrapper function invades the code in two places.</div><div>Removing that requires extra seeking to the helper function. Disabling (commenting the call) requires disabling the function too,<br>to avoid unused function warning. Much pain.</div><div><br></div><div>A single-liner with andalso/orelse is easy to write (shorter than other options) and easy to remove/disable (just jump to line + single dd or I in vim).</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, Jul 22, 2018 at 3:25 PM Viktor Söderqvist <<a href="mailto:viktor@zuiderkwast.se">viktor@zuiderkwast.se</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey everyone,<br>
<br>
I've seen these short-circuit operators with a non-boolean second<br>
operand, usually where the second operand is doing some side-effect and<br>
the return value is not important. It seems to be a trend. Example:<br>
<br>
    Msg /= undefined andalso io:format("Message: ~s~n", [Msg]),<br>
<br>
I this good or bad style?<br>
_______________________________________________<br>
erlang-questions mailing list<br>
<a href="mailto:erlang-questions@erlang.org" target="_blank">erlang-questions@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-questions" rel="noreferrer" target="_blank">http://erlang.org/mailman/listinfo/erlang-questions</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><font face="'courier new', monospace">Danil Zagoskin | <a href="mailto:z@gosk.in" target="_blank">z@gosk.in</a></font></div></div></div>