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

    +80

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    try {
     while (sender.isAlive()) {
         this.sleep(100);//static field usage(!)
     }
} catch (java.lang.InterruptedException e) {
     Log.log(Log.ERROR, this, e);
}

    типа Thread.join() не наш метод.

    Запостил: qnikst, 04 Сентября 2010

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

    • ava Lure Of Chaos 04.09.2010 19:46 # −1

      было бы нормально, если бы перед 3ей строчкой или после нее были бы какие-нибудь действия
      Ответить

      • ava absolut 04.09.2010 21:32 # 0

        Тут все действия в ожидании окончания потока.
        Не просто так про Thread.join() упомянуто.
        Ответить

        • ava Lure Of Chaos 04.09.2010 22:16 # −1

          про то и речь.
          хотя бы использование логгера уже радует.
          Ответить

    • ava Dreyk 04.09.2010 22:02 # 0

      а... а что такое static field usage? :D 100? или шо?
      Ответить

      • ava Lure Of Chaos 04.09.2010 22:18 # +1

        поработаю КО немножко (ртфм):
        в яве метод Thread.sleep(long ms); является статическим (усыпить можно только поток, в котором этот метод вызван), поэтому использование его в контексте обьекта просто плохой тон
        Ответить

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