Solved Multiple Source Packages

Hello there again bukkit, I know it's very possible to use multiple src packages, such as



So basically I've seen multiple threads on this, but still don't really understand. In my main class, I have

Code:java

public class Main extends JavaPlugin{
    Logger logger = Logger.getLogger("Minecraft");
 
    public boolean gamestarted = false;
    public LobbyTimer lobbytimerclass;
 
 
    public void onEnable(){
        lobbytimerclass = new me.reider45.KnightsVSDragons.timers.LobbyTimer(this);
        logger.info("[KvsD] enabled. Created by Reider45");
        lobbytimerclass.lobbytimer = 122;
        // Start Timer
 
        lobbytimerclass.lobbytimerstart();
        //lobbytimerclass.test();
    }

And in my LobbyTimer class as show above, I have

Code:java

public class LobbyTimer extends Main{
    public Main plugin;
 
    public LobbyTimer(Main instance){
        instance = plugin;
    }
 
    public int lobbytimer = 122;
 
    public void lobbytimerstart(){
        lobbytimer = 122;
          Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable() {
            public void run(){
                Bukkit.broadcastMessage(ChatColor.RED + "Run");
                 // Le more code
            }
        },20L, 20L);
    }
    }

Basically, i can use the lobbytimerstart(), but i get an error on

Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable()

Im assuming the problem is the
{

Code:java

(plugin, new Runnable()

I believe that I have to replace plugin with something else... but i'm not sure what. Here is the error upon startup:

[12:29:43] [Server thread/INFO]: [KnightsVSDragons] Loading KnightsVSDragons v1.0
[12:29:43] [Server thread/INFO]: [KnightsVSDragons] Enabling KnightsVSDragons v1.0
[12:29:43] [Server thread/INFO]: [KvsD] enabled. Created by Reider45
[12:29:43] [Server thread/ERROR]: Error occurred while enabling KnightsVSDragons v1.0 (Is it up to date?)
java.lang.IllegalArgumentException: Plugin cannot be null
at org.apache.commons.lang.Validate.notNull(Validate.java:203) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.validate(CraftScheduler.java:391) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.runTaskTimer(CraftScheduler.java:120) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.scheduleSyncRepeatingTask(CraftScheduler.java:116) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at me.reider45.KnightsVSDragons.timers.LobbyTimer.lobbytimerstart(LobbyTimer.java:19) ~[?:?]
at me.reider45.KnightsVSDragons.Main.onEnable(Main.java:26) ~[?:?]
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:217) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:457) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:381) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.loadPlugin(CraftServer.java:298) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.enablePlugins(CraftServer.java:280) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.reload(CraftServer.java:630) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.Bukkit.reload(Bukkit.java:279) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.command.defaults.ReloadCommand.execute(ReloadCommand.java:23) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:192) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchCommand(CraftServer.java:542) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchServerCommand(CraftServer.java:529) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.DedicatedServer.aw(DedicatedServer.java:286) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:251) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:541) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java:453) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:617) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
[12:29:43] [Server thread/INFO]: Server permissions file permissions.yml is empty, ignoring it

Any help appreciated, thanks!

 

Errrr...

Code:java

 public LobbyTimer(Main instance){
instance = plugin;
}

 

ERRRRRRR....

http://docs.oracle.com/javase/tutorial/java/package/index.html

Read. Its fundamental.

 

Not2EXceL The_Doctor_123

Still confused on what to put at the stared section :S any help?

Code:java

Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(***, new Runnable() {

Also thanks for that document Not2EXceL

 

reider45
Your main class instance..

 

I tried that and got the same error as above.

 

Did you change this?

Code:java

public LobbyTimer(Main instance){
instance = plugin;
}

 

Yes, but then I changed it back, here is the code

Code:java

package me.reider45.KnightsVSDragons.timers;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
 
import me.reider45.KnightsVSDragons.Main;
 
public class LobbyTimerClass{
    public Main plugin;
 
    public LobbyTimerClass(Main instance){
        instance = plugin;
    }
 
    public int lobbytimer = 122;
 
    public void lobbytimerstart(){
        lobbytimer = 122;
        Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable() {
            public void run(){
                Bukkit.broadcastMessage(ChatColor.RED + "Run");
            }
        },20L, 20L);
    }
}

and the main class

Code:java

package me.reider45.KnightsVSDragons;
 
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
import me.reider45.KnightsVSDragons.timers.LobbyTimerClass;
 
public class Main extends JavaPlugin{
    Logger logger = Logger.getLogger("Minecraft");
 
    public boolean gamestarted = false;
    LobbyTimerClass timerpack = new LobbyTimerClass(null);
 
    public void onEnable(){
        logger.info("[KvsD] enabled. Created by Reider45");
        timerpack.lobbytimer = 122;
        timerpack.lobbytimerstart();
    }
 
    public void onDisable(){
        logger.info("[KvsD] disabled. Created by Reider45");
    }
}

 

Umm.. you may want to learn a little bit of Algebra..

The left hand side of the equals sign is the variable you're assigning the value to..

 

FML, literally the one time I switch them around...

Reguardless, this

Code:java

package me.reider45.KnightsVSDragons.timers;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
 
import me.reider45.KnightsVSDragons.Main;
 
public class LobbyTimerClass{
    public Main plugin;
 
    public LobbyTimerClass(Main instance){
        plugin = instance;
    }
 
    public int lobbytimer = 122;
 
    public void lobbytimerstart(){
        lobbytimer = 122;
        Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable()

still gives me the error

[23:04:49] [Server thread/INFO]: [KnightsVSDragons] Loading KnightsVSDragons v1.0
[23:04:49] [Server thread/INFO]: [KnightsVSDragons] Enabling KnightsVSDragons v1.0
[23:04:49] [Server thread/INFO]: [KvsD] enabled. Created by Reider45
[23:04:49] [Server thread/ERROR]: Error occurred while enabling KnightsVSDragons v1.0 (Is it up to date?)
java.lang.IllegalArgumentException: Plugin cannot be null
at org.apache.commons.lang.Validate.notNull(Validate.java:203) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.validate(CraftScheduler.java:391) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.runTaskTimer(CraftScheduler.java:120) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.scheduler.CraftScheduler.scheduleSyncRepeatingTask(CraftScheduler.java:116) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at me.reider45.KnightsVSDragons.timers.LobbyTimerClass.lobbytimerstart(LobbyTimerClass.java:19) ~[?:?]
at me.reider45.KnightsVSDragons.Main.onEnable(Main.java:22) ~[?:?]
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:217) ~[Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:457) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:381) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.loadPlugin(CraftServer.java:298) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.enablePlugins(CraftServer.java:280) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.reload(CraftServer.java:630) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.Bukkit.reload(Bukkit.java:279) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.command.defaults.ReloadCommand.execute(ReloadCommand.java:23) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:192) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchCommand(CraftServer.java:542) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchServerCommand(CraftServer.java:529) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.DedicatedServer.aw(DedicatedServer.java:286) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:251) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:541) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java:453) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]
at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:617) [Bukkit.jar:git-Bukkit-1.6.4-R2.0-28-g373cbb7-b2945jnks]

Also, thanks for all your help so far

 

reider45
Uhum.

Code:java

public class LobbyTimer extends Main{

No. There can only be one instance of your main class in a single plugin.

 

That's not algebra, it's code... in algebra it doesn't really matter.

As for the OP:
The error says plugin can not be null. Your constructor takes the plugin. If you note your main class, you pass it null. Therefore plugin is null. Plugin can not be null...

 

Garris0n
Programming is a whole lot like Algebra. Although, less in Java due to references rather than copies(although, it is a copy of a reference when you say something is equal to something else). Algebra is all about variables and storing values in variables, and so is programming.

 

Garris0n Thanks for the explanation, I believe I'm beginning to understand it more, So how would I execute the lobbytimerstart() in the main class then..?

 

What I meant was that in algebra x=1 and 1=x mean the same thing. In programming x=1 is assigning a variable, while 1=x is a compiler error. I'm not sure if there are any languages where the reverse is true, though.

Edit:
For the OP,
You have to pass an instance of your main class to your constructor. Right now you are passing null. You then proceed to use your "plugin" variable (which you passed null to in the constructor), even though it is null. The place where you use it does not allow a null value. You should also construct the class in your onEnable to prevent any issues, though I don't think it would cause a problem in this case.

 

Garris0n
Quite true, although it is proper to write <Single Variable> = <Value> in Algebra.

reider45
Errr.. do you understand OOP?

 

I'm aware, I just meant that while it's proper in algebra it's required in code. And I think the answer to your question is a no...

 

The_Doctor_123 Not as of now, and yes I'm reading the wiki on it as I type...

 

reider45
The wiki is going to give you the most technical definition of OOP. As it stands at this moment, you need an understandable guide. This may be better.

 

Garris0n So basically my 'plugin' in the LobbyTimer class is null, alright, where would I have to move it?

 

Moving it won't fix the null pointer. You do understand what the constructor does, right? I think you should take The_Doctor_123 's advice and read that.

 

Garris0n Ok will do, thanks for all the help The_Doctor_123

I got it working :3 Thanks