Código eficiente?

[Xerelo]

Muy eficiente no es, se pueden hacer algunas mejoras, pero dependerá de cómo sean los datos y exactamente qué quieres hacer.

Por ejemplo, primero recorres dos arrays para contar el número de elementos repetidos y poder crear el array devuelto. ¿Por qué no creas uno del tamaño del de menor tamaño y evitas el for? Tu actual algoritmo tampoco devuelve un array donde todas las posiciones estén ocupadas.

Otra posible mejora, si vas a comparar números del del 1 al 25, podrías generar un array de ese tamaño, y guardar los repetidos en su posición correspondiente, elimanarías el bucle for del tamaño y el de comprobar que no es un número repetido.

Como ya he dicho, depende de la naturaleza de los datos y de los resultados.