- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
/**
* Standard QWidget Constructor
*/
SeismicSelectionDG::SeismicSelectionDG(QWidget *parent)
: QDialog(parent), ui(new Ui::SeismicSelectionDG)
{
...
}
/**
* Destructor
*/
SeismicSelectionDG::~SeismicSelectionDG()
{
if(ui) delete ui;
}
/**
* Set title to dialog box
* @param title The title for dialog box
*/
void SeismicSelectionDG::setDialogTitle(const QString& title)
{
QDialog::setWindowTitle(title);
}
Обратим внимание как мой коллега китаец тщательно документирует код. метод setDialogTitle - в особенности. А учитывая что QDialog::setWindowTitle - библиотечный метод, необходимость оборачивать его в свой метод полностью отсутствует. Также глупость в деструкторе - для delete необязательно проверять указатель на 0.
striker 09.02.2011 14:28 # +5
kitaec 09.02.2011 14:39 # +1
bugmenot 09.02.2011 14:42 # −3
striker 09.02.2011 14:43 # +3
bugmenot 09.02.2011 15:59 # −2
:-P
Lure Of Chaos 09.02.2011 18:03 # +2
guest 20.01.2017 15:04 # 0
guest 28.02.2017 07:49 # 0
guest 27.01.2017 10:12 # 0
istem 09.02.2011 15:09 # +5
Вопрос: "Где третий?"
absolut 09.02.2011 15:35 # +3
bugmenot 09.02.2011 18:08 # 0
Lure Of Chaos 09.02.2011 18:15 # 0
absolut 09.02.2011 18:16 # +1
Lure Of Chaos 09.02.2011 18:21 # 0
- ну и что?
- но он не знает китайского!
(из фильма Тень)
koodeer 10.02.2011 01:15 # +8
- ну и что?
- но он не знает пхп!
guest 20.01.2017 03:36 # 0
guest 14.02.2017 09:25 # 0
guest 10.02.2011 00:00 # −1
Откуда данные? Неужели это Вы?
absolut 10.02.2011 09:18 # 0
guest 25.02.2017 20:06 # 0
Lure Of Chaos 09.02.2011 18:04 # 0
guest 17.02.2017 00:04 # −1
pushkoff 09.02.2011 15:11 # 0
absolut 09.02.2011 18:16 # +1
p.s. капча заработала :)
Lure Of Chaos 09.02.2011 18:22 # +3
pushkoff 09.02.2011 20:35 # 0
bugmenot 09.02.2011 20:37 # 0
там немцы покупают жен
absolut 09.02.2011 22:18 # 0
т.е. перепродать
pushkoff 09.02.2011 23:06 # 0
absolut 09.02.2011 23:15 # 0
guest 03.03.2017 15:59 # 0
guest 24.02.2017 20:39 # 0
guest 14.02.2017 19:37 # 0
guest 25.02.2017 13:34 # 0
ningun 09.02.2011 17:24 # 0
Lure Of Chaos 09.02.2011 18:05 # 0
istem 09.02.2011 23:46 # 0
sneak-peek 08.03.2011 23:15 # 0
А если после этого в конструкторе будет кинуто исключение ? ui не будет освобождён. Нужно делать его std::auto_ptr<T> ui; А деструктор вообще удалить.