确定两个值是否在某个范围内的最干净、可读性最强的方法是什么？

Question

我有两个变量，float a和float b。仅当a和b都在0.0f和1.0f之间时，我想将boolean inRange变量设置为true。我可以这样做：

boolean inRange = a >= 0.0f && a <= 1.0f && b >= 0.0f && b <= 1.0f;

但是我想知道是否有更好的方法来检查这个。有没有更干净的方法，或者更有效的方法来做到这一点？

Answer 1

创建方法：

public boolean inRangeMethod(float a){
                return (a>=0.0f && a<=1.0f);
            }

之后：

boolean inRange = false;
inRange = inRangeMethod(a) && inRangeMethod(b);

Answer 2

没有比这更有效的方法了，但在我看来，这绝对是“更干净”的。实现这一点的一种方法是为边界定义常量。第二，使用辅助方法来检查单个值可以提高主方法的可读性。请参见下面的示例

public class YourClass {
    private static final float FLOOR = 0.0f;
    private static final float CEILING = 1.0f;

    public boolean inRange(float a, float b) {
        return inRange(a) && inRange(b);
    }

    private boolean inRange(float x) {
        return x > FLOOR && x < CEILING;
    }
}

Answer 3

为它创建一个方法（函数是最古老的代码组织类型是有原因的，它们可以工作！）

if(inRange(a) && inRange(b))
   ...

当然，这可以根据您的需要进行修改，但一般来说，拥有另一个具有良好名称的方法是一个极好的解决方案。此外，您没有在代码中分发1.0/0.0常量。

一个更通用的答案是创建一个RangeValidation类（假设您有其他Validation类）

RangeValidation criticalRange=new RangeValidation(0.0, 1.0);

criticalRange.validate(a);
criticalRange.validate(b);

这种模式一开始有点冗长，但它可以用作验证系统的一部分——想象许多不同类型的验证对象，它们都可以用一个简单的allValidations.validate（a）对一个值运行。标量不是很有用，但是模式可以用来验证更复杂的对象