Process et Thread

Version imprimable

Process et Thread

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:

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();

	}
}

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:

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.

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:

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 ! :)

18/10/2012, 17h05
michelp3

Bonjour Messieurs

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 !!!
18/10/2012, 17h40
tchize_

Si tu n'as pas besoin des sorties de l'application, tu peux te contenter de fermer les flux concernés.
18/10/2012, 17h49
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
18/10/2012, 17h54
tchize_

les try imbriqué sont au contraire très propres dans son cas, il s'assurent la fermeture du reader.
18/10/2012, 17h57
dotEden

Bonsoir,

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

Cordialement,

--

.eden

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

Code:

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:

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

18/10/2012, 19h04
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
18/10/2012, 19h05
michelp3

Merci à tous

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
18/10/2012, 19h11
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
18/10/2012, 19h29
wax78

Citation:

Envoyé par dotEden

@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 ??! :)
18/10/2012, 19h35
dotEden

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

Cordialement
18/10/2012, 20h18
michelp3

Merci à tous

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

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