Buenos días, cigarzonl.
Tienes la definición del método "combinacion" y la variable local en el con los mismos nombres, considero que no es una buena práctica.
El ejercicio pide:
Crea una clase (ponle el nombre que quieras) que carezca de atributos y cuyo constructor esté vacío.
tu constructor no está vacía.
El ejercicio pide:
las dos primeras letras de la primera cadena, seguidas por un espacio en blanco, seguidas de las cuatro primeras letras de la segunda cadena, seguidas de un guión medio y seguido de las 6 primeras letras de la tercera cadena.
El recuento de los carácteres extraidos empieza en 0 y excluye el carácter pasado como segundo parámetro en el método "substring()". Por lo tanto los parámetros pasados en el método "conbinacion" te tendrían que quedar así:
valor_nombre1.substring(0,2),
valor_nombre2.substring(0,4),
valor_nombre3.substring(0,6).
Saludos.