Discussion :

[stanfillirenfro]

Salut à tous!

J'essaie d'ouvrir un fichier en utilisant une fonction non-void.

Le problème est que je recois le message d'erreur suivant: "return-statement with no value, in function returning 'QStringList".

Y a -t-il une autre possibilité de stopper le processus si le fichier ne peut pas être ouvert? "break ne peut pas fonctionner ici, étant donné qu'il n'est utilisé que dans les boucles.

Voic mon code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
QStringList myWindow::open(QString &fileName)
{
QFile file(fileName);
 
if(!file.open(QIODevice::ReadOnly))
{
QMessageBox::critical(this, "Warning", "Open a file please");
return;
}
 
// Je fais quelque chose ici.
}

Merci d'avance

[koala01]

Salut,

Bien que cryptique, le message d'erreur t'indique exactement ce qui ne va pas

En effet, tu déclare explicitement que ta fonction est sensée renvoyer un objet de type QStringList. Or, on lit return; à la ligne 9 du code que tu nous présente.

Le compilateur te dis donc en gros

d'accord, mais qu'est ce que tu veux que je renvoye tu me dis de renvoyer un objet de type QStringList, mais je n'en ai pas, moi!!!

Une fois que tu as compris cela, la correction s'impose d'elle-même : il faut que tu lui fasse renvoyer une QStringList, quitte à ce qu'elle soit vide

Au choix, tu peux envisager le code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
QStringList myWindow::open(QString &fileName)
{
    QFile file(fileName);
    if(!file.open(QIODevice::ReadOnly))
    {
        QMessageBox::critical(this, "Warning", "Open a file please");
        return QStringList(); //crée explicitement une QStringList vide et la renvoie
    }
    // ici, je sais que je dois remplir une QStringList, je l'appelle "laliste" 
    QStringList laliste;
    /*   ...
     * remplir la liste avec ce qui t'intéresse
     */
    return laliste;
}

ou le code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
QStringList myWindow::open(QString &fileName)
{
QFile file(fileName);
    QStringList laliste; // Je sais que je vais renvoyer une QStringList, même
                         // si elle sera peut être vide
    if(!file.open(QIODevice::ReadOnly))
    {
        QMessageBox::critical(this, "Warning", "Open a file please");
        return laliste; // comme je n'ai rien mis dedans, la liste est vide :-)
    }
     /*   ...
      * remplir la liste avec ce qui t'intéresse
      */
    return laliste;
}

[stanfillirenfro]

Merci koala01 pour ta réponse.

J'ai fait ce que tu as proposé et ensuite, j'ai omis volontairement d'indiquer le nom du fichier. Je recois le message de la QMessageBox effectivement, mais le programme plante. Je me répète malheureusement, si le fichier ne peut donc pas être ouvert, le programme va planter! Y a -t-il un moyen d'éviter cette situation?

Merci d'avnace.

[koala01]

Merci koala01 pour ta réponse.

J'ai fait ce que tu as proposé et ensuite, j'ai omis volontairement d'indiquer le nom du fichier. Je recois le message de la QMessageBox effectivement, mais le programme plante. Je me répète malheureusement, si le fichier ne peut donc pas être ouvert, le programme va planter! Y a -t-il un moyen d'éviter cette situation?

Merci d'avnace.

Comme le dirait l'agent Mulder: "la vérité est ailleurs "

Dans notre cas, "ailleurs" ne peut être qu'à un seul endroit: dans la fonction qui appelle la fonction open . Je ne serais pas étonné outre mesure que cette fonction parte du principe que la QStringListe renvoyé n'est pas vide. Or, cela peut être le cas si tu n'a pas indiqué de nom de fichier valide. Il faut donc prendre cette possibilité en compte

[stanfillirenfro]

Merci koala01 und fois de plus.
Je peux grâce à toi marquer ce sujet comme résolu.

Merci encore pour ton aide.