Java String to Array = Crash

[JavaGamer]

Hallo,

ich habe ein kleines Tool programmiert dass eine Datei ausliest, den Inhalt als StringBuilder speichert und dann an einen String weitergibt. Danach wird dieser in ein Array eingespeichert. Leider crasht das Programm immer an dieser Stelle:

[INFO] Initialize Installer
[ERROR]java.lang.NullPointerException
	at java.util.Arrays$ArrayList.<init>(Unknown Source)
	at java.util.Arrays.asList(Unknown Source)
	at minecraftplaye.installer.handlers.FileHandler.readServer(FileHandler.java:120)
	at minecraftplaye.installer.handlers.FileHandler.getZip(FileHandler.java:72)
	at minecraftplaye.installer.gui.DescriptionGui.<init>(DescriptionGui.java:22)
	at minecraftplaye.installer.handlers.ActionHandler.actionPerformed(ActionHandler.java:23)
	at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
	at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$200(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Hier der dazu gehörige Code:

	private static void readServer(ZipFile zFile, ZipEntry entry)
	{
		try
		{
			@SuppressWarnings("resource")
			BufferedInputStream is = new BufferedInputStream(zFile.getInputStream(entry));
			StringBuilder strb = new StringBuilder();
			while(is.available() > 0)
			{
				strb.append((char)is.read());
			}
			is.close();
			
			String temp = strb.toString();
			
			System.out.println(temp);
			
			if(entry.getName().equals("Server.txt"))
				downloadServer = temp.split(System.getProperty("line.separator").toString());
			else if(entry.getName().equals("Client.txt"))
				downloadClient = temp.split(System.getProperty("line.separator"));
			
			downloadServerList = new ArrayList<String>(Arrays.asList(downloadServer));
			downloadClientList = new ArrayList<String>(Arrays.asList(downloadClient));
			
			for(int i = 0; i < downloadClientList.size(); i++)
				System.out.println(downloadClientList.get(i).toString());
			
			if(downloadServerList != null && !downloadServerList.isEmpty())
			{
				downloadClientList.addAll(downloadServerList);
				InstallationHandler.removeDuplicatedEntries(downloadServerList);
			}
			
			if(downloadClientList != null && !downloadClientList.isEmpty())
				InstallationHandler.removeDuplicatedEntries(downloadClientList);
		}
		catch(FileNotFoundException e)
		{
			Logger.log2(Level.ERROR, e, "Die Datei konnte nicht gefunden werden", "FileHandler - FileNotFoundException");
		}
		catch(NullPointerException e)
		{
			Logger.log2(Level.ERROR, e, "Die Datei konnte nicht gefunden werden oder ist null", "FileHandler - NullPointerException");
		}
		catch(IOException e)
		{
			Logger.log2(Level.FATAL, e, "Unbekannter Fehler beim Verarbeiten der Datei: " + StartUpGui.modpackDir.getAbsolutePath(), "FileHandler - IOException");
		}
	}

FileHandler.java:120

			downloadClientList = new ArrayList<String>(Arrays.asList(downloadClient));

Ich hoffe ihr könnt mir helfen!

JavaGamer

 

[JavaMeister]

Du hast eine Nullpointer Exception. Es wird exakt die Stelle ausgegeben und wie man dahin gekommen ist.

Weißt du, wie man das liest?

 

[JavaGamer]

Du hast eine Nullpointer Exception. Es wird exakt die Stelle ausgegeben und wie man dahin gekommen ist.

Weißt du, wie man das liest?

Sagen wir es mal so, teils... Also ich weiß nur dort ist etwas null, aber nicht warum oder wie ich dies beheben kann. Meine kleine Debug-Zeile gibt ja den gesamten Inhalt aus, somit weiß ich nicht was null sein soll, die Datei ist nicht null und alles wird ausgelesen und auch ausgegeben (zumindest der String temp aus dem der Array gebildet wird aus dem die Liste gebildet wird)

 

[Harry Kane]

entry.getName() kann niemals sowohl gleich "Server.txt" als auch gleich "Client.txt" sein. Wenn entry.getName() gleich "Server.txt" ist, ist es ungleich "Client.txt", und downloadClient nicht initialisiert. Sofern downloadClient nicht bereits initialisiert ist, fliegt in der von dir genannten Zeile die NPE.
Deine debug-Strategie ist noch sehr verbesserungsfähig.

 

[Joose]

Harry Kane hat dir schon die Ursache für dein Problem verraten.

Probiere in Zukunft den Debugger deiner IDE zu verwenden. Dieser kann dir Schritt für Schritt zeigen was passiert, welchen Wert ein Attribut/Variable hat usw.
Dadurch kannst du in Zukunft viel schneller deine Fehler finden

aber nicht warum oder wie ich dies beheben kann.

Wenn mal wieder eine NPE geworfen wird in deinen Code, dann schaue dir den StackTrace an und gehe zu der Zeile Code wo der Fehler passiert ist.
Dann musst du kontrollieren welches der Objekte "null" ist (wenn es nicht sowieso eindeutig ist). Am Besten geht das mit dem Debugger.
Weißt du dann welches der Objekte "null" ist musst du kontrollieren wo diese Objekt gesetzt/initialisiert/übergeben wird und an den gefunden Stellen kontrollierst du dann ob das Objekt null werden könnte usw.

Meine kleine Debug-Zeile gibt ja den gesamten Inhalt aus, somit weiß ich nicht was null sein soll, die Datei ist nicht null und alles wird ausgelesen und auch ausgegeben (zumindest der String temp aus dem der Array gebildet wird aus dem die Liste gebildet wird)

Deine "Debug-Zeile" ist vielleicht richtig, aber zwischen der Ausgabe dieser Zeile und der eigentliche Zeile wo der Fehler passiert liegen noch X Code Zeilen, sprich dort kann sich wieder alles ändern.
Außerdem ist deine "Debug-Zeile" nicht so aussagekräftig wie du denkst, da deine Variable "String tmp" nicht in der Zeile mit dem Fehler verwendet wird.

 

[JavaGamer]

Ok. Danke, hab das Problem jetzt gefixed. Werde dann wohl mein Debug-Code ein wenig überarbeiten.

 

[Neumi5694]

Kleiner Tip:
Da entry verschiedene Arbeitsabläufe auslöst, solltest du die entsprechenden Abläufe auch in Blöcke packen und die benötigten Variablen auch nur dort sichtbar definieren, so sparst du dir einige Probleme.

downloadServerList und downloadServer werden z.B. nur dann gesetzt, wenn entry den Eintrag "Server.txt" hat.
Anstatt jetzt in jeder zweiten Zeile auf null zu prüfen, solltest du alles, was damit zu tun hat, in einen Block packen, wo du sicher bist, dass sie auch gesetzt werden.