HUGE BUG! Anyone know a fix?

Okay so i am in the making of a plugin that updates lore's when you add an enchantment an item, in my case a Diamond_Sword.
Sometimes my enchantments will switch places with the enchantment above it. and then switch back. But this happens very fast, Example: The inventory checker is 1 second. every half of second (10 ticks) the enchantment will switch place's with the other enchantments... Please help

So i will post a few little screen shots and my code so its easyer to figure out what im talking about.

Screenshot 1 : http://i.imgur.com/dYJgup9.png
Then half a second later the sword will look like 'Screenshot 2'
Screenshot 2 : http://i.imgur.com/CHbyePI.png
And it keeps looping through

Code:
I wont paste the 'Strings' in the code because there too big.

Code:java

@Override
    public void onEnable() {
 
        getConfig().options().copyDefaults(true);
        saveConfig();
 
        Bukkit.getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
            @SuppressWarnings("deprecation")
            public void run() {
                for(Player all : Bukkit.getOnlinePlayers()) {
                    for(ItemStack i : all.getInventory().getContents()) {
                        if(i != null) {
                            if(i.getType() == Material.DIAMOND_SWORD) {
                                ItemMeta m = i.getItemMeta();
 
                                m.setLore(getLore(i));
                                i.setItemMeta(m);
                            }
                        }
                    }
                }
            }
        }, 25, 25);
    }
 
    public List<String> getLore(ItemStack item) {
        List<String> lore = new ArrayList<String>();
 
        lore.add(" ");
        lore.add(info);
 
        if(item.getEnchantments().containsKey(Enchantment.DAMAGE_ALL)) {
            if(item.getEnchantmentLevel(Enchantment.DAMAGE_ALL) == 1) {
                lore.add(sharpness1);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_ALL) == 2) {
                lore.add(sharpness2);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_ALL) == 3) {
                lore.add(sharpness3);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_ALL) == 4) {
                lore.add(sharpness4);
 
            } else {
                lore.add(sharpness5);
            }
        }
        //Yeah that isnt a problem :) also take alook at this that someguy sent me
        if(item.getEnchantments().containsKey(Enchantment.DAMAGE_UNDEAD)) {
            if(item.getEnchantmentLevel(Enchantment.DAMAGE_UNDEAD) == 1) {
                lore.add(smite1);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_UNDEAD) == 2) {
                lore.add(smite2);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_UNDEAD) == 3) {
                lore.add(smite3);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_UNDEAD) == 4) {
                lore.add(smite4);
 
            } else {
                lore.add(smite5);
            }
        }
 
        if(item.getEnchantments().containsKey(Enchantment.DAMAGE_ARTHROPODS)) {
            if(item.getEnchantmentLevel(Enchantment.DAMAGE_ARTHROPODS) == 1) {
                lore.add(baneofarthropods1);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_ARTHROPODS) == 2) {
                lore.add(baneofarthropods2);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_ARTHROPODS) == 3) {
                lore.add(baneofarthropods3);
 
            } else if(item.getEnchantmentLevel(Enchantment.DAMAGE_ARTHROPODS) == 4) {
                lore.add(baneofarthropods4);
 
            } else {
                lore.add(baneofarthropods5);
            }
        }
 
        if(item.getEnchantments().containsKey(Enchantment.FIRE_ASPECT)) {
            if(item.getEnchantmentLevel(Enchantment.FIRE_ASPECT) == 1) {
                lore.add(fireaspect1);
 
            } else {
                lore.add(fireaspect2);
            }
        }
 
        if(item.getEnchantments().containsKey(Enchantment.DURABILITY)) {
            if(item.getEnchantmentLevel(Enchantment.DURABILITY) == 1) {
                lore.add(unbreaking1);
 
            } else if(item.getEnchantmentLevel(Enchantment.DURABILITY) == 2) {
                lore.add(unbreaking2);
 
            } else {
                lore.add(unbreaking3);
            }
        }
 
        return lore;
    }
}

Hopefully someone there is a fix to the buggy enchants. Please let me know if there is anything that can be done to prevent this

 

hmm. why you do alllllll that code every so even if the players swords have the lore you set to it already?
You can use the EnchantItemEvent and then get the item do your if's and then set its lore, once.
also it would help you to use swich statements
Hope i kinda helped :3

 

mine-care
I don't think Switch statements are supported in Java 8.
EDIT: Typo

 

CraftCreeper6 you mean, you don't think they ARE supported?
I think they are but what I know =]

 

mine-care
Oh, yea, typo xD

 

really? Switches are not in java 8? Haven't played around with the new jdk much... do you have anything that can definitely confirm it?

 

Worry not. It was but a mere typographical misfortune ^.^

 

Abs0rbed
I couldn't find anything showing there is one...

 

CraftCreeper6 you find a lot about Java 7 because that's when Strings in switches were introduced. The tutorial page doesn't mention anything about it being removed in Java 8 (which would be quite strange if they did remove it).

 

fireblast709
I am sure I saw something about it, can't remember where though

 

I hope not, they add so much functionality, especially with the addition of String switch statements in jdk7

 

The lack of people actually downloading JDK 8 to check if they still are there worries me a bit. Anyway, I use Java 8 all the time and I can definitely confirm that switch statements are NOT removed...really why would they be?

 

Rocoty
Must've misread something.