Создание неограниченного количества элементов Опции

[BRE]

Алексей1153, я вовсе не занудствую.
Ты предлагаешь свой код человеку, который учиться. Этот код избыточен. Для чего учить человека писать избыточный код?
У тебя это вошло в привычку, но не обязательно (и, наверное, не правильно), что бы это вошло в его привычку. Отсюда и мой комментарий.

[Алексей1153]

BRE, дык, я ж ничего. У всех свой стиль, он прочитает всё, что мы тут понаписали - и выберет теперь свой стиль. Правильно рассуждаю ? ))

[DEADHUNT]

Не забывай также удалить через delete (как вариант, можно сделать оболочку для указателя, чтобы не приходилось вызывать delete вручную

зачем делать то, что уже сделали в C++0x/boost/Qt? причём данная реализация не совсем корректна (нету конструктора копирования, нету конструктора перемещения(для C++0x), и прочие мелочи)

Сообщение отредактировал DEADHUNT - 13.8.2010, 10:25

[Алексей1153]

DEADHUNT, я вообще про C++ речь веду. А где, кстати, в Qt это есть?

А про "не совсем корректность" - так вектору не нужен конструктор копирования и прочие мелочи, там оператор присваивания только нужен. Всё остальное -

"избыточность кода" © BRE


Ненужные препирательства всё это. Я подсказал путь, а решений - туча, каждому нравится своё

[DEADHUNT]

А где, кстати, в Qt это есть?

QScopedPointer/QSharedPointer

А про "не совсем корректность" - так вектору не нужен конструктор копирования и прочие мелочи, там оператор присваивания только нужен.

посмотри стандарт, там есть требования к элементам контейнера - конструктор по умолчанию, конструктор копирования, деструктор, etc.
в твоем примере получается что ты не реализовал конструктор копирования, компилятор создал его(но не так как требует семантика умного указателя).

Сообщение отредактировал DEADHUNT - 13.8.2010, 14:00

[Litkevich Yuriy]

смотри работу operator=

оператор - член класса, т.к. мы в деструкторе, то объект собрался умирать вместе с его операторами. BRE прав, тут ты переборщил с осторожностью.

хотя у меня такое тоже встречается, но тому причиной простой копипаст.

[Алексей1153]

а где сей стандарт глядеть?

А насчёт корректности работы - оно работает, одного только оператора присваивания достаточно. Правда, у меня в проекте std::vector используется, но работает без ошибок - очень хорошо и длительно тестировано

[DEADHUNT]

Link

насчёт работы это как раз из-за твоего зануления указателя в деструкторе, потому-что деструктор вызывается несколько раз вместо одного, попробуй сделать так в своём классе:

class smart_ptr {
    smart_ptr(const smart_ptr &);
....
};

или так

class smart_ptr {
    smart_ptr(const smart_ptr &) = delete;
....
};

и увидешь где используется конструктор копирования.

Сообщение отредактировал DEADHUNT - 13.8.2010, 16:48

[Litkevich Yuriy]

DEADHUNT, не мог бы ты пояснить такую строчку:

smart_ptr(const smart_ptr &) = delete;

?

[Алексей1153]

Синтаксис я тоже не понял что означает "= delete" и "= default" после прототипа функции?