Discussion :

[midou84]

bonjour a tous
voila j'ai un pgm en java qui lit un fichier .txt et mettre ces lignes dans un tableau,le fichier contient 6 lignes alors la table doit contenire 6 lignes mais il m'affiche que la taille du fichier est 15 et le tableau de sortie est null,voici le code:

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
 
import java.io.*;
 
public class  LireFichier{
 
	  static File fichier=new File("users.txt");;
	  static int i=0;	
 
     public static String []  lecture_fichier(String fichier) {
 
       int taille=fichier.length();
       System.out.println(taille);
       String[] tableau=new String[taille];
 
		try{
			InputStream ips=new FileInputStream(fichier); 
			InputStreamReader ipsr=new InputStreamReader(ips);
			BufferedReader br=new BufferedReader(ipsr);
			String  ligne;
 
			while ((ligne=br.readLine())!=null){
		    	tableau[i]=ligne;
		    	System.out.println(tableau[i]);
    	       	i++;	
    	    	}
	    	br.close();
	    	}		
		catch (Exception e){
			System.out.println(e.toString());
		    }
	  return tableau ;
 
	}
 
	public static void main (String[] args){
	 String	rep[]=lecture_fichier("utilisateur.txt");
	 	   	for(int j=0;j<rep.length;j++) { 
        	System.out.println(rep[i]);
            }
 
	 	}		
}

je ne sais pas ou est le pb ,un peut d'aide merci d'avance

[Grimly_old]

Code :

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

       String[] tableau=new String[taille];

A éviter !
Tu veux renvoyer un tableau dont tu ne connais pas la taille (tu as la taille du fichier en octets mais pas son nombre de lignes), utilise donc une liste ArrayList<String> et sa méthode .toArray() pour le retour sous forme de tableau String[].

Tes résultats seront nettement plus cohérents à mon avis.

S'il y a encore une erreur, elle m'est inconnue.

[fnobb]

Bonjour,

d'accord avec Grimly

File.length() est la taille mémoire du fichier

concernant la lecture d'un fichier dans un tableau vous avez déjà eu une réponse

lecture d'un fichier et le mettre dans un tableau

[fnobb]

Code :

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

1
2
3
4
5
6
7
8
	public static void main (String[] args){
	 String	rep[]=lecture_fichier("utilisateur.txt");
	 	   	for(int j=0;j<rep.length;j++) { 
        	System.out.println(rep[i]); <---- Pas i mais j 
            }
            
	 	}

[adiGuba]

Salut,


Désolé mais tout est loin d'être logique et surtout pas très propre...

• Tu utilises des attributs statiques abusivement ! Pourquoi as-tu besoin d'attribut static ici ??? C'est une source d'erreur et de consommation mémoire abusive (les objets ne pourront pas être libéré), mais c'est surtout inutile dans ton cas ! Utilises des variables locales
• D'ailleurs tu as un conflit de nom entre l'attribut static de type File et le paramètre String qui s'appellent tous les deux "fichier" : du coup tu récupères la longueur de la chaine et non pas la taille du fichier !
• De toute manière la taille d'un fichier ne correspond pas à son nombre de ligne, mais à son nombre de caractères...
• Si tu veux récupérer les lignes d'un fichier dans un tableau il est préférable d'utiliser une ArrayList que tu transformera ensuite en tableau...
• Tu n'utilises pas de try/finally pour fermer ton fichier (lire : Comment libérer proprement les ressources ?)

Bref ta méthode lecture_fichier() devrait plutôt ressembler à ceci :

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
	public static String[] lecture_fichier(String fichier) {
		List<String> list = new ArrayList<String>();
		try {
			BufferedReader br = new BufferedReader(new FileReader(fichier));
			try {
				String ligne;
				while ((ligne = br.readLine()) != null) {
					list.add(ligne);
				}
			} finally {
				br.close();
			}
		} catch (Exception e) {
			System.out.println(e.toString());
		}
		return list.toArray(new String[list.size()]);
	}

(au passage la méthode devrait plutôt se nommer lectureFichier() voir readFile()...)


a++

[miloux32]

Etrange aussi tes objets fichiers ....
qui un coup designe le fichier "user.txt" et une autre fois la chaine "utilisateur.txt"

[midou84]

rien n'est logique dans cette class,etant donné débutant en java tous m'apparait logique merci pour vos réponses

[19cmos83]

'lut,

tu as quasiment toutes tes variables en static ?!!!
pourquoi ? l'intêret ? la particularité de java c'est que tu es maîtres de tout c que tu écris, qui est censé être en connaissance de cause(!=javascript). Donc si tu prends la décision d'écrire tel ou tel chose de cette manière, tu dois être capable de répondre au pourquoi si l'on te questionne.

pourquoi static ?

si tu ne sais pas, fais sans.

tu utilises les tableaux aussi, c'est bien, ok, mais cependant pour t'éguiller, il y a mieux pour le remplissage de tableau dynamque.


essaie de chercher via google : Arraylist ou encore vector ou encore hashtable ... dans ton cas Arraylist ira très bien.


@ toute.


ps :

etant donné débutant en java tous m'apparait logique

et je dirais même plus, Logique !