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结束。如果为true，我们已经足够了解可以创建哪个对象。 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，如果条件执行则返回下一个）。如果存在正确的序列，则someMethod返回i +它使用的字符数（i

当将其用作循环条件时，我不喜欢在方法内部“神奇地”更改值，但我不是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 ...
}

更改/增加方法中while循环条件的值-好还是不好-

1 个答案: