Discussion :

[david06600]

Bonjour je voudrais avoir un peu d'aide concernant la serialization et lecture dans un fichier, voici mon 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
 
try
		{
    		ObjectInputStream ois = new ObjectInputStream(new FileInputStream(jobsFile));
    		while(jobsFile.length() != -1)
    		{
    			TestObject job = (TestObject)ois.readObject();
    			listJob.add(job);
    		}
		}
    	catch(IOException e)
		{
    		System.out.println("IOException : ");
    		e.printStackTrace();
		}

J'ai un EOFException qui est lancé, je me doute que cela provient de mon while, donc je voudrais savoir comment lire tout les objets du fichiers jusqu'au dernier. Qu'est ce qu'il faut mettre dans la boucle while?
Merci

[iohack]

En dessous du bloc "catch(IOException ioe)", tu rajoutes un "catch(EOFException eofe)"

[EDIT] tu devrais pas rajouter un "catch(ClassNotFoundException cnfe)" lors de ton appel à readObject ??

[le y@m's]

En dessous du bloc "catch(IOException ioe)", tu rajoutes un "catch(EOFException eofe)"

c'est au dessus, sinon, étant donné que EOFException dérive de IOException, l'exception sera attrapée dans le premier catch et ne passera jamais dans le second (d'ailleur le compilateur ne te laissera pas faire ).

[iohack]

Au temps pour moi

[le y@m's]

Tu peux utiliser la méthode available() de la classe ObjectInputStream.

Autre remarque : pense à fermer ton flux dans un bloc finally

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
try {
    ObjectInputStream ois = new ObjectInputStream(new FileInputStream(jobsFile));
    try {
        while(ois.available() > 0) {
            TestObject job = (TestObject)ois.readObject();
            listJob.add(job);
        }
    } finally {
        ois.close();
    }
} catch(IOException e) {
    System.out.println("IOException : ");
    e.printStackTrace();
}

[le y@m's]

Tu peux utiliser la méthode available() de la classe ObjectInputStream.

Je suis allez un peu trop vite , en fait cela ne marche pas, il vaut mieux utiliser la EOFException.

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
try {
    ObjectInputStream ois = new ObjectInputStream(new FileInputStream(jobsFile));
    try {
        Boolean b = true;
        while(b) {
            TestObject job = (TestObject)ois.readObject();
            listJob.add(job);
        }
    } finally {
        ois.close();
    }
} catch(EOFException e) {
    System.out.println("Fin du fichier atteint");
} catch(IOException e) {
    System.out.println("IOException : ");
    e.printStackTrace();
}

[le y@m's]

Il serait peut-être même plus élégant de gérer l'EOFException comme suit :

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
try {
    ObjectInputStream ois = new ObjectInputStream(new FileInputStream(jobsFile));
    try {
        Boolean b = true;
        while(b) {
            try {
                TestObject job = (TestObject)ois.readObject();
                listJob.add(job);
            catch(EOFException e) {
                b = false;
            } 
        }
    } finally {
        ois.close();
    }
} catch(IOException e) {
    System.out.println("IOException : ");
    e.printStackTrace();
}

Sinon, si c'est toi qui gère la sérialisation, pourquoi ne pas directement sérialiser la List de tes TestObject ? ainsi tu n'aurais plus de boucle puisque tu désérialiserais directement la List.

[david06600]

Salut l'exception ne se lance plus par contre je sérialise 2 objets dans un fichier et dans ma liste je me retrouve qu'avec le premier objet?
Voici le code complet :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
 
File jobsFile = new File("c:\\Dossiers\\TestObject.txt");
    	List listJob = new ArrayList();
 
    	if(jobsFile.length() == -1)
    	{
    		//Création d'une liste d'objet test.
    		TestObject job1 = new TestObject();
    		job1.setNom("QJMAIL");
    		job1.setEtape("JSGD_03");
    		job1.setMachine("SWR01");
    		job1.setOddjobXML(new File("C:\\Dossiers\\projet_Open\\jobSamples\\examples\\userguide\\first1.xml"));
 
    		TestObject job2 = new TestObject();
    		job2.setNom("DJACTIF");
    		job2.setEtape("JSGD_01");
    		job2.setMachine("SWR04");
    		job2.setOddjobXML(new File("C:\\Dossiers\\projet_Open\\jobSamples\\examples\\userguide\\first2.xml"));
 
    		TestObject.saveToFile(job1, jobsFile);
    		TestObject.saveToFile(job2, jobsFile);
		}
 
    	try
		{
    		ObjectInputStream ois = new ObjectInputStream(new FileInputStream(jobsFile));
    		try
			{
    			boolean b = true;
    			try
				{
    				while(b)
            		{
            			TestObject job = (TestObject)ois.readObject();
            			//String name = job.getNom();
            			//System.out.println(name);
            			listJob.add(job);
            		}
				}
    			catch(EOFException e)
				{
	    			b = false;
				}
			}
    		finally
			{
    			ois.close();
			}
 
		}
    	catch(IOException e)
		{
    		System.out.println("IOException : ");
    		e.printStackTrace();
		}
 
 
    	request.setAttribute(Constants.LIST_JOB, listJob);
 
 
        // Finish with
        return (mapping.findForward("listeDiff"));

si tu vois d'autre chose dans le code qui ne sont pas correct n'hésite pas à me le faire remarqué.

[le y@m's]

A mon avis, le problème vient du fait que tu appel deux fois ta méthode saveToFile() (dont tu aurais pu mettre le code ) et je suppose que l'un des deux objet doit être écrasé dans le fichier.

Comme je te l'ai dit dans mon précédent post, la meilleure solution est encore de sérialiser directement la List.

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
44
45
 
List listJob = new ArrayList();
try {
    File jobsFile = new File("c:\\Dossiers\\TestObject.txt");
    if(jobsFile.length() == -1) {
        //Création d'une liste d'objet test.
        TestObject job1 = new TestObject();
        job1.setNom("QJMAIL");
        job1.setEtape("JSGD_03");
        job1.setMachine("SWR01");
        job1.setOddjobXML(new File("C:\\Dossiers\\projet_Open\\jobSamples\\examples\\userguide\\first1.xml"));
        TestObject job2 = new TestObject();
        job2.setNom("DJACTIF");
        job2.setEtape("JSGD_01");
        job2.setMachine("SWR04");
        job2.setOddjobXML(new File("C:\\Dossiers\\projet_Open\\jobSamples\\examples\\userguide\\first2.xml"));
 
        List saveList = new ArrayList();
        saveList.add(job1);
        saveList.add(job2);
 
        // sérialisation de la List
        ObjectOutpuStream oos = new ObjectOutputStream(new FileOutpuStream(jobsFile));
        try {
            oos.writeObject(saveList);
            oos.flush();
        } finally {
            oos.close;
        }
    }
 
    // désérialisation de la List
    ObjectInputStream ois = new ObjectInputStream(new FileInputStream(jobsFile));
    try {
        listJob = (List) ois.readObject();
    } finally {
        ois.close();
    }
} catch(IOException e) {
    System.out.println("IOException : ");
    e.printStackTrace();
}
request.setAttribute(Constants.LIST_JOB, listJob);
// Finish with
return (mapping.findForward("listeDiff"));