if-statement - 从函数中提前返回是否比 if 语句更优雅？

Question

我和一位同事对以下哪个更优雅存在争议。我不会说谁是谁，所以是不偏不倚的。哪个更优雅？

public function set hitZone(target:DisplayObject):void
        {
            if(_hitZone != target)
            {
                _hitZone.removeEventListener(MouseEvent.ROLL_OVER, onBtOver);
                _hitZone.removeEventListener(MouseEvent.ROLL_OUT, onBtOut);
                _hitZone.removeEventListener(MouseEvent.MOUSE_DOWN, onBtDown);

                _hitZone = target;

                _hitZone.addEventListener(MouseEvent.ROLL_OVER, onBtOver, false, 0, true);
                _hitZone.addEventListener(MouseEvent.ROLL_OUT, onBtOut, false, 0, true);
                _hitZone.addEventListener(MouseEvent.MOUSE_DOWN, onBtDown, false, 0, true);
            }
        }

...或者...

public function set hitZone(target:DisplayObject):void
        {
            if(_hitZone == target)return;

            _hitZone.removeEventListener(MouseEvent.ROLL_OVER, onBtOver);
            _hitZone.removeEventListener(MouseEvent.ROLL_OUT, onBtOut);
            _hitZone.removeEventListener(MouseEvent.MOUSE_DOWN, onBtDown);

            _hitZone = target;

            _hitZone.addEventListener(MouseEvent.ROLL_OVER, onBtOver, false, 0, true);
            _hitZone.addEventListener(MouseEvent.ROLL_OUT, onBtOut, false, 0, true);
            _hitZone.addEventListener(MouseEvent.MOUSE_DOWN, onBtDown, false, 0, true);

        }

Answer 1

这是可以打破规则(即最佳实践)的情况之一。通常，您希望函数中的返回点尽可能少。这样做的实际原因是它简化了您对代码的阅读，因为您总是可以假设每个函数都会接受它的参数，执行它的逻辑并返回它的结果。为各种情况添加额外的返回往往会使逻辑复杂化，并增加阅读和完全理解代码所需的时间。一旦您的代码进入维护阶段，那么多次返回可能会对新程序员的生产力产生巨大影响，因为他们试图破译逻辑(当注释稀疏且代码不清楚时尤其糟糕)。问题随着函数的长度呈指数增长。

那么为什么在这种情况下每个人都更喜欢选项 2？这是因为您正在设置一个合约，该函数通过验证传入数据或其他可能需要检查的不变量来强制执行。构造验证的最漂亮的语法是检查每个条件，如果条件无效，则立即返回。这样您就不必通过所有检查来维护某种 isValid bool 值。

总结一下:我们真正关注的是如何编写验证代码而不是一般逻辑；选项 2 更适合验证代码。