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

Hola Chompy129:

He estado analizando tu código y lo veo muy bien, felicitaciones.
Y aprovechando que está muy bien y no veo necesidad de hacer correcciones importantes, sí me gustaría aportar otra forma de codificar la clase 'TestListaCantantesFamoso'.
En esta versión alternativa, introduzco algunos cambios para que los analices y veas si te parecen bien:

1) se ahorra un campo de instancia (boolean inicial), y se resuelve de otra manera aquello para lo cual estaba pensado originalmente.
2) Se cambia la condición while(true) por una solución más elegante, sin tener que salirse del bucle utilizando 'break'.
3) Se crea el método mostrarListaCompleta, que además de ser retulizable dentro de la clase, hace más legible el método main.
4) Se muestra la lista completa por pantalla la primera vez, y cada vez que ingresamos un nuevo cantante. Esa es la manera en que debe proceder nuestro programa (fijarse en el ejemplo de la descripción del ejercicio).

import java.util.Iterator;
import java.util.Scanner;
/**
 * Este método main testea el funcionamiento de ListaCantantesFamoso.
 * Hace uso de variables tipo Iterator.
 * @author Chompy129
 * @version 1.0
 */

public class TestListaCantantesFamoso{
   
    public static void main(String[] args){
        Scanner entradaS= new Scanner(System.in);
        ListaCantantesFamoso lista= new ListaCantantesFamoso();
        lista.anadirCantante("Michael Jackson", "Thriller");
        lista.anadirCantante("Madonna", "All I want is you");

        String nombreNuevo= "";
        String discoMasVendidoNuevo= "";
       
        System.out.println("La lista inicial contiene los siguientes datos:");
        mostrarListaCompleta(lista.getLista().iterator());
        String m= "s";
       
        while(m.equalsIgnoreCase("s")){
           
            System.out.println("Añada un nuevo cantante:");
            System.out.print("- Nombre: ");
            nombreNuevo= entradaS.nextLine();
            System.out.print("- Disco con mas ventas: ");
            discoMasVendidoNuevo= entradaS.nextLine();
            lista.anadirCantante(nombreNuevo, discoMasVendidoNuevo);           

            System.out.println("La lista actualizada contiene los siguientes datos:");
            mostrarListaCompleta(lista.getLista().iterator());
 
            System.out.println("¿Desea introducir un nuevo cantante (s/n)?");
            m= entradaS.nextLine();
           
        }

        System.out.println("Hasta luego.");

    }//Cierre de método main.
   
    private static void mostrarListaCompleta(Iterator<CantanteFamoso> it){
           
        CantanteFamoso next;
     
        while(it.hasNext()){
             next= it.next();
             System.out.print("Cantante: " + next.getNombre() + ".");
             System.out.println("       Disco con más ventas: " + next.getDiscoConMasVentas() + ".");
        }
           
    }
   
} // fin de la clase.
Analízalo y cualquier duda estamos aquí para revisarlo nuevamente.

Correcto! es una muy buena sugerencia de convenciones la que te hacen los compañeros, a tomarla en cuenta 

Hola Edu:

Estuve revisando tu código y lo veo perfecto! Por mi parte nada por corregir, veo al código muy bien estructurado, cumple al pié de la letra la consigna del ejercicio. A seguir adelante!

Hola Amateu:

Estuve revisando tu ejercicio y lo veo perfecto! Cumple con lo que pide el ejercicio.

Respecto al uso del operador '==' no es correcto usarlo con los objetos tipo String. Para comprobar igualdad de un objeto tipo String se debe usar el método 'equals' de la clase Object (a la cual pertenecen todos los objetos, incluídos los tipo String).

Ejemplo: Podemos incluír un condicional if más en tu código de la siguiente manera:

....
....
Scanner entradaEscaner = new Scanner (System.in); //Creación de un objeto Scanner
        entradaTeclado = entradaEscaner.nextLine (); //Invocamos un método sobre un objeto Scanner
       
        if(entradaTeclado.equals("java")){
            System.out.println("La palabra ingresada es 'java'");
        }
       
        if (entradaTeclado.length() < 5)
            {
            longitud = "menos de 5 ";
....
....
Lo que haría este agregado es que si la palabra ingresada es "java" nos lo indicará por consola. Si no lo fuese, no habrá ninguna indicación.

Analízalo y cualquier duda estamos aquí para reveerlo.

Hola Amateu:

Veo que es tu primer mensaje en el foro así que Bienvenido!
Estuve revisando tu ejercicio, y lo veo perfecto!

Como un agregado extra -ya que todo ha estado correctísimo- me gustaría plantearte lo siguiente a modo de "darle una vuelta más de tuerca al tema"...

Tú habías planteado en la clase Casa, el siguiente método:
public void setSalonCasa (SalonCasa valorSalonCasa)
    {
            salonCasa = valorSalonCasa;
    }
Está perfecto como está codificado, hace lo que que debe hacer.
Sin embargo quisiera mostrarte una alternativa distinta de codificación de este método:
    public void setSalonCasa(SalonCasa valorSalonCasa) {
        salonCasa.setNumeroDeTelevisores(valorSalonCasa.getNumeroDeTelevisores());
        salonCasa.setTipoSalon(valorSalonCasa.getTipoSalon());
    }
Esta forma de codificar el método setSalonCasa también realiza lo que se pide en el ejercicio.

Nos podríamos plantear la siguiente pregunta ¿ambas formas son equivalentes? ¿es indistinto codificar de una manera o de otra?... tomémonos un tiempo para pensarlo y analizarlo.

La respuesta es  que si bien ambas formas de codificación resuelven en nuestro caso particular el ejercicio que se plantea en la lección, hay diferencia entre una y otra forma.

Fíjate en lo siguiente: de la manera en que lo planteaste tu, lo que hacemos es asignarle al campo salonCasa, la referencia del objeto valorSalonCasa. Por lo tanto si se realizara cualquiera modificación al objeto valorSalonCasa, esas modificaciones se verían reflejadas cuando consultemos el objeto salonCasa (y viceversa) ¿ por qué? porque salonCasa apunta a valorSalonCasa, eso es lo que realiza la asignación: una asignación de referencias.

Ahora bien, si analizamos la segunda forma de codificar, vemos que el método lo que hace es asignar a los campos del objeto salonCasa, los valores de los campos del objeto valorSalonCasa. Pero ambos objetos siguen siendo independientes. Tanto salonCasa como valorSalonCasa referencian lugares de memorias distintos.

Todo lo dicho es aplicable también al método método setCocina.

Analízalo, y cualquier consulta estamos aqui para reveerlo.

Hola jchusmit:

Veo que es el primera post que publicas en el foro, así que me toca hacer de anfitrión y darte la bienvenida!

Como primer consejo que me gustaría darte sería sugerirte leer este hilo:
https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0
donde encontrarás la forma de insertar código en el foro de manera correcta.

Ahora bien, yendo específicamente al código que publicaste, he de decirte que está bien pensado, y si bien hay algunos errores que a continuación iremos puntualizando, has hecho un muy buen trabajo!

Veamos entonces qué correcciones habría que hacerle para mejorar el código.

1) Cómo se ha dicho en más de una oportunidad en las lecciones (y en muchos hilos del foro que recomiendo siempre leer ya que se aprende mucho de los compañeros), es una buena práctica respetar la convención que nos dice que: los nombres de los atributos y los métodos deben comenzar siempre con una letra manúscula, y los nombres de las clases siempre con una letra mayúscula.
Así pues: el atributo ´Apellido´, debería llamarse ´apellido´. El atributo ´Nombre´debería llamarse ´nombre´, etc.
Otra convención importante a respetar en este mismo sentido es la que nos dice que si bautizamos un método o un atributo con un nombre que contenga dos palabras, esa segunda palabra comenzará con una letra mayúscula. De esta forma se hace más ágil y entendible su lectura, además de darnos mayor información acerca de "qué representa" cuando se trata así de nombre compuestos.
Por lo tanto, un atributo como ´valorespe´debería llamarse ´valorEspe´, y un método como ´calmultedad´debería llamarse ´calMultEdad´, o ´setcasado´debería ser ´setCasado´, etc.

2) Siguiendo con las convenciones de un buen programador: los métodos que asignan un valor a un atributo de clase, deben bautizarse anteponiendo el prefijo ´set´. Ejemplo: ´setCasado´.
Los métodos que devuelven un valor, deben bautizarse anteponiendo el prefijo ´get´. De esta manera deberías cambiar nombres de métodos como por ejemplo el método ´Apellido´ que debería llamarse ´getApellido´, etc.

3) Fíjate en el constructor que has creado en tu código:

//constructo
public Medico2(){
Nombre = " ";
Apellido =" ";
edad = 0;
casado = false;
DI= "";
espe= "";
Aquí he de señalarte que el atributo ´espe´está muy bien inicializado. Pero fíjate que no es de la misma forma que has inicializado el atributo ´Apellido´ o el atributo ´Nombre´, donde hay un espacio entre las comillas. Ese espacio no debería existir si estamos inicializando un campo tipo String a valores por defecto.

4) La línea

if (edad <10){
debe ser reemplazada por :

if (edad <10)
para que funcione correctamentetodo el programa.


Analiza estos puntos y cualquier duda, estamos aquí para revisarlo nuevamente.

Hola JuanMontaña:

Último detalle a tener en cuenta: agregar en la signatura del método  lo que se denomina "modificador de acceso Java". Como hemos venido definiendo en el curso siempre los métodos como públicos, la signatura del método debería ser:

public String combinacion (String valor_texto1, String valor_texto2,String valor_texto3)    {
más adelante se abordará en detalle qué es esto de los modificadores de acceso Java. De momento sigamos definiendo todos nuestros métodos como públicos.

Hola JuanMontaña:

Tu código funciona correctamente. Cumple con la consigna establecida por el ejercicio, felicitaciones.

Respecto a tu consulta acerca de ¿ por qué funciona sin problemas el código sin la incorporación de la línea "import java.lang.Math;"?
Transcribo el último párrafo de la entrega Nº47 del curso donde podemos encontrar la respuesta:

Prueba ahora a eliminar la sentencia import de la primera línea y a compilar. La compilación es posible. ¿Por qué? Esto se debe a lo que comentamos relativo a que determinadas clases o paquetes se cargan automáticamente mientras que otros no se cargan a no ser que se indique específicamente. El paquete java.lang es un paquete que se carga automáticamente. Por ello podemos hacer uso de todas sus clases, como String o Math, sin necesidad de importarlo. Si escribimos la sentencia de importación no habrá mensaje de error, pero tampoco será útil ya que estamos redundando al repetir algo que hace Java automáticamente.

Hola JuanMontaña:

Estuve revisando tu ejercicio y noto que has comprendido correctamente el funcionamiento del bucle 'for'. Lo has implementado muy bien en la resolución del ejercicio planteado.
A seguir adelante!

Hola Enzo:

Perfecta la resolución del ejercicio.
A seguir adelante!