Initialisation des "input streams"

Version imprimable

Bonjour,

J'espère que cen'est pas encore un truc évident pour tout le monde, mais sur la classe suivante je suis venu à bout de toutes les erreurs de compilation sauf une.

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
package sub;
 
import java.io.*;
 
public class TsFinder{
	String fileName;
	TsFinder(String s){fileName=s;} //Constructeur avec fourniture du nom 
 
	public int getStart(){
		int inByte;
		int iBase=0;
		int iCycle=0;
		int iSuccess=0;
		boolean isFound=false;
		int offset=0;
		BufferedInputStream inStream;
 
		try{
			inStream = new BufferedInputStream (new FileInputStream(fileName));
			while((inByte=inStream.read())!=-1){
				if (isFound==true){
					if ((inByte==0x47)&(iCycle==188)){
						iSuccess++;
						iCycle=0;
					}
				}else {
					if(inByte==0x47){
						isFound=true;
						offset=iBase;
					}
				}
			}
 
		}catch (FileNotFoundException ef){
			System.out.println("File not found");
		}catch (IOException e){
			System.out.println(e);
		}finally{
			if(inStream!=null) inStream.close();
		}
		return offset;
	}
 
}

Sur la clause "finally", Eclipse me dit
1. Unhandled exception type IOException
2. The local variable inStream may not have been initialized.
3. The local variable inStream may not have been initialized.

Merci des avis de spécialistes

:)Gabier

Ta variable inStream doit être initialisé :
Code:

1 2 BufferedInputStream inStream = null;
Dans ton finally tu doit catcher l'IOException
Code:

1 2 3 4 5 6 7 8 }finally{ try{ if(inStream!=null) inStream.close(); }catch (IOException e){ System.out.println(e); } }

Citation:

Envoyé par guigui5931

Ta variable inStream doit être initialisé :

Là je trouve que le compilateur est un peu pointilleux.

Citation:

Envoyé par guigui5931

Dans ton finally tu doit catcher l'IOException

OK, mais ça devient vraiment compliqué.

Dans l'exemple ci-dessous pris dans un tuto sur ce site
et dont je me suis largement inspiré, il n'y a ni initialisation de l'input stream, ni catch multiple de IOException. Il est vrai que l'auteur met les catch après finally, ce qui paraîtt également bizarre. Mais enfin, ce que tu m'as donné résoud le problème, alors merci.
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 26 27 28 29 30 31 32 33 34 import java.io.*; public class TestBufferedInputStream { public static void main(String[] args){ BufferedInputStream bis; int byteLu; Byte[] buffer; try{ //creation d'un BufferedInputStream sur un InputStream ayant pour //source un fichier nommé source, bis=new BufferedInputStream(new FileInputStream("source")); //la methode available permet de connaitre le nombre de bytes qui //pourront étre lus d'une manière non bloquante. System.out.println(bis.available()); //lecture des données System.out.println("debut lecture"); try{ long startChrono=System.currentTimeMillis(); while((byteLu=bis.read())!=-1); System.out.println("fin lecture"); System.out.println("durée="+(System.currentTimeMillis()-startChrono)); } finally{ //ne pas oublier de refermer le flux bis.close(); } }catch(FileNotFoundException ef){ System.out.println("fichier introuvable"); } catch(IOException e){ System.out.println(e); } } }
:)Gabier

25/03/2008, 10h09
adiGuba

Salut,

Citation:

Envoyé par gabier

Là je trouve que le compilateur est un peu pointilleux.

Oui, et cela permet un code sûr et surtout l'impossibilité d'avoir une référence dans un état incorrect. Bref du code un peu plus sûr.

Citation:

Envoyé par gabier

OK, mais ça devient vraiment compliqué.

+1

L'utilisation d'un try/finally à l'intérieur d'un blog try/catch est plus souple à mon gout. De plus cela permet de limiter la porté de l'objet et de la déclarer lors de l'ouverture.

a++
26/03/2008, 07h51
gabier

Citation:

Envoyé par adiGuba

Oui, et cela permet un code sûr et surtout l'impossibilité d'avoir une référence dans un état incorrect. Bref du code un peu plus sûr.

Ouais...:roll: Pour certains objets il râle et pour d'autres il laisse faire et initialise lui-même à 0 ou NULL ... Mais le principal est de le savoir.;)

Citation:

Envoyé par adiGuba

L'utilisation d'un try/finally à l'intérieur d'un blog try/catch est plus souple à mon gout. De plus cela permet de limiter la porté de l'objet et de la déclarer lors de l'ouverture.

Pas compris, mais c'est pas grave.

Merci pour les conseils

:) Gabier
26/03/2008, 08h55
adiGuba
Citation:

Envoyé par gabier

Ouais...:roll: Pour certains objets il râle et pour d'autres il laisse faire et initialise lui-même à 0 ou NULL ... Mais le principal est de le savoir.;)

Cela ne dépend pas du type de l'objet mais de la référence que tu utilises :
- Les variables locales doivent être initialisé avant d'être utilisé.
- Les attributs d'instances sont implicitement initialisé à null.
Citation:

Envoyé par gabier

Pas compris, mais c'est pas grave.

Ben que le try/finally dans un try/catch apporte un meilleur découpage qu'un seul et unique try/catch/finally...

a++