Question

我在过去的两个月里一直在编写Java程序，但我在python和C中经验丰富的程序员。我知道我因此而犯了错误。

我来到这个问题清除Android工作室中的项目警告。

我使用带有内部类的Singleton类将所有配置参数保存在一个位置，让所有其他类访问它而不需要传递配置。

以下是我的Singleton的基本代码

public class syscfg {

    public List<CommData> Commlist;
    public static CommConfigIP4 MyCommConfig;// = new CommConfig();

    private static syscfg instance = null;
    private static boolean ConfigStat = false;

    /** JAVA singleton control methods**/
    protected syscfg(){
        // pues eso

        if(ConfigStat == false){
            Log.i("SD_app_log", "SYSCFG: Module Initialization");
            ConfigStat = true;
            MyCommConfig = new CommConfigIP4();
            init_config();
        }else{
            Log.i("SD_app_log", "SYSCFG:  Module Loaded");
        }
    }

    public static syscfg getInstance(){
        if(instance == null){
            instance = new syscfg();
        }
        return instance;
    }

    public class CommConfigIP4{
        public int discoveryPort = 30303;
        public  byte[] MyMAC;
        public  String MyType = "";
        public  String MyIP;

        public  byte[] getIPbytearray(){
//            byte[] IPout= new byte[4];
            try{
                byte[] IPout = (InetAddress.getByName(MyIP)).getAddress();
                return IPout;
            }catch (Exception e){
                return null;
            }

        }

在我的通讯java文件/类中，我有：

public class Communications {

    private syscfg CFid ;
    ...
    public Communications(Context ctx){
        ...
        CFid = syscfg.getInstance();
        init_comms(); //init_comms calls whoami
    }

    private void whoami (){
        ...
        CFid.MyCommConfig.MyType = netint.getName();
        ...
    }
}

所以，当我第一次在syscfg中拥有所有元素（变量，类和方法）时，静态android工作室显示一个警告，说静态成员通过实例引用访问。经过一些研究和文档后，我发现建议不要使用静态变量和方法，我试图将它们全部消除。但后来我在

中遇到了nullpointexception错误

CFid.MyCommConfig.MyType = netint.getName();

使用调试器，我发现CFid.MyCommConfig = null

我使用singleton来避免在syscfg类上使用static并通过实例化访问而不使用类名。

现在我的单例代码就像这里发布的CommConfigIP4静态代码一样，我再次发出警告，建议我使用：

syscfg.MyCommConfig.MyType = netint.getName();

而不是使用实例来访问配置。

这里发生了什么？我错过了什么？

谢谢，吉列尔莫

Answer 1

在你的whoami（）方法中，你可以参考：

CFid.MyCommConfig.MyType = netint.getName();

...但是，“MyCommConfig”是类“syscfg”的静态属性，而变量“CFid”是指此类的实例。换句话说，“syscfg”的所有实例（以及类定义本身）都引用了“MyCommConfig”变量的相同副本（这是静态的意思）。

因此，通过说“syscfg.MyCommConfig”来引用“MyCommConfig”变量就不那么容易了，因为这表明你指的是静态变量而不是实例。

顺便说一下，您应该考虑遵循标准Java代码约定来大写类名和变量，因为这将使您的代码对其他Java程序员更具可读性。

Answer 2

您应该将嵌套类声明为静态：

public static class CommConfigIP4

然后取消注释在外层顶部初始化静态变量的代码。

Answer 3

要避免这些警告，您应该以这种方式实现单例。我更改了一些代码以使其易于理解：

public class Syscfg {

    public Config getConfig() {
        return c;
    }

    // Obtain the same instance always
    public static Syscfg getInstance() {
        return s == null ? new Syscfg() : s;
    }

    // Important for singleton
    private Syscfg() {
        c = new Config();
    }

    private static Syscfg s;
    private Config c;

    class Config {

        public String[] getConfigs() {
            return configs;
        }

        private String[] configs = {"10.10.10.10", "userName", "userPass"}; 
    }

}

所以，如果你想知道另一个类中使用的配置，例如Test类，那么你可以使用

public class Test {

    public static void main(String[] args) {
        System.out.println(Arrays.toString(Syscfg.getInstance().getConfig().getConfigs()));
    }

}

结果是：[10.10.10.10，userName，userPass]

Java Singleton +内部类错误理解

3 个答案: