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:
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...