ejercicio CU00624B curso java clase y constructor

[jaivcaba]

adjunto el codigo, me podrian decir si esta bien.... Gracias de antemano

Aqui el codigo

Código: [Seleccionar]

/* Ejercicio CU00624B
 * EJERCICIO
Considera estás desarrollando un programa Java donde necesitas trabajar con objetos de tipo DiscoMusical.
 Define una clase DiscoMusical análoga a la que hemos visto para taxis, pero en este caso considerando
 los siguientes atributos de clase: titulo (String), autor (String), añoEdicion (int), formato (String),
 digital (boolean). Define un constructor y los métodos para poder establecer y obtener los valores de
 los atributos. Compila el código para comprobar que no presenta errores. Crea un objeto y comprueba sus
 métodos como hemos hecho con los objetos Taxi. Para comprobar la corrección de tu solución puedes
 consultar en los foros aprenderaprogramar.com.
 */

    //Atrobutos =
public class DiscoMusical{
    String titulo;
    String autor;
    int añoEdicion;
    String formato;
    boolean digital;
    // cierre de los atributos
   
    // Contructor
    public DiscoMusical (){
        titulo = "Desconocido";
        autor = "Desconocido";
        añoEdicion = 0;
        formato = "Desconocido";
        digital = false;
    }//Cierre del Constructor
   
    // Metodos
    public void setTitulo (String valorTitulo){
        titulo= valorTitulo;
    }
    public void setAutor (String valorAutor){
        autor = valorAutor;
    }
    public void setAñoEd (int valorAñoEd){
        añoEdicion = valorAñoEd;
    }
    public void setFormato (String valorFormato){
        formato= valorFormato;
    }
    public void setDigital (boolean valorDigital){
        digital = valorDigital;
    }
    // Cierre de los Metodos
   
    //Metodo para obtener los datos con Return
    public String getTitulo (){return titulo;}
    public String getAutor () {return autor;}
    public int getAñoEd () {return añoEdicion;}
    public String getFormato () {return formato;}
    public boolean getDigital () {return digital;}
}

       
   
   

[Alex Rodríguez]

Hola, lo primero y como solemos repetir, para pegar código usa el botón # del foro, púlsalo y una vez pulsado te aparecerán las etiquetas [ code ] ... [ / code]. Pega tu código entre estas dos etiquetas para que se visualice correctamente. Pulsa previsualizar y si todo está ok, pulsa publicar. Ahora te respondo sobre el código en sí...

[Alex Rodríguez]

El código está todo correcto. Lo único que te diría que es recomendable que los métodos set y get utilicen exactamente el mismo nombre de atributo que se ha usado para definir los atributos de la clase. Has escrito setAñoEd pero es más claro setAñoEdicion, ya que el nombre del atributo es añoEdicion. Lo mismo con getAñoEd. Saludos,

[jaivcaba]

Muchas gracias.. seguire tus consejos

[Zikem10]

Código: [Seleccionar]

//programa Disco musical cuyos atributos son: Titulo,Autor,Ano de edicion,Formato,Digital.

public class DiscoMusical{
    int anoedicion;
    String autor, titulo, formato;
    Boolean digital;
   
    //constructor para obtener los valores,aqui es como quiero que aparezcan las casillas en pantalla
    public DiscoMusical(){
    titulo = "";
    autor = "";
    anoedicion = 0;
         
    formato = "";
    digital = true;
}
    //Metodo para obtener los datos y guardarlos, este es un metodo de procedimiento porque se usa la palabra
    //VOID OSEA NO TENEMOS VALORES DE VUELTA
   
   
    //Metodo para establecer el valor o datos de titulo
     //setTitulo es SOLOel nombre del metodo y
    //y valorTitulo es solo el nombre definindapor el tipo en este caso String que sirve para guardar el DATO
    // ESTOS NOMBRES SOLO FUNCIONAN aqui en este procedimiento.
    public void setTitulo (String valorTitulo){
    titulo = valorTitulo;
         }
   
    //Metodo para establecer el valor o datos de autor
    public void setAutor(String valorAutor){
        autor = valorAutor;
    }
   
    //Metodo para establecer el valor o datos de ano de edicion
    public void setAnoEdicion(int valorAnoEdicion){
        anoedicion = valorAnoEdicion;
    }
   
    //Metodo para establecer el valor o datos de Formato
    public void setFormato(String valorFormato){
        formato = valorFormato;
    }
   
    //Metodo para establecer el valor o dato de Digital
    public void setDigital(Boolean valorDigital){
    digital = valorDigital;
    }

//METODO DE FUNCION SE USA RETURN Y SIRVE PARA OBTENER LOS DATOS O VALORES GUARDADOS EN EL METODO DEPROCEDIMIENTO
     

     //Metodo para obtener el titulo del objeto discomusical, no es la clase DiscoMuscical
   public String getTitulo(){return titulo;}
   
   //Metodo para obtener el autor del objeto discomusical
   public String getAutor() {return autor;}
   
   //Metodo para obtener el anodeedicion del objeto discomusical
   public int getAnoEdicion(){return anoedicion;}
   
   //Metodo para obtener el formato del objeto discomusical
   public String getFormato() {return formato;}
   
   //Metodo para obtener si es digital del objeto discomusical
   public Boolean getDigital() {return digital;}
   
}//Cierre de la Clase


[Mastermind]

Hola, pega el código usando el botón # del foro, una vez lo pulsas te aparece [ code ] y [ / code] y entre esas dos etiquetas pega el código, es la forma de luego poder copiarlo para compilarlo, saludos

[Thieba]

hola muy buena noche,

según lo que revisado del ejercicio del colega que ha iniciado este tema me parece que lo tengo bien pero si alguien me puede echar un ojo para criticarlo me encantaría.

ah otra cosa, la 's' de 'string' tiene que ser siempre en mayúscula ?me da errores siempre con esta s pero si la pongo en mayuscula me compila bien el programa.
muchas gracias a todos.

Código: [Seleccionar]

public class DiscoMusical
{//declaracion atributos
    String Titulo;
    String Autor;
    int AñoEdicion;
    String Formato;
    boolean Digital;

    //constructor
    public DiscoMusical ()
    {
        Titulo = "desconocido";
        Autor = "desconocido";
        AñoEdicion = 0;
        Formato = "desconocido";
        Digital = true;
    }//cierre constructor
   
    //metodo para establecer el titulo
    public void setTitulo (String valorTitulo)
    {
        Titulo=valorTitulo;
    }//cierre metodo
   
 
    //metodo para establecer el Autor
    public void setAutor (String valorAutor)
    {
        Autor=valorAutor;
    }//cierre metodo
   
    //metodo para establecer el año de edicion
    public void setAñoEdicion (int valorAñoEdicion)
    {
        AñoEdicion=valorAñoEdicion;
    }//cierre metodo
   
    //metodo para establecer el formato
    public void setFormato (String valorFormato)
    {
       Formato=valorFormato;
    }//cierre metodo
   
    //metodo para establecer si es digital o no el disco
    public void setDigital (boolean valorDigital)
    {
        Digital=valorDigital;
    }//cierre metodo
   
    //metododo para obtener el titulo
    public String getTitulo ()
    {return Titulo;}
   
    //metododo para obtener el autor
    public String getAutor ()
    {return Autor;}
   
    //metododo para obtener el el año de edicion
    public int getAñoEdicion ()
    {return AñoEdicion;}
   
    //metododo para obtener el formato
    public String getFormato ()
    {return Formato;}
   
    //metododo para obtener si el disco es digital o no
    public boolean getDigital ()
    {return Digital;}
}


[Ogramar]

Hola! El código debes ponerlo entre medio de las etiquetas [ code ] ... [ / code], no después porque entonces no se pega como código.

He chequeado tu código y está bien, la única cosa que tendrías que corregir es que los nombres de clases deben empezar por mayúsculas, mientras que los nombres de atributos deben empezar por minúsculas.

Por ejemplo en vez de String Titulo; debe ser String titulo;

De resto todo bien. Saludos!

[Thieba]

muchisima gracias Ogramar,
lo haré asi tal como me lo has dicho.

por el tema de la 's' de "string" sigo teniendo duda,parece que tiene que ser siempre en mayúscula.
ahora pego el codigo corrigido solo para ver como queda para hacerlo asi la proxima vez .

Código: [Seleccionar]

public class DiscoMusical
{//declaracion atributos
    String titulo;
    String autor;
    int añoEdicion;
    String formato;
    boolean digital;

    //constructor
    public DiscoMusical ()
    {
        titulo = "desconocido";
        autor = "desconocido";
        añoEdicion = 0;
        formato = "desconocido";
        digital = true;
    }//cierre constructor
   
    //metodo para establecer el titulo
    public void settitulo (String valortitulo)
    {
        titulo=valortitulo;
    }//cierre metodo
   
 
    //metodo para establecer el Autor
    public void setautor (String valorautor)
    {
        autor=valorautor;
    }//cierre metodo
   
    //metodo para establecer el año de edicion
    public void setañoEdicion (int valorañoEdicion)
    {
        añoEdicion=valorañoEdicion;
    }//cierre metodo
   
    //metodo para establecer el formato
    public void setformato (String valorformato)
    {
       formato=valorformato;
    }//cierre metodo
   
    //metodo para establecer si es digital o no el disco
    public void setdigital (boolean valordigital)
    {
        digital=valordigital;
    }//cierre metodo
   
    //metododo para obtener el titulo
    public String gettitulo ()
    {return titulo;}
   
    //metododo para obtener el autor
    public String getautor ()
    {return autor;}
   
    //metododo para obtener el el año de edicion
    public int getañoEdicion ()
    {return añoEdicion;}
   
    //metododo para obtener el formato
    public String getformato ()
    {return formato;}
   
    //metododo para obtener si el disco es digital o no
    public boolean getdigital ()
    {return digital;}
}


[Ogramar]

Ahora mejor. Cosas a mejorar:

Inicicializar los atributos booleanos a false en vez de a true, es decir, en vez de digital = true; poner digital=false; (esto normalmente, si se considerara necesario usar true se usaría).

En los nombres siempre intercarlar una mayúscula cada vez que inicie una nueva palabra. Por ejemplo en vez de settitulo usar setTitulo (e igual para los demás, en vez de getdigital sería getDigital, etc.).

Saludos!

[Thieba]

esto me gusta mucho,
gracias otra vez
pensé que si cambio una letra de los atributos ya el programa no se compilara. llegue a pensar poner así (set_titulo) pero no me atreve,

gracias Ogramar.
que tengas un excelente dia