我应该如何在不抛出异常的情况下做到这一点？

Question

我对 Java 还很陌生，我正在努力了解异常以及何时应该使用它们。我一直将它们用作错误检查的一种形式，但我遇到几个人说异常只能用于程序无法控制的事情，例如用户错误。

我有一个函数，对于给定的二维线，计算 xMin 和 xMax 之间的所有 y 值。如果直线是垂直的，此函数将引发异常，因为不可能计算垂直线上的所有 y 值。还有一个等效的函数可以在两个 y 值之间查找点，如果直线是水平的，则会引发错误。

findPointsInRangeOfX (int xMin, int xMax) throws LineIsVerticalException {
    // check if line is vertical
    // if vertical, throw an exception
    // else calculate the y values for each integer between xMin and xMax
    // return y values
}

我将此函数称为查找给定窗口内一条线上所有点的一部分，由最小和最大 x 和 y 值给出。我不检查此功能中的线是否垂直；相反，我依靠检查 findPointsInRangeOfX 并在方法周围使用 try 和 catch 块。

pointsInWindow (int xMin, int xMax, int yMin, int yMax) {
    try {
        // Find list of all the points between xMin and xMax.
        // Remove from list all points not between yMin and yMax
    }
    catch (LineIsVerticalException e) {
        // If line is vertical, cannot find points between xMin and xMax
        try {
            // Instead, find list of all points between yMin and yMax
            // Remove from list all points not between xMin and xMax
        }
        catch (LineIsHorizontalException e) {
            // This part will never be reached because the line is vertical
            // But the compiler complains if the exception isn't caught
        }
    }
}

这样好吗？我不会因为这样的错误抛出异常 - 有一条垂直线没有任何问题 - 但我用它来告诉 pointsInWindow 它需要找到 y 值而不是 x 值之间的点。我是否应该复制检查以查看 pointsInWindow 函数中的线是否垂直，而不是使用 try catch 块？如果我确实重复了检查，我应该一起摆脱 LineIsVerticalException 吗？

Answer 1

你必须坚持单一职责原则：每个方法只做一件事。 现在你的方法做两件事：检查它是否是垂直/水平的并计算一些东西。

这里的另一个注意事项：不要在程序流中使用异常。

你应该把它拆分成这样：

bool isVertical(parameters){}
bool isHorizontal(parameters){}
SomeClass CalculateVertical(parameters){}
SomeClass CalculateHorizontal(parameters){}

您的程序流程可能如下所示：

if(isVertical(something)){
 CalculateVertical(something);
else if (isHorizontal(something)){
 CalculateHorizontal(something);
}

示例实现：

SomeClass CalculateVertical(something){
 if(!isVertical(something)) { throw new IllegalArgumentException() }
 // Calculations
}

请注意，程序员不必捕获此异常。

Answer 2

或者你可以像下面这样修改：

pointsInWindow (int xMin, int xMax, int yMin, int yMax) {
    try {
        // Find list of all the points between xMin and xMax.
        // Remove from list all points not between yMin and yMax
    }
    catch (LineIsVerticalException e) {
        // If line is vertical, cannot find points between xMin and xMax
        // do something..
        }
    catch (LineIsHorizontalException e) {
        // unless LineIsVerticalException is superclass of LineIsHorizontalException,
        // this will work 
        // do something ..
        }
    }
}

Answer 3

一般来说，我尝试将异常用于意外问题（例如网络故障），而不仅仅是边界情况。然而，这是一个模糊的区别，取决于您的特定应用程序上下文。

专门针对您的问题。拆分功能怎么样。创建两个函数来检测给定的行是水平还是垂直（例如 boolean isVertical()）；。

在您的 pointsInWindow 函数中，您可以首先检查您是否正在处理垂直/水平线的特殊情况，如果没有进一步调用 findPointsInRange 方法。

尽量不要重复逻辑，因为这违反了 DRY 原则DRY principle，并且在维护代码时往往会导致进一步的问题。

希望这会有所帮助， 马库斯

Answer 4

findPointsInRangeOfX (int xMin, int xMax) throws LineIsVerticalException {
    if(isLineVeritical(xMin, xMax)){
        throw new LineIsVerticalException();
    }

    return calculateValuesBetweenMinAndMax(xMin, xMax);
}