Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > comp.lang.java.programmer > #13684 > unrolled thread

static initializer not working

Started bymike <mikaelpetterson@hotmail.com>
First post2012-04-20 05:10 -0700
Last post2012-04-20 08:42 -0400
Articles 2 — 2 participants

Back to article view | Back to comp.lang.java.programmer


Contents

  static initializer not working mike <mikaelpetterson@hotmail.com> - 2012-04-20 05:10 -0700
    Re: static initializer not working Eric Sosman <esosman@ieee-dot-org.invalid> - 2012-04-20 08:42 -0400

#13684 — static initializer not working

Frommike <mikaelpetterson@hotmail.com>
Date2012-04-20 05:10 -0700
Subjectstatic initializer not working
Message-ID<2c2614fe-33a4-457e-a0c0-e44838caf220@m16g2000yqc.googlegroups.com>
Hi,


I cannot get my registration of parser to work. When try to make a
call to get the registered parser:

PullParserFactory.instance().createParser(type);

I get a NullPointerException. When I debug I can see that the HashMap
is empty.

What am I missing?

br,

//mike

public class FindBugsParser implements AbstractPullParser{

// Register with Factory
	static {
		PullParserFactory.instance().registerParser(PluginImpl.FINDBUGS,
				new FindBugsParser());
	}

	public AbstractPullParser createParser() {
		return new FindBugsParser();
	}

}

public final class PullParserFactory {

	/**
	 * Single instance created upon class loading.
	 */
	private static  PullParserFactory INSTANCE = new PullParserFactory();

	private static final HashMap<String, AbstractPullParser>
registeredParsers = new HashMap<String, AbstractPullParser>();

	//no outside access.
	private PullParserFactory() {

	}

	public static PullParserFactory instance() {
		return INSTANCE;
	}

	public void registerParser(String type, AbstractPullParser parser) {
		registeredParsers.put(type, parser);
	}

	public AbstractPullParser createParser(String type) {

		return (AbstractPullParser) registeredParsers.get(type);
	}

}


[toc] | [next] | [standalone]


#13686

FromEric Sosman <esosman@ieee-dot-org.invalid>
Date2012-04-20 08:42 -0400
Message-ID<jmrlkh$a41$1@dont-email.me>
In reply to#13684
On 4/20/2012 8:10 AM, mike wrote:
> Hi,
>
>
> I cannot get my registration of parser to work. When try to make a
> call to get the registered parser:
>
> PullParserFactory.instance().createParser(type);
>
> I get a NullPointerException. When I debug I can see that the HashMap
> is empty.
>
> What am I missing?

     My hunch is that nothing has caused the FindBugsParser class to
be initialized at the time you make your call and get the NPE:

     - The HashMap is initially empty, and gets populated by calls
       to registerParser().

     - The only (visible) call to registerParser() occurs during the
       initialization of FindBugsParser.

     - So if FindBugsParser has not yet been initialized at the time
       of the call to instance(), the HashMap will still be empty and
       instance() will return null.

The fact that FindBugsParser.class is hanging around somewhere in
your class path is not enough to make Java load and initialize it;
Java won't load the class until it sees a need for it.

> br,
>
> //mike
>
> public class FindBugsParser implements AbstractPullParser{
>
> // Register with Factory
> 	static {
> 		PullParserFactory.instance().registerParser(PluginImpl.FINDBUGS,
> 				new FindBugsParser());
> 	}
>
> 	public AbstractPullParser createParser() {
> 		return new FindBugsParser();
> 	}
>
> }
>
> public final class PullParserFactory {
>
> 	/**
> 	 * Single instance created upon class loading.
> 	 */
> 	private static  PullParserFactory INSTANCE = new PullParserFactory();
>
> 	private static final HashMap<String, AbstractPullParser>
> registeredParsers = new HashMap<String, AbstractPullParser>();
>
> 	//no outside access.
> 	private PullParserFactory() {
>
> 	}
>
> 	public static PullParserFactory instance() {
> 		return INSTANCE;
> 	}
>
> 	public void registerParser(String type, AbstractPullParser parser) {
> 		registeredParsers.put(type, parser);
> 	}
>
> 	public AbstractPullParser createParser(String type) {
>
> 		return (AbstractPullParser) registeredParsers.get(type);
> 	}
>
> }
>
>
>


-- 
Eric Sosman
esosman@ieee-dot-org.invalid

[toc] | [prev] | [standalone]


Back to top | Article view | comp.lang.java.programmer


csiph-web