Jump to content

Recommended Posts

Posted

Hi

 

Just a comment, it's generally considered best practice to name your classes starting with a capital, and your objects starting with a lower case

 

eg

ModelRenderer muzzle

not

ModelRenderer Muzzle

 

Apart from that - I suspect the crash might be something to do with the fact that you're creating a Muzzle in the constructor of ModelPowerBeam using this, but then you call a method on the Muzzle before the ModelPowerBeam has finished constructing, i.e. the "this" is not valid yet.  I'm kind of fuzzy on the details but I'm pretty sure that's a really bad idea if the method uses the this.

 

Do you know how to use breakpoints?  That would be the most direct way to troubleshoot this.  If you don't, you really should, it makes things so much easier and won't take you long to learn

http://www.vogella.com/tutorials/EclipseDebugging/article.html

 

-TGG

 

 

Posted

Hi

 

>Also, in one of my other models I used this in the constructor and it seemed to work fine... Using breakpoints also showed that this is infact ModelPowerBeam at that point...

 

Hmmm.  It still makes me nervous.  Perhaps ModelPowerBeam only looks like it's valid, even though 'this' is not null.

http://www.javapractices.com/topic/TopicAction.do?Id=252

 

So when you're using the breakpoint and stepping through line by line, which variable is null (causes the NPE) then?

 

-TGG

Posted

Line 89-90 of ModelRenderer:

TextureOffset textureoffset = this.baseModel.getTextureOffset(par1Str);   

this.setTextureOffset(textureoffset.textureOffsetX, textureoffset.textureOffsetY);

 

You need to set your texture offsets first.

 

 

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.