Bukkit Programmierung : Was ist falsch?

[HappyMajor]

Hey Leute,
ich möchte für ein Freund ein Plugin erstellen und weiß gerade nicht mehr weiter.
Vorerst was ich möchte :
Ich möchte das wenn ein Spieler Zucker in der Hand hält und dann Rechtsklickt,das der Zucker verschwindet und der Spieler schneller wird,also ein Speed PotionEffect bekommt.

Hier ist der Code :

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent event){
    if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    Player player = event.getPlayer();
    if (event.getItem() == null) {
    return;
    } else {
    if (event.getItem().getType() == Material.SUGAR) {
		
    if (player.getInventory().getItemInHand().getAmount() == 1) {
    event.getPlayer().getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
    p.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
    } else {
    int amount = player.getInventory().getItemInHand().getAmount();
    amount--;
    player.getInventory().getItemInHand().setAmount(amount);
    p.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
    }
    }
    }
    }
    }
    }
}

[Elevenn]

Nicht if event.getItem = sugar sonder if player.getItemInHand = sugar
Bei deinem Code klappt das auch nur wenn du ein Zucker in der Hand hast.

[eagle3000]

Das muss

if(player.getItemInHand().getType() == Material.SUGAR) {
// do sth.
}

(Check, ob es das Item ist.)

bzw.

if (!(player.getItemInHand().getType() == Material.SUGAR)) {
// do sth.
}

(Check, ob es das Item NICHT ist.)

heißen. Außerdem musst du p.*** durch player.*** ersetzen.

[HappyMajor]

Danke für eure Hinweise,doch irgendwie passiert immer noch gar nichts.
Code :

package main;


import org.bukkit.Material;
import org.bukkit.craftbukkit.v1_4_R1.Overridden;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;

public class Zucker extends JavaPlugin {

	@Overridden
    public void onDisable() {
        // TODO Auto-generated method stub
    }

    @Overridden
    public void onEnable() {
        // TODO Auto-generated method stub
    }
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event){
    if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    Player player = event.getPlayer();
    if (player.getItemInHand() == null) {
    return;
    } else {
    if (player.getItemInHand().getType() == Material.SUGAR) {
		
    if (player.getInventory().getItemInHand().getAmount() == 1) {
    event.getPlayer().getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
    } else {
    int amount = player.getInventory().getItemInHand().getAmount();
    amount -=1;
    player.getInventory().getItemInHand().setAmount(amount);
    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
    }
    }
    }
    }
    }
}

eagle3000:
Wie füg ich in dem Code denn eig noch ein Rechtsklick Event ein?
Einfach mit UND(&&)?

if(player.getItemInHand().getType() == Material.SUGAR && event.getAction() == Action.RIGH_CLICK_BLOCK)) {
// do sth.
}

[ComputerBaer]

PlayerInteract wird eigentlich nur aufgerufen wenn ein Spieler einen Rechtsklick macht. (Beim Linksklick kommen die DamageEvents zum tragen)

Edit:
Was machst du denn mit dem "Overridden" da? Das sehe ich jetzt zum ersten mal und ich vermute das es mit dem Aushebeln neuen Schutzfunktion von Bukkit zu tun hat.

Originally Posted by Elevenn

Nicht if event.getItem = sugar sonder if player.getItemInHand = sugar

Mit getItem() sollte man auch an das Item in der Hand kommen.

[eagle3000]

	@EventHandler
    public void onPlayerInteract(PlayerInteractEvent event){
    if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK) {
    Player player = event.getPlayer();
    if (!(player.getItemInHand().getType() == Material.SUGAR)) {
    return;
    } else {
    if (player.getItemInHand().getType() == Material.SUGAR) {
    if (player.getInventory().getItemInHand().getAmount() == 1) {
    player.getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
    } else
    {
    int amount = player.getInventory().getItemInHand().getAmount();
    amount--;
    player.getInventory().getItemInHand().setAmount(amount);
    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
     }
    }
    }
    }
    }

Sollte so funktionieren, gucks dir aber bitte an und versteh' es.

[ComputerBaer]

Ohh, da habe ich (und alle anderen auch) wohl nicht besonders gut aufgepasst. Es ist klar das nichts passiert, weil du das Event garnicht registrierst.

@eagle3000
In diesem Fall wo die Abfrage liefern soll, dass es kein Zucker ist würde ich Ungleich (!=) bevorzugen, das ganze könnte auch sehr geringen Geschwindigkeitsvorteil bringen. Aber letztendlich ist das wohl Geschmackssache.

@HappyMajor
Ich würde den Code für bessere Lesbarkeit einrücken. Das sieht dann ungefähr so aus:

 Spoiler

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event)
    {
        if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK)
        {
            Player player = event.getPlayer();
            if (player.getItemInHand().getType() != Material.SUGAR)
            {
                return;
            }
            else
            {
                if (player.getItemInHand().getType() == Material.SUGAR)
                {
                    if (player.getInventory().getItemInHand().getAmount() == 1)
                    {
                        player.getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
                        player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
                    }
                    else
                    {
                        int amount = player.getInventory().getItemInHand().getAmount();
                        amount--;
                        player.getInventory().getItemInHand().setAmount(amount);
                        player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
                    }
                }
            }
        }
    }

Ob du den Klammern eine eigene Zeile spendierst oder sie in die selbe Zeile schreibst musst du dir aussuchen.

Hier noch ein wenig Optimiert: (Sachen die vorher nicht gingen, werden auch hier nicht gehen, der Code ist weiterhin ungetestet)
Nachtrag: Allerdings ist mir beim Optimieren ein Fehler aufgefallen, mehr dazu weiter unten.

 Spoiler

Wenn du Abfragst ob es kein Zucker ist, dann muss es beim else zwangläufig Zucker sein, also ist die zweite Abfrage überflüssig:

            if (player.getItemInHand().getType() != Material.SUGAR)
            {
                return;
            }
            else
            {
                if (player.getInventory().getItemInHand().getAmount() == 1)
                {
                    player.getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
                    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
                }
                else
                {
                    int amount = player.getInventory().getItemInHand().getAmount();
                    amount--;
                    player.getInventory().getItemInHand().setAmount(amount);
                    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
                }
            }

Spätestens jetzt sollte auffallen, dass nach dieser Abfrage überhaupt nichts mehr an Code kommt. Weshalb der if-Teil irgendwie überflüssig wird, da man das if nicht streichen kann, wird die Abfrage umformuliert und das else gestrichen.

            if (player.getItemInHand().getType() == Material.SUGAR)
            {
                if (player.getInventory().getItemInHand().getAmount() == 1)
                {
                    player.getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
                    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
                }
                else
                {
                    int amount = player.getInventory().getItemInHand().getAmount();
                    amount--;
                    player.getInventory().getItemInHand().setAmount(amount);
                    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
                }
            }
            // Für den Fall, dass du doch irgendwann einen Grund hast vorzeitig rauszuspringen, wenn es kein Zucker ist kannst du das else wieder anhängen:
            // else
            // {
            //     return;
            // }

Des Weiteren kann man erkennen, dass du eine Codezeile unnötigerweise doppelt drin hast. Solltest du den Effekt jetzt mal ändern wollen, musst du das an zwei Stellen tun. Aus dem Grund machen wir das einfach nach dem Zuckerabziehen.

            if (player.getItemInHand().getType() == Material.SUGAR)
            {
                if (player.getInventory().getItemInHand().getAmount() == 1)
                {
                    player.getInventory().removeItem(new ItemStack(event.getItem().getTypeId(), 1));
                }
                else
                {
                    int amount = player.getInventory().getItemInHand().getAmount();
                    amount--;
                    player.getInventory().getItemInHand().setAmount(amount);
                }
                
                player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
            }

So, jetzt ist mir doch tatsächlich noch ein Fehler aufgefallen. Wenn du versuchst ein Item aus dem Inventar zu entfernen, solltets du auch das Item entfernen und nicht ein neues Erstellen, dass du dann entfernst.
Bei der Gelegenheit kürze ich auch noch ein paar unnötig lange Anweisungen. (Das Reduzieren um 1 stopfe ich auch in eine Zeile, das ist aber irgendwo auch wieder Geschmackssache.)

            if (player.getItemInHand().getType() == Material.SUGAR)
            {
                if (player.getItemInHand().getAmount() == 1)
                {
                    player.getInventory().removeItem(player.getItemInHand());
                }
                else
                {
                    player.getItemInHand().setAmount(player.getItemInHand().getAmount() - 1);
                }
                
                player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
            }

Am Ende sieht der gesamte Code jetzt so aus:

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event)
    {
        if (event.getAction() == Action.RIGHT_CLICK_AIR || event.getAction() == Action.RIGHT_CLICK_BLOCK)
        {
            Player player = event.getPlayer();
            if (player.getItemInHand().getType() == Material.SUGAR)
            {
                if (player.getItemInHand().getAmount() == 1)
                {
                    player.getInventory().removeItem(player.getItemInHand());
                }
                else
                {
                    player.getItemInHand().setAmount(player.getItemInHand().getAmount() - 1);
                }
                
                player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 60, 1));
            }
            // Für den Fall, dass du doch irgendwann einen Grund hast vorzeitig rauszuspringen, wenn es kein Zucker ist kannst du das else wieder anhängen:
            // else
            // {
            //     return;
            // }
        }
    }

[HappyMajor]

ComputerBaer :
Danke du bist mein Retter!!
Ich habe völlig vergessen den Listener zu registrieren,jetzt funktioniert es.
Und danke für deine Verbesserung