Esta mal, de hecho es imposible que eso te funcione.

Lo primero, estas mezclando new y malloc sin sentido. Esta linea no tiene sentido:

 int *arreglo = new int[];

New

---

Lo segundo, sizeof es un operador que funciona en tiempo de compilación, por tanto es imposible que sepas de esa forma cual es el tamaño del arreglo dado que las llamadas de reserva de memoria se hacen en tiempo de ejecución.

Lo que estas haciendo con ese macro es basicamente algo como "Tamaño del puntero/tamaño del primer elemento del arreglo" entonces por ejemplo puntero=4 bytes, entero=4bytes. 4/4=1.

Tu macro funcionaria solamente con los arreglos estáticos, algo así:

int arreglo[2000];
cout<<sizeof arreglo/sizeof(int);
 

Y en ese caso sizeof devolvería, por ejemplo, algo como 8000/4=2000.

En el caso de los arreglos de enteros dinámicos, no hay manera portable de saber el tamaño salvo utilizando funciones especificas del sistema u manteniendo el tamaño en alguna variable/constante/define etc.


Saludos!