Esse padrão é útil em raros casos.

Um deles é para implementar o padrão Singleton, em que se esconde o constructor, e um método estático é utilizado para criar um Singleton novo ou retornar o existente. De modo simples:

<!-- language-all: c# -->

    public class MySingleton{    
    	private static MySingleton _singleton = null;
    
    	private MySingleton() { }
    
    	public static MySingleton GetInstance() {
    		if(_singleton == null)
    			_singleton = new MySingleton();
    		return _singleton;
    	}
    }

(Nota: o exemplo acima podia ser melhorado, com *double-checked locking* por exemplo para garantir *thread safety*)

Porem, na maior parte dos casos, tornar *todos* os constructores privados só traz problemas.
Impede testes unitários em isolação, não se pode fazer mocking de dependências, não se pode injectar dependências, etc.


Com todo o respeito, o teu líder devia voltar para a faculdade...