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

Hola jupertivo tu clase EntradaDeTeclado está bien planteada.

En la clase ListaCantantesFamosos tienes esto:

    public ListaCantantesFamosos (String nombre) { //Constructor: crea una lista de nombres vacía
        nombreDeLaLista=nombre;
        listadenombres=new ArrayList<String>();
        System.out.println ("La lista se llama: "+nombreDeLaLista);
    }//Creamos el objeto de tipo ArrayList
Si ves comentarios en otros hilos verás que se recomienda no incluir System.out.println dentro de constructores ni métodos porque la responsabilidad de un constructor es inicializar los atributos de un objeto, no debe hacer nada más que esto (en el caso de métodos, debe cumplir con su responsabilidad y nada más que eso).

En cambio aquí sí lo tienes bien planteado:
    public String getNombre (int posicion) {
        if (posicion>=0 && posicion<listadenombres.size()) {
            return listadenombres.get(posicion);}
        else {return "No existe nombre para la posición solicitada";} }
Este método devuelve un String, que puede ser el nombre de la lista, o si no existe un String que es un aviso de que no existe nombre para la posición solicitada.

Tu clase con el main está muy bien, en pocas líneas haces uso de los objetos y diriges el programa.

Supongo que te habrás dado cuenta de que aunque tu código funciona bien no has cumplido con lo que pedía el enunciado:

Crea una clase denominada ListaCantantesFamosos que al ser inicializada contenga un ArrayList con tres Strings que sean el nombre de cantantes famosos. Crea una clase test con el método main que inicialice un objeto ListaCantantesFamosos, pida dos cantantes famosos más al usuario, los añada a la lista y muestre el contenido de la lista por pantalla.

Salu2

Hola, pega el código con el que estés trabajando para poder verlo, de la forma en que se indica en https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0

Salu2

Hola emilanesi lo primero por ser nuevo es pedirte que leas esto https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0
donde se encuentran indicaciones sobre cómo pegar código en el foro.

Tu ejercicio está bien resuelto. Verás que en muchos ejercicios usamos nombre = ""; que significa cadena vacía, en lugar de nombre = "Desconocido"; pero no está mal como tú lo has hecho.

Esto creo que lo puedes mejorar:

System.out.println ("Ha cambiado la condición del médico de nombre: " + nombre + " a " + valorCasado);

Prueba a ver el resultado y verás que el mensaje que sale es poco entendible. Seguro que puedes idear una forma de mostrar un mensaje más entendible.

Salu2

Gracias!

Hola Dan, todo correcto! Sobre la última pregunta se pueden hacer algunos comentarios, puedes leerlos en https://www.aprenderaprogramar.com/foros/index.php?topic=1978

Salu2

Hola, una clave primaria en la práctica profesional raramente es una clave compuesta, y menos por un nombre , apellidos y nombre de centro de trabajo.

Lo normal es que la clave primaria sea una columna única, por ejemplo idProfesores para la tabla profesores e idCentros para la tabla de centros. Si no tienes restricciones, esto sería lo que te recomendaría

Salu2

Sí, gracias por la explicación, creo que le puede ser útil a quien lea el código para entender mejor lo que hace.

Salu2

Hola ahora sí he podido ejecutar y visualizar lo que ocurre. Lo que he visto es que pasan varias cosas extrañas:

- Las pelotas rebotan por la parte de arriba del div tanto en las coordenadas iniciales del div como en algunos momentos en otras coordenadas

- Las pelotas no rebotan por la parte de abajo del div en movimiento, aunque a veces el rebote por abajo se produce en la parte inferior de la pantalla y a veces en un punto intermedio de la parte inferior.

No veo un sitio fijo para el rebote de las pelotas, y tampoco veo un patrón aparente que permita decir dónde van a rebotar.

El problema parece estar en cómo se fija el punto de rebote de las pelotas pero voy a intentar mirar el código con más detenimiento a ver si logro encontrar el problema

Salu2

Hola, primero mi enhorabuena porque has desarrollado un gran trabajo. Cosas que te puedo comentar para mejorar. Las clases nómbralas siempre en singular. En vez de DiscosMusicales sería DiscoMusical. Ten en cuenta que cuando hables con otro programador dirás: "he creado un objeto DiscoMusical" pero no sonará bien "He creado un objeto DiscosMusicales".

En el menún efectivamente patinas un poco ¿Por qué? Por la repetición de código. Para evitar esa repetición puedes usar colecciones de objetos ó arrays, pero esto se ve más adelante en el curso, así que ahora no te diría que corrijas nada, sino que esperes a avanzar en el curso y dentro de un tiempo revises este ejercicio para eliminar la repetición de código, seguro que un poco más adelante ya vas a ser capaz de ver cómo.

Lo importante ahora es que has sido capaz de mejorar tu código, corregir muchas cosas defectuosas y aplicar muchos conceptos que se han estudiado y lo has hecho correctamente, así que otra vez mi enhorabuena y a seguir así.

Salu2!

Hola Framonteu, tu declaración de atributos está bien, y tu constructor también. Pero te faltan los métodos getter y setter

Define un constructor y los métodos para poder establecer y obtener los valores de los atributos.

Sobre los constructores: una clase es como un molde para crear objetos. Cada vez que creas un objeto, lo harás con este molde. El constructor sirve para decir los valores iniciales que tendrán los atributos de los objetos. Tal y como tú has definido el constructor todos los objetos que se creen tendrán como nombre Jorge y como apellidos Pérez. Pero esto no es lo más adecuado. Para ver cómo debe hacerse léete el hilo https://www.aprenderaprogramar.com/foros/index.php?topic=1437

Salu2