Sistema.currentTimeMillis() vs new Date() vs Calendario.getInstance().getTime()

Mirando el JDK, innermost constructor para Calendar.getInstance() tiene esto:

public GregorianCalendar(TimeZone zone, Locale aLocale) {
    super(zone, aLocale);
    gdate = (BaseCalendar.Date) gcal.newCalendarDate(zone);
    setTimeInMillis(System.currentTimeMillis());
}

Así que ya hace automáticamente lo que usted sugiere. El constructor por defecto de Date tiene esto:

public Date() {
    this(System.currentTimeMillis());
}

Así que realmente no es necesario obtener la hora del sistema específicamente a menos que desee hacer algunas matemáticas con él antes de crear su objeto de Calendario/Fecha con él. También tengo que recomendar joda-time para usar como reemplazo de las propias clases de calendario/fecha de Java si su propósito es trabajar con date cálculos mucho.

Si estás USANDO una fecha, entonces te aconsejo encarecidamente que uses jodatime, http://joda-time.sourceforge.net / . Usar System.currentTimeMillis() para campos que son fechas suena como una muy mala idea porque terminarás con un montón de código inútil.

Tanto la fecha como el calendario están seriamente borked, y Calendar es definitivamente el peor intérprete de todos ellos.

Te aconsejo que utilices System.currentTimeMillis() cuando realmente estás operando con milisegundos, por ejemplo así

 long start = System.currentTimeMillis();
    .... do something ...
 long elapsed = System.currentTimeMillis() -start;

Prefiero usar el valor devuelto por System.currentTimeMillis() para todo tipo de cálculos y solo usar Calendar o Date si realmente necesito mostrar un valor que es leído por humanos. Esto también evitará el 99% de sus errores de horario de verano. :)

En mi máquina intenté comprobarlo. Mi resultado:

Calendar.getInstance().getTime() (*1000000 times) = 402ms
new Date().getTime(); (*1000000 times) = 18ms
System.currentTimeMillis() (*1000000 times) = 16ms

No se olvide de GC (si usa Calendar.getInstance() o new Date())

Dependiendo de su aplicación, es posible que desee considerar el uso de System.nanoTime() en su lugar.

He intentado esto:

        long now = System.currentTimeMillis();
        for (int i = 0; i < 10000000; i++) {
            new Date().getTime();
        }
        long result = System.currentTimeMillis() - now;

        System.out.println("Date(): " + result);

        now = System.currentTimeMillis();
        for (int i = 0; i < 10000000; i++) {
            System.currentTimeMillis();
        }
        result = System.currentTimeMillis() - now;

        System.out.println("currentTimeMillis(): " + result);

Y el resultado fue:

Fecha(): 199

CurrentTimeMillis (): 3

System.currentTimeMillis() es obviamente el más rápido porque es solo una llamada de método y no se requiere ningún recolector de basura.