6

Tengo que hacer un trabajo y tengo una punto que no se exactamente si saldría bien o no.

Os cuento es un juego de cartas y tengo un atributo de la clase carta que es año de creación. para no andar con follones lo he puesto como String.

Ahora en el SET quiero verificar que la cadena que yo meto sea numérica y de 4 digitos.

Lo de la cadena numérica había pensado algo así

public void setAñoCreacion(String añoCreacion) {
    if(Integer.parseInt(añoCreacion)
        this.añoCreacion = añoCreacion
    }
}

Pero no me admite porque dice que no puede pasarse a boolean no entiendo porque no o es numérica o no lo es. Gracias por echarme una mano.

7 respuestas 7

6

En la sentencia if (condición), condición tiene que ser siempre un trozo de código equivalente a un booleano.

En tu código, utilizas la funcion parseInt para la condición:

if (Integer.parseInt(añoCreacion)) {
    this.añoCreacion = añoCreacion;
}

La función parseInt devuelve un dato de tipo int, por eso recibes el error:

Type mismatch: cannot convert from int to boolean

El if espera un boolean y tú le estás pasando un int. Hay varias formas de solucionar este problema, una de ellas podría ser utilizar la función StringUtils.isNumeric.

Para comprobar que el año tenga 4 dígitos, te sugiero que uses la función .length() de String, y finalmente el código quedaría así:

if (StringUtils.isNumeric(añoCreacion) && añoCreacion.length() == 4) {
    this.añoCreacion = añoCreacion;
}

Para usar la función isNumeric, tienes que añadir la librería correspondiente.

import org.apache.commons.lang3.StringUtils;
1
4

Hola para hacer más efectiva la comprobación de ambas cosas te dejo el siguiente ejemplo:

  String variableCorrecta = "2019";
  String variableValida = "abcd";
  String variableInvalida = "abcde";
  System.out.println("Probando con Variable Correcta: " + isValid(variableCorrecta));
  System.out.println("Probando con Variable Valida: " + isValid(variableValida));
  System.out.println("Probando con Variable Invalida: " + isValid(variableInvalida));

  public boolean isValid(String anio){
    if(anio.length()==4){
      try{
        System.out.println("Anio" + Integer.parseInt(anio));
        return true;
      }catch(Exception e){
        System.out.println("cumple la primer condición pero no es un entero");
        return false;
      }
    }else{
      System.out.println("No cumple la primer condición de tener una longitud de 4");
      return false;
    }
  }

Espero te sirva, saludos.

2
  • no puedo poner print dentro de una clase o un metodo salvo en el main
    – unknow
    Commented el 8 mar. 2019 a las 10:49
  • @carlosdiaz no hace falta que uses los System.out.println, creo que los ha escrito para que entiendas mejor el flujo. Si quieres loggear información en la consola fuera del main, puedes utilizar algún Logger, por ejemplo el org.apache.log4j.Logger.
    – user119854
    Commented el 8 mar. 2019 a las 12:15
3

Podrías comprobarlo con una expresión regular:

public void setAñoCreacion(String añoCreacion) {
    if(añoCreacion.matches("\\d{4}") { //el string está formado por 4 dígitos
        this.añoCreacion = añoCreacion;
    } else {
      throw new InvalidParameterException("...");
}

Por otro lado, te recomiendo leer por qué no es bueno usar ñ o acentos en el código fuente.

6
  • Gracias ya he corregido lo de la ñ antes de leer el comentario jaj es un error de novato :P
    – unknow
    Commented el 8 mar. 2019 a las 10:29
  • Si se van a usar años negativos la expresión regular sería -?\d{4}
    – JDev
    Commented el 8 mar. 2019 a las 10:29
  • 2
    Por este medio doy la bienvenida a @carlosdiaz a Stackoverflow. En la pregunta leo esto: para no andar con follones lo he puesto como String y me pregunto si precisamente lo que crees que te libera de follones lo que hace es liarte más. Si algo ES numérico decláralo como tal y controla el código donde tengas que controlarlo, por ejemplo, cuando el usuario ingresa los datos. Declarar un miembro de una clase de un tipo que no es el suyo me parece erróneo desde el punto de vista de la lógica y del diseño. La POO intenta precisamente que lo que programamos se ajuste a lo real.
    – A. Cedano
    Commented el 8 mar. 2019 a las 10:34
  • @A.Cedano siemrpe me han dicho qeu si es numerico pero no vas operar con el como un numero de telefono lo declare como String y ponerle DAte si solo va ser el año me parece ms coomodo pero dime tu idea
    – unknow
    Commented el 8 mar. 2019 a las 10:48
  • @carlosdiaz pues quien te haya dicho eso esta totalmente equivocado. Un tipo debe ser siempre declarado como su forma natural. Si es un numero es un numero. Si es un char un char, etc ....
    – JDev
    Commented el 8 mar. 2019 a las 11:47
2

Una forma sencilla siguiendo lo que estabas usando es convertir a integer y comprobar el número de dígitos en función del valor:

try {
      if (Integer.parseInt(anoCreacion)>999) {
          this.anoCreacion = anoCreacion
      } 
  } catch (NumberFormatException nfe) { }

También puedes añadir como condición adicional que sea menor de 10000 en caso de que pudiera haber más de 4 dígitos.

2
public void setAñoCreacion(String añoCreacion) {
    if(Integer.parseInt(añoCreacion)
        this.añoCreacion = añoCreacion
    }
}
  1. En tu bloque de código le falta un paréntesis
  2. El error es porque Integer.parseInt(String) devuelve un entero, te faltaría compararlo con algo.
  3. En el if, si introduces una cadena que no es numérica, te salta una excepción de parsing, deberías de:
try {
 Integer.parseInt(añoCreacion);
 this.añoCreacion = añoCreacion;
} catch (NumberFormatException ex) {
  // mensaje de error
}
2

Puedes dejar el setter de la siguiente forma:

public void setAñoCreacion(int añoCreacion) {
    this.añoCreacion = añoCreacion
}

Y hacer tus verificaciones en otra parte, como por ejemplo en el evento keyPressed, ahi puedes verificar simplemente si la tecla presionada es numérica, así garantizas que el valor obtenido es numérico, después con una condición verificas que el valor sea de 4 dígitos y lo envías al setter

1

Para comprobar la longitud puedes usar el metodo length de String.

if(añoCreacion.length() == 4){
.....
}

Eso si yo no haria estas comprobaciones en el set sino antes.

19
  • Antes donde? a mi las comprobaciones me las han enseñado hacer en el set, por ejemplo que el precio sea mayor que 0. y lo de que sea numérica esta bien?
    – unknow
    Commented el 8 mar. 2019 a las 10:10
  • 3
    Un set unicamente debe asignar el valor, no debe hacer comprobaciones. Si lo haces en el set estas mezclando logica con modelo. El encargado de las comprobaciones tiene que ser el que crea el objeto, no el propio en si.
    – JDev
    Commented el 8 mar. 2019 a las 10:16
  • @JDev Si un setter simplemente asigna el valor no hay valor añadido: ¿Qué diferencia hay con exponer el atributo como público? Los setters están para controlar las modificaciones de los atributos Commented el 8 mar. 2019 a las 10:19
  • 1
    @PabloLozano es que un setter si tiene valor añadido pues dentro de el puedes modificar otros valores de la clase dependiendo del nuevo valor pero NUNCA debes hacer comprobaciones que hagan que no se asigne el valor.
    – JDev
    Commented el 8 mar. 2019 a las 10:23
  • @JDev Estoy de acuerdo en no silenciar el error, pero se puede hacer la comprobación y lanzar una excepción si el el valor no es válido Commented el 8 mar. 2019 a las 10:26

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.