Problème avec QFtp

Bonjour,

J'ai un problème avec mon code pour l'envoi de fichier FTP.

J'essaye d'envoyer des fichiers répartis dans un dossier puis un sous dossier.
Mon problème c'est qu'il envoi les fichiers dans le sous dossier, mais n'envoie jamais ceux du dossier.

Au niveau du déplacement dans les dossiers je pense ne pas mettre trompé pourtant, j'ai testé sans utilisé l'FTP pour voir les chemins parcourus et visiblement il passe par tous.
De plus, j'ai utilisé des fichiers pour vérifier les passages avec l'FTP et d'après les contrôles il est censé envoyé tout les fichiers.

Bref, voici le code de la fonction d'envoi :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

void FTP::envoi(QStringList list, QString dossier) { ftp->mkdir(dossier); ftp->cd(dossier);   for(int i=0; i<list.count(); i++) { QFileInfo *fileInfo = new QFileInfo(list.value(i)); if(!(fileInfo->isDir())) { QFile *file = new QFile(fileInfo->filePath()); ftp->put(file,fileInfo->fileName()); QMessageBox::information(this,dossier,"FILE : "+fileInfo->filePath()); } else { QDir *dir = new QDir(fileInfo->filePath()); envoi(watchList(dir),fileInfo->fileName()); ftp->cd(".."); } } }

---

Merci d'avance de votre aide.

Bonsoir,

Le fait que ta fonction parcourt le premier dossier dépend surtout de la façon dont elle est appelée la première fois.

Par contre,

• tu alloues des pointeurs sans les libérer => fuite de mémoire,
• QDir et QFileInfo ne devraient pas être alloués dynamiquement,
• QDirIterator permet de récupérer la liste des fichiers d'un répertoire récursivement sans récursion dans le code,
• tu ouvres tous les fichiers en même temps, alors que tu pourrais ouvrir et fermer les fichiers dans le slot connecté au signal commandFinished. Ça permettait en plus de gérer les erreurs éventuelles au moment où elles arrivent.

On peut passer un chemin complet à put et mkdir (ex: ftp->mkdir("toto") puis ftp->mkdir("toto/tata") puis ftp->put(..., "toto/tata/fichier.txt")), donc sans utiliser cd à chaque fois.

Merci pour cette réponse.

Alors j'ai rectifié mes erreurs, fuite de mémoire, QDir, QFileInfo et j'ai utilisé QDirIterator.
Par contre, bien que ton idée me paraisse intéressante pour l'utilisation des fichiers, je ne vois pas comment faire...
J'avoue que l'utilisation des slots et signaux avec la class QFtp, ce n'est pas forcément ce que j'ai le plus compris...

Je vais regarder pour trouver comment faire mais si tu as des explications ce serait avec plaisir.

Merci d'avance.

EDIT : Je n'ai pas encore trouvé pour les signaux et slots mais il y a quelque chose que je ne comprends pas.
J'ai transformé mon code de cette manière :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

void FTP::envoi() { QDirIterator dir("/*chemin du dossier*/", QDirIterator::Subdirectories); ftp->mkdir(dir.path()); while(dir.hasNext()) { if(dir.fileName() != QString(".") && dir.fileName() != QString("..") && dir.fileName() != QString("")) { QFileInfo fileInfo(dir.filePath()); if(fileInfo.isDir()) { ftp->mkdir(dir.filePath()); } else { QFile *file = new QFile(dir.filePath()); ftp->put(file,dir.filePath()); //file->close();   } } dir.next(); } }

---

Je ne doute pas qu'il y ait encore des problèmes, mais mon soucis majeur et que j'ai réussi, sur une vingtaine d'essais, à avoir deux fois le résultat escompté, soit tout mes fichiers étaient transférés . La plupart du temps, les fichiers transférés sont uniquement ceux du sous dossier, et d'autres fois je n'ai rien du tout.

Bref, je suppose que des erreurs sont encore là et engendre le problème alors merci d'avance de votre aide...

Tu peux par exemple avoir la structure suivante:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

class FTP : public QObject ou QWidget { ... private: // réutilisable après close()/setFileName() QFile currentFile; // non réutilisable, donc un pointeur QScopedPointer<QDirIterator> dirIterator;   public slots: void envoi(const QString & dossier) { // lance le premier put ou mkdir } private slots: void onCommandFinished(int id, bool error) { if (error) { // Affiche l'erreur puis quitte return; }   if (ftp->currentCommand() == QFtp::Put || ftp->currentCommand() == QFtp::Mkdir) { // Lance le fichier ou répertoire suivant s'il y en a un } } };

---

EDIT:

Citation:

---

Bref, je suppose que des erreurs sont encore là et engendre le problème alors merci d'avance de votre aide...

---

Il faut vérifier qu'il n'y a pas eu d'erreur lors d'une des opérations, parce que si c'est le cas toutes les opérations suivantes déjà présentes dans la file d'attente sont annulées.
L'erreur est indiquée par QFtp::error() ou QFtp::errorString() dans le slot connecté au signal commandFinished(int,bool).

Dans mon slot appelé par le signal commandFinished je teste le booléen et affiche l'erreur avec QFtp->error() si elle a lieu.

J'ai également le signal QFtp::done de connecté.

Malheureusement je n'ai aucun retour comme quoi une erreur s'est produite...

Je ne sais pas si c'est dû à ça, mais d'après le code de QFtp et après quelques tests, si QFtp n'arrive pas à ouvrir le fichier lui-même, il arrête de traiter les commandes suivantes, sans signaler aucune erreur.
Essaye d'ouvrir les fichiers et de tester leur bonne ouverture avant de les passer à ftp->put.

J'ai rajouté ces lignes là autour du put :

Code:

---

1 2 3 4 5 6

QFile *file = new QFile(dir.filePath()); if (!file->open(QIODevice::ReadOnly | QIODevice::Text)) { ftp->put(file,dir.filePath()); file->close(); }

---

Et depuis, je pense qu'il parcours mieux les dossiers dans le sens où avant il me transféré bien les fichiers du sous dossier mais s'arrêtait là pour la plupart du temps, or là, il m'indique mes deux sous dossier. Donc je pense qu'il parcourt tout, mais, il ne me transfert aucun fichier.

Il ne faut pas fermer le fichier juste après put, sinon c'est toujours QFtp qui l'ouvrira.

En enlevant le file->close(), il me transfère uniquement les fichiers du premier sous dossier qui est en même temps le premier élément du dossier racine...

Je viens de remarquer que tu t'es aussi trompé sur la condition. QFile::open renvoie true en cas de succès, et je ne crois pas que ce soit très utile d'ouvrir le fichier en mode texte, donc:

Code:

---

1 2 3 4 5

if (file->open(QIODevice::ReadOnly)) { ftp->put(file,dir.filePath()); } else { QMessageBox::information(0, "Erreur de lecture", file->errorString()); }

---

Tu détruis bien les QFile dans le slot connecté à commandFinished ?

Merci de ton investissement, malheureusement je n'ai plus accès à mon code avant vendredi matin.

J'ai quand même l'impression que je ne fais pas ce qu'il faut dans mon code.

Dès vendredi matin, je change ce que tu m'as dis et je rectifie mon code.
Si je peux au moins le tester avant et que ça fonctionne ou que j'ai toujours un problème je réagirai avant vendredi main.

En attendant, encore merci de ton aide.

J'ai corrigé ma condition, bête erreur de copier/coller...

Je n'ai aucun message d'erreur lors de l'ouverture des fichiers.

Sans destruction, j'obtiens 7fichiers sur 9 téléchargés du premier sous-dossier.
Quand je détruis le fichier dans la fin d'execution, l'application plante.

Il faut que je rajoute dans le slot de fin d'execution :

Code:

---

delete file;

---

??

Plutôt quelque chose comme ça:

Code:

---

1 2 3 4 5 6 7

/* dans le slot connecté à commandFinished */ if (ftp->currentCommand() == QFtp::Put) { QFile *file = static_cast<QFile*>(ftp->currentDevice()); if(file) { file->deleteLater(); } }

---