Buffing Chain Armour

I'm new to coding and to some this might seem like a stupid problem to try to sovle but it keeps saying I'm missing a class or an invert at the end... What should I put then?

Code:java

private final List<Material> chainSet = Lists.newArrayList(
        Material.CHAINMAIL_HELMET, Material.CHAINMAIL_CHESTPLATE,
        Material.CHAINMAIL_LEGGINGS, Material.CHAINMAIL_BOOTS
);
 
    @EventHandler
    public void onEntityDamage(EntityDamageEvent event) {
        Entity entity = event.getEntity();
 
        if (entity instanceof Player) {
            PlayerInventory inventory = ((Player) entity).getInventory();
            double modifier = 1;
 
            // Damage reduction is 1 - 0.57 ^ (number of equipped pieces)
            for (ItemStack armorPiece : inventory.getArmorContents()) {
                if (armorPiece != null && chainSet.contains(armorPiece.getType())) {
                    modifier *= 0.57D;
                    armorPiece.setDurability((short) 0);
                }
            }
 
            if (modifier < 1) {
                int original = event.getDamage();
                event.setDamage((int) Math.round(event.getDamage() * modifier));
 
                System.out.println("Damage reduced from " + original + " to " + event.getDamage());
            }
        }
    }
}
}

 

Help would be appreciated...

Anyone?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

You have extra brackets at the end.

 

The } on line 30 should go away.

 

That helped and then it revealed three more errors I solved two of them and now I'm stuck on the final one... on line 23 it says there needs to be something in the () for event.getDamage()... What should I put there?

 

Slash9211 This might work:

Code:java

event.getDamage().getAmount();

 

JD_Guy17

I tried that and got this as an errer "Error53, 49) java: double cannot be dereferenced"

 

Your modifier isn't a double. Try 1.0 as your modifier.

 

JBoss925 still doesn't work...

Anymore ideas on how to fix it?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

What is armor piece?

 

Slash9211 Change:

Code:java

int original = event.getDamage().getAmount();
//to
double original = event.getDamage().getAmount();
//or
int original = (int) event.getDamage().getAmount();

A double is a number like 1.0 (so it isn't always a whole number). int has to always be a whole number

 

I tried both JD_Guy17 and its still putting a red underline under what I underlined in the quotation marks "event.getDamage() ; ) and its saying this http://imgur.com/XPGCB52
What should I do this is whats been throwing me off lately...

JBoss925 it's the armor the player is wearing...

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Let me see your method for it. Perhaps it's incorrect? Idk thinking of fixes.

 

JBoss925 I'll just post all of my code...

Code:java

package com.Slash9211.GuardArmour.com;
 
 
import com.google.common.collect.Lists;
import org.bukkit.Material;
import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.entity.EntityDamageEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.PlayerInventory;
import org.bukkit.plugin.java.JavaPlugin;
import java.util.List;
 
/**
* Created by Slash9211 on 4/6/2014.
*/
public class GuardArmour extends JavaPlugin {
 
@Override
public void onEnable() {
getLogger().info("Plugin Enabled And Is Working");
}
 
@Override
public void onDisable() {
getLogger().info("Plugin Disabled And Is Not Longer Working");
}
 
private final List<Material> chainSet = Lists.newArrayList(
Material.CHAINMAIL_HELMET, Material.CHAINMAIL_CHESTPLATE,
Material.CHAINMAIL_LEGGINGS, Material.CHAINMAIL_BOOTS
);
 
@EventHandler
public void onEntityDamage(EntityDamageEvent event) {
Entity entity = event.getEntity();
 
if (entity instanceof Player) {
PlayerInventory inventory = ((Player) entity).getInventory();
double modifier = 1;
 
// Damage reduction is 1 - 0.57 ^ (number of equipped pieces)
for (ItemStack armorPiece : inventory.getArmorContents()) {
if (armorPiece != null && chainSet.contains(armorPiece.getType())) {
modifier *= 0.57D;
armorPiece.setDurability((short) 0);
}
}
 
if (modifier < 1) {
int original = (int) event.getDamage().getAmount();
event.setDamage((int) Math.round(original * modifier));
 
 
}
}
}
}

That's all I have and http://imgur.com/XPGCB52 is the problem at the end where it says event.getDamage() the () is underlined red...