1. C++ / Говнокод #8814

    +993

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if(pObject == NULL)
{
        //code
}else if(pObject != NULL && pObject->getProperty() == Class::VALUE)
{
       //code
}

    Идентификаторы надуманные, но суть такова

    Запостил: google-moogle, 14 Декабря 2011

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

    • ava kyzi007 14.12.2011 08:43 # +3

      А в чем прикол?
      Ответить

      • ava google-moogle 14.12.2011 09:35 # 0

        Зачем вторая проверка на pObject != NULL ?
        Ответить

        • ava kyzi007 14.12.2011 10:27 # +4

          Что то мне подсказывает что такой фигни в любом проекте найти дофига можно, нисмешно.
          Ответить

    • ava defecate-plusplus 14.12.2011 09:03 # +3

      а #14/88 считается гетом?
      Ответить

    • ava TarasB 14.12.2011 10:45 # −2

      Комбинаторная избыточность?
      Ответить

    • ava Psionic 14.12.2011 17:46 # 0

      Когда уже народ выучит что вход иф может быть не только через оператор типа bool?
      Ответить

      • ava roman-kashitsyn 14.12.2011 17:48 # 0

        а что плохого в == NULL?
        Ответить

      • ava TarasB 15.12.2011 14:53 # +2

        Когда уже народ выучит, что для небулевых выражений читаемее не сокращать?
        Ответить

        • ava Steex 19.01.2012 14:41 # 0

          Дело вкуса, в общем-то. Я обычно не пишу != NULL, но при этом оставляю == NULL.
          А код, подобный приведенному выше обычно появляется после изменения существующего кода. Понадобилось в какой-то момент времени обработать нулевой указатель — вот и дописали секцию.
          Ответить

          • ava guest 15.09.2012 15:11 # 0

            проще изначально на ==NULL проверять, а не бред всякий писать, как автор того дивного кода, 2 прорки, это избыточность
            Ответить

    • ava Steve_Brown 15.12.2011 17:27 # 0 

      if (pObject && pObject->IsSomething())
  Foo();
if (pObject == NULL)
  LogWarning(...);
if ((i > 0 && pObject != NULL) || IsValid)
  Bar(...);
if (pObject)
  pObject->Release();

      Как-то так - вроде эстетично.
      Ответить

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