Grails中service的线程安全的小例子

阅读更多
不小心弄出一个线程不安全的Service
class ExcelImpService extends AbstractExcelImporter {
    ExcelImportService excelImportService

    List imp(FileStore excelFileStore, Map config) {
        String excelPath = excelFileStore.path
        assert new File(excelPath).exists()
        this. read(excelPath)
        println workbook.getSheetName(0)
        println workbook.getSheetName(0)  //停一下
        excelImportService.columns(workbook, config)
    }
}

因为默认依赖注入的service是单实例的,所以会出现下面的结果
---sout---
浏览器1:company
浏览器2:expenseGroup
浏览器2:expenseGroup
浏览器1:expenseGroup  (属性对象被别的线程修改了,应该还是company才对)

这个问题还是很隐晦的,而且线程安全问题一般不容易测试,所以要尽量从理论上消灭在萌芽状态
public abstract class AbstractExcelImporter extends imexporter.AbstractImexporter {
Workbook workbook= null  // 这个继承过来的属性就是隐患,每次调用read方法,都会修改之   
    ...略...
}
经验:service如果有属性,要么搞成final的,否则就要格外留意方法中对其修改的操作。

解决办法:
开始想把ExcelImpService改成一个普通类,但是不利于获得其它service的支持(excelImportService)。。。

把service声明为session或request级别的,体验一下基于使用条件的线程安全。
    /**
     * default(singleton) is NOT thread safe, since extends workbook prototype from AbstractExcelImporter
     * request -- 同一浏览器的同一tab是线程安全的
     * session -- 不同(厂商)浏览器间是线程安全的,同一浏览器的不同tab是不安全的(但一般没人那么无聊吧,所以这就够了)
     */
    static scope = 'session'

你可能感兴趣的:(grails,service,线程安全)