Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\'))

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 08:11:02 GMT

angelyouyou — Tue, 20 Jul 2021 08:11:02 GMT

@J-Hilk said in Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')):

toNativeSeparators

Thanks.That is OK.

void Text2PcapWidget::on_toolButton_BrowserCapStoragePath_clicked()
{
    QString pcapStoragePath = QFileDialog::getExistingDirectory(this, UI_RES_CONTROL_TEXT2WIDGET_PCAP_DIAG_TITLE, DIAG_DEFAULT_DIR);
    pcapStoragePath = QDir::toNativeSeparators(pcapStoragePath);
    if (pcapStoragePath.isEmpty()) {
        return;
    }

    ui->lineEdit_CapStoragePath->setText(pcapStoragePath);
}

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 07:22:21 GMT

J.Hilk — Tue, 20 Jul 2021 07:22:21 GMT

@JonB The first time I used QDir, I was so disappointed with it, after I noticed that exists and mkdir/mkpath aren't static , that I never bothered to check what else actually is static. 😉

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 07:08:33 GMT

JonB — Tue, 20 Jul 2021 07:08:33 GMT

@J-Hilk
Since the method takes a parameter of the path to convert, QString QDir::toNativeSeparators(const QString &pathName), you can see it's a static rather than an instance method! Along with a whole bunch of "utility" methods in QDir for acting on a specified path :)

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 07:02:09 GMT

J.Hilk — Tue, 20 Jul 2021 07:02:09 GMT

@Chris-Kawa oh my goodness, QDir::toNativeSeparators is static 🤦‍♂️
I never realised!

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 06:59:49 GMT

Chris Kawa — Tue, 20 Jul 2021 06:59:49 GMT

As mentioned above Qt uses / for all of its path handling. Use QDir::toNativeSeparators to convert the path for displaying in native format.

void Text2PcapWidget::on_toolButton_BrowserCapStoragePath_clicked()
{
    QUrl pcapStorageUrl = QFileDialog::getExistingDirectoryUrl(this, UI_RES_CONTROL_TEXT2WIDGET_PCAP_DIAG_TITLE, QUrl(DIAG_DEFAULT_DIR));
    QString pcapStoragePath = pcapStorageUrl.toLocalFile();
    if (pcapStoragePath.isEmpty()) {
        return;
    }

    ui->lineEdit_CapStoragePath->setText(QDir::toNativeSeparators(pcapStoragePath));
}

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 06:50:38 GMT

angelyouyou — Tue, 20 Jul 2021 06:50:38 GMT

@J-Hilk
Thanks for your help.
I have used getExistingDirectoryUrl and toLocalFile, but the same case.


void Text2PcapWidget::on_toolButton_BrowserCapStoragePath_clicked()
{
    QUrl pcapStorageUrl = QFileDialog::getExistingDirectoryUrl(this, UI_RES_CONTROL_TEXT2WIDGET_PCAP_DIAG_TITLE, QUrl(DIAG_DEFAULT_DIR));
    QString pcapStoragePath = pcapStorageUrl.toLocalFile();
    if (pcapStoragePath.isEmpty()) {
        return;
    }
    QDir::fromNativeSeparators(pcapStoragePath);

    ui->lineEdit_CapStoragePath->setText(pcapStoragePath);
}

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 06:38:53 GMT

J.Hilk — Tue, 20 Jul 2021 06:38:53 GMT

@angelyouyou thats normal, Qt as cross platform framework, uses internally / as path separators and will also always return those to you, until otherwise specified. I would suggest using getExistingDirectoryUrl:

QUrl pcapStoragePath = QFileDialog::getExistingDirectoryUrl(this, UI_RES_CONTROL_TEXT2WIDGET_PCAP_DIAG_TITLE, DIAG_DEFAULT_DIR);
    if (pcapStoragePath.isEmpty()) {
        return;
    }

    ui->lineEdit_CapStoragePath->setText(pcapStoragePath.toLocalFile());

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Fri, 23 Jul 2021 06:43:38 GMT

jsulm — Fri, 23 Jul 2021 06:43:38 GMT

@angelyouyou Qt internally uses / instead of \. So, you can use paths with / with Qt, even on Windows. Else, you can replace / with \ (https://doc.qt.io/qt-5/qstring.html#replace-3).

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 06:22:53 GMT

angelyouyou — Tue, 20 Jul 2021 06:22:53 GMT

@jsulm
sorry. I have updated my picture to compare the different.

The wireshark path is obtained with the below code:

void Text2PcapWidget::on_toolButton_BrowserWiresharkPath_clicked()
{
    QString wireshakSelectedPath = QFileDialog::getOpenFileName(this, UI_RES_CONTROL_TEXT2WIDGET_WS_DIAG_TITLE, DIAG_DEFAULT_DIR);
    if (wireshakSelectedPath.isEmpty()) {
        return;
    }

    QMessageBox::information(this, UI_RES_CONTROL_MESSAGEBOX_INFO, wireshakSelectedPath);
    ui->lineEdit_WiresharkPath->setText(wireshakSelectedPath);
}

Reply to Why QFileDialog::getExistingDirectory returned non native directory?(pcap directory below with '/', not '\')) on Tue, 20 Jul 2021 05:37:54 GMT

jsulm — Tue, 20 Jul 2021 05:37:54 GMT

@angelyouyou What exact folder did you select for pcap? Is it on network drive?