C++ 使用 Qt 重命名文件会导致 SIGSEGV

Question

我刚刚使用一个名为“Sound Juicer”的程序将我所有的 CD 复制到了我的计算机上。它工作正常，它为每个艺术家创建一个文件夹，并为每个专辑创建另一个文件夹。当然，在这些文件夹中还有 mp3 文件。

问题是我想要曲目编号、艺术家和曲目标题作为我歌曲的名称。 Sound Juicer 所做的是在代表“Disk 1 Title”的文件前面添加 d1t。

我是一名程序员，所以我用这个问题来练习一下。这有效：

void MainWindow::rename( const QString & text )
{
    static int _files = 0;

    QDir dir( text );
    QFileInfoList a = dir.entryInfoList( QDir::Files | QDir::Dirs );
    for( int i = 2; i < a.size(); i++ )
    {
        static QDir tmp;

        if( a.at( i ).isDir() )
            rename( a.at( i ).absoluteFilePath() );

        if( a.at( i ).fileName().startsWith( "d1t" ) || a.at( i ).fileName().startsWith( "d2t" ) )
        {
            QString newFile = a.at( i ).fileName().remove(0,3);
            tmp = a.at( i ).dir();

            if( !tmp.rename( a.at( i ).fileName(), newFile ) )
                qDebug() << "Failed";

            _files++;
        }
    }
}

它检查一个目录，选择第一个文件或目录并检查它是什么。如果它是一个目录，它会调用自己（递归）并重新开始，直到他找到一些文件或不再存在目录。如果找到一个文件，它会重命名它并将文件计数器加 1。

然而，它只重命名了前 2 或 3 个目录中的所有文件。之后，它引起了 SIGSEGV。有谁知道怎么回事？

我的目录示例：

1 个目录（“Sum 41”）-> 1 个子目录（“All Killer No Filler”）-> 文件“d1t01.Sum 41 - Introduction to Destruction.mp3”等...... 2 子目录（“Blah Blah”）-> 文件 ...

2 个目录（“Shinedown”）-> 1 个子目录（“疯狂之声”）-> 文件 d1t01。 Shinedown - Devour.mp3 等...

3 目录（“Guns N' Roses”）-> 子目录（“Blah Blah”）-> 文件 ... 子目录（“Blah”）-> 文件 ...

Answer 1

使用 static 你试图告诉编译器你只想要整个程序中的一个实例，但你把它放在一个 for 循环中。那不是很干净。在这种情况下，静态是无用的。由于静态初始化混乱问题，静态也会导致问题。因此，如果您真的不需要静态初始化，请将其删除并使其成为局部变量或类变量。

重写您的方法，以便更好地阅读它，我看到您递归调用该方法，这可能会导致无限循环。

另一个问题是你的文件变量，它的目的是什么？

void MainWindow::rename( const QString & text ) {
    int files = 0;

    QDir dir(text); // does QDir also accept wrong paths?
    QFileInfoList list = dir.entryInfoList(QDir::Files|QDir::Dirs); // does it return a list in all cases?
    foreach (QFileInfo entry, list) {
        if (entry.isDir()) {
            //is everything always ok when doing this?
            // POTENTIAL infinite loop 
            rename(entry.absoluteFilePath());
        }
        else {
            QString fileName = entry.fileName();
            if(fileName.startsWith("d1t") || fileName.startsWith("d2t")) {
                if (!entry.dir().rename(fileName, fileName.remove(0,3))) qDebug() << "Failed";
                files++;
            }
        }
    }
}

Answer 2

我在 fonZ 和 Frank Osterfeld 的帮助下以某种方式修复了它。我终于创建了一个 GUI 来选择一条路径。不过，我已经做到了。这段代码查找所有目录而不会导致无限循环或溢出（到目前为止）。问题在于 QDir 和 QFileInfo 获取路径的函数。我玩了一会儿，结果出来了：

void MainWindow::rename( const QString& path )
{
    //If invalid path return
    if( path.isEmpty() || ( !QDir( path ).exists() ) )
        return;

    //entryInfoList( QDir::NoDotAndDotDot ) doesn't work. 
    QFileInfoList fileList = QDir( path ).entryInfoList();

    foreach (QFileInfo entry, fileList ) {

        //Eliminating wrong paths
        if( entry.isDir() ){
            if( entry.filePath().endsWith(".") )
                continue;

            //Start function again with new directory
            rename( entry.filePath() );
        }
        else{

            QString fileName = entry.fileName();
            if( fileName.startsWith( "d1t" ) || fileName.startsWith( "d2t" ) ){
                //Remove those characters
                fileName.remove( 0, 3 );

                //If renaming is successful, increment the successful files
                //If not, increment the failed files and print an error
                if( entry.dir().rename( entry.fileName(), fileName ) )
                    files++;
                else{
                    addError( "Could not rename " + entry.fileName() + " to " + fileName );
                    filesFailed++;
                }

            }
        }

    }
}