[NOW WORKING] Using NMS and OBC Without Your Plugins Breaking [No Reflection Involved]

Code:

if(VerisonHandler.is1_7_2()){
//1.7.2 NMS
}else if(VersionHandler.is1_7_5()){
//1.7.5 NMS
}

In order for this to be any use you have to use reflection or have multiple craftbukkits in your build path. bigteddy98

 

Yeah, I know.

 

BungeeTheCookie The problem is, I can have multiple CraftBukkit, but users won't..

Like if you imported 1.7.2 NMS class, and the plugin is loaded by CraftBukkit 1.6.4, no matter are you using an if/else statement, the plugin simply crashed because you CraftBukkit will return a NoClassDefFoundError!..

 

TigerHix
I thought something like that would happen, because as soon as the class loads, it would try to do that.
A (possible?) solution would be to have all Versioned NMS in their own classes, and initialize classes (no static methods) when needed.
It'd be very clunky, though. Reflection ftw

 

Why would you get the NoClassDefFoundError even if you decide to not use an import and use net.minecraft.server.v1_7_1R.Class?

 

BungeeTheCookie if the class you call contains any mention of a not found class, it will automatically fail. The way I worked around this was make three classes for each of the versions I was working with, and put the if statement before I call the classes, then call the one that matches the version.

 

BungeeTheCookie I beg u haven't tried yourself. Refers to ccrama's explanation.
In conclusion, this class doesn't really does what you stated in the title..

 

BungeeTheCookie Also the class won't work with every 1.7 version, as there are three different versions of NMS for 1.7. I edited the code as so so it would work:

Code:java

public static boolean isR1(){
        if(Bukkit.getServer().getClass().getPackage().getName().contains("R1")){
            return true;
        }
        return false;
    }
 

That way you can have support for each version of NMS. Using your class won't work if you directly refer to NMS on different versions of 1.7.

 

ccrama
Ok thanks! Could you provide me an example of using three different classes for the initilization so I can put it in my thread and not mislead people?

TigerHix
I have failed your mission. Don't worry, I will find a solution! COOKIE POWA

 

As we have all seen now this code is not going to work without reflection, maybe you should change the title and remove the [No Reflection Involved] part

 

bigteddy98 ccrama TigerHix CaptainBern
I have added more checks so now it should function more properly

 

BungeeTheCookie
Instead of :

Code:java

public static boolean is1_2_5(){
    if(Bukkit.getVersion().contains("1.2.5")){
        return true;
    }
    return false;
}

You should do :

Code:java

public static boolean is1_2_5(){
    return Bukkit.getVersion().contains("1.2.5");
}

 

What exactly are the benefits of using this over reflection?

 

1. People who are not experts at reflection, like me, would actually find this beneficial. It saves them the time from learning all of the things about reflection. People would use this for making private plugins mostly, and if they are smart enough to use NMS, they are smart enough to update it when a new CraftBukkit version comes out.

2. Reflection reflects light, which can darken your room, making it hard to code because you are thinking Mini Godzilla has teamed up with The Boogie Man and are going to come and destroy you in the dark. Thus, reflection should be abolished for all of Eternity.

 

Thanks for enlightenment. (hue)

 

[NOW WORKING]

thx god you understood that, only 30 more posts was required for this...

 

@ranGan.
Any time.

RawCode
Lolz