Discussion :

[forum]

Bonjour,

Je vous propose un nouvel élément à utiliser : Classe de gestion transfert FTP

C'est simplement une classe de gestion de transfert de fichier via ftp / ftps en csharp.

Elle gère notamment le download, uploade, append de fichier.

Elle donne aussi des infos comme la taille d'un fichier, le contenu d'un dossier etc...

Vous pourrez aussi renommer des éléments, vérifier si un élément existe etc...



rappel important :

Vous ne pourrez pas vous connecter en sftp avec cette classe.

sftp utilise le protocole ssh et pas ftp.



---

Developpé sous Visual Studio 2017, C# .



Version 1.0

Qu'en pensez-vous ?

[Max]

Salut

Y a selon moi beaucoup de choses à améliorer.

Code :

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

1
2
3
4
 
GestFtp gestFtp = new GestFtp();
string foo = null;
bool bar = gestFtp.Create_Directory(foo);

• Ton code vient de lancer une NullReferenceException non gérée
• En l'état, ta classe devrait être statique (je te laisse te demander pourquoi)

Code :

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

public string Cnx_LocalDefault_FolderPath = @"C:\temp\";

• Pas moyen que ce genre de chose se trouve en dur

Code :

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

1
2
System.Net.FtpWebResponse Ftp_Response = (System.Net.FtpWebResponse)Ftp_Request.GetResponse();
Ftp_Response.Close();

• Tu devrais regarder les blocs using (cf. doc) plutôt que de t'embêter avec les Close() (dans ton cas c'est valable pour toutes tes réponses FTP, tes streams etc.)

Code :

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

1
2
3
4
5
6
7
8
 
Ftp_Request.GetResponse();
// ...
catch (Exception ex)
{
	Ftp_Error(ex);
	return false;
}

• Tu devrais te contenter de catcher les exceptions référencées sur la doc (donc ici InvalidOperationException et WebException). En catchant l'exception de plus haut niveau, tu peux masquer des erreurs qui ne sont pas de ton ressort et qui devraient remonter directement à l'appelant

Code :

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

1
2
3
4
5
public bool Cnx_UseBinary = true;                         // Bascule entre transfert de données binaire(True) ou ascii(False).
public bool Cnx_UsePassive = true;                         // Cnx_UsePassive = False si le processus de transfert de données de l'application cliente est à l'écoute d'une connexion sur le port de données
                                                                   // Cnx_UsePassive = True si le client doit initialiser une connexion sur le port de données.
 
public bool Cnx_KeepAlive = true;                         // Activation des connexions persistantes

• Tout ça, soit t'es sur une classe statique et ça se passe en paramètre d'entrée de toutes les méthodes qui font une connexion FTP, soit t'es sur une classe normale et idem dans chaque méthode ou alors dans un constructeur (et d'autres constructeurs qui ont des valeurs par défaut), mais pas figé comme ça
• Plutôt que de renvoyer des booléens partout, la classe FtpWebResponse elle a plein d'infos utiles comme StatusCode ou StatusDescription
• Tu veux que ce soit utilisé dans un contexte pro, alors ajoute impérativement la possibilité de passer à travers un proxy
• Tu devrais faire un tour sur les conventions de nommage en C#, ça faciliterait la lecture de tes sources pour tout le monde
• Dans son ensemble, ta classe n'apporte rien de plus que ce qu'on peut trouver sur la doc présente sur le MSDN

Il y a certainement des choses à ajouter, mais je m'arrête là

[Fab2bprog]

Salut,

Comme je te l'ai dit dans une autre discussion, il s'agit d'une classe csharp totalement classique et simple à comprendre, mais qui fais le job.
Je vais essayer de lui apporter quelques corrections pour aller dans le cens de tes remarques.
Je vais aussi essayer de lui apporter dans les prochaines versions quelques méthodes complémentaires comme la distinction entre dossier et fichier
( liste de dossiers uniquement, ou de fichiers uniquement, ou fichier avec une certaines extensions ), ce qui n'est pas possible ( si j'ai bien compris ) de faire de façon directe par l'usage de fonction toute faite.
S'il existe des choses genre:

Code :

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

request.Method = WebRequestMethods.Ftp.ListDirectoryDetails;

Rien ne me dit que la chaine ftp, sera strictement la meme selon le type de serveur et son système.
La lettre "d" indique que c'est un directory mais la chaine d'info de permission (drwx) est elle toujours au meme endroit ? ( en claire : est ce que c'est équivalent au ls -l qui lui est posix ) …

Il faut que je me renseigne. Mais si tu as des idées n'hésite pas.