java - 减少方法中的 for 循环次数

我正在尝试减少此方法中 for 循环的数量，但我不确定如何做到这一点，同时仍然保持逻辑机智，而不是转向另一种方法。我想这样做是为了减少 McCabe Cycolmatic Complexity ，它随着每个循环增加 1。所以我想将方法​​减少 1。

  private void method(int page)
    {
        for (int i = 0; i < LINES_PER_PAGE; i++)
        {
            nextLine[i] = null;
        }
        try
        {
            Scanner temp = new Scanner(fileToPrint);
            for (int i = 0; i < page - 1; i++)
            {
                skipAPage(temp);
            }
            for (int i = 0; (i < LINES_PER_PAGE) && (temp.hasNext()); i++)
            {
                nextLine[i] = expandTabs(temp.nextLine());
            }
        } catch (FileNotFoundException e)
        {
            e.printStackTrace();
        }
    }

最佳答案

    for (int i = 0; i < LINES_PER_PAGE; i++) {
        nextLine[i] = null;
    }

这是完全没有必要的，因为任何对象的默认值都是 java 中的 null。

在剩下的两个循环中，您调用了两种不同的方法，而您的第二个循环也取决于参数，因此我不确定它们是否可以合并为一个循环。

由于 nextLine 似乎是在方法之外声明的，我建议您使用 List<Sometype> 并将来自 expandTabs(temp.nextLine()); 的元素添加到其中，每次进入该方法时都清除列表 list.clear() 。

试试这个代码。使用适当的类型来声明 List<String> 。您可以使用 List 方法从 myList.get(index) 访问元素。

List<String> myList = new ArrayList<>();

private void method(int page) {
  try {
    myList.clear();            // Clear the list.
    Scanner temp = new Scanner(fileToPrint);
    for (int i = 0; i < page - 1; i++) {
      skipAPage(temp);
    }
    for (int i = 0; (i < LINES_PER_PAGE) && (temp.hasNext()); i++) {
      myList.add(expandTabs(temp.nextLine()));    // Add the elements to the list.
    }
  } catch (FileNotFoundException e) {
    e.printStackTrace();
  }
}

关于java - 减少方法中的 for 循环次数，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/33000427/