java if else嵌套代码重构

java if else 如果嵌套很深，代码的易读性差和冗余，遇到多个判断条件，适当提前 return，可以让代码结构清晰。

我们下看一个反例，代码如下：

public static boolean isInArea(double latitue, double longitude, double areaLatitude1, double areaLatitude2, double areaLongitude1, double areaLongitude2) {    if (isInRange(latitue, areaLatitude1, areaLatitude2)) {        if (areaLongitude1 * areaLongitude2 > 0) {            if (isInRange(longitude, areaLongitude1, areaLongitude2)) {                return true;            } else {                return false; // 冗余            }        } else {            if (Math.abs(areaLongitude1) + Math.abs(areaLongitude2) < 180) {                if (isInRange(longitude, areaLongitude1, areaLongitude2)) {                    return true;                } else {                    return false; // 冗余                }            } else {                double left = Math.max(areaLongitude1, areaLongitude2);                double right = Math.min(areaLongitude1, areaLongitude2);                if (isInRange(longitude, left, 180) || isInRange(longitude, 0, right)) {                    return true;                } else {                    return false; // 冗余                }            }        }    } else {        return false;    }}public static boolean isInRange(double point, double left, double right) {    if (point >= Math.min(left, right) && point <= Math.max(left, right)) {        return true;    } else {        return false;     }}

该代码的功能就是判断一个GPS point是否在给定的矩形区域内，代码逻辑上没有问题，大量的if else 导致代码不简洁。

如下我重写了这两个方法，减少冗余的判断，提前用return 返回，减少使用else语句。

优化后的代码如下：

public static boolean isInArea(double latitue, double longitude, double minLatitue, double maxLatitue, double minLongitude, double maxLongitude) {    // located in latitude range.    if (!isInRange(latitue, minLatitue, maxLatitue)) {        return false;    }    // Eastern or Western Hemisphere.    if (minLongitude * maxLongitude > 0) {        return isInRange(longitude, minLongitude, maxLongitude);    }    // One is Eastern, the other is Western, span zero.    if (Math.abs(minLongitude) + Math.abs(maxLongitude) < 180) {        return isInRange(longitude, minLongitude, maxLongitude);    }    // One is Eastern, the other is Western ,span 180.    return isInRange(longitude, maxLongitude, 180) || isInRange(longitude, minLongitude, -180);}private static boolean isInRange(double point, double left, double right) {    return point >= Math.min(left, right) && point <= Math.max(left, right);}

优化后代码简洁很多，结构也清晰点。

当有大量的if else 嵌套可以考虑如下方法让代码结构整洁：

1. 在if中return,减少else

2. 利用三元表达是减少判断

3. 减少上面的判断冗余