Кресты / Говнокод #5606 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
  23. 23
  24. 24
  25. 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.

Запостил: kitaec kitaec, (Updated )

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

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

Из-за тебя ушел bormand, guest!

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

    ↑
    8
    ↓