Question

If I write this:

$('#SomeDiv').click(DoSomething);

function DoSomething() {

  if (SomeCondition === true) {
     return false;
  }

  // more code here
}

regardless of whether I put return; or return false; or return true; the code doesn’t throw exceptions and the function execution stops.

Which is the best option?

Answer 1

If those are the only options, use return; in this case. (read below)

If the function normally returns something (calculates something, gets some value, etc) then you definitely don’t want to return anything, because you might confuse the caller.

If your function doesn’t normally return anything, then it might not hurt to return anything you like, but it might still confuse callers.

I would personally rather just put an else after the if, and not use the return;. And if the function gets too large, just retractor it a bit.