[bisq-network/bisq-core] Fix startup bug (#106)

Manfred Karrer notifications at github.com
Tue May 1 13:50:18 UTC 2018


Regarding https://bisq.slack.com/archives/C8MU98VSL/p1525179418000275

I usually follow the strategy to do the normal use case first and handle exceptional cases in the else clauses. I also try to avoid returns during the code path as it can become harder to understand where the method breaks.

I know some are preferring the other style to do the validation first and break with return statements and sometimes I used that as well. Not super strongly opinion about that but as I used the first style in most of the code I tried to stick with that.   

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq-core/pull/106#issuecomment-385675054
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20180501/47158098/attachment-0002.html>


More information about the github mailing list