crossplatform.ru

Здравствуйте, гость ( Вход | Регистрация )

> Упрощение кода, написание того или иного кода более понятно и просто
AD
  опции профиля:
сообщение 5.8.2008, 16:26
Сообщение #1


Профессионал
*****

Группа: Участник
Сообщений: 2003
Регистрация: 4.2.2008
Из: S-Petersburg
Пользователь №: 84

Спасибо сказали: 70 раз(а)




Репутация:   17  


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

Если разрешите, то могу начать.
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
 
Начать новую тему
Ответов
Red Devil
  опции профиля:
сообщение 6.8.2008, 11:59
Сообщение #2


Студент
*

Группа: Участник
Сообщений: 68
Регистрация: 6.6.2008
Из: Saint-Petersburg
Пользователь №: 194

Спасибо сказали: 1 раз(а)




Репутация:   3  


void ProjectView::AddDirs(const QDir & Dir, QTreeWidget * pParent)
{
    QStringList DirsList = Dir.entryList(QDir::Dirs);
    foreach(QString Dirname, DirsList)
    {
        if(Dirname == "." || Dirname == "..")
            continue;
      
        QTreeWidget * pItem = new QTreeWidgetItem(pParent);              
        pItem->setText(0, dirname);                          
        pItem->setIcon(0, folderIcon);                        
        StartDir(QDir(dir.absoluteFilePath(dirname)), pItem);  
    }                                                                                
}

void ProjectView::AddFiles(const QDir & Dir, QTreeWidgetItem * pParent)
{
    QStringList FilesList = Dir.entryList(QDir::Files);

    foreach(QString File, FilesList)
    {
        QTreeWidget * pItem = new QTreeWidgetItem(pParent);
        pItem->setData(0, Qt::UserRole, QVariant(Dir.absolutePath() + "/" + File));
        pItem->setText(0, File);
        pItem->setText(1, Dir.absoluteFilePath(file));
        pItem->setIcon(0, fileIcon);
    }
}

void ProjectView::CreateItems(const QDir & Dir)
{
    AddDirs(Dir, rootItem);
    AddFiles(Dir, rootItem);
}

void ProjectView::StartDir(const QDir & Dir, QTreeWidgetItem * pParent)
{
    AddFiles(Dir, pParent);
    AddDirs(Dir, pParent);
}


Сообщение отредактировал Red Devil - 6.8.2008, 12:01
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение

Сообщений в этой теме
- AD   Упрощение кода   5.8.2008, 16:26
- - void*   ну вот у меня такая дилема. Нижеприведенный код по...   5.8.2008, 18:46
- - ViGOur   Ну я дуаю ты сам мог бы догадаться, если бы был вн...   5.8.2008, 21:59
- - void*   ViGOur, насчет entryInfoList я знал, просто мне ка...   5.8.2008, 22:21
- - Red Devil   void ProjectView::AddDirs(const QDir ...   6.8.2008, 11:59
- - void*   упростил однако из двух функций сделал четыре хо...   6.8.2008, 12:49
- - ViGOur   void createItems(const QDir &dir, QTreeWid...   6.8.2008, 14:22
- - void*   ViGOur, спасибо! примерно такое у меня и верте...   6.8.2008, 17:26
- - Tonal   2 ViGOur Я бы всё таки выделил из этого кода функц...   7.8.2008, 8:09
- - ViGOur   Цитата(Tonal @ 7.8.2008, 9:09) Я бы всё т...   7.8.2008, 8:27
|- - AD   Выдалась возможность, может сможете и мне упростит...   7.8.2008, 9:10
- - Tonal   Вынеси длинные проверки с next -> status... в о...   7.8.2008, 12:02
|- - AD   Цитата(Tonal @ 7.8.2008, 13:02) Вынеси дл...   7.8.2008, 12:55
- - Tonal   Без тега кода всяко только уменьшилась! По к...   8.8.2008, 15:21
- - Tonal   Чё непонятно-то?   10.8.2008, 17:31
|- - AD   Цитата(Tonal @ 10.8.2008, 18:31) Чё непон...   10.8.2008, 22:06
- - Tonal   ЦитатаДай - понимаю, курить - понимаю, дай курить ...   11.8.2008, 8:05
|- - AD   Цитата(Tonal @ 11.8.2008, 9:05) После про...   11.8.2008, 8:48
- - Tonal   1) После проверок где... Код if(...) isC...   11.8.2008, 11:07
|- - AD   Теперь это выглядит более внятно и легче править. ...   11.8.2008, 14:54
- - Red Devil   Кодclass ScrTrack: public TImageClass { privat...   12.8.2008, 10:12
- - rich   Может кому поможет: "Рефакторинг. Улучшение с...   13.12.2008, 0:03
- - Admin   rich, а ты темой не ошибся? Причем тут упрощение к...   13.12.2008, 0:52
- - alex977   Цитата(Admin @ 13.12.2008, 0:52) rich, а ...   13.12.2008, 11:28
- - rich   Цитата(Admin @ 13.12.2008, 0:52) rich, а ...   14.12.2008, 18:41


Быстрый ответОтветить в данную темуНачать новую тему
Теги
Нет тегов для показа


6 чел. читают эту тему (гостей: 6, скрытых пользователей: 0)
Пользователей: 0




RSS Текстовая версия Сейчас: 29.11.2024, 16:04