[TUT]How to create the /I <Item> Command | Using HashMaps

Create a new Class called HashMapz

Code:

public class HashMapz {
public static HashMap<String, Material> items = new HashMap<String, Material>();
public static void setItems() {
items.put("apple", Material.APPLE);
items.put("cake", Material.CAKE);
     }
}

The Command

Code:

if (cmdLabel.equalsIgnoreCase("i") && args.length==1 ){
			Material type = HashMapz.items.get(args[0].toLowerCase());
			if (type == null) {
				sender.sendMessage(R + "Invalid item name");
				return true;
			}
			ItemStack item = new ItemStack(type, type.getMaxStackSize());
			p.getInventory().addItem(item);
			sender.sendMessage(ChatColor.AQUA + "You have gave yourself " + args[0]);
			return true;
		}

In the Main Class Under onEnabled enter this:

Code:

	HashMapz.setItems();

in the plugin.yml Register the command

Code:

commands:
  I:
   description: Spawn a Item
   usage: /I [ItemName]

 

Short, sweet, simple.

 

Hmm But this doesnt support data values?

 

You can import data values for the string "1", Material.Stone get it?

 

Code:java

// Package Declaration
package me.iffa.trashcan.commands.general;
 
// TrashCan Imports
import me.iffa.trashcan.commands.TrashCommand;
import me.iffa.trashcan.utils.MessageUtil;
 
// Bukkit Imports
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
 
/**
 * Represents /item.
 * 
 * @author iffamies
 */
public class ItemCommand extends TrashCommand {
 
    /**
     * Constructor of ItemCommand.
     * 
     * @param label Command label
     */
    public ItemCommand(String label) {
        super(label);
    }
 
    /**
     * {@inheritDoc}
     */
    @Override
    public boolean executeCommand(CommandSender cs, String[] args) {
        if (!(cs instanceof Player)) {
            MessageUtil.sendMessage(cs, "Sorry, only players can use this command.");
            return true;
        }
        Player player = (Player) cs;
        if (!player.hasPermission("trashcan.general.item")) {
            MessageUtil.sendMessage(player, ChatColor.RED + "You don't have permission!");
            return true;
        }
        if (args.length < 1) {
            return false;
        } else {
            String[] itemAndData = args[0].split(":");
            int item = 0;
            try {
                item = Integer.parseInt(itemAndData[0]);
            } catch (NumberFormatException ex) {
                // Item was not an int, do nothing
            }
            Material itemMat = item == 0 ? Material.matchMaterial(itemAndData[0]) : Material.getMaterial(item);
            int damage = 0;
            if (itemMat == null) {
                MessageUtil.sendMessage(player, ChatColor.RED + "Invalid item '" + itemAndData[0] + "'!");
                return true;
            }
            if (itemAndData.length == 2) {
                try {
                    damage = Integer.parseInt(itemAndData[1]);
                } catch (NumberFormatException ex) {
                    // Data was not an int, complain here
                    MessageUtil.sendMessage(player, ChatColor.RED + "Data must be a valid number!");
                    return true;
                }
            }
            ItemStack itemStack = new ItemStack(itemMat, 1, (short) damage);
            if (args.length == 1) {
                player.getInventory().addItem(itemStack);
            } else {
                int amount = 1;
                try {
                    amount = Integer.parseInt(args[1]);
                } catch (NumberFormatException ex) {
                    MessageUtil.sendMessage(player, ChatColor.RED + "Invalid item amount!");
                    return true;
                }
                itemStack.setAmount(amount);
                player.getInventory().addItem(itemStack);
            }
            MessageUtil.sendMessage(player, ChatColor.GREEN + "You've been given " + itemStack.getAmount() + " of " + itemMat.toString() + " (" + itemMat.getId() + ")!");
            return true;
        }
    }
 
    /**
     * {@inheritDoc}
     */
    @Override
    public void sendUsage(CommandSender cs) {
        MessageUtil.sendMessage(cs, ChatColor.GRAY + "Usage: /item <item[:damage]> [amount]");
    }
}

It's a shame I'm not allowed to release this plugin. grr

 

Why do that much?

 

It's great that you show people what code they need, but you don't explain anything. How are people supposed to learn what this does if there are no explanations?

 

Thats a great Question. Its like hard for me to Explain but easy for me to show rather

 

Can you show us how to do Item Data Values too?
(For reference purposes)

 

Pretty much the same thing but the String is a Number rather than a Word example:

Code:

items.put("1", Material.CAKE);

Because that string will be what ever the user Types

 

I think he is talking about data values for different types of wool, slabs, logs, etc.

 

Yes. That.

 

Actually I have a new code for items :] Its 100% to find any item and its less code.

 

If you're going to take this HashMap approach, wouldn't it be easier to store names with item IDs, then create ItemStacks with this constructor.

Doing it this way also means you could just use regex to recognize if the user specifies a number, so that instead of hashmapping ("1",1), you could just do new ItemStack(args[1], args[2], 0) and plop that in the PlayerInventory. You'd have to make sure that the number actually corresponds to a real item, but thats the basic approach.

You could also use regex to recognize "number:number" to deal with item damage.

Sorry this isn't very detailed; I'm on my iPod ATM.

 

This has problems, it would require someone to enter every item in order to work.
It's so much easier to do something like this, where it tries to make an itemstack, and if it can't, it continues.

Code:

@Override
public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
        if (cmd.getName().equals("i")) {
            if (sender instanceof Player) {
                Player p = (Player) sender;
                try {
                    p.getInventory().addItem(new ItemStack(Integer.parseInt(args[0]), Integer.parseInt(args[1])));
                } catch (Exception e) {
                    p.sendMessage("Sorry, I couldn't find that item!");
                }
            } else {
                log.info("Sorry, this command is for players only!");
            }
        }
        return true;
}

 

public static Material getMaterial(String name) {
Material m = null;
try {
m = Material.getMaterial(Integer.parseInt(name));
if (m != null) return m;
} catch (Exception ex) {};
name = name.toLowerCase();
//custom names
m = items.get(name);
if (m != null) return m;
//matching
m = Material.matchMaterial(name);
if (m != null) return m;
//looping
name = name.replace("_", "");
for (Material mm : Material.values()) {
if (mm.toString().replace("_", "").equalsIgnoreCase(name)) {
return mm;
}
}
return null;
}
}

 

There isn't a code option for nothing.

 

it isn't better to use enums instead of Hashmap?

 

I would not, hashmaps are more reliable.

 

I think its the oposite way, you can use enums whit multiply threads, whitout more code, but this cant be done whit HashMaps, also enums are inited automatic

 

I know now, this is a long time ago.

 

Does anyone have a hash map with all the items in it? and working? I'm not lazy. Just asking xD

 

Whilst I'm unsure why no one has suggested this, Material has a matchMaterial method, why not just use that?

 

Archelaus
Iffa actually did this in his code, but I found the method not working well for mossy cobblestone few months ago

 

Why not just use my code here?

Code:java

 
//Author = davejavu
if (cmdLabel.equalsIgnoreCase("i") && args.length == 1){
Material mat;
if (args[0].matches("-?\\d+(.\\d+)?"){
mat = Material.geyById(Integer.parseInt(args[0]));
}else{
mat = Material.matchMaterial(args[0]);
}
player.getInventory().setItem(player.getInventory().firstEmpty(), mat).setAmount(64);
player.sendMessage(ChatColor.GOLD + "Item given");
}

 

So many ways to do this, I posted a nice one above as well. Yet, good job! Like yours too.

 

Wait why can't you?