Question

I have an InputStream that gets passed to an InputSource:

int resID = getResources().getIdentifier("c" + getCfrFile(), "raw", getPackageName());  
InputStream ins  = getResources().openRawResource(resID); 
InputSource xmlSource = new InputSource(ins);   

xmlTableRowLoader = new AcmTableRowLoader(handler, xmlSource, this, pathBuilder(false), "title");

How do I properly close() the InputStream? When I try to close it:

ins.Close();

I get an java.lang.NullPointerException because the stream gets closed before the “xmlSource” is finished with it. Also the object “xmlTableRowLoader” extends Thread.

Answer 1

That depends on how AcmTableRowLoader is using it. If it is only using it in its constructor, which I doubt from what you said, then closing it in the method above might be valid. However, since it appears that AcmTableRowLoader is using it in places other than the constructor… I would suggest having the AcmTableRowLoader create and close the stream itself since it knows when it is done with it.

Or in short, close it after you have closed the xmlSource. Hard to tell exactly where / how without more context. Looks like you just jumped the gun.