Foros aprenderaprogramar.com
Didáctica y divulgación de la programación

Y ya que estoy por aquí, comentarte que el ejercicio lo veo bien. Sigue avanzando con el curso, a medida que vayas haciendo más ejercicios irás cogiendo más soltura. Saludos,

Hola, el código funciona bien pero hay muchos detalles a comentar y corregir.

- Lo primero (ya se ha comentado varias veces en el foro pero lo repetimos) : cuando se vaya a pegar código pulsar el botón # del editor del foro. Una vez aparezcan las etiquetas [ code ] ... [ / code] pegar el código entre esas dos etiquetas.

- Cuando se tenga el mensaje preparado pulsar en Previsualizar para ver si todo sale bien. Si todo está bien pulsar en publicar, si no corregir y volver a pulsar en previsualizar.

Sobre el código en sí:

- El nombre de la clase no es adecuado por dos cosas: los nombres de clase deben empezar en mayúsculas, no en minúsculas (no es obligatorio pero así lo hacen todos los programadores). Y por otro lado el nombre de clase debe ser alusivo a lo que representa la clase: esta clase representa un médico, por tanto el nombre de clase podría ser Medico ó MedicoConCalculador pero no calculoedad porque eso no indica qué representa la clase.

- En el constructor: lo más normal es iniciar los valores a 0 si son numéricos, "" si son Strings, false si son booleanos, etc.

Por tanto en vez de:

casado=true ;
    edad=29;
    dd=5;

Lo más normal sería:

casado=false ;
    edad=0;
    dd=0;


Los nombres de métodos deben intercalar mayúsculas cada vez que empieza una palabra. Por ejemplo debe ser getNombre en lugar de getnombre.

Los nombres de métodos deben ser descriptivos de lo que hace el método. getced no indica bien qué es lo que hace el método. Mejor sería getCedula, así queda más claro.

Los prefijos set y get están reservados para métodos que establecen o devuelven atributos.

El nombre setmult no es adecuado porque no es un método para establecer un atributo.

En este método tienes esto:

public void  setmult()
{
    int aa;
    aa=edad%5;
    dd=5-aa;
   System.out.println("FALTA "+ dd +" AÑOS PARA QUE SEA MULTIPLO DE 5");
   
}

aa es un nombre de variable que no describe para qué sirve esa variable. Tienes que intentar que los nombres sean descriptivos, por ejemplo en vez de aa poner añosQueFaltanParaMultiplo5

Lo mismo ocurre con dd

Otra cosa: dd es un valor que se calcula a partir de la edad. dd no debe ser un atributo de la clase. Un médico debe saber su edad, debe llevar esa información (como atributo), pero no tiene por qué llevar dd, eso no es importante para representar al médico.

Repitiendo ejercicios y haciendo más te irás acostumbrando a muchas de las cosas comentadas, adelante y a por ello

Creo que ya te han respondido aquí: https://www.aprenderaprogramar.com/foros/index.php?topic=1161

Y una vez más, recordar que no se dupliquen las preguntas en distintos foros o un moderador podrá bloquear al usuario duplicador...

Hola toni, respuestas y código las veo correctas.

A la pregunta: ¿Es posible sin utilizar una invocación a super ni un método get?


Podríamos responder con que es tanto válido:

{ return 925 + super.añosConsolidados * 33.25d + 47.80d * añosEmerito; }

Como:

{ return 925 + añosConsolidados * 33.25d + 47.80d * añosEmerito; }


Esto siempre que añosConsolidados esté declarada como protected, si se declara private no es posible el acceso de una forma ni de otra, ya que aparecería el mensaje de error "añosConsolidados has private access in Profesor"

Saludos

Hola, también te recomendaría que pienses como opción en el curso tutorizado on-line de fundamentos de programación nivel I que ofrecemos en esta web (ver http://aprenderaprogramar.com/index.php?option=com_content&view=article&id=64&Itemid=87). Este curso es de pago, pero tienes un programa de estudios bien definido y un tutor que te guiará. El interés de este curso es no perderse en una maraña de conceptos y contenidos, sino tomar la base con la ayuda de un libro pensado para el aprendizaje de los fundamentos de programación y de alguien experimentado y que eso sirva de trampolín para después seguir progresando de forma autónoma. Es cierto que muchas personas aprenden de forma autodidacta, pero la opción de contar con este tipo de ayuda es algo que conviene valorar. No existe una solución mágica igual para todas las personas, cada uno debe pensar que es lo más adecuado para sí mismo. Saludos.

No, al título no le veo nada de malo... Es decir, cualquiera puede poner un título y hablar de lo que quiera, por ejemplo podríamos hablar del hacking de cuentas de facebook o de la gente que se dedica a ello si quisiéramos, y de las técnicas de hacking que se utilizan para ello. Pero una cosa es hablar sobre algo, que sin pasarse no tiene nada de malo, incluso podrías exponer que es un medio para detectar fallos de seguridad, y otra cosa es proponer cometer un delito, una inmoralidad, promoverlos, etc. Mucha gente se piensa que internet es el todo vale... 

Hola, dongo comentarte que vb6 es una versión que ya no tiene soporte de microsoft ya que ha sido reemplazada por nuevas versiones (Visual Studio con VB 2005, 2008, 2010, 2012, etc.), sin embargo fue vb6 tuvo tanta aceptación que aún sigue siendo utilizado por miles de personas y empresas. Se puede instalar siguiendo las instrucciones de: https://www.aprenderaprogramar.com/foros/index.php?topic=286

A Jigsaw comentarle que trate de dividir los temas de forma que no se mezclen cosas, por ejemplo abrir un tema sobre una cuestión concreta de visual basic y otro tema sobre una cuestión concreta de php, si no lo hacemos así es casi imposible localizar los temas dentro del foro. Este tema ya está abierto así y da igual, pero si en el futuro no divide los temas los moderadores podremos bloquearle  . También que explique con el mayor detalle posible cuál es el problema, ya que revisar un proyecto puede llevar muchas horas, tendría que especificar concretamente cuál es el problema, en qué archivo y en qué líneas está el problema, de otra forma imposible revisar... Espero se pueda comprender 

Saludos

Creo que al mismo tiempo que escribía por otro lado te ha respondido dongo, creo que en su respuesta tienes más información ya que como ves te explica la posibilidad de conversión. Y además dice una cosa muy cierta: para una lista de 12 meses es más lógico usar un array de tamaño fijo porque los de tamaño variables están pensados para cuando la colección puede aumentar y a no ser que como dice dongo un meteorito nos impacte los 12 meses van a ser fijos.

Saludos

Hola, lo primero que no veo claro es por qué declaras abstract class Inmobiliaria en lugar de public abstract class Inmobiliaria

Después tengo que decirte que tus preguntas van en la buena dirección: estás viendo que hay una falta de coherencia como esa creación de un Array para cada tipo de inmueble.

El problema que veo, no queriendo criticar sino ayudar, es que lo que ocurre en el fondo es que están manejando conceptos que no acaban de entender y eso es algo así como pretender caminar en la oscuridad sin darse golpes contra las paredes. Sería más razonable buscar una linterna y aprender cómo usarla para poder avanzar. Inicialmente te puede retrasar un poco respecto a los compañeros que avanzan dándose porrazos, pero en cuanto aprendas a usar la linterna los alcanzarás y dejarás atrás. Eso en Java vendría siendo como aprender las bases de Java, por eso te hacía referencia a lo importante que es un curso sobre los fundamentos de Java.

Hablar de fundamentos llevaría mucho tiempo y eso es una tarea que tendrás que pensar tú mismo si acometes. Centrándonos en el problema que comentas, los objetos Inmobiliaria sí pueden y deben conocer métodos... Para ello se utilizan distintas técnicas, quizás es largo de explicar y ver algo de código sea más útil.

Aquí te dejo un ejemplo de cómo se puede acceder a métodos de subclases de una clase abstracta, básicamente dos técnicas (habría más, pero creo que éstas son válidas): sobreescritura de métodos abstractos y uso de instanceof. En el caso de sobreescritura, el compilador determina dinámicamente qué método es el que debe aplicar. Con instanceof, debes hacer casting para acceder al método de interés. Espero te sirva de ayuda...

CLASE INMOBILIARIA


public abstract class Inmobiliaria {
   protected String nombre;
   protected String direccion;
   protected float metros;
   protected int dormitorios;
   protected String estado;
   protected static int cod = 0;
 
   //constructor
   public Inmobiliaria(String nombre, String direccion, float metros, int dormitorios, String estado) {
        this.nombre = nombre; this.direccion = direccion;
        this.metros = metros; this.dormitorios = dormitorios;
        this.estado = estado; cod++;
    }
   
   protected abstract double precio();
   protected abstract boolean ascensor();
}


CLASE PISO
public class Piso extends Inmobiliaria{
   
    private String ubicacion;
    private String balcon;
    private int planta;
    private static final float pvpMetro = 300;
    //constructor
    public Piso(String nombre, String direccion, float metros, int dormitorios, String estado,String ubicacion, String balcon, int planta) {
        super(nombre, direccion, metros, dormitorios, estado);   
        this.ubicacion = ubicacion;
        this.balcon = balcon;
        this.planta = planta;
    }

    @Override
    protected double precio() {
    double precioTotal = 0;
      precioTotal = this.metros*pvpMetro;
      return precioTotal;
    }
   
    public boolean ascensor(){
        System.out.println ("Soy un ascensor piso - método sobreescrito");
        boolean ascensor = false;
        return ascensor;
    }
   
    public boolean ascensor2(){
        System.out.println ("Soy un ascensorian2 de piso - Método sólo de piso");
        boolean ascensor2 = false;
        return ascensor2;
    }
}

CLASE PISOS2
public class Pisos2 extends Inmobiliaria{
   
    private String ubicacion2;
    private String balcon2;
    private int planta2;
    private static final float pvpMetro = 300;
    //constructor
    public Pisos2(String nombre, String direccion, float metros, int dormitorios, String estado,String ubicacion, String balcon, int planta) {
        super(nombre, direccion, metros, dormitorios, estado);
        this.ubicacion2 = ubicacion;
        this.balcon2 = balcon;
        this.planta2 = planta;
    }

    @Override
    protected double precio() {
    double precioTotal = 0;     
      precioTotal = this.metros*pvpMetro;   
      return precioTotal;
    }
   
    public boolean ascensor(){
        System.out.println ("Soy un ascensor pisos2");
        boolean ascensor = false;       
        return ascensor;
    }
   
    public boolean dimeTrue() { return true;}   
}

CLASE TEST
import java.util.ArrayList;
import java.util.Iterator;
public class Test{
    public static void main (String [] args) {
        Inmobiliaria i1 = new Piso("juan", "calle 1", 3.22f, 4, "vendido", "norte", "si", 2);
        Inmobiliaria i2 = new Pisos2("jose", "calle flu", 6.22f, 2, "comprado", "sur", "no", 7);
        i1.ascensor();
        i2.ascensor();
        System.out.println ("Hemos accedido a los métodos ascensor de objetos Inmobiliaria...");
        ArrayList <Inmobiliaria> lista1 = new ArrayList<Inmobiliaria> ();
        lista1.add(i1);
        lista1.add(i2);
        Iterator<Inmobiliaria> it = lista1.iterator();
        while ( it.hasNext() ) {           //Utilizamos el método hasNext de los objetos tipo Iterator
            Inmobiliaria m1 = it.next();

            System.out.println ("Analizando elemento... " );
            m1.ascensor();                //Utilizamos el método next de los objetos tipo Iterator
            if (m1 instanceof Piso) {((Piso)m1).ascensor2();} else {
                System.out.println ("El objeto analizado no es tipo Piso");
            }
        }   
    } 
}

Pequeño comentario: la clase Piso y la clase Pisos2 heredan de la clase abstracta inmobiliaria.

En la clase test creamos un array de objetos Inmobiliaria y accedemos a los métodos propios de Piso y Pisos2...

Hola, el código funciona bien y resuelve lo que pide el ejercicio. Lo que te diría que hay que mejorar es lo siguiente:

ListaCantantesFamosos lista1 = new ListaCantantesFamosos("Cantantes introducidos por usuario");

¿Qué es el parámetro? Es el nombre de la lista de cantantes que estás creando

Sin embargo en el constructor de la clase ListaCantantesFamosos escribes lo siguiente:

public ListaCantantesFamosos(String nombreCantante)

cuando sería más correcto

public ListaCantantesFamosos(String nombreDeLaLista) ó public ListaCantantesFamosos(String tituloDeLaLista)

El problema de poner ahí nombreCantante es que crea confusión: una persona que revise el código no entenderá qué es lo que quieres hacer. El código debe ser claro y consistente, así es como lograrás que sea "bien valorado" por quien lo revise. En este caso tienes una inconsistencia...

Saludos.