[Resource] Get Item ID From String

After an hour or so of research and trial and error, I figured out a pretty compact way (21 lines) to get the ID of an item through a string. It uses JSoup so it's pretty much useless unless there's a way to implement JSoup into a plugin, in which case I learned something new, and you're welcome I think this is pretty cool either way.

Code:java

package com.github.KrazyTraynz;
 
 
import org.jsoup.Jsoup;
import org.jsoup.nodes.Document;
import org.jsoup.nodes.Element;
 
import java.io.IOException;
import java.util.Map;
import java.util.TreeMap;
 
public class StringID {
 
    Map<String, String> ids = new TreeMap<String, String>(String.CASE_INSENSITIVE_ORDER);
 
    public void setPairs(){
        Document doc = null;
 
        try{    
            doc = Jsoup.connect("[url]http://minecraft-ids.grahamedgecombe.com[/url]").get(); //REMOVE THE URL TAGS
        }catch(IOException ex){
            ex.printStackTrace();
        }
 
        for(Element table : doc.select("table")){
            for(Element body : table.select("tbody")){
                for(Element row : body.select("tr")){
                    String name = row.select("td").get(2).text();
                    String ID = row.select("td").get(0).text();
 
                    ids.put(name, ID);
                }
            }
        }
    }
 
        public String getID(String s){
            if(ids.containsKey(s)){
                return ids.get(s);
            }
            return "";
        }
 
        public Object getName(String s){
            for(Map.Entry entry : ids.entrySet()){
                if(s.equals(entry.getValue())){
                    return entry.getKey();
                }
            }
            return "";
        }
}
 

Call setPairs() in your onEnable(), and getID() and getName() can be called anywhere else. Both of the "get" methods do exactly what you think they do. getID() is case-insensitive for convenience, so don't worry about capitalization.

 

Calling this Method from the Server Thread is deadly...

 

A better method would be to download all that stuff and put them in a hashmap. You can code it to output the final code. @krazytrynz

 

Georgeto
Never really put into consideration what this could do to a server

filoghost
So just modify the method to put everything into a HashMap, and create a separate method to call it?

 

Exactly. So you don't have to check the website every time A list of formatted materials would be very useful.

 

Updated to be thread-safe, thanks filoghost

 

The site now offers the list in JSON and TSV format. Using one of those, instead of scraping the HTML, would make this code much more robust in the face of any markup changes I make to the HTML in the future.

To parse the TSV you could do something along these lines in setPairs() instead:

Code:

URL url = new URL("http://minecraft-ids.grahamedgecombe.com/items.tsv");
try (BufferedReader reader = new BufferedReader(new InputStreamReader(url.openStream(), StandardCharsets.UTF_8))) {
    String row;
    while ((row = reader.readLine()) != null) {
        row = row.trim();
        if (row.isEmpty()) /* skip blank lines */
            continue;
 
        String[] cols = row.split("\t");
        String name = cols[2];
        String id = cols[0];
        String metadata = cols[1];
        String idAndMetadata = metadata.equals("0") ? id : (id + ":" + metadata);
        ids.put(name, idAndMetadata);
    }
}

 

Or you could use the server files.

 

I don't understand why this is even needed. The Material class already has the item id's stored and has a method to retrieve a Material by name.

Code:

int id= Material.getMaterial(name.toUpperCase()).getId();

getId() is depricated however it still works, and isn't the only method to get the id from the material, so why reference a site and parse it to obtain values that already exist?

 

That method is pretty bad. It's not very user friendly and doesn't recognize a lot of common item names.

 

krazytraynz
This class isn't thread safe... Your HashMap would need to be switched to a ConcurrentHashMap

 

xTrollxDudex What does that do?>

 

That's because its not an alias database.
If you want to add name aliases to items, use internal list, the website it self doesn't contain that many names, so its also quite pointless for this.
Besides, the original purpose of this is to get item ID, which is perfectly covered by material enum. krazytraynz you do realise newbies will try to use it because its in the resource section? Hence it needs to be coded as well as possible, and using external libraries for something that can be done with material enum is quite pointless.