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 MySingleton GetInstance() {
if(_singleton == null)
_singleton = new MySingleton();
return _singleton;
}
}
Porem, na maior parte dos casos, um constructor privado 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...