[OPINION] Lightweight Updater

What do you guys think?

Code:java

package org.pato.tnttag.util;
 
import java.io.InputStream;
import java.net.MalformedURLException;
import java.net.URL;
 
import javax.xml.parsers.DocumentBuilderFactory;
 
import org.pato.tnttag.core.TNTTag;
import org.w3c.dom.Document;
import org.w3c.dom.Node;
import org.w3c.dom.NodeList;
 
public class Updater {
 
private TNTTag plugin;
private URL filesFeed;
private String name;
private String version;
private String link;
private String description;
private String pubDate;
private String fullName;
private String host = "[URL]http://dev.bukkit.org[/URL]";
private String query = "/bukkit-plugins/";
private String file = "/files.rss";
private String URL;
private Updater.updateResult result;
public enum updateResult{
AVAILABLE,
FAIL_DBO,
UP_TO_DATE
}
 
public Updater(TNTTag plugin, String APIKey){
this.URL = host + query + APIKey + file;
this.plugin = plugin;
try {
this.filesFeed = new URL(URL);
} catch (MalformedURLException e) {
e.printStackTrace();
}
}
 
public void checkUpdate(){
try {
InputStream input = this.filesFeed.openConnection().getInputStream();
Document document = DocumentBuilderFactory.newInstance().newDocumentBuilder().parse(input);
 
Node latestFile = document.getElementsByTagName("item").item(0);
NodeList children = latestFile.getChildNodes();
 
this.version = children.item(1).getTextContent().replaceAll("[a-zA-Z]", "");
this.name = children.item(1).getTextContent().replaceAll(" v" + this.version, "");
this.fullName = children.item(1).getTextContent();
this.link = children.item(3).getTextContent();
this.description = children.item(5).getTextContent();
this.pubDate = children.item(7).getTextContent();
 
if (plugin.getDescription().getVersion() != version){
result = updateResult.AVAILABLE;
return;
}
} catch (Exception e) {
result = updateResult.FAIL_DBO;
return;
}
result = updateResult.UP_TO_DATE;
return;
}
 
 
 
public String getFullName() {
return fullName;
}
 
public String getLatestName() {
return name;
}
 
public String getDescription() {
return description;
}
 
public String getPubDate() {
return pubDate;
}
 
public String getVersion(){
return this.version;
}
 
public String getLatestFileLink(){
return this.link;
}
 
public updateResult getResult(){
return result;
}
 
}

One way to use it (Obviously goes on the onEnable()):

Code:java

 private void checkUpdate() {
Updater updater = new Updater(this, "73538");
updater.checkUpdate();
result = updater.getResult();
switch(result){
case AVAILABLE:
name = updater.getLatestName();
version = updater.getVersion();
pubDate = updater.getPubDate();
link = updater.getLatestFileLink();
fullName = updater.getFullName();
this.log.log(Level.INFO, "============================================");
this.log.log(Level.INFO, "An update is available:");
this.log.log(Level.INFO, fullName + " is available for download at");
this.log.log(Level.INFO, link);
this.log.log(Level.INFO, "============================================");
TNTTag.versionDiff = true;
break;
case UP_TO_DATE:
this.log.log(Level.INFO, "TNT Tag is up to date.");
TNTTag.versionDiff = false;
break;
case FAIL_DBO:
this.log.log(Level.INFO, "The updater could not contact dev.bukkit.org.");
break;
}
}

Bump

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

 

I like it. If i hadn't already began using Gravitys updater, i would use this.

 

ImPhantom
Rhanks! Does not use Thread so very lightweihht.