Discussion :

[sousou12]

Bonjour,

je veux lire un fichier txt en java. ma fonction java marche très bien mais le problème est le suivant :
j'écris ma fonction dans une classe java puis j'appelle cette fonction de puis une autre classe mais malheureusement j'ai une erreur merci d'avance

code de classe lire fichier:

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
package file;
 
import java#io#BufferedReader;
import java#io#FileInputStream;
import java#io#InputStream;
import java#io#InputStreamReader;
 
public class FileManipulation 
{
	/**
         * Constructor of the class, */
	/*public FileManipulation## {
			}
	*///lecture du fichier texte de communication
	public String readfile ##
	{
		String sCurrentLine="";		
		String chaine="";
		String fichier ="D:/communicationfile#txt";
 
		try
		{
			InputStream ips=new FileInputStream#fichier#; 
			InputStreamReader ipsr=new InputStreamReader#ips#;
			BufferedReader br=new BufferedReader#ipsr#;
			String ligne;
			while ##ligne=br#readLine###!=null#
			{
				System#out#println#ligne#;
				sCurrentLine=ligne;
				chaine+=ligne+"\n";
			}
			br#close##; 
 
		}		
		catch #Exception e#
		{
			System#out#println#e#t#String###;
		}
		return 	sCurrentLine;
	}
 
}

code d'appeler de cette classe:

Code :

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

1
2
3
FileManipulation fileManipulation = null;
         String request=fileManipulation.readfile# #;
				discr#request#;

[Népomucène]

C'est illisible. Il faut mettre ton code dans les balises codes prévues à cet effet.

Sinon pourquoi n'utilises-tu pas la classe Scanner ?

[sousou12]

merci pour la reponse voici le code
code de classe lecture fichier

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
package file;
 
import java.io.BufferedReader;
import java.io.FileInputStream;
import java.io.InputStream;
import java.io.InputStreamReader;
 
public class FileManipulation 
{
	/**
         * Constructor of the class, */
	/*public FileManipulation() {
			}
	*///lecture du fichier texte de communication
	public String readfile ()
	{
		String sCurrentLine="";		
		String chaine="";
		String fichier ="D:/communicationfile.txt";
 
		try
		{
			InputStream ips=new FileInputStream(fichier); 
			InputStreamReader ipsr=new InputStreamReader(ips);
			BufferedReader br=new BufferedReader(ipsr);
			String ligne;
			while ((ligne=br.readLine())!=null)
			{
				System.out.println(ligne);
				sCurrentLine=ligne;
				chaine+=ligne+"\n";
			}
			br.close(); 
 
		}		
		catch (Exception e)
		{
			System.out.println(e.toString());
		}
		return 	sCurrentLine;
	}
 
}

voici le code d'appelle

Code :

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

1
2
3
FileManipulation fileManipulation=null;
    String request=fileManipulation.readfile( );
     disc(request);

[ludomacho]

bonjour,

définie ta méthode readFile comme static,

Code :

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

1
2
 
public static String readfile ()

ainsi tu peux l'appeler depuis une autre classe de la façon suivante

Code :

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

1
2
3
 
     String request=FileManipulation.readfile( );
     disc(request);

[sousou12]

Merci à tous il marche tres bien

[Népomucène]

Merci à tous il marche tres bien

Ben là, je suis un peu étonné
à priori, la méthode ne retourne que la dernière ligne, mais bon, c'est peut-être voulu.
Quant à la variable "chaine", je ne vois pas à quoi elle sert.
Enfin, avec la classe Scanner, le code aurait pris 2 fois moins de lignes ...

[thelvin]

Ben là, je suis un peu étonné
à priori, la méthode ne retourne que la dernière ligne, mais bon, c'est peut-être voulu.
Quant à la variable "chaine", je ne vois pas à quoi elle sert.

Effectivement...

Enfin, avec la classe Scanner, le code aurait pris 2 fois moins de lignes ...

Non. En écrivant bien le code, il prendrait deux fois moins de lignes.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public String readfile() throws IOException {
  String fichier ="D:/communicationfile.txt";
 
  try(InputStream ips = new FileInputStream(fichier)) {
    InputStreamReader ipsr=new InputStreamReader(ips);
    BufferedReader br=new BufferedReader(ipsr);
    String chaine = ""; 
    String ligne;
    while((ligne=br.readLine())!=null) {
      chaine += ligne+"\n";
    }
    return chaine;
  }
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public String readfile() throws IOException {
  String fichier ="D:/communicationfile.txt";
 
  File file = new File(fichier);
  try(Scanner scanner = new Scanner(file)) {
    String chaine = ""; 
    while(scanner.hasNextLine()) {
      chaine += scanner.nextLine()+"\n";
    }
    return chaine;
  }
}

Deux lignes en moins parce qu'on économise la création du InputStreamReader et la déclaration String ligne;. C'est pas une révolution non plus -_-°.
Bon, je préfère son hasNextLine() au lieu de vérifier si la ligne est null. Mais quand on utilise un Scanner, il est pas immédiatement évident qu'on veut juste tokeniser les lignes. Alors qu'un BufferedReader ne "tokenise" que ça, donc la question se pose pas.

[Népomucène]

<CHIPOTAGE>

J'ai dis 2 fois moins de ligne en reprenant texto l'exemple donné dans la faq :

Code :

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

1
2
3
4
5
6
7
8
9
10
public String readfile() throws Exception {
     String filePath ="D:/communicationfile.txt";
     Scanner scanner=new Scanner(new File(filePath));
     String chaine = ""; 
     while (scanner.hasNextLine()) {
          chaine += scanner.nextLine()+"\n";
     }
     scanner.close();
     return chaine;
}

</CHIPOTAGE>

En revanche, dans ton exemple, tu utilises un FileInputStream plutôt qu'un objet File.
Quel avantage as-t-on dans ce cas précis ?

[thelvin]

Aucun. Je m'étais figuré que Scanner n'implémente pas AutoCloseable et donc qu'on ne peut pas le mettre dans le try-with-resource.
Je corrige mon exemple, ce qui fait gagner une ligne de plus. C'est vrai que ça commence à compter.

En tout cas l'exemple que tu cites ne s'assure pas que scanner.close() est bien appelé. Pas génial à garder en FAQ. Il faudrait que ce soit dans un finally, au moins.