Discussion :

[michelp3]

Bonjour, en revoyant les threads, j'ai examiné le tuto suivant

http://ydisanto.developpez.com/tutor.../runtime-exec/

J'ai essayé de le tester mais process dans les 2 new Thread() pose problème

Eclipse me signale une erreur à 2 endroits --> process en rouge ds code, quelque chose m'échappe ...

Voici le code , MERCI

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
package thread;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class CopyOfCl_Process_developpez {

	public static void main(String[] args) {
		// lancement de la commande
		 Runtime rt = Runtime.getRuntime();
		 
		 try {
			 String[] argument = { "cmd ", "/c ", "java -jar C:/Users/Utilisateur/JAVA/ESSAI/TOTONERO/Toto.jar" };
			 final Process process  = rt.exec(argument);
		     } 
		 catch (IOException e1) {
		 } 
	
	// Consommation de la sortie standard de l'application externe dans un Thread separe
	new Thread() {
		public void run() {
			try {
				BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
				String line = "";
				try {
					while((line = reader.readLine()) != null) {
						// Traitement du flux de sortie de l'application si besoin est
					}
				} finally {
					reader.close();
				}
			} catch(IOException ioe) {
				ioe.printStackTrace();
			}
		}
	}.start();

	// Consommation de la sortie d'erreur de l'application externe dans un Thread separe
	new Thread() {
		public void run() {
			try {
				BufferedReader reader = new BufferedReader(new InputStreamReader(process.getErrorStream()));
				String line = "";
				try {
					while((line = reader.readLine()) != null) {
						// Traitement du flux d'erreur de l'application si besoin est
					}
				} finally {
					reader.close();
				}
			} catch(IOException ioe) {
				ioe.printStackTrace();
			}
		}
	}.start();

	}
}

[tchize_]

et bien lisez le message d'erreur: la variable process n'existe pas. Et pour cause, elle n'existe que dans une petite portion de votre code qui n'inclue pas le thread:

Code :

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

1
2
3
4
5
6
		 try {
			 String[] argument = { "cmd ", "/c ", "java -jar C:/Users/Utilisateur/JAVA/ESSAI/TOTONERO/Toto.jar" };
			 final Process process  = rt.exec(argument);
		     } 
		 catch (IOException e1) {
		 }

Dès qu'on a fini le try, elle n'existe plus. La durée de vie d'une variable en java, c'est celle du bloc qui la contient.

[wax78]

Il y'a aussi eclipse qui te souligne process en jaune aussi ... signifiant que process est instancié puis jamais utilisé ce qui devrait te mettre la puce a l'oreille...

En effet tu déclare ton process dans un Try Catch() ce qui le rends "invisible" en dehors...
Hors tu vas devoir l'utiliser en dehors du try catch (pas possible) ou tout simplement mettre les 2 thread aussi dans ce try catch.

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
public class CopyOfCl_Process_developpez
{
 
	public static void main(String[] args)
	{
		// lancement de la commande
		Runtime rt = Runtime.getRuntime();
 
		try
		{
			String[] argument = { "cmd ", "/c ", "java -jar C:/Users/Utilisateur/JAVA/ESSAI/TOTONERO/Toto.jar" };
			final Process process = rt.exec(argument);
 
			// Consommation de la sortie standard de l'application externe dans un Thread separe
			new Thread()
			{
				public void run()
				{
					try
					{
						BufferedReader reader = new BufferedReader(new InputStreamReader(process.getInputStream()));
						String line = "";
						try
						{
							while ((line = reader.readLine()) != null)
							{
								// Traitement du flux de sortie de l'application si besoin est
							}
						} finally
						{
							reader.close();
						}
					} catch (IOException ioe)
					{
						ioe.printStackTrace();
					}
				}
			}.start();
 
			// Consommation de la sortie d'erreur de l'application externe dans un Thread separe
			new Thread()
			{
				public void run()
				{
					try
					{
						BufferedReader reader = new BufferedReader(new InputStreamReader(process.getErrorStream()));
						String line = "";
						try
						{
							while ((line = reader.readLine()) != null)
							{
								// Traitement du flux d'erreur de l'application si besoin est
							}
						} finally
						{
							reader.close();
						}
					} catch (IOException ioe)
					{
						ioe.printStackTrace();
					}
				}
			}.start();
		} catch (IOException e1)
		{
		}
 
	}
}

ou encore declaré process comme variable static de la classe et ne pas utiliser final.

Edit : Doh trop lent !

[michelp3]

J'ai donc installé mes 2 threads à l'intérieur du try comme vous me l'avez

recommandé et le problème est OK

mais ... je suis à coté de la plaque car mon Toto.jar qui est appelé doit seulement affichier une mini fenetre graphique avec HelloWorld

Dans ce cas, les 2 threads sont sans objet en l'état mais que faut-il conserver ?

Je suis perdu !!!

[tchize_]

Si tu n'as pas besoin des sorties de l'application, tu peux te contenter de fermer les flux concernés.

[dotEden]

Bonsoir,

@michelp3 :

• Je ne comprend pas trop le sens de ta question, que veux-tu conserver ?

• Pour tes threads je te conseille vivement de les intégrer dans une classe à part.

• J'essayerais de faire de mon mieux pour te créer un truc propre en rentrant chez moi

@wax78 : Je ne pense pas qu'il soit correct de mettre un try sur un si grosse portion de code, et encore pire d'imbriquer les try...


Cordialement,

--

.eden

[tchize_]

les try imbriqué sont au contraire très propres dans son cas, il s'assurent la fermeture du reader.

[dotEden]

Bonsoir,

Ne peut-il pas mettre le finally directement dans le try parent ?


Cordialement,

--

.eden

[tchize_]

non, en général avec les streams on a la structure

Code :

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

1
2
3
4
5
6
7
8
9
10
try{
   Reader reader = ....;
   try{
      .... operations
   } finally {
      reader.close();
   }
} catch (IOException e){
 //....
}

Parce que celle-ci est plus ch**

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
 
   Reader reader;
   try{
       reader = ....
       .... operations
   } catch (IOException e){
       ....
   } finally {
      try{
         if (reader!=null) //faut tester du coup 
             reader.close();
      } catch (IOException e){
         // ben oui, le close lance des ioexceptions
      }
   }
   reader = null; //plus besoin

[dotEden]

Bonsoir,

J'aurais plutôt vu en encapsulant le la lecture du flux dans une fonction qui throws l'IOException, mais c'est vrai que ta façon de faire est au final bien plus simple, moi je suis peut-être encore trop attachée au notion qui m'ont été apporté par mes études (on avait pas le droit à écrire des try imbriqués).

Cordialement,

--

.eden

[michelp3]

En fait je pensais que l'exécution d'une appli extérieure nécessitait obligatoirement de gérer d'autres threads

Dans mon cas où j"affiche simplement une fenetre avec helloworld ces threads sont sans objet

LES threads evoqués ds mon exemple servent si je recois des infos du programe externe : ex : un fichier texte, ce qui n'est pas le cas --> ils sont inutiles

Dites moi si je me trompe

[dotEden]

Si, tes threads sont dans un objet anonyme (un objet que tu ne pourras plus utiliser), sinon, tes threads servent tout de même à récupérer les éventuelles erreurs de ton programme appelé et des différents messages qui pourront être envoyé par l'application.

Cordialement

[wax78]

@wax78 : Je ne pense pas qu'il soit correct de mettre un try sur un si grosse portion de code, et encore pire d'imbriquer les try...

.eden

D'ou l'eventuelle 2 ème solution qui évite tout mettre dans le "try"", et je n'ais pas parlé d'imbriquer des try ??!

[dotEden]

Ah oui excuse moi, je n'avais pas lu la dernière phrase

Cordialement

[michelp3]

j'ai éclairci certaines notions et c'est bien !!!

Bonne soirée ou bonne journée , cela dépend