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

Hola!

Estuve analizando tu ejercicio y lo bien, cumple con la consigna. Sin embargo me gustaría hacer unas recomendaciones para mejorar el código.

Respecto de la clase ListaCantantesFamosos la veo muy bien. Pero he de sugerirte seguir las convención de cómo nombrar en este caso los atributos. Por ejemplo:
siguiendo con la convención que ya se ha tratado en otro hilo, el nombre del atributo tipo ArrayList no deberia llamarse 'listadecantantes' sino 'listaDeCantantes'. Y el parámetro 'valor_nombre' debería llamarse 'valorNombre'.
Si bien la forma de nombrar que has elegido no dará ningún error de compilación y el código funcionará correctamente, es bueno acostumbrarse a seguir estas convenciones de programación.

Respecto de la clase testListaCantantesFamosos.
En la línea:

ListaCantantesFamosos listadecantantes = new ListaCantantesFamosos("Nombres introducidos");
se llama al constructor de la clase ListaCantantesFamosos pasando como parámetro la cadena "Nombres introducidos". Esa cadena String es lo que será el nombre de la lista. Podría tener más sentido quizás bautizar la lista como : "Cantantes que me gustan" o "Cantantes de habla inglesa". Si bien cada uno puede elegir el nombre que más desee para su lista, solo quería asegurarme que estabas conciente de que esa cadena "Nombres introducidos" sería el nombre de la lista.

Al bucle:
while(i <5){
        System.out.println ("El cantante "+ (i+1) + " es " + listadecantantes.getNombre(i));
        i++;
           }
no lo veo bien. Si vas a utilizar un contador, sería más lógico usar un bucle for antes que un while, de la siguiente manera:

for (int = 0; i<5;i++){
System.out.println ("El cantante "+ (i+1) + " es " + listadecantantes.getNombre(i));

}

Hola!

Correctas tus dos primeras respuestas.
Respecto a la tercera respuesta verás, el método pop se puede considerar tipo mixto debido a que este método puede realizar tanto modificaciones como consultas en la pila.

Saludos.

Hola Adrián:

Está muy bien tu ejercicio, felicitaciones! Muy bien usada la clase Random.

Ten en cuenta como creo lo hemos conversado en otro hilo, en programación tratamos de evitar el uso de la letra 'ñ' para nombrar atributos, clases o métodos. Así, palabras como 'año' las escribimos como 'anno' o 'anio'.

Saludos, a seguir adelante!

Hola Adrián:

Veo muy bien tu código, cumple con la consigna del ejercicio.
Quisiera hacerte solamente dos observaciones:

1) En la clase Test, haría una modificación para no dejar el 'else if" vacío.

public class Test {

    static ListaCantantesFamosos lista;

    public static void main (String[] args) {

        lista = new ListaCantantesFamosos();
        lista.addCantante(new CantanteFamoso ("Madonna","All I want is you"));
        lista.addCantante(new CantanteFamoso ("Jorge Negrete","Jalisco"));
        System.out.println("\fMostrando lista actual\n");
        lista.getLista();
        opcionCantante();
    }

    public static void opcionCantante () { //metodo que nos pregunta por mas cantantes

        EntradaDeTeclado respuestaUsuario, nuevaEntrada;
        CantanteFamoso cantante;

        do {
            System.out.println("\nDesea ingresar otro cantante? Y/N");
            respuestaUsuario = new EntradaDeTeclado();
            //if que contempla respuesta "Y", respuesta "N" o cualquier otra respuesta
            if(respuestaUsuario.getEntrada().equalsIgnoreCase("y")) {
                cantante = new CantanteFamoso();
                System.out.println("\nIngresa un nuevo cantante");
                nuevaEntrada = new EntradaDeTeclado();
                cantante.setCantante(nuevaEntrada.getEntrada());
                System.out.println("Ingresa el disco mas vendido de "+nuevaEntrada.getEntrada());
                nuevaEntrada = new EntradaDeTeclado();
                cantante.setDisco(nuevaEntrada.getEntrada());
                lista.addCantante(cantante);
                System.out.println("\nMostrando lista actualizada\n");
                lista.getLista();
            } else if (!respuestaUsuario.getEntrada().equalsIgnoreCase("n")) {
                System.out.println("Por favor ingresa Y o N");
              }
        } while (!respuestaUsuario.getEntrada().equalsIgnoreCase("n"));
        System.out.println("\nMostrando lista final\n");
        lista.getLista();
        System.out.println("\nPrograma finalizado");
    }
}
2) En la clase ListaCantantesFamosos renombraría la clase getLista() por un nombre más acorde, como podría ser por ejemplo mostrarLista() ya que lo que el método hace es "mostrar la lista", y no "devolver la lista" que es lo que se interpretaría con getLista().

A seguir adelante!

Hola Adrián:

Hago mea culpa: se me pasó por alto que estaba declarado como campo de instancia, en un golpe de vista no lo ví y creí que nunca lo habías hecho. Perdón, creo que necesito cambiar los cristales de mis anteojos jejeje 

Es absolutamente correcto tal como lo hiciste declarándolo como campo de instancia ya que es un atributo utilizado por más de un método. Luego si, inicializarlo en el Construcor.

public class EntradaDeTeclado {
    private String entradaTeclado;

    public EntradaDeTeclado ()  { //Constructor
        entradaTeclado="";
        pedirEntrada(); //Llamada al método interno
    } //Cierre del constructor
A seguir adelante entonces!

Hola Adrián:

1) Cuando pensamos en crear una clase hay que preguntarse "qué es" o "para qué servirá" esa clase. Acto seguido consideraremos qué atributos serán necesarios  (sus propiedades) y con qué métodos se regirá el comportamiento de los objetos que pertenezcan a esta clase.
 
Supongamos la clase Persona con atributos: nombre, apellido, género, edad, altura, peso, y profesión. Y sus métodos: establecer y retornar cada atributos, y mostrar por consola dichos atributos.
No sería correcto establecer un método dentro de la clase Persona que devuelva un índice con fines estadísticos donde se relacione el género, con la edad, el peso, y altura (por más que todos sean atributos del objeto tipo Persona).
No podemos delagar a la clase (Persona) lo que parece ser responsabilidades de otra clase.

Lo que se debería hacer en ese caso -estamos suponiendo- es crear una nueva clase llamada por ejemplo CalculadorEstadisticoDePoblacion que contenga un atributo ArrayList de objetos tipo Persona, y donde uno de sus métodos sea justamente ese método que devuelve un índice que relaciona el género, la edad, el peso, y la altura de cada objeto del ArrayList.

Entonces, volviendo a nuestro caso de interés que es la clase EntradaDeTeclado: si el objeto es tipo EntradaDeTeclado, el atributo entradaTeclado debe ser establecido únicamente por lo que tipeamos por teclado. No podemos establecer el atributo entradaTeclado ni con un parámetro recibido, ni tomando un valor leído en un archivo, ni por el valor leído en un puerto, etc etc. Debe existir una correlación entre el nombre de la clase y lo que ella representa, debe existir una coherencia entre qué es, y para qué sirve.
No debemos responsabilizar a esta clase, con responsabilidades que seguramente son atribuibles a otra clase.

Lo de la reutilización del código está perfecto! Esa es la idea: crear clases bien diseñadas que luego las podamos volver a utilizar en otros programas.
Una clase que gestione las entradas por teclado siempre será útil y te ahorrará tiempo de programación.
Por eso quiero que se entienda bien: la observación no está relacionada por el hecho de que no se utilicen todos los métodos de la clase, sino que apuntaba a lo conceptual de qué responsabilidades tiene una clase.
Si tu clase EntradaDeTeclado tuviera también métodos para leer y devolver no solo objetos tipo String, sino otros métodos para leer y devolver objetos tipo Long, tipo Float, tipo Integer, etc, estaría muy bien! por más que no se utilizaran siempre.

2) Codificaste el Constructor de la clase de la siguiente manera:

public EntradaDeTeclado ()  {
        entradaTeclado="";
        pedirEntrada(); //llamada al metodo interno
    }
a la línea :

entradaTeclado="";
sería mas correcto escribirla:

private String entradaTeclado="";
donde queda establecido el ámbito 'private', y establecido el tipo 'String' del atributo entradaTeclado.

Hola Adrián:

Te olvidaste de subir la clase EntradaDeTeclado. Publícala así puedo ver el código completo.

Hola Luis:

Veo perfecto tu ejercicio! A seguir adelante!
Saludos.

Hola!


Estuve revisando tu ejercicio y está muy bien. Has usado herramientas que aún no se han visto en el curso como el tema del casting (lección nº89).

Voy hacerte algunas sugerencias para tener en cuenta. Pero insisto, tu ejercicio esta muy bien y cumple con la consigna del ejercicio.

Dentro del bucle do while, utilizas el condicional if con su respectivo else. Dentro del else vuelves a utilizar un condicional if:

do{
            entradaTeclado.setEntrada();
            numeroAux=numeroAux.valueOf(entradaTeclado.getEntrada());
            if((int)numeroAux>numeroAleatorio){
                System.out.print("No has acertado: el número es menor. Prueba otra vez : ");
            }else{
                if((int)numeroAux<numeroAleatorio){
                    System.out.print("No has acertado: el número es mayor. Prueba otra vez : ");
                }
            }
        }while((int)numeroAux!=numeroAleatorio);
Podríamos resolverle directamente con un condicional if y un else if, de la siguiente manera:

do{
            entradaTeclado.setEntrada();
            numeroAux=numeroAux.valueOf(entradaTeclado.getEntrada());
            if((int)numeroAux>numeroAleatorio){
                System.out.print("No has acertado: el número es menor. Prueba otra vez : ");
            }else if((int)numeroAux<numeroAleatorio){
                     System.out.print("No has acertado: el número es mayor. Prueba otra vez : ");
            }
        }while((int)numeroAux!=numeroAleatorio);

Respecto a la clase EntradaTeclado. Hemos conversado un poco sobre ella en una publicación anterior

https://www.aprenderaprogramar.com/foros/index.php?topic=2880.0

y mi sugerencia va nuevamente en esa misma dirección. ¿Y cuál es la nueva razón para sostener esto? Pues que en POO siempre pensamos en la modularidad y la reutilización del código como una de nuestras premisas.
Entonces, esta vez que necesitamos que la entrada por teclado sea un tipo int en vez de un tipo String, bien podríamos agregar un nuevo método a la clase GestionadorEntradasTeclado para que cumpla esa necesidad. Definiríamos un nuevo método en esa clase que se llame recogerInt.
De esta manera con un solo objeto gestionamos todo lo que recibimos por teclado :

import java.util.Scanner;

public class GestionadorEntradasTeclado{

    /**Devuelve un objeto tipo String que corrsponde al valor ingresado por el usuario a través del teclado.
     * @return retorna un objeto tipo String.
     */
     public String recogerString(){
        Scanner scannerTeclado = new Scanner (System.in);
        String valorTeclado = scannerTeclado.nextLine();
        return valorTeclado;
    }
   
    /**Devuelve un valor tipo int que corresponde al valor ingresado por el usuario a través del teclado.
    * @return retorna un valor tipo int.
    */
    public int recogerInt(){
       Scanner scannerTeclado = new Scanner (System.in);
       int valorTeclado = scannerTeclado.nextInt();
       String valorTmp = scannerTeclado.nextLine(); // para vaciar el buffer.
       return valorTeclado;
    }
     
} // Fin de la clase
Igualmente he de decirte que has utilizado muy bien en tu código el método valueOf para convertir el tipo String en tipo Integer.

Saludos.

Hola Luis:

Esta muy bien tu ejercicio. Tendrías que agregar los métodos set para cada uno de los campos de instancia.
Además en el Constructor faltó recibir el parámetro valorCasado, así al campo casado (tipo boolean) no se le tiene que asignar el valor por defecto false.