首页 > 解决方案 > 更改/增加方法内的while循环条件的值 - 好或坏 -

问题描述

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

问题的本质是“隐藏”用作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))) .

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

标签: javastringwhile-loopinterpreter

解决方案


使用可变全局变量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 ...
}

推荐阅读