如何有条件地更新计数器算法Java

Question

这个问题有点混乱，所以我将尝试通过代码和示例进行解释。

如果满足条件，我如何减少计数器？听起来很简单吧？但是，如果该方法在 for 循环中不断被调用，以便传递给它的值被重置怎么办？

示例： 我有一个包含几页的文档，但是其中一些页面是空白的，我不想计算它们。

对于第一页为空白的 5 页文档。我想完成以下行为：

page 1: nothing...
page 2: 1 of 4
page 3: 2 of 4
page 4: 3 of 4
page 5: 4 of 4

目前它包括空白页作为页数的一部分。

这是一个代码示例：

static main() {
    // et cetera
    for(Page page : pages) {
        int pageNumber = page.getPageNo();
        int pageCount = page.getTotalPageCount();
        form.getTemplate().writePageNumber(page, pageNumber,pageCount);
    }
}

因此，在主程序中，我在 for 循环中调用了一个名为 writePageNumber() 的方法，在该循环中我传入了 currentPageNumber 和 totalPage 计数。

public boolean writePageNumber(Page page, int currentPageNo, int totalPageCount) {
    if ( !page.isBlank() ) {
        this.write(currentPageNo, totalPageCount);
    }
    else if ( page.isBlank() ){
        currentPageNo--;
        totalPageCount--;
    }

    // et cetera
}

现在在 writePageNumber() 方法中，我获取页面上下文并进行处理。我检查页面是否不空白然后我写totalPages的currentPage，如果它是空白那么我减少currentPage No和totalPage计数......以便不计算空白页面......这是正确的方法去做吧？不确定

由于该方法是在for循环中调用的，我递减后的值会被重置，如何维护递减的值？

可能有更好的方法来做到这一点，任何帮助将不胜感激。

编辑：

为了清楚起见，问题在于 writePageNumber() 是从 for 循环中调用的，因此下次调用 writePageNumber 时，值会被重置，因此在第二次迭代期间 else if 条件不再为真，第一个条件为真所以写错页码

Answer 1

问题是在您当前的代码中，变量currentPageNo 和totalPageCount 在堆栈上，这意味着它们的值在方法返回后被丢弃。解决您的问题的一种方法是将writePageNumber 的主体提取到您的for 循环中。因此，与其拥有类似的东西，

Page page = /* ... */
int totalPages = 5;
for (int currentPage = 1; currentPage <= totalPages; i++) {
    writePageNumber(page, currentPage, totalPages);
};

您的代码可能如下所示：

Page page = /* ... */
int totalPages = 5;
for (int currentPage = 1; currentPage <= totalPages; i++) {
    if (!page.isBlank()) {
       // ...
    } else {
      currentPage--;
      totalPages--;
    }
    // ...
}

编辑：您提到不破坏代码模式，但在这种情况下您可能不得不妥协。目前，您期望writePageNumber 是一个函数，它接受两个整数参数，修改它们，并为下一个函数调用以某种方式返回它们的更新值。您不能从 Java 中的函数返回多个值。您需要让这些值在方法调用中保持不变，因此它们需要在方法范围之外声明（或者，就像我建议的那样，您将方法提取到循环中），这意味着使用一个字段.

Answer 2

我认为问题在于：
您有数 (5) 个页面对象；每个对象都有自己的currentPage 和totalPageCount，如下所示：
对象 1：Page: current = 1; total = 5;
对象 2：Page: current = 2; total = 5;
对象 3：Page: current = 3; total = 5;
对象 4：Page: current = 4; total = 5;
对象 5：Page: current = 5; total = 5;

在您的代码中，您正在更新current 和total（不在对象中，而只是在它们正在使用的上下文中本地），但这些值不会在其余对象中更新； for 循环遍历这些未更新的剩余对象。
我建议你这样做：

private int blanks = 0;
public boolean writePageNumber(Page page, int currentPageNo, int totalPageCount) {
    if ( page.isBlank() ) {
        blanks++;
        // etcetera
    } else {
        this.write(currentPageNo - blanks, totalPageCount - blanks);
    }

    // et cetera
}

编辑

您应该首先遍历所有页面以获得总空白页，以获得固定的totalPageCount，然后删除上面totalPageCount - blanks中的- blanks，否则您将获得如下数据：

对象 1：Page: current = 1; total = 5;
对象 2：Page: current = BLANK; total = 5;
对象 3：Page: current = 2; total = 4;
对象 4：Page: current = BLANK; total = 4;
对象 5：Page: current = 3; total = 3;

注意总数是如何不断变化的。
还有一点需要注意的是，totalPageCount 不是页面的属性，因此不应该属于 Page 对象，而是属于 book 对象。看看这让事情变得多么复杂？！