有没有我能写的比一堆 else if 语句更有效、更干净的代码？

如何解决有没有我能写的比一堆 else if 语句更有效、更干净的代码？

我更像是一个中级 android 开发人员，我想学习如何编写更干净有效的代码，我在我的应用程序上遇到了这个问题，需要检查一堆语句，例如变量是否为每个单独的评分低于或高于某个值。

private void rateStock(double peRatio) {
    //For Dividend Stocks
    if(currentDiv != 0 && fiveYearDiv != 0){
        double dividendDiff = currentDiv - fiveYearDiv;
        if (peRatio <= 20 && peRatio > 0  && dividendDiff >= 0.01 && payoutRatio <= 0.65) {
            int stockratingDividend = 5;
            addItems(stockratingDividend);
        } else if (peRatio > 20 && peRatio <= 25 && dividendDiff >= 0.005 && payoutRatio <= 0.75) {
            int stockratingDividend = 4;
            addItems(stockratingDividend);
        } else if (peRatio > 25 && peRatio <= 30 && dividendDiff >= 0.001 && payoutRatio <= 0.85) {
            int stockratingDividend = 3;
            addItems(stockratingDividend);
        } else if (peRatio > 30 && peRatio <= 35 && payoutRatio <= 0.95) {
            int stockratingDividend = 3;
            addItems(stockratingDividend);
        } else if( peRatio > 35 &&  peRatio <= 40 && currentDiv > fiveYearDiv && payoutRatio >= 100) {
            int stockratingDividend = 2;
            addItems(stockratingDividend);
        } else if( peRatio > 40 || peRatio < 0 && fiveYearDiv > currentDiv  && payoutRatio >= 100) {
            int stockratingDividend = 1;
            addItems(stockratingDividend);
        } else {
            int stockratingDividend = 0;
            addItems(stockratingDividend);
        }
    }
}

正如您在上面的代码中看到的那样，它非常混乱和混乱，甚至无法正常工作，因为它必须检查的条件太多，并且通常不适合任何 else if 语句并且只返回 0

我知道这可能很多，但至少有人可以指导我写一篇关于如何编写更有效和干净的代码而不是一堆乱七八糟的 else if 语句的帖子吗？

解决方法

您的参数的可能值并未完全由 if/else 分支中的选项表示。因此，最好的方法是实现一个由自己决定的类，如果它负责计算您的单个结果值 stockRatingDividend。此草图（如果未设置最小值/最大值的边界，则没有 getter、构造函数和无处理）将是：

public class SRDResolver {
    public class SRDRolver {
        private int peRatioMin;
        private int peRatioMax;
        private double dividendDiffMin;
        private double dividendDiffMax;
        private double payoutRatioMin;
        private double payoutRatioMax;

        private double stockRatingDividend;

        public boolean hasSolution(int peRatio,double dividendDiv,double payoutRatio){
            return peRatio > peRatioMin && peRatio <= peRatioMax  && dividendDiv > dividendDiffMin && dividendDiv <= dividendDiffMax && payoutRatio > payoutRatioMin && payoutRatio <= payoutRatioMax;
        }
    }

    
}

然后你把实例放在一个列表中。然后，您通过使用 filter(i-> i.hasSolution(...).findFirst() 流式传输该列表获得正确的实例，结果为您提供了一个 Optional。如果它存在，则采用它的 stockRatingDividend，否则使用回退值 0。这样，如果需要，您可以轻松添加新实例，而不会弄乱 if/else。您可能会在将实例放入列表时添加一些检查以确保实例的数字空间跨度不重叠。

对于未来的流浪者，有同样的问题，我自己想出了一个解决方案！

我所做的是将其拆分为不同的类并在第一个类 ValueLists.java 中我创建了所有范围并为列表创建了一个 getter。

public List<Range> getPeRatioRanges() {

        Range range20 = new Range(0,20);
        Range range25 = new Range(21,25);
        Range range30 = new Range(26,30);
        Range range35 = new Range(31,35);
        Range range40 = new Range(36,40);

        List<Range> list = new LinkedList<>(Arrays.asList(range40,range35,range30,range25,range20)); // add in reverse order cuz index = rating

        return list;
    }

在另一个类 CheckRatingOfStock.java 中，我创建了一个简单的循环来遍历范围并获得评级。

 List<Range> peRatioRanges = getPeRatioRanges();

            for (int i = 0; i < peRatioRanges.size(); i++) {
                Range range = peRatioRanges.get(i);
                int intPERatio = (int) rawPEratio;

                if (range.contains(intPERatio)) {
                    rating = i + 1;
                }
            }

这里范围的索引是评级+1，所以如果它的0-20它的评级是5，因为列表是颠倒的。通过这种方式，使用 2 个代码块，我避免了大量的来回和大量 else if 语句的混乱，并且还使其更易于维护和清洁，至少在我看来是这样。

这可以作为一种方法来实现：

static int getIndexOfRange(int intPERatio,List<Range> ranges)
{
    for (int i=0; i< ranges.size(); i++) 
       if (ranges.get(i).contains(intPERatio)) 
           return i+1;                         //if found,return immediatelly
              
    return -1;   //if it reaches this line,means the item was not found
 }               //return -1 as representation of this

以下两者可以结合，因为它们做同样的事情。

else if (peRatio > 25 && peRatio <= 30 && dividendDiff >= 0.001 && payoutRatio <= 0.85) {
    int stockRatingDividend = 3;
    addItems(stockRatingDividend);
} else if (peRatio > 30 && peRatio <= 35 && payoutRatio <= 0.95) {
    int stockRatingDividend = 3;
    addItems(stockRatingDividend);
}

此外，我建议将 if-else 块重构为返回 stockRatingDividend 的新方法。这将使股息计算逻辑与其他逻辑分开，这实际上会使代码更易于理解。

if(currentDiv != 0 && fiveYearDiv != 0){
    double dividendDiff = currentDiv - fiveYearDiv;
    addItems(getStockRatingDividend(/*arguments*/));
}