轉自開源中國:https://my.oschina.net/u/3777556/blog/3011167
大年初二，朋友問了我一個技術的問題(朋友實在是好學，佩服！)
該問題來源知乎(synchronized鎖問題)：
開啓10000個線程，每個線程給員工表的money字段【初始值是0】加1，沒有使用悲觀鎖和樂觀鎖，但是在業務層方法上加了synchronized關鍵字，問題是代碼執行完畢後數據庫中的money 字段不是10000，而是小於10000 問題出在哪裏？
Service層代碼：
SQL代碼(沒有加悲觀/樂觀鎖)：
用1000個線程跑代碼：
簡單來說：多線程跑一個使用synchronized關鍵字修飾的方法，方法內操作的是數據庫，按正常邏輯應該最終的值是1000，但經過多次測試，結果是低於1000。這是爲什麼呢？
一、我的思考
既然測試出來的結果是低於1000，那說明這段代碼不是線程安全的。不是線程安全的，那問題出現在哪呢？衆所周知，synchronized方法能夠保證所修飾的代碼塊、方法保證有序性、原子性、可見性。
講道理，以上的代碼跑起來，問題中Service層的increaseMoney()是有序的、原子的、可見的，所以斷定跟synchronized應該沒關係。
(參考我之前寫過的synchronize鎖筆記：Java鎖機制瞭解一下)
既然Java層面上找不到原因，那分析一下數據庫層面的吧(因爲方法內操作的是數據庫)。在increaseMoney()方法前加了@Transcational註解，說明這個方法是帶有事務的。事務能保證同組的SQL要麼同時成功，要麼同時失敗。講道理，如果沒有報錯的話，應該每個線程都對money值進行+1。從理論上來說，結果應該是1000的纔對。
(參考我之前寫過的Spring事務：一文帶你看懂Spring事務！)
根據上面的分析，我懷疑是提問者沒測試好(hhhh，逃)，於是我也跑去測試了一下，發現是以提問者的方式來使用是真的有問題。
首先貼一下我的測試代碼：
@RestController
public class EmployeeController {
@Autowired
private EmployeeService employeeService;
@RequestMapping("/add")
public void addEmployee() {
for (int i = 0; i < 1000; i++) {
new Thread(() -> employeeService.addEmployee()).start();
}
}
}
@Service
public class EmployeeService {
@Autowired
private EmployeeRepository employeeRepository;
@Transactional
public synchronized void addEmployee() {
// 查出ID爲8的記錄，然後每次將年齡增加一
Employee employee = employeeRepository.getOne(8);
System.out.println(employee);
Integer age = employee.getAge();
employee.setAge(age + 1);
employeeRepository.save(employee);
}
}
簡單地打印了每次拿到的employee值，並且拿到了SQL執行的順序，如下(貼出小部分)：
從打印的情況我們可以得出：多線程情況下並沒有串行執行addEmployee()方法。這就導致對同一個值做重複的修改，所以最終的數值比1000要少。
二、圖解出現的原因
發現並不是同步執行的，於是我就懷疑synchronized關鍵字和Spring肯定有點衝突。於是根據這兩個關鍵字搜了一下，找到了問題所在。
我們知道Spring事務的底層是Spring AOP，而Spring AOP的底層是動態代理技術。跟大家一起回顧一下動態代理：
public static void main(String[] args) {
// 目標對象
Object target ;
Proxy.newProxyInstance(ClassLoader.getSystemClassLoader(), Main.class, new InvocationHandler() {
@Override
public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
// 但凡帶有@Transcational註解的方法都會被攔截
// 1... 開啓事務
method.invoke(target);
// 2... 提交事務
return null;
}
});
}
(詳細請參考我之前寫過的動態代理：給女朋友講解什麼是代理模式)
實際上Spring做的處理跟以上的思路是一樣的，我們可以看一下TransactionAspectSupport類中invokeWithinTransaction()：
調用方法前開啓事務，調用方法後提交事務
在多線程環境下，就可能會出現：方法執行完了(synchronized代碼塊執行完了)，事務還沒提交，別的線程可以進入被synchronized修飾的方法，再讀取的時候，讀到的是還沒提交事務的數據，這個數據不是最新的，所以就出現了這個問題。
三、解決問題
從上面我們可以發現，問題所在是因爲@Transcational註解和synchronized一起使用了，加鎖的範圍沒有包括到整個事務。所以我們可以這樣做：
新建一個名叫SynchronizedService類，讓其去調用addEmployee()方法，整個代碼如下：
@RestController
public class EmployeeController {
@Autowired
private SynchronizedService synchronizedService ;
@RequestMapping("/add")
public void addEmployee() {
for (int i = 0; i < 1000; i++) {
new Thread(() -> synchronizedService.synchronizedAddEmployee()).start();
}
}
}
// 新建的Service類
@Service
public class SynchronizedService {
@Autowired
private EmployeeService employeeService ;
// 同步
public synchronized void synchronizedAddEmployee() {
employeeService.addEmployee();
}
}
@Service
public class EmployeeService {
@Autowired
private EmployeeRepository employeeRepository;
@Transactional
public void addEmployee() {
// 查出ID爲8的記錄，然後每次將年齡增加一
Employee employee = employeeRepository.getOne(8);
System.out.println(Thread.currentThread().getName() + employee);
Integer age = employee.getAge();
employee.setAge(age + 1);
employeeRepository.save(employee);
}
}
我們將synchronized鎖的範圍包含到整個Spring事務上，這就不會出現線程安全的問題了。在測試的時候，我們可以發現1000個線程跑起來比之前要慢得多，當然我們的數據是正確的：
還有一個解決辦法,將事務隔離級別從默認變成讀未提交,則也是可以的.
<tx:method name="tranFor*" propagation="REQUIRED" isolation="READ_UNCOMMITTED" rollback-for="java.lang.Exception"/>