Como bien dice @gbianchi, es un gravísimo error intentar meter la propiedad Genero
de Pelicula
en un enum.
En primer lugar, Genero
es una entidad en sí misma, y debe ser tratada como tal. Imagina por ejemplo que se necesite una descripción del Genero
o que el modelo de datos se extienda, y surja una nueva entidad o lógica que agrupe varios géneros. Será harto complicado adaptar tu código en cualquiera de esos casos.
Más aún, supongamos, como suele ocurrir, que una misma película pueda pertenecer a varios géneros ... Comprenderás que el haber optado por un enum no es otra cosa que capar tus datos.
Tienes dos opciones, y eso ya dependerá de otros asuntos contextuales que no vienen al caso aquí:
Opción 1:
Declarar Genero
como una propiedad de Pelicula
. Y en ese caso, cuando creas la instancia de Pelicula
debes pasar una instancia de Genero
.
Esta sería la clase para tu entidad Pelicula
:
public class Pelicula {
private String titulo, director, sinopsis;
private int anyo;
private Genero genero;
public Pelicula(){ /*vacio*/}
public Pelicula(String titulo, String director, int anyo, String sinopsis, Genero genero) {
this.titulo = titulo;
this.director = director;
this.sinopsis = sinopsis;
this.anyo = anyo;
this.genero = genero;
}
public Genero getGenero() {
return genero;
}
@Override
public String toString(){
return String.format("Título: %s\nDirector: %s\nSinopsis: %s\nAño: %d\nGénero:\t %s",
titulo, director, sinopsis, anyo, genero.toString());
}
}
Y esta sería la clase para la entidad Genero
:
public class Genero {
private int idGenero;
private String description;
public Genero (int idGenero, String description) {
this.idGenero=idGenero;
this.description=description;
}
public int getIdGenero() {
return idGenero;
}
public void setIdGenero(int idGenero) {
this.idGenero = idGenero;
}
public String getDescription() {
return description;
}
@Override
public String toString(){
return String.format("Id: %d Descripción: %s",
idGenero, description);
}
}
En este modelo de datos, la instancia de Pelicula
tendría que construirse así:
Pelicula mPelicula=new Pelicula("Un Titulo", "Un Director", 2021, "Una sinopsis", new Genero(1, "Este es el Género 1, cuya característica es..."));
Genero mGenero=mPelicula.getGenero();
System.out.println(mPelicula.toString());
Salida:
Título: Un Titulo
Director: Un Director
Sinopsis: Una sinopsis
Año: 2021
Género: Id: 1 Descripción: Este es el Género 1, cuya característica es...
Opción 2:
Declarar la propiedad Genero
solamente por su id en Pelicula
:
class Pelicula {
private String titulo, director, sinopsis;
private int anyo;
private int genero;
public Pelicula(){ /*vacio*/}
public Pelicula(String titulo, String director, int anyo, String sinopsis, int genero) {
this.titulo = titulo;
this.director = director;
this.sinopsis = sinopsis;
this.anyo = anyo;
this.genero = genero;
}
public int getGenero() {
return genero;
}
//etc
La instanciación se haría asi:
Pelicula mPelicula=new Pelicula("Un Titulo", "Un Director", 2021, "Una sinopsis", 1);
Y, si quieres obtener más cosas de Genero
, como la descripción o lo que fuera, tendrás que programar código fuera de tus clases de entidad, para no estropearlas, donde accedas al Genero
por su id y obtengas lo que necesites.
Lo que es importante entender aquí es que, optar por un enum no hace otra cosa que capar tus datos. En el caso de una pelicula que pueda tener más de un género, simplemente cambias la declaración de la propiedad por una lista y ya lo tienes resuelto.
Veamos el caso de la opción 1. Sólo la cambia la clase Pelicula
, en lugar de un objeto simple Genero
, tendremos una lista de objetos de ese tipo y en consecuencia, el método toString()
debe adaptarse a la nueva realidad:
class Pelicula {
private String titulo, director, sinopsis;
private int anyo;
private List<Genero> generos;
public Pelicula(){ /*vacio*/}
public Pelicula(String titulo, String director, int anyo, String sinopsis, List<Genero> generos) {
this.titulo = titulo;
this.director = director;
this.sinopsis = sinopsis;
this.anyo = anyo;
this.generos = generos;
}
public List<Genero> getGeneros() {
return generos;
}
@Override
public String toString(){
StringBuilder sb=new StringBuilder();
sb.append(String.format("Título: %s\nDirector: %s\nSinopsis: %s\nAño: %d\n\nGéneros:\n",
titulo, director, sinopsis, anyo));
for(Genero g : generos){
sb.append(g.toString());
sb.append("\n");
}
return sb.toString();
}
}
La instancia de la clase se crearía con algo así más o menos:
List<Genero> generos = new ArrayList<>();
generos.add(new Genero(1,"Este es el género 1"));
generos.add(new Genero(8,"Este es el género 8, surgido porque bla, bla, bla"));
generos.add(new Genero(81,"Este es el género 81, llamado así porque se relaciona con los géneros 1 y 8"));
Pelicula mPelicula=new Pelicula("Un Titulo", "Un Director", 2021, "Una sinopsis", generos);
La salida de este código es:
Título: Un Titulo
Director: Un Director
Sinopsis: Una sinopsis
Año: 2021
Géneros:
Id: 1 Descripción: Este es el género 1
Id: 8 Descripción: Este es el género 8, surgido porque bla, bla, bla
Id: 81 Descripción: Este es el género 81, llamado así porque se relaciona con los géneros 1 y 8
Y en Genero
nada ha cambiado.
Como podrás ver, el uso de un enum te hará cuesta arriba todo esto. En cuanto a la opción 1 o la opción 2, eso depende de ti, del contexto en el que estás y de lo que quieres lograr. Lo que sí te aconsejo es que pienses en grande cuando programas. Es decir, aunque ahora tus películas sean de un solo Genero
, declara una lista, así, sólo tendrás que agregar nuevos elementos si algún día tu aplicación evoluciona.