Упрощение кода, написание того или иного кода более понятно и просто Опции

[AD]

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

Если разрешите, то могу начать.

[void*]

ну вот у меня такая дилема. Нижеприведенный код показывает в дереве (QTreeWidget'e) содержимое определенной папки методом рекурсии, однако состоит это дело из двух функций, и мне почему-то кажется что можно это как-то сделать и через одну функцию:

код

void ProjectView::createItems(const QDir &dir) { //параметром является нужная директория

     QStringList listDirs = dir.entryList(QDir::Dirs);

     foreach(QString dirname, listDirs) {
                               if(dirname == "." || dirname == "..") continue;
                               curItem = new QTreeWidgetItem(rootItem); //curItem и rootItem объявлены в классе ранее
                               curItem->setText(0, dirname);                    //rootItem является главным итемом, к нему 
                               curItem->setIcon(0, folderIcon);                 //присоединяются все остальные
                               startDir(QDir(dir.absoluteFilePath(dirname)), curItem); //curItem -  просто единый указатель 
     }                                                                                     //использующийся для создания всех итемов


     QStringList listFiles = dir.entryList(QDir::Files);

     foreach(QString file, listFiles) {
                               curItem = new QTreeWidgetItem(rootItem);
                               curItem->setData(0, Qt::UserRole, QVariant(dir.absolutePath() + "/" + file));
                               curItem->setText(0, file);
                               curItem->setText(1, dir.absoluteFilePath(file));
                               curItem->setIcon(0, fileIcon);
                               //curItem->setFlags(...);
     }
}
void ProjectView::startDir(const QDir& dir, QTreeWidgetItem *parent) { //рекурсивная функция
     QStringList listFiles = dir.entryList(QDir::Files);
     foreach(QString filename, listFiles) {
                               curItem = new QTreeWidgetItem(parent);
                               curItem->setData(0, Qt::UserRole, QVariant(dir.absolutePath() + "/" + filename));
                               curItem->setText(0, filename);
                               curItem->setText(1, dir.absoluteFilePath(filename));
                               curItem->setIcon(0, fileIcon);
     }

     QStringList listDirs = dir.entryList(QDir::Dirs);
     foreach(QString subdir, listDirs) {
                               if(subdir == "." || subdir == "..") continue;
                               curItem = new QTreeWidgetItem(parent);
                               curItem->setText(0, subdir);
                               curItem->setIcon(0, folderIcon);
                               startDir(QDir(dir.absoluteFilePath(subdir)), curItem);
     }
}

[ViGOur]

Ну я дуаю ты сам мог бы догадаться, если бы был внимательней.
Не нужно два цикла, вместо entryList используй entryInfoList в и foreach получай QFileInfo с помощью которого ты можешь узнать файл это или директория (isFile или isDir).

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

p.s. как сделаешь код здесь выложи, чтобы покритиковали если будет что.

[void*]

ViGOur, насчет entryInfoList я знал, просто мне казалось что так нагляднее. А так - пока что повозиться с этим кодом нет времени и свободных рук просто для начала сделал попроще и нагляднее, а потом уже можно будет как-нибудь оптимизировать.

[Red Devil]

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

[void*]

упростил однако из двух функций сделал четыре
хотя так оно конечно возможно понятнее

[ViGOur]

void createItems(const QDir &dir, QTreeWidgetItem *parent = 0)
{ 

    QFileInfoList fiList = dir.entryInfoList();

    foreach(QFileInfo fi, fiList) 
    {
        QTreeWidgetItem *curItem = 0;
        if( fi.isDir())
        {
            QString szDirName = fi.fileName();
            if( szDirName=="." || szDirName=="..")
                continue;

            curItem = new QTreeWidgetItem(parent);
            curItem->setText(0, szDirName);
            curItem->setIcon(0, folderIcon);
            
            createItems( fi.absoluteFilePath(), curItem);
        }
        else if( fi.isFile())
        {
            QString szFileName = fi.fileName();
            curItem = new QTreeWidgetItem(parent);
            curItem->setData(0, Qt::UserRole, QVariant(dir.absolutePath() + "/" + szFileName));
            curItem->setText(0, szFileName);
            curItem->setText(1, dir.absoluteFilePath(szFileName));
            curItem->setIcon(0, fileIcon);
        }
    }
}

Вызывается так:

createItems( QDir( "c:\\temp\\"), rootItem);

[void*]

ViGOur, спасибо! примерно такое у меня и вертелось в голове, только никак не мог выразить это в коде

[Tonal]

2 ViGOur
Я бы всё таки выделил из этого кода функции создания итема для файла и для директории. Тогда код получается максимально наглядный и все занимаются своим делом:

QTreeWidgetItem* createDirItem(QTreeWidgetItem& parent, const QFileInfo& fi);
QTreeWidgetItem* createFileItem(QTreeWidgetItem& parent, const QFileInfo& fi);
void createItems(const QDir &dir, QTreeWidgetItem *parent = 0) { 

    QFileInfoList fiList = dir.entryInfoList();

    foreach(QFileInfo fi, fiList) 
    {
        QTreeWidgetItem *curItem = 0;
        if( fi.isDir())
        {
            QString szDirName = fi.fileName();
            if( szDirName=="." || szDirName=="..")
                continue;

            createItems(fi.absoluteFilePath(), createDirItem(*parent, fi));
        }
        else if( fi.isFile())
        {
            createFileItem(*parent, fi);
        }
    }
}

Сообщение отредактировал Tonal - 7.8.2008, 8:12

[ViGOur]

Я бы всё таки выделил из этого кода функции создания итема для файла и для директории

Согласен, так как могут понадобится дополнительные манипуляции с папками и файлами.