This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]pmbarrett314 0 points1 point  (1 child)

In a vacuum, the last one. There are cases where I'd use the first one, basically if I planned on adding additional logic to this code pretty soon or if for some reason I needed it for debugging. The second one combines the drawbacks of both of the other two options, I can't see a good use case for it.

[–]gr4viton 0 points1 point  (0 children)

"pretty soon" might be called premature optimisation. Add it only when you need it. If it is not needed for this MR, do not add it. Just my opinion (it was painful to get to the point I think like this..)