今天我提交我的远程分支合并到git orgin/master的请求，亮哥给了我一些指导。


亮哥started a discussionon the diff

web/datacloud-web/app/com/mininglamp/datacloud/web/handler/db/DBUserHandler.scala

37

36}.flatMap {3837_.headOption match {3938case Some(x) => Future.failed(UserAlreadyExists(x.username))40
- case None => Future.successful(u.username)
39+ case None => {
40+ db.run {
41+ DataCloudUsers.forceInsert(u)
42+ }1

Master亮哥@Liangliang 21 minutes ago

这里如果是多线程的话，db还没成功，可能程序就返回Future.successful了。所以db.run{}.onComplete里返回successful


其实这样写在多线程环境中不能同步，用onComplete()方法做一个完美连接，就能保持同步。我现在在db.run后面没有用这个方法而直接返回了Future.successful(u.username)，这样确实不合理，要保持连贯性，或者用flatMap也可以。


顺便了解了一下


newscala.collection.mutable.LinkedHashMap[String,Any]()withscala.collection.mutable.SynchronizedMap[String,Any]

在scala中实现SynchronizedMap就可以实现同步,或者concurrent.Maptrait也可以.

cocurrentHashMap引入了分段锁的概念,不是锁整个HashMap,效率极大提高.


syncronized关键字也是一种方式:someMap.synchronized { someMap += ("someKey", "anyValue") }


http://www.scala-lang.org/docu/files/collections-api/collections_11.html


http://stackoverflow.com/questions/7952553/is-scalas-pattern-matching-regex-thread-safe


正则表达式由于继承类的关系,有些线程安全,有些则不是.scala.util.matching.Regex依赖于java.util.regex.Pattern,是线程安全的,但scala.util.matching.Regex.Match和scala.util.matching.Regex.MatchIterator不是线程安全的,scala.util.matching.Regex.MatchData是上述两个类的一部分,所以也找不到线程安全的实例.

因此在多线程环境中正则表达式匹配最好用scala.util.matching.Regex实现.


查了半天还是没查到match关键字和map是否线程安全,网上说的map是Map集合,不是map方法,可能需要查看一下底层实现.


后来经梁兄指导,每个消息都是独立的,内部肯定都会线程安全.