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

    +23

    1. 1
    2. 2
    if(this==NULL)
  return;

    Запостил: HaskellGovno, 04 Июля 2012

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

    • ava defecate-plusplus 04.07.2012 09:13 # +5

      (с) bormand
      http://ideone.com/2mCis
      Ответить

      • ava Xom94ok 04.07.2012 20:00 # +5

        Нравится мне раздел этого сайта, посвященный C++. В нем можно найти такие жемчужины, после которых мой код выглядит узенькой тропинкой на минном поле.
        Ответить

    • ava Psionic 04.07.2012 09:54 # +2

      Вот как толко народ не извращается в борьбе с ацес вилолайшен.
      Ответить

      • ava bormand 04.07.2012 10:06 # +3

        Ага. Лучше ацесс виолейшн, чем скрытый баг. Че с ним бороться...
        Ответить

        • ava HaskellGovno 04.07.2012 21:38 # 0

          На самом деле тут есть одно но. Я не уверен, что это говнокод. Потому что он используется по другому. Есть некий шаблон интрузивного списка, так вот это кусок кода из функции foreach. Думаю понятно, что если указатель на интрузивный список пуст, то значит и сам список пуст, поэтому list->foreach(functor); должен ни разу не вызвать functor.
          Ответить

          • ava bormand 05.07.2012 06:13 # 0

            Ясно, ну тогда логика в этом есть.
            Ответить

          • ava Psionic 05.07.2012 09:31 # +1

            Ну незнаю, я подобные вещи в стречал в коде где люди пытались спасти прогу от отправления в страну вечной охоты.
            Ответить

    • ava diok 05.07.2012 02:55 # 0

      Фантомный объект.
      Ответить

    • ava ctm 05.07.2012 06:54 # 0

      а вот интересно, на каких компиляторах это работать не будет?
      ЗЫ: в дельфях таким образом метод Free у TObject реализован. видимо чтобы можно было вызывать для инициализированных в нулл указателей. Разрыв шаблона был, когда увидел 1-й раз такое.
      Ответить

      • ava bormand 05.07.2012 07:43 # +2

        По идее должно на всех, главное не написать это внутри виртуального метода.
        Ответить

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