Jawa / Говнокод #11477 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
  23. 23
  24. 24
  25. 25
  26. 26
/* ------------------------------------------------------------ */
    private boolean isJspServlet ()
    {
        if (_servlet == null)
            return false;
        
        Class c = _servlet.getClass();
        
        boolean result = false;
        while (c != null && !result)
        {
            result = isJspServlet(c.getName());
            c = c.getSuperclass();
        }
        
        return result;
    }
    
    
    /* ------------------------------------------------------------ */
    private boolean isJspServlet (String classname)
    {
        if (classname == null)
            return false;
        return ("org.apache.jasper.servlet.JspServlet".equals(classname));
    }

Копался в исходниках Jetty

Запостил: konsoletyper konsoletyper, (Updated )

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

  • ava someone # Ссылка на оригинал 0 

    return _servlet != null && JspServlet.class.isAssignableFrom(_servlet.getClass());


    Fixed?
    Ответить

    • ava konsoletyper # Ссылка на оригинал 0 ⇈

      На самом деле, всё несколько сложнее, если учитывать архитектуру класслоадеров. Должно быть вроде
      if (_servlet == null)
            return false;
try {
    Class<?> cls = Class.forName("org.apache.jasper.servlet.JspServlet", false,
            _servlet.getClass().getClassLoader());
    return cls.isInstance(_servlet);
} catch (ClassNotFoundException ex) {
    return false;
}


      Но всё равно, код Jetty поражает рядом дурацких вещей вроде ненужного сравнения с null в isJspServlet(String) или паскалеподобным стилем выхода из цикла.
      Ответить

    • ava 3.14159265 # Ссылка на оригинал 0 ⇈

      Всё праильно, но почему не упростить еще?
      return _servlet != null && _servlet instanceof JspServlet; 

//А если еще и вспомнить что instanceof проверяет на null, то достаточно

return _servlet instanceof JspServlet;
      Ответить

        • ava bormand # Ссылка на оригинал 0 ⇈

          Т.е. null instanceof любого типа? Ну какая-то логика в этом есть (ссылка на объект любого класса может быть нуллом), но это же неюзабельно...

          P.S. http://ideone.com/MYkvT
          Ответить

          • ava 3.14159265 # Ссылка на оригинал 0 ⇈

            >ссылка на объект любого класса может быть нуллом
            >instanceof
            А инстанс не может.
            Да и вообще надо помнить о SQLных nullах. Они даже друг дружке не равны.
            Ответить

            • ava bormand # Ссылка на оригинал 0 ⇈

              > А инстанс не может.
              Тогда уж так: "null не может быть инстансом [никакого класса]".
              Ответить

                  • ava roman-kashitsyn # Ссылка на оригинал 0 ⇈

                    кстати, мануал к scala.Nothing: Nothing is - together with Null - at the bottom of Scala's type hierarchy.

                    Nothing is a subtype of every other type (including Null); there exist no instances of this type. Although type Nothing is uninhabited, it is nevertheless useful in several ways. For instance, the Scala library defines a value scala.collection.immutable.Nil of type List[Nothing]. Because lists are covariant in Scala, this makes scala.collection.immutable.Nil an instance of List[T], for any element of type T.

                    Another usage for Nothing is the return type for methods which never return normally. One example is method error in sys, which always throws an exception.
                    Ответить

                  • ava HaskellGovno # Ссылка на оригинал 0 ⇈

                    >А у void'а вообще нет инстансов 😉
                    http://ideone.com/S4HaU
                    void f(void){
        return void();//Создаем и возвращаем инстанс вйода.
};
                    результат: успешно	     время: 0.01s    память: 2676 kB     возвращаемое значение: 0
                    Ответить

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓