Problem bei process.start() ExitValue

[xerion21]

Hallo zusammen,

erstmal nen frohes neues Jahr

Ich habe leider im neuen Jahr schon ein Problem.
Da ich noch ein relativer Frischling im JavaBereich bin, habe ich wahrscheins ein einfaches Problem.

Ich habe folgenden Prozess ( dieser funktioniert einwandfrei) :

ProcessBuilder pb = new ProcessBuilder();
      pb.command().add( "cmd" );
      pb.command().add( "/c" );
      pb.command().add( "***");

Jedoch bei dem pb.ExitValue() bekomme ich eine Fehlermeldung, hier der Code:

Process p = null;
      try {
         p = pb.start();
         
      } catch ( IOException e ) {
         e.printStackTrace();
      }
      
      Scanner s = new Scanner( p.getInputStream() );
      // System.out.println( s.next() );
      s.close();   
      errorlevelcopy = p.exitValue();

Die Fehlermeldung sieht wie folgt aus:

Exception in thread "AWT-EventQueue-0" java.lang.IllegalThreadStateException: process has not exited
at java.lang.ProcessImpl.exitValue(ProcessImpl.java:442)
at scripts.SystemScripts.install7Zip(SystemScripts.java:58)
at gui.ButtonActionListener.checkSelected(ButtonActionListener.java:128)
at gui.ButtonActionListener.actionPerformed(ButtonActionListener.java:176)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2018)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2341)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
at java.awt.Component.processMouseEvent(Component.java:6505)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3320)
at java.awt.Component.processEvent(Component.java:6270)
at java.awt.Container.processEvent(Container.java:2229)
at java.awt.Component.dispatchEventImpl(Component.java:4861)
at java.awt.Container.dispatchEventImpl(Container.java:2287)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4832)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4492)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4422)
at java.awt.Container.dispatchEventImpl(Container.java:2273)
at java.awt.Window.dispatchEventImpl(Window.java:2719)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:735)
at java.awt.EventQueue.access$200(EventQueue.java:103)
at java.awt.EventQueue$3.run(EventQueue.java:694)
at java.awt.EventQueue$3.run(EventQueue.java:692)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:87)
at java.awt.EventQueue$4.run(EventQueue.java:708)
at java.awt.EventQueue$4.run(EventQueue.java:706)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:705)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:91)

Hat jemand eine Idee, warum dieser Quellcode die Fehlermeldung produziert?

Danke schonmal

Gruß

Xerion21

EDIT:
Wenn ich den Quellcode so umschreibe scheint es zu gehn:

try {
         p = pb.start();
         errorlevelcopy = p.waitFor();
         
      } catch ( IOException | InterruptedException e ) {
         e.printStackTrace();
      }

 

[turtle]

process has not exited

Das ist doch eindeutig, oder?

Du startest einen externen Prozess, wartest aber nicht auf sein Ende.

Da läuft also dein Thread 1, der den Thread 2 startet. Während T1 bereits lauffähig im Speicher ist, muss T2 dies erst schaffen. Dies braucht "etwas" länger, während dessen läuft T1 bereits weiter und möchte den Exit-Status abfragen. Kann er aber noch gar nicht,da T2 noch nicht beendet (wahrscheinlich noch nicht einmal vollständig gestartet).

Füg mal ein

p.waitFor();

PS: Bis zum Ende lesen, schadet nicht. Hast du anscheinend schon gemacht.

 

[xerion21]

Das ist doch eindeutig, oder?

Du startest einen externen Prozess, wartest aber nicht auf sein Ende.

Da läuft also dein Thread 1, der den Thread 2 startet. Während T1 bereits lauffähig im Speicher ist, muss T2 dies erst schaffen. Dies braucht "etwas" länger, während dessen läuft T1 bereits weiter und möchte den Exit-Status abfragen. Kann er aber noch gar nicht,da T2 noch nicht beendet (wahrscheinlich noch nicht einmal vollständig gestartet).

Füg mal ein

p.waitFor();

PS: Bis zum Ende lesen, schadet nicht. Hast du anscheinend schon gemacht.

ja ich habe getExitValue() durch waitFor() ersetzt.
Was interessant war, dass es teilweise ( ab und zu ) funktioniert hat und mal nicht. Aber waitFor() gibt ja auch den ErrorLevel aus, also perfekt

Merci

 

[turtle]

Bei mir muss der Start-Thread mindestens 11 Millisekunden waren, bevor er den Exitcode abfragen kann.

Das war "nur" ein "cmd /c dir" Kommando, kann also bei anderen Prozessen abweichen.

Wichtig ist sich nur, sich bewusst zu machen, das da zwei unabhängige Prozesse laufen und man "irgendwie" synchronisieren muss.

 

[xerion21]

Bei mir muss der Start-Thread mindestens 11 Millisekunden waren, bevor er den Exitcode abfragen kann.

Das war "nur" ein "cmd /c dir" Kommando, kann also bei anderen Prozessen abweichen.

Wichtig ist sich nur, sich bewusst zu machen, das da zwei unabhängige Prozesse laufen und man "irgendwie" synchronisieren muss.

das heißt Prozess A ist der sage ich mal "StandardProzess" und der andere Prozess ist der "p-Prozess", diese laufen nebereinander her.

Diese Wartezeit bringst du mit einem sleep ein? oder mit welchem Befehl?

Danke für deine Hilfe

 

[turtle]

Yep. Im StandardProzess

Thread.sleep(11); // 10 geht nicht

Dies ist aber schlechter Stil, da dies vom Rechner, Auslastung (und Tidenhub ;-) abhängig ist. Besser waitFor() wie wir ja schon wissen