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

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

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

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

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

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

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

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 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);

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

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

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