Jump to content

[1.12.1] Help with 2 small errors that i dont know how to solve.


Recommended Posts

Hey guys,


I am quite unexperienced but i am learning what i can. But i cant seem to solve two errors maybe you guys can help me?


private boolean hasArmor(EntityPlayer player)
net.minecraft.item.ItemStack[] armor = player.inventory.armorInventory;
if ((armor != null) && (
	(armor[0] != null) || (armor[1] != null) || (armor[2] != null) || (armor[3] != null))) {
	return true;


net.minecraft.item.ItemStack[] armor = player.inventory.armorInventory; is giving me the following error >: cannot convert from NonNullList<ItemStack> to ItemStack[]


Next error


/*     */   public float[] getRotationsNeeded(Entity entity) {
/* 197 */     if (entity == null)
/* 198 */       return null;
/* 199 */     double diffX = entity.posX - this.mc.player.posX;
/*     */     double diffY;
/*     */     ;//double diffY;
/* 202 */     if ((entity instanceof EntityLivingBase)) {
/* 203 */       EntityLivingBase entityLivingBase = (EntityLivingBase)entity;
/*     */       
/*     */ 
/*     */ 
/*     */ 
/*     */ 
/* 209 */       diffY = entityLivingBase.posY + entityLivingBase.getEyeHeight() * 0.9D - (this.mc.player.posY + Minecraft.getMinecraft().player.getEyeHeight());
/*     */ 
/*     */     }
/*     */     else
/*     */     {
/*     */ 		
/* 215 */       diffY = (entity.boundingBox.minY + entity.boundingBox.maxY) / 2.0D - (this.mc.player.posY + Minecraft.getMinecraft().player.getEyeHeight()); }
/* 216 */     double diffZ = entity.posZ - this.mc.player.posZ;
/* 217 */     double dist = MathHelper.sqrt(diffX * diffX + diffZ * diffZ);
/*     */     
/* 219 */     float yaw = (float)(Math.atan2(diffZ, diffX) * 180.0D / 3.141592653589793D) - 90.0F;
/* 220 */     float pitch = (float)-(Math.atan2(diffY, dist) * 180.0D / 3.141592653589793D);
/* 221 */     return new float[] {this.mc.player.rotationYaw + 
/*     */     
/* 223 */       MathHelper.wrapDegrees(yaw - this.mc.player.rotationYaw), this.mc.player.rotationPitch + 
/*     */       
/*     */ 
/* 226 */       MathHelper.wrapDegrees(pitch - this.mc.player.rotationPitch) };
/*     */   }

diffY = (entity.boundingBox.minY + entity.boundingBox.maxY) / 2.0D - (this.mc.player.posY + Minecraft.getMinecraft().player.getEyeHeight()); } is giving me "The field Entity.boundingBox is not visible".





Link to comment
Share on other sites

I know is sometimes sounds mean to tell people "learn Java". But your problems really are simple Java things that have not much to do with modding specifically. So I really suggest learning Java more solidly. I suggest a very good, simple book called Java In Easy Steps. I like the book because it is very short but still covers the major topics. So you can get through it quickly, and it isn't boring or "scary" like some of those "Java for Dummies" types of books.


Your first problem is very clear, you're trying to assign a type to a value that is not of that type. The return type of the armor inventory is a NonNullList<ItemStack> and you're trying to put that value into an ItemStack[] (an array of ItemStack). If you really need an array you can convert -- in fact the NonNullList probably has a method to help you do that. But you can also just keep it as a NonNullList and use list type of methods for looping through and checking values.


Your second problem is also very clear, you are trying to use a field or method which does not have public scope. That means you cannot access it from outside the class (or package or hierarchy depending on what the scope actually is). You really need to study the concept of scope in programming. In particular Java has public, protected, private (plus some other cases like local) that can control "visibility". To fix this problem, the first thing you need to do is see if there is a public "getter" method for the private field. For example, maybe there is a getBoundingBox() or similar method available. 

Check out my tutorials here: http://jabelarminecraft.blogspot.com/

Link to comment
Share on other sites

3 hours ago, Anonymous1611 said:

Entity.boundingBox is not visible

This used to be a public value and now it's private. There should be a getter function, if not, you can use reflection.

3 hours ago, Anonymous1611 said:

cannot convert from NonNullList<ItemStack> to ItemStack[]

The type changed from one version to another when ItemStacks were made to never be allowed to be null (instead using .isEmpty() so that zero-size stacks weren't treated as items, but rather nothing). You'll need to update your code to match the new system.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.


Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.


DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

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.

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.

  • Create New...

Important Information

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