1. C# / Говнокод #12799

    +138

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    private static Dictionary<object, object> _cache = new Dictionary<object, object>();
.........
if( _cache.ContainsKey( key ) )
{
  lock( syncRoot )
  {
    _cache[key] = value;
    return;
  }
}
_cache.Add( key, value );

    Бараньи мозги.

    Запостил: grobotron, 25 Марта 2013

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

    • ava landanan 25.03.2013 13:30 # 0 

      lock (syncRoot)
   _cache[key] = value;
      Ответить

    • ava taburetka 25.03.2013 15:08 # −1

      Правильно, за такое бутылкой и по башке...
      Ответить

    • ava 3.14159265 25.03.2013 15:26 # +1

      Atomic AddOrUpdate они не проходили.
      Ну и помимо неатомарности еще и производительность такого однопоточного кеша оставляет желать лучшего.
      А ведь даже в шарпе (в котором перечислений не было до 3-ей версии, а аналога великолепной ConcurentHashMap до 4-ой ) таки удосужились написать ConcurrentDictionary.
      Потому тут джва варианта:
      - типичный нубский говнокеш
      - код времен .NET 3.5 или раньше
      Ответить

    • ava Lure Of Chaos 26.03.2013 11:14 # +1

      бараньи мозги - это очень вкусно!
      Ответить

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