-
Notifications
You must be signed in to change notification settings - Fork 8
añado Multimedia (abstracta) y Persona #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
nuevas clases Multimedia (abstracta) y Persona |
|
|
||
| public class Artista { | ||
| String nombre; | ||
| public class Artista extends Persona{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Te falta un espacio
|
|
||
| private String id; | ||
| private String titulo; | ||
| private String calidadString; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
¿Crees que la calidad va asociada al Contenido o al Fichero?
|
|
||
| public class Director { | ||
| String nombre; | ||
| public class Director extends Persona{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Espacio
|
|
||
| public abstract class Multimedia extends Contenido { | ||
|
|
||
| String calidadString; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tienes repetida la variable
|
|
||
| @Override | ||
| public String getCalidadString() { | ||
| // TODO Auto-generated method stub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Estos comentarios se tienen que borrar
|
|
||
| Director director; | ||
| Actor[] actoresPrincipales; | ||
| Persona[] actoresPrincipales; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En verdad aquí deben ser Actores, no Personas
|
|
||
| @Override | ||
| public String getCalidadString() { | ||
| // TODO Auto-generated method stub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Idem
|
Los comentarios admiten markdown como sabes. Cuando hagas el comentario ve poniendo cada cosa que aporta en una línea para que quede más claro que como lo has escrito |
añado clase Multimedia (abstracta) con un método denominado getCalidad
y super clase Persona