1. Java / Говнокод #6770

    +78

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    public static DBFactory getInstance() throws SQLException
	{
		synchronized (DBFactory.class)
		{
			        if (_dbinstance == null)
				_dbinstance = new DBFactory();
		}
		return _dbinstance;
	}

    Серийный и беспощадный доступ к инстансу ДБ-Фэктори -_-
    Причем, данный код фурычит в многопоточной среде, где порой по 20-30 потоков одновременно обращаются к Фэктори, чтобы взять коннект с пула.

    Запостил: VirtualVoid, 27 Мая 2011

    Комментарии (9) RSS

    • ava sayidandrtfm 27.05.2011 19:32 # +2

      В очередь, сукины дети, в очередь!(с)
      Ответить

    • ava Lure Of Chaos 28.05.2011 23:55 # 0

      а в чем говно? по мне, грамотный синглтон
      Ответить

      • ava ingenuus 29.05.2011 00:03 # 0

        Может в том, что надо было сделать так:
        if(_dbinstance == null) {
    synchronized(DBFactory.class) {
         _dbinstance = new DBFactory();
    }
}
        Ответить

        • ava Lure Of Chaos 29.05.2011 00:35 # 0

          нет, нельзя, ибо между проверкой и присвоением другой поток может инициализировать. нужно именно в критической секции и проверять и присваивать
          Ответить

          • ava Eugene 30.05.2011 07:51 # +1

            существует техника "Double-Checked Locking":
            if(_dbinstance == null) {
    synchronized(DBFactory.class) {
        if(_dbinstance == null) {
             _dbinstance = new DBFactory();
        }
    }
}

            но она тоже не работает, т.к. компилятор может поменять порядок инициализации DBFactory и _dbinstance.
            Как написано тут: http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html
            There is no way to make it work without requiring each thread that accesses the helper object to perform synchronization.
            Лучше использовать 
            class DBFactory {
    static DBFactory singleton = new DBFactory();
}

            т.к. в этом случае инициализация гарантирована самой средой
            Ответить

            • ava VirtualVoid 30.05.2011 12:16 # 0

              Насколько я помню, паттерн D.C.L можно смело применять при использовании модификатора Volatile.
              А иначе точно каша получится.

              Но тем не менее это нонсенс, синглтоны как раз для этой задачи.
              Ответить

        • ava VirtualVoid 29.05.2011 00:38 # 0

          Можно, но не нужно.
          Такую задачу проще решить с применением синглтона, чем проверкой-созданием экземпляра.
          Ответить

      • ava Skipy 30.05.2011 16:12 # 0

        При такой нагрузке делать отложенную инициализацию, мягко говоря, нерационально. К тому же таким образом. Надо _dbinstance было создать в статическом блоке инициализации - и вообще без синхронизации можно обойтись было бы.
        Ответить

    • ava guest8 09.04.2019 10:50 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий