Minecraft PC IP: play.cubecraft.net

DrHam

Forum Veteran
Sep 15, 2015
3,615
4,166
518
Kepler-452b
I mean, I could direct all the dev attention to this, but then it would delay bigger more important projects :p
I know :3
Just wondering, what are JrDevelopers doing right now, if you can tell it? (@502Errors is working in the next Minerware update, right?)
 

OrangeGE

Forum Expert
Jan 21, 2014
4,681
2,369
313
24
From what I see, your plugin may work, however the suitability of it to other devs could be concerning. For example, in one if statement you return the boolean as true if the if statement triggers, and in the rest you don't. Your indentation is not in line too, which could cause some confusion and at the end of the onCommand you declare the boolean as true even if all of the 'if' statements are false, when really if none of the if statements are triggered you should return the result as a false.

I also don't think the sout saying "Done!" is that necessary, and you can also use getLogger.info("text") for a console message (same with getLogger.warn("text") etc too).

Sorry if some of this made no sense, I am a bit tired and I don't really know why I even replied to this thread xD
 

Thaus

Forum Expert
Jul 6, 2016
2,719
2,063
273
In a chicken coop
From what I see, your plugin may work, however the suitability of it to other devs could be concerning. For example, in one if statement you return the boolean as true if the if statement triggers, and in the rest you don't. Your indentation is not in line too, which could cause some confusion and at the end of the onCommand you declare the boolean as true even if all of the 'if' statements are false, when really if none of the if statements are triggered you should return the result as a false.

I also don't think the sout saying "Done!" is that necessary, and you can also use getLogger.info("text") for a console message (same with getLogger.warn("text") etc too).

Sorry if some of this made no sense, I am a bit tired and I don't really know why I even replied to this thread xD
It sorta doesn't make sense to me but I haven't done coding in years. I think you are about 90% correct.
 

DrHam

Forum Veteran
Sep 15, 2015
3,615
4,166
518
Kepler-452b
From what I see, your plugin may work, however the suitability of it to other devs could be concerning. For example, in one if statement you return the boolean as true if the if statement triggers, and in the rest you don't. Your indentation is not in line too, which could cause some confusion and at the end of the onCommand you declare the boolean as true even if all of the 'if' statements are false, when really if none of the if statements are triggered you should return the result as a false.

I also don't think the sout saying "Done!" is that necessary, and you can also use getLogger.info("text") for a console message (same with getLogger.warn("text") etc too).

Sorry if some of this made no sense, I am a bit tired and I don't really know why I even replied to this thread xD
I just made this in a fast and bad way, so, yeah, it's not well coded, I know xD
Save this criticizism when I post Prison Riot :P

I don't even know why I coded this lmao
 
Members Online

Team online

Latest profile posts

Apparently I hack IN PARKOUR LOLOLOL
IMG_6919.png

😂🤣

After uttering those 3 words, the dude left and gave me a free win xD
A picture of my cat sleeping having kitty dreams
1000053259.jpg
kfccaleb wrote on WayneX4's profile.
You can now have a follow after I played with you!
How do you make spoilers with a name?
Top Bottom