Global State and Singletons Dependency injection

Question

This is a problem I face a lot of times when I am designing a new app. I'll use a sample problem to explain this.

I am writing simple game, so I want to hold a list of players. I have few options...

1. Use a static field in some class

private  static ArrayList<Player> players = new ArrayList<Integer>();  
public Player getPlayer(int i){
    return players.get(i);
}

but this a global state

2. Or I can use a singleton

class PlayerList{
    private PlayerList instance;
    private PlayerList(){...}
    public PlayerList getInstance() {
        if(instance==null){
            ...
        }
        return instance;
    } 
 }

but this is bad because it's a singleton

3. Dependency injection

class Game {
    private PlayerList playerList;
    public Game(PlayerList list) {
        this.list = list;
    }
    public PlayerList getPlayerList() {
        return playerList;
    }
}

this seems good but it's not.

If any object outside Game need to look at PlayerList (which is the usual case) I have to use one of the above methods to make the Game class available globally. so I just add another layer to the problem. I didn't actually solve anything.

What is the optimum solution? (currently I use Singleton approach)

Answer 1

That's why DI Containers manage lifecycle. Let the Playerlist be a singleton in terms of container lifecycle. Gives you full testability of components and let's the container (not you) get its hands dirty.

Answer 2

The idea behind dependency injection is as the name states to inject dependencies. So whatever object needs to know about the player list will be injected with it. Usually, it makes a lot of sense to use dependency injection as extensively as possible before switching to dependency lookup or some other mechanism. This will also make it possible to extend the game later to have different player lists for different levels or whatever extension you might think about.