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

Hola Beginer, el código está bien. Unicamente hay una cosa que no lo está: no debes tener un atributo especialidad y otro oldEspecialidad a no ser que tengas necesidades específicas para hacerlo, y el ejercicio no dice nada de esto.

Para recuperar un atributo de la clase no te hace falta usar getEspecialidad();

Puedes hacerlo simplemente escribiendo especialidad

El método setEspecialidad podrías dejarlo así:

    public void setEspecialidad (String valorEspecialidad){
        System.out.println ("La especialidad del doctor: " + nombre + " " + apellido + " es: " + especialidad);
        System.out.println ("Ha cambiado la especialidad del doctor: " + nombre + " " + apellido + " De: " +especialidad + "  A: " + valorEspecialidad);
        especialidad = valorEspecialidad;
    }


En resumen:

- Eliminar el atributo oldEspecialidad por no ser necesario

- Modificar el método setEspecialidad, no es necesario hacer uso de oldEspecialidad

Saludos

Hola toni, poco que comentar respecto a posibles mejoras del código. Como ya se ha hecho en otras ocasiones, felicitarte porque además de resolver el ejercicio estás ampliándolo con posibilidades creadas por tí. Además el diseño del código se ve cada vez más eficiente y usando los conceptos y diseño propios de la programación orientada a objetos.

Hay una cosa que me ha llamado la atención, la interface Construcción vacía y que después escribas public interface Instalacion extends Construccion, supongo que sería alguna idea que tenías en la cabeza o alguna prueba sobre cómo relacionar interfaces entre sí.

Creo que estás alcanzando, o ya has alcanzado, el final del curso de programación Java desde cero, que puedes considerar que has seguido brillantemente. No sé qué trayectoria tienes pensado seguir, pero comentarte que nos gustaría que siguieras colaborando con nosotros en los foros ahora como "profesor" ayudando y respondiendo a aquellas personas que, igual que tú y todos en su día incluido yo, están comenzando en el mundo de la programación. También existen otras posibilidades de colaboración y participación con la web, pero todo depende de la voluntad y posibilidades de las personas, ya que detrás de esto no hay ninguna gran empresa, sino simplemente eso, personas, y tanto los foros como el resto de la web funcionan gracias a eso.

A seguir adelante, un abrazo. 

Hola, debido a la cantidad de consultas recibidas sobre el tema dejo aquí esta información.

La visualización de errores en PHP hay que activarla si no está activada y queremos visualizar los mensajes de error en vez de una pantalla en blanco. Puede intentarse de distintas maneras, aquí indico algunas:

1) Contacta con el servicio de hosting y pide que te activen la depuración de errores php (php debugging)

2) Accede al fichero php.ini en el servidor y busca en él error_reporting y display_errors y cambia sus valores. No siempre es posible acceder al fichero php.ini (los responsables del servidor puede que no te tengan habilitado el acceso)

3) Crea un fichero cuyo nombre debe ser .httaccess (tal cual he escrito, el nombre del fichero empieza con un punto), o edítalo si ya existe y añade estas líneas:

php_flag  display_errors        on
php_value error_reporting       2039

4) Accede al cPanel o equivalente del servidor, busca la opción php debugging y actívala

Saludos,

Para mi gusto sí es un poco "excesivamente grande". Si la puedes dejar con una altura en torno a 100px te lo agradezco pues así no agrandará tanto los posts. Saludos!

Tranquilo dongo, aquí no damos largas a nadie... a no ser casos extremos o molestos, si algún día te decides dedicar al hackeo por favor utiliza otra web 

Hola, si no sabes por dónde empezar es que posiblemente no has estudiado cosas básicas.

Para adquirir las nociones básicas de programación recomendamos comenzar con este curso: http://aprenderaprogramar.com/index.php?option=com_content&view=category&id=28&Itemid=59

No es necesario estudiarlo todo, sólo algunas partes. Puedes empezar leyendo hasta la entrega CU00111A y consultando las dudas que te surjan.

Saludos!

Hola sniper, he revisado tu código y está perfecto. Hay una cosa que sí merecería ser mejorada relacionada con los métodos para mostrar información, pero lo verás tú mismo cuando llegues a la entrega CU00690, donde se explica la sobreescritura de métodos. Enhorabuena!

Nosotros como es lógico recomendamos nuestros materiales:

Artículo para empezar a leer algo: http://aprenderaprogramar.com/index.php?option=com_content&view=article&id=28&Itemid=73

Curso gratuito para aprender a programar: http://aprenderaprogramar.com/index.php?option=com_content&view=category&id=28&Itemid=59

Cursos tutorizados on line (para empezar recomendamos el de fundamentos de programación): http://aprenderaprogramar.com/index.php?option=com_content&view=article&id=64&Itemid=87

Libro para aprender a programar: http://aprenderaprogramar.com/index.php?option=com_content&view=article&id=97&Itemid=106

No recomendamos Java ni programación web para empezar, de acuerdo con nuestros criterios didácticos.

Saludos

Hola, tienes varias cosas que mejorar en este código.

En el constructor:

    public Medico (){
        nombre = "";
        apellidos = "";
        edad = 20;
        casado = false;
        id = "me00000";
        especialidad= "cirujano";
    }

Lo normal es inicializar a valores false, 0 ó vacío, por tanto un médico recién creado no tendría 20 años de edad, sino el valor 0 que sirve para saber que no ha sido establecida su edad.

Lo más normal sería esto:

    public Medico (){
        nombre = "";
        apellidos = "";
        edad = 0;
        casado = false;
        id = "";
        especialidad= "";
    }

Y el problema principal: no tienes métodos get y set. El enunciado te pide:

"los métodos para poder establecer y obtener los valores de los atributos"

Los métodos que has definido no están bien planteados.

Vuelve a leer los contenidos del curso y los ejemplos del curso sobre los métodos get y set.

Fíjate también en cómo lo han resuelto otros compañeros en el foro: https://www.aprenderaprogramar.com/foros/index.php?topic=1132

Una vez hayas releído y revisado lo ideal es que trates de escribir tú mismo el código, si eres capaz de hacerlo será que habrás entendido bien los conceptos, que es lo interesante...

Saludos

Hola, yo también he mirado y probado el código y funciona bien, pero tengo una matización que hacer en función de lo que dice el enunciado:

Crea un programa Java que permita “jugar a adivinar un número” como se expone a continuación. El programa debe iniciarse indicando “Se ha generado un número aletatorio entero entre 1 y 100, intente adivinarlo”. El usuario introducirá un número y si el número aleatorio generado por el ordenador es menor deberá indicarse “No has acertado: el número es menor. Prueba otra vez”. Si el usuario introduce un número menor que el número aleatorio deberá indicarse “No has acertado: el número es mayor”. El programa terminará cuando el usuario introduzca el número aleatorio que había escogido el ordenador.

En teoría, debías crear un número aleatorio entre 1 y 100 (en tu caso has decidido que 100 sea un valor flexible, a lo que no le veo problema).

El problema lo veo aquí:

  public void addNumeros(int cantidad)
  {
    for(int i = 1; i <= cantidad; i++)
    {
      listaDeNum.add(i);
    }
  }

Realmente no estás generando un número aleatorio entre 1 y cantidad, sino que estás creando un ArrayList de enteros entre 1 y cantidad, para luego elegir aleatoriamente un número de entre los números existentes en ese ArrayList.

Ahora te planteo las siguientes preguntas:

¿Realmente es necesario crear la lista para después elegir el número?

¿Si tuviéramos que elegir un aleatorio entre decimales qué ocurriría?

¿Sería eficiente el proceso si el aleatorio hubiera que elegirlo entre 1 y 100000000000?

Donde quiero ir a parar es que aunque en este contexto no tenga demasiada importancia (de hecho el tiempo que puede consumir es inapreciable para números pequeños) el diseño falla en cuanto a eficiencia: hacer algo, que consume recursos como memoria y tiempo, sin que realmente sea necesario.

El código lo veo bien trabajado, pero esta es la impresión que he sacado en cuanto al diseño, aunque quizás me equivoque en algo, por ello creo que lo mejor es que lo revises tú mismo y trates de valorar lo comentado. Saludos.