Adding default extension to the file

[rookee]

Hi,

I'm trying to save the files with a default extension. With the following code snippet, the file does save with the specified file name but it doesn't attach the extension to it. Can someone please help me. Thanks in advance.

QString fileName = QFileDialog::getSaveFileName(this, tr("Save Logs"),"/Tmp/",tr("*.csv"));
           if(!fileName.isEmpty())
           {
               QFile f(fileName);
               if(!fileName.endsWith(".csv"))
                   fileName = fileName + ".csv";
 f.open(QIODevice::WriteOnly);

QString fileName = QFileDialog::getSaveFileName(this, tr("Save Logs"),"/Tmp/",tr("*.csv"));
           if(!fileName.isEmpty())
           {
               QFile f(fileName);
               if(!fileName.endsWith(".csv"))
                   fileName = fileName + ".csv";
 f.open(QIODevice::WriteOnly);

To copy to clipboard, switch view to plain text mode 

[anda_skoa]

You are modifying fileName after you've used it to initialize "f".

Cheers,
_

[rookee]

Could you please elaborate. How do I use fileName instead of f and check if the user entered extension, if the extension is not entered attach the extension and save it.

[d_stranz]

if ( !fileName.isEmpty() )
{
    if ( !fileName.endsWith(".csv") )
       fileName += ".csv";
 
    QFile f( fileName );
    f.open( QIODevice::WriteOnly);
 
    // ...
}

if ( !fileName.isEmpty() )
{
    if ( !fileName.endsWith(".csv") )
       fileName += ".csv";
    
    QFile f( fileName );
    f.open( QIODevice::WriteOnly);

    // ...
}

To copy to clipboard, switch view to plain text mode 

[anda_skoa]

Could you please elaborate.

C++ is an imperative language.
Each statement potentially changes the state of the machine.
Statements are executed in order of sequence in the code.

so

QFile f(fileName);
fileName = ...;

QFile f(fileName);
fileName = ...;

To copy to clipboard, switch view to plain text mode 

first executes the creation of "f", then the modification of fileName.

How do I use fileName instead of f and check if the user entered extension

You currently don't use f to check for the extension so that questions does not make any sense.

if the extension is not entered attach the extension and save it.

You do that already.

Cheers,
_

[rookee]

Thanks a lot d_stranz and anda_skoa. I really appreciate it.