Dealing with NullPointerExceptions

One of the most common problems that begginers run into (and some 'experts' though they can handle it i should hope) is the dreaded NullPointerException.  One of the most frustrating things about the NPE is that it doesn't say what was null.  All you get is a line number in the stack trace so if you're doing a lot on that line, it's not always clear.  There are various ideas about how to clean that up in java 7.  There are other solutions here and there as well so we'll see if any of that makes it into Java 7 or not.  But we don't have Java 7 yet so none of that really helps.  So here's my methodology/suggestions for dealing with them now. First, let's list why NPEs happen.  That will help you find many NPEs just by looking at the code.  The most common (and obvious to the seasoned developer) is that you didn't initialize a variable.  Now, local variables must be initialized so the compiler helps you out a little there.  However, instance and class fields do not so you'll need to be careful there.  Also, you can silence the compiler complaints about uninitialized local variables by setting them to null.  Obviously, if you don't reassign these references before trying to use them, you'll get an NPE.

Another source is method return values.  If you can't see the code being called, there's no real guarantee that you'll get a non-null reference back out of it.  To be safe, all these values should be check for nulls before using them.  Of course, your own methods might have bugs in them such that an null return might not be obvious.  Or it might be intentional.  In any case, you need to be mindful of the risks of using return values.

Dealing with them is simple enough though apparently not that obvious to a beginner.  As I mentiond earlier, the exact line is mentioned in the stack trace in the error logs or on the console.  Given the discussion above, you can often spot the offending reference just by looking at that line of code.  But if you do a lot of things one a line of code like I usually do, it can sometimes be less than obvious.  The solution is simple enough here, too:  break the line down into simpler bits.  If you have chained method calls, for example, create local variables for each step and print out the results:

foo.bar().bob(getDoug()).dude(getCar());

This becomes:

System.out.println("foo = " + foo);
Bar bar = foo.bar();
System.out.println("bar = " + bar);
Doug doug = getDoug();
System.out.println("doug = " + doug);
Bob bob = bar.bob(doug);
System.out.println("bob = " + bob);

We can essentially rule out the results of getCar() as that would result in an NPE on another line if that null gets passed into dude().  But this should be enough to highlight the exact value that is null.  Once you correct that, you can recombine all that back into online if you like.  There you go.  NPE found and fixed.

It's nothing fancy or complex.  Just a little legwork to get you over the hump.  This sort of thing becomes second nature to seasoned programmers but isn't always the most obvious to beginners.  I hope it helps some of you on your way.

Basic Thread Advice

I'm not a concurrency expert and you won't find anything terribly profound if you consider yourself an experienced multithreaded developer.  But I did want to share a simple, basic tip for beginners.  Often when a beginner decides that threads are the solution to whatever problem he's facing, he extends Thread.  When told that extending Thread is a bad idea and that he should implement Runnable instead, it often falls on deaf ears.  Most of the reasoning given is that there's no point in extending Thread and that you use up your one extension for no gain.  This is all true but not entirely compelling.  This morning I ran across a much more compelling argument:  restarting a thread. Threads, once stopped, can not be restarted so you have to create a new Thread and start that one.  Normally even this may not be that big of a deal.  But in this particular case this morning, the developer had state he wanted to preserve.  Since his Thread could not be restarted, he'd have to create a new Thread, copy over that state, then start the new one.  If he'd just used a Runnable, he could have simply started a new Thread with that Runnable and be done.  Faced with that realization, he changed his code to use Runnables instead and is now a happy camper.

So, again, don't extend Thread.  You gain nothing from it and tie your hands in more than one way.

How Best to Prepare and Serve Crow

I've been very vocal about my distaste for maven in the past.  Most of my preference for other tools is the large, spiky, opaque interface that maven offers.  Figuring out how to customize a maven build is an exercise in frustration at best.  I was talking to a maven dev (no clue how "senior" the guy was) and he couldn't answer a question I had (I think it was about finding out what plugins/targets a pom offered).  He fumbled about looking for an answer.  And that just typifies why I dislike maven so much. On the other hand, there are many things to like as well.  IDE integration has finally reached a usable point for me.  IDEA has top notch support for basing a project definition off of it, finally, which is a big win if your project has dependency changes frequently.  It also makes hetergeneous development environments much more portable among many other advantages.  Maven still has its Golden Path on which life is all rainbows and butterflies and deviating from that path is starts to look like a descent into Emyn Muil.  But still, the world is slowly moving that way in large part because the ant guys seem to have stopped any sort of evolution of their own offering.

So it's my intention to move all my stuff to maven now.  We use it at work so I'm already using it most of the time anyway.  Many of my complaints are born of frustration/ignorance of the details of the tool and there's only way to cure that.  I knew that despite all my kvetching that this day would come.  And here it is.

So I have one question for the tube generation:  what's the best way to cook crow?