Discussion :

[CliffeCSTL]

Bonjour,

Je souhaite récupérer le contenu d'un fichier, dans un répertoire FTP, en java :

Code :

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

private static String getStringContentFromFile(String fileToGet)

fileToGet est le nom du fichier, exemple "toto.txt"

Voila mon code, qui semble être correct :

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
16
17
18
19
20
21
22
23
24
25
26
27
private static String getStringContentFromFile(String fileToGet) throws SocketException, IOException {
    String fileContent = null;
 
    // Connection
    FTPClient ftpClient = new FTPClient();
    ftpClient.connect(server);
    ftpClient.login(user, pass);
 
    // On change de répertoire
    ftpClient.enterLocalPassiveMode();
    ftpClient.changeWorkingDirectory(directory);
 
    fileContent = convertStreamToString(ftpClient.retrieveFileStream(fileToget));
 
    // Fermeture connection
    ftpClient.logout(); ftpClient.disconnect();
 
    return fileContent;
}    
 
private static String convertStreamToString(java.io.InputStream is) {
    java.util.Scanner scanner = new java.util.Scanner(is);
    java.util.Scanner s = scanner.useDelimiter("\\A");
    String stringToReturn = s.hasNext() ? s.next() : "";
    scanner.close();
    return stringToReturn;
}

Pouvez-vous vérifier si je n'ai rien oublié ?

mon code doit retourner "null" si le fichier n'existe pas.
A quoi sert ftpClient.enterLocalPassiveMode(); ?

Merci.

[wax78]

Voila mon code, qui semble être correct :

Pouvez-vous vérifier si je n'ai rien oublié ?

Lance ton code et regarde ce qu'il fait, on ne vas pas le faire pour toi...

A quoi sert ftpClient.enterLocalPassiveMode(); ?

A dire au serveur que tu veux passer en mode passif pour les transferts.

[CliffeCSTL]

Merci.

Quelqu'un d'autre ?

[eulbobo]

Merci.

Quelqu'un d'autre ?

Oui, il manque le test pour renvoyer null si le fichier n'existe pas.

[CliffeCSTL]

Oui, il manque le test pour renvoyer null si le fichier n'existe pas.

Exact.

Au niveau de la connexion FTP, tout est correct ?

[eulbobo]

Là comme ça j'aurai tendance à dire oui... Après, tu pourrais aussi récupérer à chaque étape le code de retour du serveur pour savoir si c'est utile d'aller plus loin.
En gros, chaque méthode renvoie normalement un Objet ou un code qui te permet de dire si tout s'est bien passé ou s'il y a eu une erreur.

J'aime cette lib pour ça : de base, ça ne renvoie pas des exceptions de tous les côtés pour tout et n'importe quoi, et ça te laisse gérer les problèmes toi même avec les codes retours qui t'intéressent ! (en gros, ils te permettent de ne pas faire une gestion par exception, et ça c'est bien).

Disconnect à la fin c'est bien aussi ! Tout le monde n'y pense pas. Mais c'est nécessaire.