我有一个在另一个线程上运行的无限while循环。在这个while循环中,我使用的是String。对于这个String,我也有一个二传手。当有人调用setter时,我希望它等待while循环的当前迭代完成,然后将其更改为下一次迭代。
这是我尝试过的:
import java.util.concurrent.TimeUnit;
public class Test {
private String testString = "aaaaaaa";
private Test() {
new Thread(() -> {
while (true) {
synchronized (testString) {
System.out.println(testString);
// Simulating a thread blocking operation
try {
TimeUnit.SECONDS.sleep(10);
} catch (InterruptedException exception) {
exception.printStackTrace();
}
System.out.println(testString);
}
}
}).start();
}
private void setTestString(String testString) {
synchronized (testString) {
this.testString = testString;
}
}
public static void main(String[] args) {
Test test = new Test();
try {
TimeUnit.SECONDS.sleep(5);
} catch (InterruptedException exception) {
exception.printStackTrace();
}
test.setTestString("bbbbbbbb");
}
}
预期输出:
aaaaaaa
aaaaaaa
bbbbbbbb
bbbbbbbb
bbbbbbbb
bbbbbbbb
...
实际输出:
aaaaaaa
bbbbbbbb
bbbbbbbb
bbbbbbbb
bbbbbbbb
bbbbbbbb
...
为什么setTestString
方法没有等待?我在做什么错了?
答案 0 :(得分:1)
问题在于您的线程每次都在读取更新的实例变量。如果希望循环的迭代看到一致的值,则应该只读取一次实例变量的值。另外,由于您是从一个线程中编写它,而从另一个线程中读取它,因此您需要使其变得易失或在同步方法中进行读写。
private volatile String testString = "aaaaaaa";
private Test() {
new Thread(() -> {
while (true) {
// volatile read
String localTestString = testString;
System.out.println(localTestString);
// Simulating a thread blocking operation
try {
TimeUnit.SECONDS.sleep(10);
} catch (InterruptedException exception) {
exception.printStackTrace();
}
System.out.println(localTestString);
}
}).start();
}
当有人调用setter时,我希望它等待while循环的当前迭代完成
那不是一个好的解决方案。如果让set方法等待,则不能保证在循环的当前迭代完成后对其进行调度。您可以自己查看:将set方法更改为
private void setTestString(String testString) {
synchronized (this.testString) {
this.testString = testString;
}
}