Message d'erreur: Return-statement with no value

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:

---

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

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

Citation:

---

Envoyé par le compilateur, qui n'a décidément aucune imagination

d'accord, mais qu'est ce que tu veux que je renvoye :question: 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:

---

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:

---

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; }

---

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.

Citation:

---

Envoyé par 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.

---

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

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 ;)

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

Merci encore pour ton aide.