我是新人，关于Java线程的问题，很简单的例子，可是我就是调不出来，不知道哪里错了

0 0

我是新人，关于Java线程的问题，很简单的例子，可是我就是调不出来，不知道哪里错了5

我想试一下synchronized关键字是不是有效?所以写了一个很简单的类，但是就是不能调到数字按序排列。不知道错在哪里了。

数字序列类：

package com.testthread;

public class UnsafeSequence {
	private int value = 0;
	public synchronized int getValue(){
		value = value+1;
		return value;
	}
}

线程类

package com.testthread;

public class TestThread extends Thread {
	private UnsafeSequence us;

	public TestThread(UnsafeSequence us, String threadname) {
		super(threadname);
		this.us = us;
	}

	@Override
	public void run() {
		String classname = this.getClass().getSimpleName();
		String threadname = currentThread().getName();
		for (int i = 0; i < 5; i++) {
			System.out.println(classname + "[" + threadname + "]:"
					+ us.getValue());
		}
	}
}

Main类

package com.testthread;

public class MainClass {
	public static void main(String[] args) throws InterruptedException {
		System.out.println("Main started");
		UnsafeSequence us = new UnsafeSequence();
		TestThread at = new TestThread(us,"at");
		TestThread bt = new TestThread(us,"bt");
		
		at.start();
		bt.start();
		System.out.println("Main ended");
	}
}

可是结果是：

Main started
Main ended
TestThread[bt]:2
TestThread[bt]:3
TestThread[at]:1
TestThread[at]:5
TestThread[at]:6
TestThread[at]:7
TestThread[at]:8
TestThread[bt]:4
TestThread[bt]:9
TestThread[bt]:10

我想让数字按序排列，可是数字没有按序排列，请问哪里写错了，谢谢

问题补充：我的机器是多核cpu，会不会和多核cpu有关呢？

Java thread

2013年1月29日 22:20

slargychen
14
0 0 5

4个答案按时间排序按投票排序

0 0

采纳的答案

你真要输出1-10的话，应该在UnsafeSequence.getValue里面输出value的值，肯定就是1-10了，因为System.out.println的执行和getValue的执行是两个方法的调用，getValue方法调用返回以后，不一定它就马上执行了System.out.println，因为这个时候可能又转到别的线程执行去了，加synchronized唯一能保证的是，getValue无论多少个线程同时调用，他一定会序列化执行，也就是在这里输出value的值，一定是递增的。

2013年1月29日 23:55

lbfhappy
281
0 0 15

添加评论

0 0

你的问题是这样的楼上的已经说的很清楚了   我在简单说一下
若多个线程同时跑到
    System.out.println(classname + "[" + threadname + "]:"
                    + us.getValue());
这句话这时方法getValue（）会有锁所以只能一个线程进去但是并不能保证谁先输出结果所以顺序会乱

修改：
   1.去掉 getValue（）方法前的 Synchronized
   2.synchronized (us) {
System.out.println(classname + "[" + threadname + "]:"
+ us.getValue());
     }
（在输出那句话上用synchronized锁定这样保证一个一个的进入）

2013年1月30日 11:44

502220545
87
0 1 7

添加评论

0 0

public class UnsafeSequence {
	private AtomicInteger value = new AtomicInteger(0);

	public int  getValue() {
		return value.incrementAndGet();
	}

	public static void main(String[] args) {
		System.out.println("Main started");
		UnsafeSequence us = new UnsafeSequence();
		TestThread at = new TestThread(us, "at");
		TestThread bt = new TestThread(us, "bt");

		at.start();
		bt.start();
		System.out.println("Main ended");

	}
}

class TestThread extends Thread {
	private UnsafeSequence us;

	public TestThread(UnsafeSequence us, String threadname) {
		super(threadname);
		
		this.us = us;
	}

	@Override
	public void run() {
		String classname = this.getClass().getSimpleName();
		String threadname = currentThread().getName();
		for (int i = 0; i < 10; i++) {
			synchronized (us) {
				System.out.println(classname + "[" + threadname + "]:" + us.getValue());
			}
			
		}
	}
}

2013年1月30日 11:38

angeltping
10
0 0 1