Java Auf anderes Programm warten

[JavaGamer]

Hallo,

ich habe mir einen kleinen Installer geschrieben und habe nun das Problem, dass dieser Datein downloadet und direkt nach dem ersten Download alles pausieren soll, eine gedownloadete .jar starten soll und sobald diese geschlossen ist den Download der anderen Dateien fortsetzten soll.

Bisher habe ich schon dies geschrieben, komme jetzt aber am pausieren und fortsetzten des Threads nicht weiter:

	/**
	 * @param downloadList The list of all download links
	 * @param destination The destination file to save the files in. 
	 * 
	 * @throws MalformedURLException wrong URL
	 * 
	 * @since 1.0
	 */
	public static void downloadFiles(final ArrayList<String> downloadList, final File destination)
	{
		DownloadGui.setProgressMinMax(DownloadGui.getAllBar(), 0, downloadList.size() + 1);
		DownloadGui.setProgressMinMax(DownloadGui.getCurrentBar(), 0, 100);
		
		Runnable download = new Runnable()
		{
			@Override
			public void run() 
			{
				String url = "http://files.minecraftforge.net/maven/net/minecraftforge/forge/" + DescriptionGui.getMcVersion() 
						+ "-" + DescriptionGui.getForgeVersion() + "/forge-" + DescriptionGui.getMcVersion() 
						+ "-" + DescriptionGui.getForgeVersion() + "-installer.jar";
				
				try
				{
					downloadFile(new URL(url), PathHandler.getPathToMinecraft());
					
					try 
					{
						Runtime.getRuntime().exec(new String[]{"java","-jar",forge.getName()});
					} 
					catch (IOException e)
					{
						Logger.log2(Level.FATAL, e, "Problem beim Starten des Forge-Installers!", "InstallationHandler - MalformedURLException");
					}
					
					if(forge != null && forge.exists())
						forge.delete();
				} 
				catch (MalformedURLException e1)
				{
					Logger.log2(Level.FATAL, e1, url, "InstallationHandler - MalformedURLException");
				}
				
				for(int i = 0; i < downloadList.size(); i++)
				{
					try 
					{
						downloadFile(new URL(downloadList.get(i).toString()), destination);
					} 
					catch (MalformedURLException e) 
					{
						Logger.log2(Level.FATAL, e, downloadList.get(i).toString(), "InstallationHandler - MalformedURLException");
					}
					DownloadGui.setProgress(DownloadGui.getAllBar(), i + 1);
				}
			}
		};
	}

Ist diese Zeile eigentlich so richtig?

Runtime.getRuntime().exec(new String[]{"java","-jar",forge.getName()});

Die Date forge liegt übrigens an einem anderen Ort als diese Jar!
Zudem sieht das leider meiner Meinung nach sehr unübersichtlich aus...wüsste da zufällig jemand wie ich das ändern kann?

JavaGamer

 

[Joose]

Ist diese Zeile eigentlich so richtig?

Runtime.getRuntime().exec(new String[]{"java","-jar",forge.getName()});

Schaut richtig aus, lässt sich aber auch leicht testen indem du einen Dummy Wert für "forge.getName()" einträgst und den Code ausführst.

Zudem sieht das leider meiner Meinung nach sehr unübersichtlich aus...wüsste da zufällig jemand wie ich das ändern kann?

Wenn der Code unübersichtlich wird, liegt ein Fehler im Konzept bzw. der Architektur vor.

Bevor du nun weitermachst (mit Thread pausieren usw.) würde ich dir raten deinen Code zu überarbeiten
-> UI und Logik trennen!
Die Methode "downloadFile" (bzw. "downloadFiles") sollte nichts anderes machen als die Datei zu laden und ein Event schmeißen bezüglich Fortschritt.
Aber die Methode sollte nicht die UI bearbeiten (Werte setzen usw.)

 

[JavaGamer]

Schaut richtig aus, lässt sich aber auch leicht testen indem du einen Dummy Wert für "forge.getName()" einträgst und den Code ausführst.

Wenn der Code unübersichtlich wird, liegt ein Fehler im Konzept bzw. der Architektur vor.

Bevor du nun weitermachst (mit Thread pausieren usw.) würde ich dir raten deinen Code zu überarbeiten
-> UI und Logik trennen!
Die Methode "downloadFile" (bzw. "downloadFiles") sollte nichts anderes machen als die Datei zu laden und ein Event schmeißen bezüglich Fortschritt.
Aber die Methode sollte nicht die UI bearbeiten (Werte setzen usw.)

Also eigentlich macht es ja auch nichts anderes, es gibt die Werte an eine andere Methode weiter und die setzt die Werte dann. Nur die Werte müssen ja weitergegeben werden.
Und übersichtsmäßig stören mich eher die billionen Klammern (warum kann man z.B. nicht bei try und catch die Klammern weg lassen wie bei if und else, würde es meiner Meinung nach übersichtlicher machen).

Dummy-Wert?

 

[Joose]

Also eigentlich macht es ja auch nichts anderes, es gibt die Werte an eine andere Methode weiter und die setzt die Werte dann. Nur die Werte müssen ja weitergegeben werden.

Lass deine Methode ein Event werfen mit dem Fortschritt. Jedes UI Element welches an dem Event interessiert ist kann es abonnieren und darauf reagieren wenn es will.

Nein gerade Klammern und Code Blöcke machen Code übersichtlicher und lesbarer. Probiere deine 2 try/vatch Blöcke zusammen zufassen.
Deine Methode download Files ist jetzt spezifisch auf einen Anwendungsfall ausgerichtet. Achte darauf das die Methode downloadFiles wirklich nur das macht was der Name sagt -> Dateien laden.
Bei dir wird jetzt immer ein Thread gestartet und eine spezifische Datei geladen und dann erst die eigentliche Liste abgearbeitet.

 

[JavaGamer]

Lass deine Methode ein Event werfen mit dem Fortschritt. Jedes UI Element welches an dem Event interessiert ist kann es abonnieren und darauf reagieren wenn es will.

Nein gerade Klammern und Code Blöcke machen Code übersichtlicher und lesbarer. Probiere deine 2 try/vatch Blöcke zusammen zufassen.
Deine Methode download Files ist jetzt spezifisch auf einen Anwendungsfall ausgerichtet. Achte darauf das die Methode downloadFiles wirklich nur das macht was der Name sagt -> Dateien laden.
Bei dir wird jetzt immer ein Thread gestartet und eine spezifische Datei geladen und dann erst die eigentliche Liste abgearbeitet.

Ok und wie lasse ich eine Methode ein Event werfen dass den Progress weiter gibt? Da ich hab keine Ahnung wie ich es anders machens soll, außer wenn ich die Variable direkt aufrufe ohne extra Methode dafür, wäre dann aber auch nicht dass was du meinst.

Also dann weiß ich nicht wie ich es machen soll, da es muss erst eine Datei gedownloadet werden, diese dann ausgeführt werden und wenn diese Datei (es ist eine .jar) beendet wurde, erst dann sollte der Rest gedownloadet werden wenn downloadlist != null ist, sonst werden Datein einfach aus der .zip rauskopiert, für das ich eine unfertige Methode habe.

package minecraftplaye.installer.handlers;

import java.io.BufferedInputStream;
import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
import java.net.HttpURLConnection;
import java.net.MalformedURLException;
import java.net.URL;
import java.net.URLConnection;
import java.util.ArrayList;
import java.util.HashSet;
import java.util.zip.ZipEntry;
import java.util.zip.ZipFile;

import minecraftplaye.installer.gui.DescriptionGui;
import minecraftplaye.installer.gui.DownloadGui;
import minecraftplaye.installer.gui.StartUpGui;
import minecraftplaye.installer.lib.Level;

public class InstallationHandler 
{
	
	private static ZipFile zFile;
	private static ZipEntry clientEntry, serverEntry;
	
	static File forge = null;

	public InstallationHandler() 
	{
		if(StartUpGui.client.isSelected() && FileHandler.getDownloadList(FileHandler.downloadClientList))
			downloadFiles(FileHandler.downloadClientList, StartUpGui.targetDir);
		else if(StartUpGui.server.isSelected() && FileHandler.getDownloadList(FileHandler.downloadServerList))
			downloadFiles(FileHandler.downloadServerList, StartUpGui.modpackDir);
		else
		{
			if(StartUpGui.client.isSelected() && getZipFile() != null && getClientEntry() != null)
			{
				copyFiles(true);
			}
			else if(StartUpGui.server.isSelected() && getZipFile() != null && getServerEntry() != null)
			{
				copyFiles(false);
			}
		}
	}
	
	/**
	 * Copies all needed files for the installation
	 * 
	 * @param isClient true if its the client, false if its the server
	 * 
	 * @since 1.0
	 */
	private static void copyFiles(boolean isClient)
	{
		if(isClient)
		{
			
		}
		else
		{
			
		}
	}
	
	/**
	 * @param downloadList The list of all download links
	 * @param destination The destination file to save the files in. 
	 * 
	 * @throws MalformedURLException wrong URL
	 * 
	 * @since 1.0
	 */
	public static void downloadFiles(final ArrayList<String> downloadList, final File destination)
	{
		DownloadGui.setProgressMinMax(DownloadGui.getAllBar(), 0, downloadList.size() + 1);
		DownloadGui.setProgressMinMax(DownloadGui.getCurrentBar(), 0, 100);
		
		for(int i = 0; i < downloadList.size(); i++)
		{
			try 
			{
				downloadFile(new URL(downloadList.get(i).toString()), destination);
			} 
			catch (MalformedURLException e)
			{
				Logger.log2(Level.FATAL, e, downloadList.get(i).toString(), "InstallationHandler - MalformedURLException");
			}
			DownloadGui.setProgress(DownloadGui.getAllBar(), i + 1);
		}
	}
	
	/**
	 * Downloads the file from the given url and saves the file in the file-destination.
	 * 
	 * @param websiteFile The URL from that the file should be downloaded.
	 * @param destination The destination file to save the file in.
	 * 
     *  @since 1.0
	 */
	static void downloadFile(URL websiteFile, File destination)
	{
		BufferedInputStream in = null;
		FileOutputStream fout = null;
		URLConnection connection = null;
		int minValue = 0;
		int maxValue = 0;
		try 
		{
			connection = (URLConnection) websiteFile.openConnection();
			
			maxValue = connection.getContentLength();
			
			in = new BufferedInputStream(connection.getInputStream());
			fout = new FileOutputStream(StartUpGui.modpackDir + websiteFile.getFile());
			
			DownloadGui.setLabel(websiteFile.getFile());
			forge = new File(websiteFile.getFile());
			
			final byte data[] = new byte[1024];
			int cout;
			while((cout = in.read(data, 0, 1024)) != -1)
			{
				fout.write(data, 0, cout);
				minValue = in.read();
				DownloadGui.setProgress(DownloadGui.getCurrentBar(), (minValue * 100 ) / maxValue);
			}
			
			in.close();
			fout.close();
		} 
		catch (MalformedURLException e)
		{
			e.printStackTrace();
		} 
		catch (IOException e) 
		{
			e.printStackTrace();
		}
	}
	
	/**
	 * Removes every duplicated entry from the given list.
	 * 
	 * @param arrayList The ArrayList<String> from that duplicated entries should be removed
	 * 
	 * @since 1.0
	 */
	public static void removeDuplicatedEntries(ArrayList<String> arrayList)
	{
		HashSet<String> hashSet = new HashSet<String>(arrayList);
		arrayList.clear();
		arrayList.addAll(hashSet);
	}

	/**
	 * @return the clientEntry
	 */
	public static ZipEntry getClientEntry()
	{
		return clientEntry;
	}
	
	/**
	 * @param clientEntry the clientEntry to set
	 */
	public static void setClientEntry(ZipFile zFile, ZipEntry clientEntry) 
	{
		InstallationHandler.zFile = zFile;
		InstallationHandler.clientEntry = clientEntry;
	}
	
	/**
	 * @return the serverEntry
	 */
	public static ZipEntry getServerEntry() 
	{
		return serverEntry;
	}
	
	/**
	 * @param serverEntry the serverEntry to set
	 */
	public static void setServerEntry(ZipFile zFile, ZipEntry serverEntry)
	{
		InstallationHandler.zFile = zFile;
		InstallationHandler.serverEntry = serverEntry;
	}
	
	/**
	 * @return the zip file
	 */
	public static ZipFile getZipFile()
	{
		return zFile;
	}
}

Keine Ahnung wo hin mit diesem unfertigen Code...:

		Runnable download = new Runnable()
		{
			@Override
			public void run() 
			{
				String url = "http://files.minecraftforge.net/maven/net/minecraftforge/forge/" + DescriptionGui.getMcVersion() 
						+ "-" + DescriptionGui.getForgeVersion() + "/forge-" + DescriptionGui.getMcVersion() 
						+ "-" + DescriptionGui.getForgeVersion() + "-installer.jar";
				
				try
				{
					downloadFile(new URL(url), PathHandler.getPathToMinecraft());
					
					Runtime.getRuntime().exec(new String[]{"java","-jar",forge.getName()});
					
					if(forge != null && forge.exists())
						forge.delete();
				} 
				catch (MalformedURLException e1)
				{
					Logger.log2(Level.FATAL, e1, url, "InstallationHandler - MalformedURLException");
				}
				catch (IOException e)
				{
					Logger.log2(Level.FATAL, e, "Problem beim Starten des Forge-Installers!", "InstallationHandler - MalformedURLException");
				}
			}
		};

 

[arilou]

Mal zurück zum ursprünglichen Problem dieser Anfrage:

Process p = Runtime.getRuntime().exec( ... );
p.waitFor();

Dein Programm wartet auf das Ende des Aufrufs mittels .waitFor() .
Sofern der Unterprozess jedoch mehr als 64k Output in stdout (oder stderr) schreibt, wartet dein Programm jedoch bis Sankt Nimmerlein, da es die entsprechenden Stream-Puffer des Prozesses leeren muss...

 

[JavaGamer]

Mal zurück zum ursprünglichen Problem dieser Anfrage:

Process p = Runtime.getRuntime().exec( ... );
p.waitFor();

Dein Programm wartet auf das Ende des Aufrufs mittels .waitFor() .
Sofern der Unterprozess jedoch mehr als 64k Output in stdout (oder stderr) schreibt, wartet dein Programm jedoch bis Sankt Nimmerlein, da es die entsprechenden Stream-Puffer des Prozesses leeren muss...

Und was ist stdout und stderr? Und wie kann ich dann verhindern dass das Programm bis zum Sankt Nimmerleintag wartet?

 

[Joose]

O

Und was ist stdout und stderr? Und wie kann ich dann verhindern dass das Programm bis zum Sankt Nimmerleintag wartet?

Die Streams für Ausgabe und Fehler.
Zu Programmieren gehört auch dazu sich über das eine oder ander zu informieren, vor allem wenn man neu in diesem Gebiet/Thema ist. Lies dir diesbezüglich Tutoruals durch und probiere herum ( auch mit eigenen kleinen Testprogrammen)

 

[arilou]

Und was ist stdout und stderr? Und wie kann ich dann verhindern dass das Programm bis zum Sankt Nimmerleintag wartet?

Zu stdout und stderr empfehle ich die Wikipedia, ggf. auch Google, das ist absolutes Grundwissen.

Wie du "Sankt Nimmerlein" vermeidest? Indem du einen Thread startest, der die entsprechenden Datenströme des Process' p verarbeitet (es genügt, wenn er sie ausliest und dann wegwirft...)

 

[JavaGamer]

Danke.