Foros aprenderaprogramar.com
Aprender a programar => Aprender a programar desde cero => Mensaje iniciado por: jogonzalezmor en 04 de Julio 2014, 06:46
-
He aquí mi código.
public class ExtraeLetras
{
// instance variables - replace the example below with your own
/**
* Constructor de objects de la clase ExtraeLetras
*/
public ExtraeLetras()
{
// initialise instance variables
}//Fin del constructor
public String combinaCadenas(String valorCadena1, String valorCadena2, String valorCadena3)
{
// put your code here
String cadena1 = valorCadena1;
String cadena2 = valorCadena2;
String cadena3 = valorCadena3;
String combinacion = valorCadena1.substring(0, 2)+ " " + valorCadena2.substring(0, 4) + " - " + valorCadena3.substring(0, 6);;
return combinacion;
}//Fin del método combinaCadenas
}//Fin de la clase
-
Está bien aunque hay una cosa que no es correcta.
Cuando vi valorCadena3.substring(0, 6);; pensé que iba a dar un error por los dobles punto y coma, pero no da error ::) así que eso lo puedes tener en cuenta pero da igual
Lo que no está bien es:
String cadena1 = valorCadena1;
String cadena2 = valorCadena2;
String cadena3 = valorCadena3;
¿Para qué declaras estas variables si luego no las usas? Eso sería lo que se llama "código inútil" y hay que tratar de que en nuestras clases y programas no quede ese tipo de código porque al final complica la corrección de los programas y puede dar lugar a problemas a la larga.
Salud2s