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

    +929.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    page_t* make_page()
{
 page_t* p = new page_t;
 // заполнение
 return p;
 delete(p);
}

    я в шоке... не знаю что хотел сказать аффтар (недоступен)... не могу даже прокомментировать ЭТО %)

    Запостил: zaufi, 11 Февраля 2010

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

    • ava Barmaglot 11.02.2010 18:26 # +6.2

      Автор где-то услышал, что ресурсы нужно создавать и удалять в одном месте. А так как удалять объект по логике нельзя - переносим delete за return :-)
      +
      Ответить

      • ava nil 12.02.2010 12:02 # 0

        В одном месте, это стопудов внутри составного оператора!
        {
        new
        delete
        }
        Ответить

      • ava zaufi 12.02.2010 16:29 # +4

        из моего прошлого общения с аффтаром я думаю что врядли он вообще чтото слышал или читал про хорошие практики в С++... есть мнение что ему вообще неведомо "что такое хорошо и что такое плохо" в С++ и в программировании вообще... все что он понимает это "работает" или "не работает" -- и не важно (вообще, абсолютно, тотально не важно!) каким способом это достигнуто (к примеру, съеданием всей доступной памяти или на 100% занятым ЦПУ)... когда начинаешь с ним разговаривать мысли об убийстве приходят довольно скоро ;(
        Ответить

    • ava del 11.02.2010 18:41 # 0

      Вау, даже не знаю, что и сказать )
      Ответить

    • ava TarasB 11.02.2010 19:16 # +0.2

      Это повышает читабельность кода.
      Ответить

    • ava guest 11.02.2010 21:14 # +1.2

      Сильно.
      Отдал ресурс - удали.
      Хотя, однажды, я, будучи в трезвом уме и здравой памяти написал что-то в духе
      StringStream->Free();
return StringStream->Size;

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

    • ava nil 11.02.2010 23:13 # +6

      «Перед прочтением сжечь»
      Ответить

    • ava guest 11.02.2010 23:32 # 0

      ужас
      а вдруг ексепшн во время заполнения?
      неужели так трудно приучить себя использовать умные указатели?
      Ответить

    • ava guest 12.02.2010 11:05 # 0

      Гениально)))))))
      Ответить

    • ava guest 12.02.2010 17:15 # 0

      Чувак просто забыл написать конструкцию try catch finally :D
      Ответить

      • ava mutanabbi 12.02.2010 18:15 # 0

        Боюсь чувак не знаком с этой конструкцией. Тут эльдорадо говнокода у этого чувака.
        Ответить

        • ava gost 26.09.2020 22:20 # 0

          С ней никто не знаком. В «C++» нет никакого «finally».
          Ответить

          • ava bormand 26.09.2020 22:25 # 0

            BOOST_SCOPE_EXIT наиболее похож по смыслу на finally. Но он нинужен.
            Ответить

    • ava ccoder 03.03.2010 11:55 # 0

      Вот из-за таких писак и приходится часами искать в проектах, которые приходилось дорабатывать, где же вылетает программа!
      Ответить

    • ava guest 01.04.2010 00:39 # +1

      Видимо это комментарий на память, что надо делать с возвращаемым значением :)
      Ответить

    • ava Myxa 27.09.2020 00:57 # 0

      В языках с генераторами было бы что-то типа: 
      {
 page_t* p = new page_t;
 yield p; // вернём новый объект на первой итерации
 delete(p); // и удалим на второй
 yield nullptr;
}
      Увы, в «C++» генераторов нет...
      Ответить

      • ava OCETuHCKuu_nemyx 27.09.2020 01:01 # 0

        Есть ли языки с генераторами и без гц?
        Ответить

        • ava bormand 27.09.2020 01:06 # 0

          Кресты.
          Ответить

          • ava Myxa 27.09.2020 01:11 # 0

            А как в них выглядят генераторы?
            Ответить

            • ava OCETuHCKuu_nemyx 27.09.2020 01:13 # 0

              Наверно он имеет в виду парашу с итераторами.
              Ответить

              • ava OCETuHCKuu_nemyx 27.09.2020 01:14 # 0

                Ванишед.
                Ответить

              • ava bormand 27.09.2020 01:15 # 0

                А сейчас бустовые корутины есть:
                void foo(boost::coroutine<int>::push_type &sink) {
    while (true)
        sink(42);
}
                Ответить

                • ava rotoeb 27.09.2020 01:19 # 0

                  Переведи на "PHP". Минут пять смотрел на этот код и нихуя в итоге не понял. Какие-то четвероточия, блядь, какие-то амперсанды, и всё это как-то по-дебильному расставлено.
                  Ответить

                • ava bormand 27.09.2020 01:26 # +2

                  1.5 секунды конпеляции этой поебени на почти 5ГГц проце - охуенно, конечно. И ошибки от опечаток в экран не влезают. Походу с корутинами лучше дождаться с++20.
                  Ответить

                  • ava CHayT 27.09.2020 20:38 # +2

                    Метушня течёт в нашем коде, раскаляя процы.
                    Ответить

                    • ava defecate-plusplus 27.09.2020 21:06 # +1

                      это что-то из азиатской культуры? цой?
                      Ответить

                      • ava bormand 27.09.2020 21:21 # +1

                        > азиатской культуры

                        Угу. И этим самым boost'ом, ты Семён, пронзишь небеса!
                        Ответить

          • ava OCETuHCKuu_nemyx 27.09.2020 01:11 # 0

            А почему петух написал, что увы, в с++ генераторов нет?
            Ответить

            • ava bormand 27.09.2020 01:13 # +1

              Пока драфт, но пилят.

              Будет что-то в духе:
              co_generator<int> foo() {
    co_yield(42);
    co_yield(100500);
}
              Ответить

    • ava MAKAKA 29.09.2020 16:49 # 0

      а не будет ворнинга, что код после ретурна?
      Ответить

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