Skip to main content
se añadieron 16 caracteres en el cuerpo
Origen Enlace
Marce Puente
  • 3.2k
  • 2
  • 14
  • 21

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

private String Obra; private int EntradasVendidas;

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.

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 == "" ) {
        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");
    }
}

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.

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.

Origen Enlace
Marce Puente
  • 3.2k
  • 2
  • 14
  • 21

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 == "" ) {
        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");
    }
}

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.