Discussion :

[isa3000]

Bonjour,
J'ai créé un programme qui permet de récupérer des données via un port série et d'enregistrer ces données dans un fichier txt.
Le programme compile bien, aucune erreur ne s'affiche dans la console mais le fichier n'est pas créé.
Je n'arrive pas à trouver mon erreur.
Merci de votre aide.

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
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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
 
import java.io.*;
import javax.comm.*;
import com.sun.comm.Win32Driver;
 
public class connection {
	CommPortIdentifier portId;
	static SerialPort port;
	static InputStream in;
	static BufferedReader br;
	BufferedWriter bw;
	String donnees;
	String s;
	public connection() {
		//initialisation du driver
		Win32Driver w32Driver= new Win32Driver();
		w32Driver.initialize();
 
		//récupération du port
		try{
			portId=CommPortIdentifier.getPortIdentifier("COM2");
		}catch(NoSuchPortException ex){}
 
		//ouvrir le port
		try {
			port=(SerialPort)portId.open("Mon_Appli", 20000);
			}
		catch (PortInUseException ex) {}
 
		//paramétrer le port
		try{
			port.setFlowControlMode(SerialPort.FLOWCONTROL_NONE);
			port.setSerialPortParams
					(9600,
					SerialPort.DATABITS_8,
					SerialPort.STOPBITS_1,
					SerialPort.PARITY_NONE);
		}catch (UnsupportedCommOperationException ex){}
 
 
		//gérer le flux d'entrée 
		try {
			in = port.getInputStream();
		} catch (IOException ex) {}
 
		//lire les données
		try {
			br = new BufferedReader(new InputStreamReader(port.getInputStream()));
			s = br.readLine();
 
		} catch (IOException e) {e.printStackTrace();}
 
		//écrire les données
		fichier("c:\\test.txt",s);
 
		close();
 
	}
 
	//écrire les données dans un fichier
	public void fichier(String monFichier, String mesDonnees)
	{
		FileWriter fw;
		try {
			fw = new FileWriter(monFichier, true);
		BufferedWriter bw = new BufferedWriter(fw);
		bw.write(mesDonnees);
		bw.newLine();
		bw.close();
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
 
 
	public static void close()
	{
		try {
			in.close();
			br.close();
		} catch (IOException e) {
			e.printStackTrace();
		}
		port.close();
	}
 
 
	public static void main(String[] args) 
	{
		new connection();
	}
}

[nnovic]

C'est normal qu'aucune erreur ne s'affiche dans la console, vu que tu attrapes toutes les exceptions sans les traiter. Quand tu fais un catch, le minimum c'est au moins d'afficher l'exception dans la console...

[isa3000]

en fait au bout d'un moment il m'affiche quand meme une erreur :

Exception in thread "main" java.lang.NullPointerException
at connection.<init>(connection.java:32)
at connection.main(connection.java:92)

je vais rajouter du texte pour les exceptions

[gulain]

Bonjour,
pense à utiliser la balise [code].
En java, la convention usuelle est de donner aux classes un nom commençant par une majuscule, et aux instances un nom commençant par une minuscule.

Il faut que tu améliores ta gestion des exceptions, d'une part écrivant au moins sur la console lorsque tu en attrapes une, d'autre part en évitant de juxtaposer des blocs try catch non indépendant.
Par exemple

Code :

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

1
2
3
4
5
6
7
8
9
10
//récupération du port
try{
portId=CommPortIdentifier.getPortIdentifier("COM2");
}catch(NoSuchPortException ex){}
 
//ouvrir le port
try {
port=(SerialPort)portId.open("Mon_Appli", 20000);
}
catch (PortInUseException ex) {}

Si ton premier try attrape une exception, quand on passera dans le deuxième, port risque d'être null. Il vaut mieux imbriquer les try catch, ou bien faire un seul try avec plusieurs catch.

Pour ta gestion des entrées sorties, pense à bien fermer les flux (et donc à utiliser des blocs try finally, comme indiqué ici dans la faq java).

[isa3000]

J'ai mis un seul try et un finally avec des throws exceptions pour la classe connection. Il n'y a toujours pas de création de fichier.

J'ai vérifié, il reste bloqué au s = br.readLine(); La ligne du dessus passe (j'ai essayé en mettant des System.out.println("aa"); pour cette vérification).

[isa3000]

C'est bon j'ai réussi à le faire fonctionner. Maintenant il ne me retourne que la première ligne. Il faut que je voye comment tout récupérer.