Config is other classes

Help i get this error





Code:

Code:

package xyz.CheeseNips;

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;


public class Teamspeak extends JavaPlugin {

    public void onEnable() {
        getCommand("website").setExecutor(new website());
   
   
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
       
        if (!(sender instanceof Player)) {
            sender.sendMessage(ChatColor.YELLOW + getConfig().getString("TeamSpeak"));
            return true;
        }
       
        Player player = (Player) sender;
       
        if (cmd.getName().equalsIgnoreCase("ts")) {
            player.sendMessage(ChatColor.GOLD + getConfig().getString("TeamSpeak")); }
        return false;
           
        }

}

]


Other class where i get errors:

Code:

package xyz.CheeseNips;

import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

public class website implements CommandExecutor{

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandlabel, String[] arg) {
       
        if (!(sender instanceof Player)) {
            sender.sendMessage(ChatColor.YELLOW + getConfig().getString("Website"));
            return true;
        }
       
        Player player = (Player) sender;
       
        if (cmd.getName().equalsIgnoreCase("website")) {
            player.sendMessage(ChatColor.GOLD + getConfig().getString("Website"));
        }
        return false;
    }

}

 

because your "website" class doesn't have the method "getConfig". Also please follow naming conventions.

 

@teej107 What would be the method for it sorry i'm new too making plugin ?

 

New to plugins or new to Java? I suggest you learn the latter before attempting to use the Bukkit API.

 

Code:

getCommand("website").setExecutor(new website(this));

and in the other class:

Code:

package xyz.CheeseNips;

import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

public class website implements CommandExecutor{
   (PluginName) plugin;

  
  public Website((Plugin Name) instance){
     plugin=instance;
   }
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandlabel, String[] arg) {
      
        if (!(sender instanceof Player)) {
            sender.sendMessage(ChatColor.YELLOW + getConfig().getString("Website"));
            return true;
        }
      
        Player player = (Player) sender;
      
        if (cmd.getName().equalsIgnoreCase("website")) {
            player.sendMessage(ChatColor.GOLD + getConfig().getString("Website"));
        }
        return false;
    }

}

And now you can just plugin.getConfig() in other class. I use this.
I had same problem.

 

@epe07 What is with the casting? Just pass JavaPlugin or your main class. That way you can access the JavaPlugin methods like getConfig.

 

I didnt mean casting :3

I did forget it. Just without them. (I mean just plugin name)

 

Your CommandExecutor class needs to have "extends JavaPlugin" in the class declaration.

 

@PixelBeaver
No. You should only have one class that extends JavaPlugin, which would be the main class.

 

public void config(){
plugin.getConfig();
}

 

@DoggyCode Please don't write him any code.

He knows extremely little (possibly nothing) about Java at this point, he should be learning that, not copy-pasting someone else's code

 

@Zombie_Striker I meant that if he wanted to use getConfig() in that class he would have to make it extend JavaPlugin

 

@PixelBeaver wrong. Add a pass the JavaPlugin class to the class you are trying to use the methods in.

 

@henderry2019 I'm just stating what works for me, just trying to help

 

A way that I can reference to the getConfig() method is by putting

Code:

public static MainClassName plugin;

This allows you to reference your main class and use it's methods, such as getConfig(). Whenever you want to use the getConfig() method, you'll have to use it like this:

Code:

plugin.getConfig().otherstuffhere

Hopefully this helps!

 

True, it's just terrible practice. Static shouldn't be used as an access anywhere modifier.

 

@teej107 Hey, at least it works!

 

Just because you can do something doesn't mean you should.

 

You know, that's why the people here hates theBCBroz channel. The code presented there just barely works, is very inefficient and uses a lot of bad code style, like statics.

 

@teej107 @mcdorli
If you're so against using the variable method "static," you can also just use:

Code:

public MainClassName plugin;

 

Encapsulation issues aside, I am not against using the "variable method" static. Quite a few people on here abuse and misuse it.

 

Oh okay, I understand then.