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

    • было бы нормально, если бы перед 3ей строчкой или после нее были бы какие-нибудь действия
      Ответить
      • Тут все действия в ожидании окончания потока.
        Не просто так про Thread.join() упомянуто.
        Ответить
        • про то и речь.
          хотя бы использование логгера уже радует.
          Ответить
    • а... а что такое static field usage? :D 100? или шо?
      Ответить
      • поработаю КО немножко (ртфм):
        в яве метод Thread.sleep(long ms); является статическим (усыпить можно только поток, в котором этот метод вызван), поэтому использование его в контексте обьекта просто плохой тон
        Ответить

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