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

    +166

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    MyClass * obj = (MyClass*)true;
while (obj)
{
    obj = GetObj();
    // ...
}

    Запостил: glook, 18 Мая 2010

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

    • ava pushkoff 19.05.2010 12:57 # −3

      плюсанул...
      но иногда такой код бывает оправданным... допустим, если перед вызовом GetObj нужно выполнить какое либо тействие, типа MoveNext()...
      Ответить

      • ava unfunk 19.05.2010 17:38 # +3

        уж лучше do-while написать, чем-так
        Ответить

        • ava pushkoff 19.05.2010 19:56 # −4

          при do..while() надо проверять на 0 первый объект, отсюда условие в цикле, что - плохо...
          я не говорю это это не говнокод, мне просто однажды пришлось написать такую конструкцию, так как при других раскладах, получалась либо дополнительные проверки на досрочный выход, либо использование функций получения очередного элемента в нескольких местах... (еще был вариант с goto, но религия не позволяет)...
          Ответить

          • ava guest 28.05.2010 14:49 # +1

            Но так и здесь же тоже нужна проверка, что GetObj() не вернул NULL!

            MyClass * obj;
while (obj = GetObj() ) {
// ...
}

            Вот и все, и не надо изобретать Слонопотама.
            Ответить

            • ava guest 28.05.2010 15:55 # −1

              компилятор С++ не любит такой код, тут будет warning
              надо так
              MyClass * obj;
              while ( (obj = GetObj()) != 0 )
              {
              // ...
              }
              С уважением
              Pushkoff
              З.Ы. Не могу залогиниться, прокся перестала пускать https трафик от яндекса.
              Ответить

              • ava guest 28.05.2010 16:07 # −1

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

                • ava guest 28.05.2010 16:10 # −1

                  Скобки сильно защищают лишней траты нервов...
                  Ответить

                  • ava guest 28.05.2010 16:14 # 0

                    ну это если не знать precedence
                    Ответить

                  • ava guest 28.05.2010 16:15 # −1

                    Ты эти сказочки школьникам рассказывай, а пока что С++ опять говном упоролся...
                    Ответить

                • ava guest 28.05.2010 16:14 # 0

                  дык в православном святотроицком си так же
                  Ответить

              • ava guest 31.05.2010 20:29 # 0

                MyClass * obj = GetObj();
                while (obj)
                {
                // код ...
                obj = GetObj();
                }

                Немного гормоничнее, не находите?
                Ответить

              • ava glook 29.06.2012 19:00 # 0

                Хватит и так:

                while ((obj = GetObj()))
                {
                }
                Ответить

        • ava absolut 19.05.2010 21:36 # 0

          Вообще не понятно, если основная работа цикла идет после GetObj() (о чем свидетельствует комментарий с троеточием), то почему условие выхода перед GetObj()? В таком контексте действительно разумнее do..while, тело выполнится хотя бы раз, чего видимо и добивались преобразованием от true.
          Ответить

          • ava pushkoff 20.05.2010 16:53 # 0

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

    • ava Говногость 19.05.2010 14:04 # 0

      Зачем это кому-то могло понадобиться?
      Ответить

      • ava ISith 24.05.2010 01:05 # 0

        Затем, что в любой почти конторе есть пачка долбоебов, которые считают, что такой код ог, и активно внедряют его в проекты =)
        Ответить

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