x264.exe aus Java aufrufen - wie mit Pause bei Fehler?

Sicher das Du das Auslesen richtig machst?
Ich greife unter C++ auf den StandardError-Output zu um die x264 Ausgaben zu erhalten.

Cu Selur

Das, was ihr hier besprecht, klingt so, als ob es in der "Software-Entwicklung" besser aufgehoben wäre.

Und noch eins, sasuke9999: Bitte in Zukunft etwas mehr Mühe geben bei der Wahl eines aussagekräftigen Beitragstitels.

Eine frage: Wofür brauchst du das cmd /k start am Anfang?
Solang du keine Shell-Befehle ausführen willst, ist das ziemlich überflüssig und du handelst dir unnötig eine Abhängigkeit von "cmd.exe" ein.

Außerdem sollte man Pfadangaben immer schön braf in Anführungszeichen packen, da sie Leerzeichen enthalten könnten!
Also "irgendwas \"" + pfad + "\" irgendwas" anstatt "irgendwas " + pfad + " irgendwas".

Ich würde es also so versuchen:

Zitat

Process p=Runtime.getRuntime().exec("\" + pfad + "\" " + parameter + " \"" + ziel + "\" \"" + quelle + "\"");

Wenn unbedingt mit cmd.exe dann ist es glaub ich so richtig:

Zitat

Process p=Runtime.getRuntime().exec("cmd.exe /k \"start \"" + pfad + "\" " + parameter + " \"" + ziel + "\" \"" + quelle + "\"\"");

(Man muss für cmd.exe also alles, was nach dem /k kommt nochmals in Anführungszeichen packen, auch wenn der Befehl selbst wiedrum Anführungszeichen enthält)

vermutlich erleuchtend: http://oreilly.com/pub/h/1092
Anmerkungen:
1. .getInputStream() liest den Standard Input
.getErrorStream() liest den Standard Error <- dieser ist hier gefragt
2. der Prozess und der Inputreader sollten in eigenen Threads laufen

Cu Selur

Einen sehr interessanten Artikel zum Thema "Stdout/Stderr eines Konsolen-Prozess umleiten" gibt es hier:
http://support.microsoft.com/kb/190351/de

Ist zwar nicht Java, sondern für C++ bzw. Win32, aber könnte trotzdem hilfreich sein, um das Prinzip zu verstehen...

Wenn Pfadnamen Leerzeichen enthalten - auch wenn dies nur möglicherweise der Fall ist - muss man sie in Anführungszeichen setzen. Das gilt sowohl für den Pfad zu deiner x264.exe, als auch für die Pfade die du an x264 übergibst. Du solltest deshalb schon darauf achten alle Pfade in Anführungszeichen zu setzen, sonst sind Probleme vorprogrammiert!

Beispiel:

Zitat

"C:\Users\Hans Wurst\Desktop\Projektordner\x264.exe" --qp16 --brframe --output "C:\Users\Hans Wurst\Desktop\Projektordner\Ziel.mkv" "C:\Users\Hans Wurst\Desktop\Projektordner\Quelle.avs"

Macht als Java String mit korrekt maskierten Anführungszeichen dann:

Zitat

String cmdln = "\"C:\\Users\\Hans Wurst\\Desktop\\Projektordner\\x264.exe\" --qp16 --brframe --output \"C:\\Users\\Hans Wurst\\Desktop\\Projektordner\\Ziel.mkv\" \"C:\\Users\\Hans Wurst\\Desktop\\Projektordner\\Quelle.avs\"";

Natürlich werden die Pfade in deinem fertigen Programm nicht "hardcoded" sein :zunge:

Und wenn du mit cmd.exe arbeitest, was man zur Parameter-Übergabe sicher NICHT braucht, dann muss NOCHMAL der gesamte Befehl in Anführungszeichen also:

Zitat

cmd.exe /k ""C:\Users\Hans Wurst\Desktop\Projektordner\x264.exe" --qp16 --brframe --output "C:\Users\Hans Wurst\Desktop\Projektordner\Ziel.mkv" "C:\Users\Hans Wurst\Desktop\Projektordner\Quelle.avs""

Den Java String spar ich mir mal an dieser Stelle

Zitat von sasuke9999

Ich hab es mit den "\" ausprobiert klappte aber leider nicht. Ohne cmd fürht er erst gar nicht die exe aus. Man bruacht scheinbar "cmd start".

Nö, das kannst du mir ruhig glauben. Ich hab schon das ein oder andere front-end gebastelt

Korrekt wäre meiner Meinung nach außerdem:

Zitat

Process application = Runtime.getRuntime().exec("cmd /k \"start \"" + pfad + "\" " + parameter + " \"" + ziel + "\" \"" + quelle + "\"\"");

Oder eben einfacher:

Zitat

Process application = Runtime.getRuntime().exec("\"" + pfad + "\" " + parameter + " \"" + ziel + "\" \"" + quelle + "\"");

Vlleicht lässt du dir die von deinem Programm generierte Kommandozeile mal von Java auf der Console ausgeben, um sie dann von Hand in der Command Shell zu testen...

Klappt bei mir 1a, wenn ich es so mache :cool:

public class Test {	public static void main(String[] args) throws IOException {		String pfad = "C:\\Program Files (x86)\\MPlayer for Windows\\MPlayer.exe";		String parameter = "-framedrop -vo gl";		String ziel = "F:\\ViDeOz\\Music\\Soulfly - Back To The Primitive (Live @ Area4 Festival 2008).avi";		Process application = Runtime.getRuntime().exec("\"" + pfad + "\" " + parameter + " \"" + ziel + "\"");		try {			application.getInputStream().close();			application.getOutputStream().close();			application.getErrorStream().close();			application.waitFor();		} catch (InterruptedException e) {			e.printStackTrace();		}		System.out.println("Exit Code: " + application.exitValue());	}}

Alternativ auch mit deiner heiß geliebten cmd.exe

public class Test {
	public static void main(String[] args) throws IOException {
		String pfad = "C:\\Program Files (x86)\\MPlayer for Windows\\MPlayer.exe";
		String parameter = "-framedrop -vo gl";
		String ziel = "F:\\ViDeOz\\Music\\Soulfly - Back To The Primitive (Live @ Area4 Festival 2008).avi";

		Process application = Runtime.getRuntime().exec("cmd.exe /k \"\"" + pfad + "\" " + parameter + " \"" + ziel + "\"\"");

		try {
			application.getInputStream().close();
			application.getOutputStream().close();
			application.getErrorStream().close();
			application.waitFor();
		} catch (InterruptedException e) {
			e.printStackTrace();
		}

		System.out.println("Exit Code: " + application.exitValue());
	}
}

Noch etwas übersichtlicher ist es vllt so

public class Test {
	public static void main(String[] args) {
		Process application = null;


		String pfad = "C:\\Program Files (x86)\\MPlayer for Windows\\MPlayer.exe";
		String parameter = "-framedrop -vo gl";
		String ziel = "F:\\ViDeOz\\Music\\Soulfly - Back To The Primitive (Live @ Area4 Festival 2008).avi";


		String cmdln = "\"" + pfad + "\" " + parameter + " \"" + ziel + "\""; 		
		System.out.println("Commandline:\n" + cmdln + "\n");


		try {
			application = Runtime.getRuntime().exec("cmd.exe /k \"" + cmdln + "\"");
		} catch (IOException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
		}

		if(application == null) System.exit(-1);

		try {
			application.getInputStream().close();
			application.getOutputStream().close();
			application.getErrorStream().close();
		} catch (IOException e1) {
			// TODO Auto-generated catch block
			e1.printStackTrace();
		}


		System.out.println("Process created, waiting...");


		try {
			application.waitFor();
		} catch (InterruptedException e) {
			e.printStackTrace();
		}

		System.out.println("Exit Code: " + application.exitValue());
	}
}

Zitat von sasuke9999

Hm bei mir gehts damit auch nicht.

Exit Code: -1
wird ausgegeben und x264.exe wird nicht gestartet.

Wäre nun interessant, was x264 zu bemängeln hat

Deshalb: Lass dir die von deinem Programm generierte Commadline ausgeben und starte sie manuell in der Eingabeaufforderung...

So, hier das komplette Programm mit Weiterleitung von Stdout/Stderr in separaten threads:

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




public class Test {
	public static void main(String[] args) {
		Process application = null;


		String pfad = "C:\\Program Files (x86)\\MPlayer for Windows\\MPlayer.exe";
		String parameter = "-framedrop -vo gl -noquiet";
		String ziel = "F:\\ViDeOz\\Music\\Soulfly - Back To The Primitive (Live @ Area4 Festival 2008).avi";


		String cmdln = "\"" + pfad + "\" " + parameter + " \"" + ziel + "\""; 		
		System.out.println("Commandline:\n" + cmdln + "\n");


		try {
			application = Runtime.getRuntime().exec("cmd.exe /k \"" + cmdln + "\"");
		} catch (IOException e1) {
			e1.printStackTrace();
		}

		if(application == null) System.exit(-1);


		ReadThread read_std = new ReadThread(application.getInputStream(), "STDOUT");
		ReadThread read_err = new ReadThread(application.getErrorStream(), "STDERR");


		try {
			application.getOutputStream().close();
		} catch (IOException e1) {
			e1.printStackTrace();
		}

		System.out.println("Process created, waiting...\n");


		read_std.run();
		read_err.run();


		try {
			application.waitFor();
		} catch (InterruptedException e) {
			e.printStackTrace();
		}

		try {
			read_std.join();
			read_err.join();
		} catch (InterruptedException e) {
			e.printStackTrace();
		}

		System.out.println("\n\nProcess terminated with ExitCode: " + application.exitValue());
	}

	private static class ReadThread extends Thread
	{
		private final InputStream stdout;
		private final String prefix;

		ReadThread(InputStream stdout, String prefix){
			this.stdout = stdout;
			this.prefix = prefix;
		}

		@Override
		public void run() {
			if(stdout == null) return;

			BufferedReader in = new BufferedReader(new InputStreamReader(stdout)); 			
			String currentLine = null;

		    try {
				while((currentLine = in.readLine()) != null ) System.out.println(prefix + ": " + currentLine) ;
			} catch (IOException e) {
				e.printStackTrace();
			} 

			try {
				in.close();
			} catch (IOException e) {
				e.printStackTrace();
			}
		}

	}
}

Viel Spass damit

*gig* wirkt ein bissel wie der Anfang eines sx264 in Java (hatte Anfangs auch überlegt Java zu nehmen, wollte dann aber wieder meine C++ Kenntnisse etwas auffrischen und mich deshalb gegen Java entschieden).
Viel Spaß wenn Du dazu noch einen BitrateCalculator schreibst,... Overheadberechnungen sind ein echter Killer. *gig*

Zitat von sasuke9999

Supi. Jetzt gibt er was aus

STDOUT:
STDOUT: C:\Users\***\Desktop\testx264>
STDERR: x264 [error]: invalid argument: qp = p

:ani_lol: :zunge:

Übrigens kannst du

Zitat

application = Runtime.getRuntime().exec("cmd.exe /k \"" + cmdln + "\"");

durch

Zitat

application = Runtime.getRuntime().exec(cmdln);

ersetzen und es wird immer noch tun