remove directory empty or not empty

[munna]

You will need to write a recursive function to do this.

Something like this

void remove(const QString &path)
{
       QFileInfo fileInfo(path);
       if(fileInfo.isDir()){
              QDir dir(path);
              QStringList fileList = dir.entryList();
              for(int i = 0; i < fileList.count(); ++i){
                      remove(fileList.at(i));
              }
              rmpath(path);
       }
       else{
             QFile::remove(path);
       }
}

void remove(const QString &path)
{
       QFileInfo fileInfo(path);
       if(fileInfo.isDir()){
              QDir dir(path);
              QStringList fileList = dir.entryList();
              for(int i = 0; i < fileList.count(); ++i){
                      remove(fileList.at(i));
              }
              rmpath(path);
       }
       else{
             QFile::remove(path);
       }
}

To copy to clipboard, switch view to plain text mode 

[raphaelf]

Hi,
I have a folder under c:\temp with the name aa1.
This folder contains a folder qq1. And qq1 contains a file.
I was not able to delete aa1, qq1 and the file.
What is wrong here:

QString pathTest = "C:/temp/aa1";
		QFileInfo fileInfo(path);       
		if(fileInfo.isDir())
		{              
			QDir dir(pathTest);
			QMessageBox::information(this,"pathTest",pathTest);              
			QStringList fileList = dir.entryList();
 
			for(int i = 0; i < fileList.count(); ++i)
			{                 
				QMessageBox::information(this,"fileList.at(i)",fileList.at(i));
				remove(fileList.at(i));              
			}              
			rmpath(pathTest);       
		}       
		else
		{             
			QFile::remove(pathTest);      
		}

QString pathTest = "C:/temp/aa1";
		QFileInfo fileInfo(path);       
		if(fileInfo.isDir())
		{              
			QDir dir(pathTest);
			QMessageBox::information(this,"pathTest",pathTest);              
			QStringList fileList = dir.entryList();
			              
			for(int i = 0; i < fileList.count(); ++i)
			{                 
				QMessageBox::information(this,"fileList.at(i)",fileList.at(i));
				remove(fileList.at(i));              
			}              
			rmpath(pathTest);       
		}       
		else
		{             
			QFile::remove(pathTest);      
		}

To copy to clipboard, switch view to plain text mode 

[wysota]

You are not calling QDir::rmdir() anywhere, you only delete files.

[munna]

try replacing

rmpath(pathTest);

rmpath(pathTest);

To copy to clipboard, switch view to plain text mode 

with

dir.rmdir(path);

dir.rmdir(path);

To copy to clipboard, switch view to plain text mode 

Also, I see that the function is not recursive.

[raphaelf]

Hi

Why can i not delete artist1 under c:\temp??The folder artist1 contains a folder album1 and album1 contains a file a.jpg.

What is wrong?

QString pathTest = "C:/temp/artist1";
		QFileInfo fileInfo(pathTest);       
		if(fileInfo.isDir())
		{              
			QDir dir(pathTest);
			QMessageBox::information(this,"pathTest",pathTest);              
			QStringList fileList = dir.entryList();
 
			for(int i = 0; i < fileList.count(); ++i)
			{                 
				QMessageBox::information(this,"fileList.at(i)",fileList.at(i));
				remove(fileList.at(i));              
			}              
			dir.rmdir(pathTest);        
		}       
		else
		{             
			QFile::remove(pathTest);      
		}

QString pathTest = "C:/temp/artist1";
		QFileInfo fileInfo(pathTest);       
		if(fileInfo.isDir())
		{              
			QDir dir(pathTest);
			QMessageBox::information(this,"pathTest",pathTest);              
			QStringList fileList = dir.entryList();
			              
			for(int i = 0; i < fileList.count(); ++i)
			{                 
				QMessageBox::information(this,"fileList.at(i)",fileList.at(i));
				remove(fileList.at(i));              
			}              
			dir.rmdir(pathTest);        
		}       
		else
		{             
			QFile::remove(pathTest);      
		}

To copy to clipboard, switch view to plain text mode 

[munna]

Something like this should work.

void someFunction()
{
     QString pathTest = "C:/temp/artist1";
     remove(pathTest);
}
 
void remove(const QString &path)
{
    QFileInfo fileInfo(path);
    if(fileInfo.isDir()){
        QDir dir(path);
        QStringList fileList = dir.entryList();
        for(int i = 0; i < fileList.count(); ++i){
            remove(fileList.at(i));
        }
        dir.rmdir(path);
    }
    else{
         QFile::remove(path);
    }
}

void someFunction()
{
     QString pathTest = "C:/temp/artist1";
     remove(pathTest);
}

void remove(const QString &path)
{
    QFileInfo fileInfo(path);
    if(fileInfo.isDir()){
        QDir dir(path);
        QStringList fileList = dir.entryList();
        for(int i = 0; i < fileList.count(); ++i){
            remove(fileList.at(i));
        }
        dir.rmdir(path);
    }
    else{
         QFile::remove(path);
    }
}

To copy to clipboard, switch view to plain text mode 

I thought the code was pretty much self-explanatory

[raphaelf]

Hi munna

Have you test it?I am not able to delete any folder..

Are you shure this should work?Its not working:

void Manage::deleteArtist()
{
.
.
QString artist = ui.delete_artist_cb->currentText();
QString path;
	QSqlQuery select_sound_path("select value from config_tbl where config ='dsm_path'");
						while(select_sound_path.next())
						{
							path = select_sound_path.value(0).toString();
						}
	QString artistDirectory(path + "/" + artist);
removeArtistDirectory(artistDirectory);
}
 
void Manage::removeArtistDirectory(const QString &artistDirectory)
{    
	QMessageBox::information(this,"","remove function");
	QMessageBox::information(this,"",artistDirectory);
	QFileInfo fileInfo(artistDirectory);    
	if(fileInfo.isDir())
	{        
		QDir dir(artistDirectory);        
		QStringList fileList = dir.entryList();        
		for(int i = 0; i < fileList.count(); ++i)
		{            
			QMessageBox::information(this,"",fileList.at(i));
			remove(fileList.at(i));        
		}        
		dir.rmdir(artistDirectory);    
	}    
	else
	{         
		QFile::remove(artistDirectory);    
	}
}

void Manage::deleteArtist()
{
.
.
QString artist = ui.delete_artist_cb->currentText();
QString path;
	QSqlQuery select_sound_path("select value from config_tbl where config ='dsm_path'");
						while(select_sound_path.next())
						{
							path = select_sound_path.value(0).toString();
						}
	QString artistDirectory(path + "/" + artist);
removeArtistDirectory(artistDirectory);
}

void Manage::removeArtistDirectory(const QString &artistDirectory)
{    
	QMessageBox::information(this,"","remove function");
	QMessageBox::information(this,"",artistDirectory);
	QFileInfo fileInfo(artistDirectory);    
	if(fileInfo.isDir())
	{        
		QDir dir(artistDirectory);        
		QStringList fileList = dir.entryList();        
		for(int i = 0; i < fileList.count(); ++i)
		{            
			QMessageBox::information(this,"",fileList.at(i));
			remove(fileList.at(i));        
		}        
		dir.rmdir(artistDirectory);    
	}    
	else
	{         
		QFile::remove(artistDirectory);    
	}
}

To copy to clipboard, switch view to plain text mode 

[munna]

Your removeArtistDirectory function is not recursive

Inside the for loop, replace

remove(fileList.at(i));

remove(fileList.at(i));

To copy to clipboard, switch view to plain text mode 

with

removeArtistDirectory(fileList.at(i));

removeArtistDirectory(fileList.at(i));

To copy to clipboard, switch view to plain text mode 

By the way, is this code not giving you any error?