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

    +110

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    public class CatalogCache : IDIsposable
{
    public void Dispose()
    {
        if (_loadThread != null)
        {
            if (_loadThread.IsAlive)
                _loadThread.Abort();
        }

        if (_updateThread != null)
        {
            if (_updateThread.IsAlive)
                _updateThread.Abort();
        }
}

    Запостил: abatishchev, 05 Апреля 2012

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

    • ava lucidfoxGovno 05.04.2012 12:52 # +5

      и опять без LINQ
      додиезники такие ламеры...
      Ответить

      • ava abatishchev 05.04.2012 12:54 # +7

        Специально для тебя:
        new[] { loadThread, updateThread  }
    .Where(t => t != null && t.IsAlive)
    .AsParallel()
    .ForEach(t => t.Abort());
        Ответить

        • ava roman-kashitsyn 05.04.2012 13:32 # +3

          LINQ, LINQ... В той же Scala это выглядит примерно также:
          Array(loadThread, updateThread).filter(t => t != null && t.isAlive).foreach(_.stop())
          Ответить

        • ava Говногость 05.04.2012 16:03 # +6

          >.AsParallel()
          ILOLD
          Ответить

          • ava roman-kashitsyn 05.04.2012 16:29 # +6

            Породим два потока чтобы грохнуть два потока
            а то двое на одного не честно
            Ответить

          • ava 3.14159265 05.04.2012 19:01 # +5

            Я об таком бездумном использовании .AsParallel() уже говорил.
            И пояснял на пальцах, почему оно обычно совсем не нужно.

            >Породим два потока чтобы грохнуть два потока
            Так это ж многопоточно! И быстрей в джва раза!!!
            Ответить

    • ava roman-kashitsyn 05.04.2012 13:12 # +3

      В этом вашем .NET делать аборты тоже считается дурным тоном?
      в java это называется Thread.stop() и давным давно задепрекейчено
      Ответить

      • ava abatishchev 05.04.2012 13:50 # 0

        аборт треда на диспоузе класса - я думаю да
        Ответить

        • ava roman-kashitsyn 05.04.2012 14:04 # +1

          аборт треда вообще (а вдруг у него там файлы открытые)
          у нас хорошим тоном считается примерно такая организация задачи для потока при ручной имплементации: 
          while(!currentThread().isInterrupted() && hasThingsToDo()) {
    doTheStuff();
}
cleanUp();
          Однако даже так сейчас мало кто пишет, ведь есть Executors framework.
          Ответить

          • ava lucidfoxGovno 05.04.2012 14:20 # −2

            у додиезников хорошим тоном считается если компилируется и если дали MVP
            Ответить

          • ava 3.14159265 05.04.2012 19:11 # 0

            >Executors framework
            Охеренно спроектированная вещь.
            На мой взгляд java.util.concurent одна из самых удобных вещей, что есть по дефолту в яве.
            Ответить

    • ava TarasB 05.04.2012 16:31 # 0

      А кто первый заметит, что одной } не хватает?
      Ответить

      • ava Steve_Brown 05.04.2012 16:56 # −1

        Да судя по IDIsposable, по крайней мере часть кода набита руками.

        Другой вопрос, а как правильно? Ну, если поток сделан, как выше показано, то задать условие для завершения и дальше Join.
        Не хватает _loadThread = null; , т.к. Dispose может вызваться и дважды.
        Другой момент, что ведь в Dispose нельзя обращаться к объектам, они ведь могут быть уже подчищены сборщиком мусора.
        Ответить

        • ava TarasB 05.04.2012 21:09 # +1

          > Другой момент, что ведь в Dispose нельзя обращаться к объектам, они ведь могут быть уже подчищены сборщиком мусора.

          ЧОЧО пока объектом кто-то владее, уборщик его не трогает
          Ответить

          • ava roman-kashitsyn 05.04.2012 21:13 # +1

            GC может собирать целые "острова" объектов, ссылающихся друг на друга
            Ответить

          • ava rat4 05.04.2012 21:14 # +1

            "Э, уборщик, пошёл отсюда, не трогай мои объекты!"
            Ответить

    • ava Lure Of Chaos 06.04.2012 00:00 # +1

      > if (_updateThread.IsAlive)
      > _updateThread.Abort();
      жертва аборта...
      Ответить

    • ava guest8 09.04.2019 13:01 # −999

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

    • ava Novi4oK 24.08.2021 02:22 # 0

      Нет, главное к подружкам своим, таким же потаскушкам, она меня не ревновала, к проституткам каких иногда приводил прямо домой, в те дни, когда она была «занята» – тоже, а тут, к матери родной ревнует!
      Ответить

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