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.