学习Java时看Java编程思想时的一个问题

[GFive555]

public class EvenGenerator {

	private volatile boolean canceled = false;
	private int currentEvenValue = 0;
	public int next() {
		++currentEvenValue;
		Thread.yield();
		++currentEvenValue;
		return currentEvenValue;
	}
	public void cancel(){
		canceled = true;
	}
	public boolean isCancel(){
		return canceled;
	}
}
public class EvenChecker  implements Runnable{

	private EvenGenerator even;
	private final int id ;
	public EvenChecker(EvenGenerator even,int id){
		this.even = even;
		this.id = id;
		
	}
	public void run() {
		while(!even.isCancel()){
			int val = even.next();
			
			if(val%2 != 0){
				System.out.println(val +  "  not even");
				even.cancel();
			}
			System.out.println(val + "  even");
			if(count++ >20){
				return;
			}
		}
		
	}
	

}
public class Init {
	public static void main(String[] args) {

		ExecutorService exs = Executors.newCachedThreadPool();
		for(int i=0;i<3;i++)
			exs.execute(new EvenChecker(new EvenGenerator(),i));
		exs.shutdown();
	}
}

书上说这样可能出来奇数，但是我运行了半天都没有一个奇数而且每次是三个偶数同时出来

[soton_dolphin]

你创建了三个线程，每个线程都有一个独立的EvenChecker, 每个EventChecker都有独立的EvenGenerator. evenGenerator.next() 返回的是双数啊。再看看代码是不是敲错了

[GFive555]

引用 1 楼 soton_dolphin 的回复:

你创建了三个线程，每个线程都有一个独立的EvenChecker, 每个EventChecker都有独立的EvenGenerator. evenGenerator.next() 返回的是双数啊。再看看代码是不是敲错了

是我改了一下代码，在原来的代码EvenChecker里面有一个test方法

public static void test(EvenGenerator gen){
		ExecutorService exs = Executors.newCachedThreadPool();
		for(int i=0;i<3;i++){
			exs.execute(new EvenChecker(gen, 10));
		}
		exs.shutdown();
	}

主函数为

public static void main(String[] args) {

		EvenChecker.test(new EvenGenerator());
	}

这样就有奇数 ，这是为啥啊？？？？

[GFive555]

产生的结果为什么也这么奇怪？ 4 even 5 not even 3 not even 3 even 7 not even 5 even 7 even

[soton_dolphin]

引用 2 楼 GFive555 的回复:

[quote=引用 1 楼 soton_dolphin 的回复:] 你创建了三个线程，每个线程都有一个独立的EvenChecker, 每个EventChecker都有独立的EvenGenerator. evenGenerator.next() 返回的是双数啊。再看看代码是不是敲错了

是我改了一下代码，在原来的代码EvenChecker里面有一个test方法

public static void test(EvenGenerator gen){
		ExecutorService exs = Executors.newCachedThreadPool();
		for(int i=0;i<3;i++){
			exs.execute(new EvenChecker(gen, 10));
		}
		exs.shutdown();
	}

主函数为

public static void main(String[] args) {

		EvenChecker.test(new EvenGenerator());
	}

这样就有奇数 ，这是为啥啊？？？？[/quote] 因为test里面共享了一个EvenGenerator

[GFive555]

引用 4 楼 soton_dolphin 的回复:

[quote=引用 2 楼 GFive555 的回复:] [quote=引用 1 楼 soton_dolphin 的回复:] 你创建了三个线程，每个线程都有一个独立的EvenChecker, 每个EventChecker都有独立的EvenGenerator. evenGenerator.next() 返回的是双数啊。再看看代码是不是敲错了

是我改了一下代码，在原来的代码EvenChecker里面有一个test方法

public static void test(EvenGenerator gen){
		ExecutorService exs = Executors.newCachedThreadPool();
		for(int i=0;i<3;i++){
			exs.execute(new EvenChecker(gen, 10));
		}
		exs.shutdown();
	}

主函数为

public static void main(String[] args) {

		EvenChecker.test(new EvenGenerator());
	}

这样就有奇数 ，这是为啥啊？？？？[/quote] 因为test里面共享了一个EvenGenerator[/quote] 嗯，明白了，谢谢。 但是那个结果里为什么会有7 是偶数这样的结果

[galiniur0u]

因为test共享了这个EvenGenerator，而EvenGenerator的next方法非线程安全的，所以会产生奇数的情况

[GFive555]

自学Java可能问了一些比较无聊的问题，感谢大神的回答。谢谢！！