-1

No se como dar valor en el metodo a Obra de si esta asignada se debe lanzar una excepción. Está declarada obra como string. Copio el enunciado, no logro el poder tener el objeto string Obra como único.

Para poder asignar una obra y posteriormente terminarla habrá que implementar los métodos asignarObra y terminarObra. El primero recibirá como parámetro el nombre de la obra y el segundo no recibirá ningún parámetro.

  • Si el parámetro del método asignarObra es incorrecto, deberá lanzarse una excepción y se detendrá el proceso de instanciación del objeto. En este caso, las posibles excepciones que podrían lanzarse son:

  • Si el método recibe un null en obra, la excepción será de tipo NullPointerException y el mensaje de error de la excepción debería ser del tipo "X". si el método recibe una cadena vacía en obra, la excepción será de tipo IllegalArgumenException y el mensaje de error de la excepción debería ser del tipo "X".

  • Si se intenta asignar una obra pero ya había una obra asignada, se debería lanzar una excepción de tipo IllegalStateException con el mensaje "El teatro ya tiene una obra asignada".

  • Del mismo modo, si se intenta terminar una obra sin que haya ninguna asignada, se lanzaría el mismo tipo de excepción IllegalStateException, aunque en este caso con el mensaje "El teatro no tiene una obra asignada que se pueda finalizar".

La parte principal de mi código.

// Atributos de objeto constantes (representan características inmutables)
private int codigoTeatro;                         // Codigo del teatro
private final String NombreTeatro;                         // Nombre del teatro    
private final int Aforo;                                // Aforo del teatro


// Atributos de objeto variable (representan el estado del vehiculo en un instante dado)
   
private String Obra;                                       // Obra que se está representando
private int EntradasVendidas;                           // Numero de entradas vendidas

Y el código que llevo es el siguiente:

public void asignarObra (String Obra) throws NullPointerException,IllegalStateException ,IllegalArgumentException {
    
    if (this.Obra == null){
        throw new NullPointerException ("El nombre de la obra no puede ser nulo");}
        
    if (this.Obra == ""){
        throw new IllegalArgumentException ("El nombre de la obra no puede ser cadena vacía");
    }    
    
    if (this.Obra != ""  ){
        throw new IllegalStateException ("El teatro ya tiene una obra asignada");}

}
    
   
public void terminarObra() throws IllegalArgumentException {
    
if (this.Obra != null){
    throw new IllegalStateException ("El teatro no tiene una obra asignada que se pueda finalizar");}

}
6
  • Cuál es la pregunta?
    – Alfabravo
    Commented el 5 ene. 2023 a las 18:10
  • No se como dar valor en el metodo a Obra de si esta asignada se debe lanzar una excepción. Está declarada obra como string.
    – antonio
    Commented el 5 ene. 2023 a las 18:13
  • 1
    No sabemos donde guardas las obras... pero basicamente tenes que hacer esa comparacion..
    – gbianchi
    Commented el 5 ene. 2023 a las 18:14
  • // Atributos de objeto variable (representan el estado del vehiculo en un instante dado) private String Obra; // Obra que se está representando private int EntradasVendidas; // Numero de entradas vendidas
    – antonio
    Commented el 5 ene. 2023 a las 18:15
  • El código va EN la pregunta, yendo a editar. En los comentarios es infumable y no se ve si tiene algún error. La mejor forma de contar lo que cuentas en el inicio de la pregunta es mostrando el código. En la pregunta.
    – Alfabravo
    Commented el 5 ene. 2023 a las 18:34

1 respuesta 1

1
private int codigoTeatro;                         
private final String NombreTeatro;         
private final int Aforo;                              

private String Obra; 
private int EntradasVendidas;


public void asignarObra( String Obra ) throws NullPointerException,IllegalStateException ,IllegalArgumentException { 
  if ( Obra == null ) {
      throw new NullPointerException( "El nombre de la obra no puede ser nulo" );
  }
  if ( Obra.equals( "" )) {
      throw new IllegalArgumentException ( "El nombre de la obra no puede ser cadena vacía" );
  } 
  if ( ! this.Obra.equals( "" )){
      throw new IllegalStateException ("El teatro ya tiene una obra asignada"); 
  }

}

public void terminarObra() throws IllegalArgumentException {
    if ( this.Obra == null ) {
        throw new IllegalStateException ("El teatro no tiene una obra asignada que se pueda finalizar");
    }
}

Lo llevabas bien encaminado, pero te equivocaste con los this, yo personalmente, prefiero modificar los nombres para no usarlos, por ejemplo:

void setNombre( String elNombre ) {
    nombre = elNombre;
}

Esto me evita las posibles confuciones... y escribo menos. Dentro de asignaObra, primero haces las verificaciones de Obra, osea el dato que se pasa como parámetro de la función, y lanzas de ser necesario, la excepción correspondiente. Luego haces lo correspondiente con this.Obra, osea que el objeto Obra que tienes, este o no en null, y obras en consecuencia. Luego en terminarObra, verificas que this.Obra no sea null.

3
  • Me sigue sin funcionar quitando los dos primeros this
    – antonio
    Commented el 5 ene. 2023 a las 20:59
  • Perdon @antonio, cometí el error de copiar tu código sin pensar, estabamos comparando cadenas con ==, y se deben comparar con equals, edite mi respuesta, lo he probado y funciona. Commented el 6 ene. 2023 a las 4:36
  • Muchas gracias!! Solucionado
    – antonio
    Commented el 6 ene. 2023 a las 10:05

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.