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

    +123

    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
    if(elementNode != null) {
	//attachGrowingEffectAnimation(elementNode);
	String shadowMode = elementNode.getUserData(JMEScene3D.SHADOW_MODE);
	if(shadowMode != null) {
		if(shadowMode.equals(JMEScene3D.SHADOW_MODE_RECEIVE)) {
			shadowNodesMap.put(ShadowMode.Receive, elementNode);
		}
	}
	
	if(elementNode != null) {
		pyElementsNode.attachChild(elementNode);
	}
}

    Ну правда, а вдруг сам по себе станет не null?

    Запостил: someone, 27 Декабря 2012

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

    • ava 3.14159265 27.12.2012 14:50 # +2

      shadowMode.equals может быть с побочным эффектом.
      Ответить

      • ava inkanus-gray 27.12.2012 14:51 # 0

        А я бы ожидал подвоха в следующей строке.
        Ответить

        • ava 3.14159265 27.12.2012 15:24 # 0

          Если говорить серъезно, то там закомментирован кусок.
          //attachGrowingEffectAnimation(elementNode );
          Ответить

          • ava someone 27.12.2012 15:53 # +3

            Этот закомментированный кусок всё равно не может устанавливать переменную в null.
            Ответить

            • ava 3.14159265 29.12.2012 16:50 # +1

              Ну а если его раскомментировать то сможет. Ваш КЭП
              Ответить

              • ava wissenstein 30.12.2012 18:54 # 0 

                public class NullPointerTest {

    private void attachGrowingEffectAnimation(String elementNode) {
        elementNode = null;
    }
    
    @Test
    public void testNullIsNotSetInsideFunction() throws Exception {
        String elementNode = "non-null";
        attachGrowingEffectAnimation(elementNode);
        assertNotNull(elementNode);
    }
}

                отрабатывает успешно.
                То есть, таки не сможет
                Ответить

                • ava bormand 30.12.2012 19:11 # +1

                  А если elementNode это член, а не локальная переменная - то сможет: https://ideone.com/xjoyYB
                  Ответить

                  • ava wissenstein 30.12.2012 19:17 # +1

                    Ну, да… Я не учёл, что контекст тоже вполне может оказаться говнокодом
                    public class NullPointerTest {
    
    private String elementNode;

    private void attachGrowingEffectAnimation(String elementNode) {
        this.elementNode = null;
    }
    
    @Test
    public void testNullIsSetInsideFunction() throws Exception {
        elementNode = "non-null";
        attachGrowingEffectAnimation(elementNode);
        assertNull(elementNode);
    }
}
                    Ответить

                    • ava bormand 30.12.2012 19:20 # 0

                      Кстати, а кто-нибудь в курсе, поддерживает ли ideone JUnit?
                      Ответить

                  • ava wissenstein 30.12.2012 19:26 # +1

                    И, кстати, этот пример (https://ideone.com/xjoyYB) как раз не показывает возможности обnullения elementNode.
                    Ответить

                    • ava bormand 30.12.2012 19:30 # +1

                      Ну да, я второпях this забыл, и занулил аргумент вместо члена, а на результат даже не посмотрел ;(
                      И на старуху бывает проруха.
                      Ответить

      • ava guest 18.01.2013 10:43 # 0

        почему?
        Ответить

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