java - 更改/增加方法内 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 < j，并且我们得到 i = j)。

当将其用作循环中的条件时，我不喜欢在方法内部“神奇地”更改值，但我不喜欢 if(i < (j = someMethod(i, str) )) 任何一个。

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

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 ...
}