更改/增加方法内的while循环条件的值 - 好或坏 -

Question

很抱歉，如果以前有人问过类似的问题，我无法通过搜索找到满意的答案。

问题的本质是“隐藏”用作 while 循环中条件的值的增量/更改是否是不好的做法。

int i = 0; // global/class variable

while(i < str.length()) {
    someMethod(i);
}

其中 someMethod 递增 i。

我正在扫描一些字符串并根据字符调用不同的方法。该方法获取字符串的当前索引作为参数，读取/处理一些字符，创建一个对象，并返回新的当前索引。

例子：

int i = 0;

while(i < str.length()) {
    if(condition(str.charAt(i))) i = someMethod(i, str);
    else if(condition2(str.charAt(i))) i = someOtherMethod(i, str)
    else ...
}

然而事情并没有那么简单。在某些情况下，condition() 方法必须先处理一些字符，然后才能以 true 或 false 结束。 如果为真，我们已经足够了解要创建哪个对象。 someMethod 或多或少已经过时了。我们可以直接在 condition() 中创建对象，但我们需要告诉循环 i 的新值。

解决方案是将 i 声明为全局/类变量，并在 condition() 中设置正确的值。然而，我们“隐藏”了值 i 的变化。

一种可能的解决方案是将方法本身作为条件的一部分。

int i = 0; 
int j = 0;

while(i < str.length()) {
    if(i < (j = someMethod(i, str))) i = j; 
    else if(i < (j = someOtherMethod(i, str))) i = j; 
    else..
}

如果不存在正确的字符序列，someMethod 将原样返回 i（i == j，然后执行下一个 if 条件）。如果存在正确的序列，someMethod 返回 i + 它使用的字符数（i

当方法用作循环中的条件时，我不喜欢在方法内部“神奇地”改变值，但我不喜欢 if(i

我是一名学生，我只是想了解你将如何以最易读/可维护的方式编写/解决这个问题？

Answer 1

使用可变全局变量int 可能不是一个好的解决方案：它可以在类中的任何地方出错，因此会破坏实际逻辑。
依赖中间局部变量的方式使读取变得更加复杂，原因并不好。

因此，将调用返回分配给i 的第一种方式听起来更好：

int i = 0;

while(i < str.length()) {
    if(condition(str.charAt(i))) i = someMethod(i, str);
    else if(condition2(str.charAt(i))) i = someOtherMethod(i, str)
    else ...
}

作为替代方案，您可以使用表示索引的可变包装类。
它避免返回新索引，因为它是可变的，因此您不需要在调用者方法中重新分配它。

您也可以使用AtomicInteger，但使用起来可能有点复杂。

public class Index {

    private int value;

    public void set(int value) {
        this.value = value;
    }

    public int get() {
        return value;
    }    
}

并使用它：

String str = ...;

for(Index index = new Index(); index.get() < str.length();) {       
    if(condition(str.charAt(index.get())))  someMethod(index, str);
    else if(condition2(str.charAt(index.get()))) someOtherMethod(index, str);
    else ...
}