Говнокод #11251 — C++ — Говнокод.ру

C++ / Говнокод #11251

−40
1. 1
```
*new
```
Самая соль.

Запостил: Говногость, 20 Июня 2012

Tweet
Комментарии (22) RSS
- HaskellGovno 20.06.2012 13:04 # +1
  
  Утечка памяти.
  Ответить
  - bormand 20.06.2012 13:15 # +4
    
    А может там в каком-нибудь методе delete this есть ? ;)
    Ответить
    - Steve_Brown 20.06.2012 14:32 # 0
      
      (*new CFoo).Release();
      
      CFoo *pFoo = &*new CFoo; // заведомо бессмысленно
      
      Ну, можно, конечно, ссылку инициализировать, типа
      
      CFoo &f = *new CFoo; ... delete &f;
      Ответить
      
      bormand 20.06.2012 15:05 # +5
      
      Как-то так:
      
      class Foo { public: void suicide() { delete this; } static Foo &create() { return *new Foo(); } private: Foo(); ~Foo(); }; ... Foo &x = Foo::create(); ... do something with x ... x.suicide();
      
      UPD: Естественно, смысла в этом нет :)
      Ответить
      
      HaskellGovno 20.06.2012 15:07 # +6
      
      class СЭМО { public: void suicide() {
      Ответить
- TarasB 20.06.2012 14:20 # +5
  
  Можно контекст?
  Ответить
- Xom94ok 20.06.2012 19:34 # +8
  
  Если new перегружен с каким-нибудь счетчиком созданных объектов - всё OK, но поговорить с автором по душам (по-доброму) всё-таки надо. А то, может, ему общения не хватает.
  Ответить
  - HaskellGovno 20.06.2012 20:52 # +1
    
    Вы хотели сказать, что new перегружен на возврат умного указателя? Не, это по стандарту не возможно.
    Ответить
    - Xom94ok 20.06.2012 21:36 # 0
      
      Сам объект, создаваемый new может иметь функционал умного указателя или тупо хранить в статическом контейнере указатели на созданные объекты, хотя это действительно была глупая идея. А вот передать управление временем жизни другому объекту - уже не выглядит как бред. Так организовали Qt: дочерние виджеты удаляются вместе с их родителями.
      Я всего лишь пытаюсь с оптимизмом посмотреть на этот код :-) TarasB прав, нужно смотреть контекст.
      Ответить
      
      Steve_Brown 21.06.2012 10:43 # +1
      
      А что...
      
      class MyWidget : QWidget { ... QPushButton &someButton_; MyWidget( QWidget *parent ) : QWidget(parent), someButton_(*new QPushButton("&Some...")) { someButton_.setToolTip("Do something"); ... } }
      
      :)
      Ответить
- guest 21.06.2012 23:36 # −6
  
  показать все, что скрытоКГ/АМ! (спойлер: ОП - ХУЙ!)
  Ответить
- guest 22.06.2012 18:16 # +6
  
  То чувство когда узнаёшь о какой-то новой неебической хуйне в С++ каждый день.
  Ответить
- Kirinyale 25.06.2012 12:33 # 0
  
  Вопрос стиля на самом деле.
  
  Мы так регулярно ссылки возвращаем из методов с самодокументирующим (по кодстайлу) названием типа create. RAII, подсчёта ссылок и прочих необходимых по контексту прелестей это не отменяет.
  
  Возвращение/передача ссылки вместо указателя, в частности, символизирует невозможность NULL (т.е. отсутствие необходимости его проверять на каждом шагу или думать перед вызовом функции, можно ли ей его передать, если очень хочется).
  Ответить
  - bormand 25.06.2012 12:38 # +2
    
    > символизирует невозможность NULL
    Скажем мягче - невозможность при вменяемости программиста. Т.к. нехороший человек может намеренно или случайно сделать return *p, где p == NULL и получится нулевая ссылка.
    Ответить
    - Kirinyale 25.06.2012 12:43 # 0
      
      Я же написал "символизирует", а не "гарантирует". :) А от невменяемых программистов никакие средства не спасут - с такими просто не нужно работать.
      
      В любом случае, баги могут быть (и будут) при любом стиле, но ссылка в интерфейсе метода - отличный способ обойтись без лишних комментариев по поводу NULL.
      Ответить
      
      bormand 25.06.2012 12:48 # 0
      
      Ну да. И еще немаловажна ее иммутабельность.
      Ответить
    - defecate-plusplus 25.06.2012 12:59 # 0
      
      не совсем так
      return *p - заставит убиться прямо тут (если сигналы не ловить, не рассматриваем)
      а вот int & a = a; - как раз невалидная ссылка, которую можно вернуть - получается отложенная пуля в ногу
      Ответить
      
      bormand 25.06.2012 13:02 # +2
      
      Эм, а можно пруф в стандарте, по которому *p (а не доступ к его полям и методам) выдает исключение или аборт?
      
      http://ideone.com/2mCis
      Ответить
      
      defecate-plusplus 25.06.2012 13:21 # +1
      
      я всегда отталкивался только от использования http://ideone.com/ijx71
      что же, выходит можно возвращать ссылки в виде return *p;
      пруф в стандарте, что можно -
      5.3.1 Unary operators /1 ... Note: a pointer to an incomplete type (other than cv void ) can be dereferenced. The lvalue thus obtained can be used in limited ways (to initialize a reference, for example); this lvalue must not be converted to an rvalue, see 4.1.
      Ответить
      
      HaskellGovno 24.08.2012 21:00 # 0
      
      Что-то ты сегодня не то нашел.
      Тут про incomplete type говорится. Для complete type другое поведение может быть.
      Ответить
      
      defecate-plusplus 24.08.2012 23:10 # +1
      
      сегодня?
      пруф тогда привел неверный - торопился, видать, а найти подходящий пункт в пфд найти не удалось
      собственно, это, как оказывается, было не так трививально:
      http://bit.ly/NPeziu
      http://bit.ly/Hi3Okf
      Ответить
- 3.14159265 07.07.2012 14:09 # +3
  
  КРИСТЕЦ
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Где здесь C++, guest?!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

C++ / Говнокод #11251

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

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