Problem mit Sets

[EuadeLuxe]

Hallo an alle java-forum.org-User.

Wieder einmal sitze ich an einem Javaprogramm und stoße hierbei auf ein Problem. Ich will in einer Set alle Objekte speichern doch seltsamerweise gibt mir der Compiler eine NullPointerException. Die Objekte sind vom Typ einer von mir definierten Schittstelle (IDeadlinePlugin). Das Programm dient eigentlich dazu Plugins zu laden.

Hier die Stelle die laut dem Compiler den Fehler beinhaltet.

public void initPlugins() {
		Iterator<IDeadlinePlugin> iterator = this.getPlugins();
		if(!iterator.hasNext())
		{
			logger.info("No plugins loaded.");
		}
		while(iterator.hasNext())
		{
			IDeadlinePlugin plugin = iterator.next();
			logger.info("initializing " + plugin.getName());
			plugin.onLoad();
			if(plugin != null)
			{
				plugins.add(plugin);
			}
		}
	}

Hier die Fehlermeldung:

Sep 27, 2012 3:35:54 PM com.TestPlugin.plugin.TestPlugin onLoad
Information: Test Plugin was enabled.
Exception in thread "main" java.lang.NullPointerException
	at com.ProjectDeadline.plugin.DefaultPluginService.initPlugins(DefaultPluginService.java:56)
	at com.ProjectDeadline.ProjectDeadline.loadPlugins(ProjectDeadline.java:36)
	at com.ProjectDeadline.ProjectDeadline.main(ProjectDeadline.java:28)

Was kann ich tun?

 

[FerFemNemBem]

Mahlzeit,

welche Zeile ist denn bei Dir die 56?

Gruss, FFNB.

 

[EuadeLuxe]

Diese Zeile:

plugins.add(plugin);

Definiert wurde plugins als Feld, so:

private static Set<IDeadlinePlugin> plugins;

 

[Kjubert]

plugins

ist ja immernoch nur deklariert, nicht instanziiert - zumindest in diesem Ausschnitt nicht.
Aber da müsste ja die IDE schon meckern, hm?
Naja vielleicht liegts ja an sowas?!

[EDIT]
irgendwo

plugins = new Set<IDeadlinePlugin>();

????????
[/EDIT]

 

[Marco13]

Nebenbei:

plugin.onLoad();
if(plugin != null)
{
    plugins.add(plugin);
}

Er wird IMMER in den if-Block reingehen - FALLS er nicht vorher mit einer NPE abkachelt... (Man kann Eclipse sagen, dass es in so einem Fall eine Warning generieren soll)

 

[EuadeLuxe]

Vielen Dank für die vielen Antworten.

Ich habe es jetzt auch mit einer Hashmap versucht. Wieder dieselbe Fehlermeldung. Vllt. hilft noch der Aufruf dieser Methode und die ganze Klasse weiter.

Die Klasse PluginFactoryLoader:

package com.ProjectDeadline.plugin;

import java.io.File;
import java.io.IOException;
import java.util.logging.Level;
import java.util.logging.Logger;

public class PluginFactoryLoader 
{
	public static IPluginService createPluginService()
	{
		addPluginsToClasspath();
		return DefaultPluginService.getInstance();
	}
	
	public static void addPluginsToClasspath()
	{
		try
		{
			//add the plugin directory to classpath
            PluginLoader.addDir(new File("plugins"));
		} catch (IOException ex)
        {
            	Logger.getLogger(PluginFactoryLoader.class.getName()).log(Level.SEVERE, null, ex);
        }
	}
}

Die Klasse DefaultPluginService:

/**
 * 
 */
package com.ProjectDeadline.plugin;

import java.util.*;
import java.util.logging.*;

import com.ProjectDeadline.command.Command;

/**
 * @author TeamDeadline
 *
 */
public class DefaultPluginService implements IPluginService
{
	private static DefaultPluginService pluginService;
	private ServiceLoader<IDeadlinePlugin> serviceLoader;
	private Logger logger = Logger.getLogger(DefaultPluginService.class.getName());
	
	private static HashMap<Command, IDeadlinePlugin> commands = null;
	
	private DefaultPluginService()
	{
		//load plugins
		serviceLoader = ServiceLoader.load(IDeadlinePlugin.class);
	}
	
	public static DefaultPluginService getInstance()
	{
		if(pluginService == null)
		{
			pluginService = new DefaultPluginService();
		}
		return pluginService;
	}
	
	public Iterator<IDeadlinePlugin> getPlugins() {
		return serviceLoader.iterator();
	}

	public void initPlugins() {
		Iterator<IDeadlinePlugin> iterator = this.getPlugins();
		if(!iterator.hasNext())
		{
			logger.info("No plugins loaded.");
		}
		while(iterator.hasNext())
		{
			IDeadlinePlugin plugin = iterator.next();
			logger.info("initializing " + plugin.getName());
			plugin.onLoad();
			
			commands.put(plugin.usedCommands[0], plugin);
		}
	}
	
	public static HashMap<Command, IDeadlinePlugin> getPluginList()
	{
		return commands;
	}
}

Und zu guter Letzt noch die Main-Klasse:

/**
 * Setting the project preferences
 */
package com.ProjectDeadline;

import java.util.*;

import com.ProjectDeadline.Gui.GuiMain;
import com.ProjectDeadline.command.Command;
import com.ProjectDeadline.plugin.DefaultPluginService;
import com.ProjectDeadline.plugin.IDeadlinePlugin;
import com.ProjectDeadline.plugin.IPluginService;
import com.ProjectDeadline.plugin.PluginFactoryLoader;

/**
 * @author TeamDeadline
 *
 */
public class ProjectDeadline {
	private static GuiMain guiMain;
	
	private static HashMap<Command, IDeadlinePlugin> commands;
	
	/**
	 * @param args
	 */
	public static void main(String[] args) {
		loadPlugins();
		guiMain = new GuiMain("Project Deadline");
		guiMain.setVisible(true);
	}
	
	private static void loadPlugins()
	{
		IPluginService pluginService = PluginFactoryLoader.createPluginService();
		pluginService.initPlugins();
		commands = DefaultPluginService.getPluginList();
	}
	
	public static IDeadlinePlugin getCommandExecutor(Command cmd, String[] args)
	{
		IDeadlinePlugin plugin;
		plugin = commands.get(cmd);
		
		if(plugin != null)
		{
			return plugin;
		}
		
		return null;
	}
}

 

[SlaterB]

vielleicht liefert DefaultPluginService.getPluginList() null zurück?
einfach mit System.out.println() anschauen, ob die richtige Reihenfolge eingehalten ist (Zuweisung vor Zugriff)
und auch ein respektabler Wert in die Variable reinkommt

edit:
ach so, das ist deine Klasse, tja, da ist commands auch null, du musst dir angewöhnen auch Objekte zu erstellen

hier
> guiMain = new GuiMain("Project Deadline");
> guiMain.setVisible(true);
würde die zweite Zeile ohne die erste auch NullPointerException geben, ist doch leicht einzusehen,
null != gut

 

[EuadeLuxe]

Hallo SlaterB,

Leider werde ich aus deiner Antwort nicht schlau (ich halt.... ).
Der Fehler geschieht bei der Anweisung 'commands.put(plugin.usedCommands[0], plugin);'.

 

[SlaterB]

private static HashMap<Command, IDeadlinePlugin> commands = null;
->
private static HashMap<Command, IDeadlinePlugin> commands = new HashMap<Command, IDeadlinePlugin>();

 

[EuadeLuxe]

Hab´s ausgetestet aber es klappt immer noch nicht. Ich glaube ich werde noch mal alles umschreiben müssen....

 

[SlaterB]

tja, vielleicht hilft auch genauer Fehlemeldungen zu nennen usw.,
kann auch sein dass es an den Plugins vom ServiceLoader liegt, welche hier unbekannt sind

bei jeder Zeile a la
> commands.put(plugin.usedCommands[0], plugin)
musst du, falls es noch um NullPointerException geht, im Zweifelsfall alles ausschreiben:

System.out.println("ist commands null? "+(commands == null));
System.out.println("ist plugin null? "+(plugin == null));
if (plugin != null) {
  System.out.println("ist plugin.usedCommands null? "+(plugin.usedCommands == null));
}
// usw.
commands.put(plugin.usedCommands[0], plugin)

kompliziert, aber so ist das wenn du nicht alle Stellen vorher sicherst/ mit klaren Kopf andere Untersuchungen vornimmst