Singleton Class return new Instance every time

Question

I'm trying to make a singleton class, but it returns me a new instance every time. Here is my code.

Did I make any mistake? Why does it return a new instance every time?

It should be the same I think.

public final class SingletonA {     
    private static SingletonA instance;

    private SingletonA(){
    }

    public static SingletonA getInstance() {    
        if(instance==null)
        {
            System.out.println("RETURNING NEW INSTANCE OF SingletonA Class");
            return new SingletonA();
        }
        System.out.println("RETURNING OLD INSTANCE OF SingletonA Class");
        return instance;
    }
}

public class SingletonB {
    public static void main(String[] args) {
        SingletonA one = SingletonA.getInstance();
        System.out.println("1."+one+" | "+one.hashCode());

        SingletonA two = SingletonA.getInstance();
        System.out.println("2."+two+" | "+two.hashCode());

        SingletonA three = SingletonA.getInstance();
        System.out.println("3."+three+" | "+three.hashCode());
    }
}

// HERE IS THE OUTPUT
RETURNING NEW INSTANCE OF SingletonA Class
1.oop.singleton.SingletonA@2a139a55 | 705927765
RETURNING NEW INSTANCE OF SingletonA Class
2.oop.singleton.SingletonA@15db9742 | 366712642
RETURNING NEW INSTANCE OF SingletonA Class
3.oop.singleton.SingletonA@6d06d69c | 1829164700

instance in the first line has to be assigned to = new SingletonA() instead of returning new SingletonA() every time. — Arunav Sanyal, Jun 19 '16 at 19:04

score 2 · Answer 1 · answered Jun 19 '16 at 18:50

2

Change like below

if(instance==null) {
    System.out.println("RETURNING NEW INSTANCE OF SingletonA Class");
    instance = new SingletonA();
}

answered Jun 19 '16 at 18:50

Beniton Fernando

1,533
1
14
21

score 2 · Answer 2 · answered Jun 19 '16 at 18:52

You have to assign the instance otherwise the check will always yield true, thus returning a new instance:

public static SingletonA getInstance() {    
    if(instance==null)
    {
        System.out.println("CREATING NEW INSTANCE OF SingletonA Class");
        instance = new SingletonA();
    }
    System.out.println("RETURNING INSTANCE OF SingletonA Class");
    return instance;
}

score 2 · Answer 3 · answered Jun 19 '16 at 18:54

here is not correct because you are returning new instances of the SingletonA and NEVER assigning it to the object instance..

 public static SingletonA getInstance() {    
        if(instance==null)
        {
            System.out.println("RETURNING NEW INSTANCE OF SingletonA Class");
            return new SingletonA();
        }
        System.out.println("RETURNING OLD INSTANCE OF SingletonA Class");
        return instance;
    }

you mean for sure

 public static SingletonA getInstance() {    
        if(instance==null)
        {
            System.out.println("RETURNING NEW INSTANCE OF SingletonA Class");
            instance = new SingletonA();
        }
        System.out.println("RETURNING OLD INSTANCE OF SingletonA Class");
        return instance;
    }

score 2 · Answer 4 · answered Jun 19 '16 at 19:06

You should initialize instance to new SingletonA() just before returning in the test for null check.

public final class SingletonA {     
    private static SingletonA instance;

    private SingletonA(){
    }

    public static SingletonA getInstance() {    
        if(instance==null)
        {
            System.out.println("RETURNING NEW INSTANCE OF SingletonA Class");
            // You are missing this assignment. 
            instance = new SingletonA();
            return instance; 
        }
        System.out.println("RETURNING OLD INSTANCE OF SingletonA Class");
        return instance;
    }
}

score 0 · Answer 5 · answered Dec 13 '21 at 21:13

0

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

answered Dec 13 '21 at 21:13

Sourav singh yuvraj

1
2

1

How does your answer improve on the other answers? Please [edit] your question to provide details. – Mark Stewart Dec 15 '21 at 18:26

Singleton Class return new Instance every time

5 Answers5

Linked