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

Buenas agustin, para escribir en los foros es importante leer https://aprenderaprogramar.com/foros/index.php?topic=1460.0

No uses este foro para consultas porque sólo está destinado a lo que se comenta en https://aprenderaprogramar.com/foros/index.php?topic=17.0

Hay un hilo donde se comenta un problema similar al tuyo en la url https://www.aprenderaprogramar.com/foros/index.php?topic=486.0

Salu2

Buenas Sosender, lee por favor https://aprenderaprogramar.com/foros/index.php?topic=1460.0 donde hay indicaciones sobre cómo poner título a los temas, insertar código, etc. Ahí podrás ver que este título que has puesto no es adecuado (procederé a corregirlo yo mismo). Tenlo por favor en cuenta para próximos temas que abras.

El esquema de llaves está bien construido. Si se quisiera, podríamos aumentar el nivel de detalle. Por ejemplo dentro del primer texto en el contenido de la página, podríamos indicar que hay un link (etiqueta a) abriendo unas llaves a partir de este elemento; dentro del tercer párrafo p podríamos indicar que hay una imagen abriendo una llave y así sucesivamente.

El esquema de cajas me parece perfecto.

Salu2!

Buenas, para quien revise este ejercicio, puede verse una explicación a los resultados que se obtienen en https://aprenderaprogramar.com/foros/index.php?topic=3969.0

Salu2

Buenas elendur14, por favor lee https://aprenderaprogramar.com/foros/index.php?topic=1460.0 y sigue las recomendaciones que ahí se dan a la hora de poner título a los temas. Incluye varias palabras clave que correspondan hasta alcanzar el espacio disponible para el título (es importante incluir el nombre del lenguaje y más palabras clave; en este título te ha faltado incluir Java) y pon el código de los ejercicios (por ejemplo CU00667B) al final del título del tema, no al principio.

Algunos comentarios. En el constructor de ListaCantantesFamosos en lugar de listaCantantesFamosos = new ArrayList<>(); deberías haber escrito listaCantantesFamosos = new ArrayList<CantanteFamoso>(); por ser preferible siempre especificar el tipo de objeto que contiene la lista.

La inicialización de Strings por convenio se hace a cadena vacía, no a un espacio en blanco. Es decir, en lugar de
nombreCantante = " ";
discoConMasVentas = " ";

Debe ser (para atenernos al convenio):
nombreCantante = "";
discoConMasVentas = "";

Ten en cuenta que aunque parezca lo mismo, no es lo mismo una cadena vacía (longitud 0) que un espacio en blanco (longitud 1).

Respecto a si es recomendable hacer esto: tempCantante = it.next(); la respuesta es que sí, puedes usarlo siempre que te resulte útil y lo hagas de una forma coherente, como en este caso.

Por otro lado, tu código está muy bien. Lo único que te recomendaría es que como ejercicio pruebes a hacerlo más compacto de la siguiente manera: en la clase con el main, en lugar de crear los dos objetos y luego usar los métodos set, prueba a hacerlo directamente usando el constructor.
Y en el bucle prueba a en lugar de crear el cantante y hacer los set para luego añadir a la lista, hacerlo todo directamente en el lista1.addCantanteFamoso

Finalmente animarte a que nos ayudes a responder en los foros. Revisando ejercicios de compañeros adquirirás destreza en la revisión de código y serás cada vez más rápido en su análisis, aparte de que nos ayudarás.

Salu2

Buenas, para quien revise este ejercicio, cumple con lo que se pedía y funciona correctamente.

Hay un detalle que no veo correcto. Se incluye un método

    public String setEntrada() {
        entradaTeclado = sc.nextLine();
        return entradaTeclado;
    }

Los métodos set, por convención, no llevan return. Por tanto este nombre de método no es correcto de acuerdo con la convención que se suele seguir en programación Java.

De resto todo bien.

Salu2

Buenas, por favor mirar este hilo donde se explica cómo poner título a los temas en el foro: https://aprenderaprogramar.com/foros/index.php?topic=1460.0 es importante que pongas un título como se indica. Este lo corregiré yo.

La cosa como bien dices se va complicando un poco, pero sólo un poco. Creo que lo más importante es que avances comprendiendo lo que vas haciendo, aunque vayas un poco más lento. A veces asimilar las cosas requiere un poco de tiempo. Y una vez vas asentando los conceptos la complejidad se va diluyendo como un azucarillo en el agua porque cada vez tienes más experiencia...

El ejercicio está bien resuelto, y me gusta cómo has planteado el código pues aunque podría haberse hecho un poco más compacto, la lógica está muy bien autodescrita por el propio código y los nombres de variables. Animo. Salu2

Buenas, para quien revise este ejercicio decir que está bien resuelto y es un buen ejemplo de herencia en JavaScript con prototype. El código anterior hay que montarlo en archivos para que funcione. Para que resulte más fácil de probar lo dejo aquí todo unido para que pueda probarse con un solo archivo:

<!DOCTYPE html>
<html lang="es">
<head>
<meta charset="UTF-8">
<title>Ejercicios (CU01147E) </title>
<style type="text/css">
body{
background: green;
font-family: Arial, Helvetica, sans-serif;
}
button{
margin: 100px;
background-color:blue;color:white;width:120px;
border-radius: 5px;
}
</style>
<script type="text/javascript">
function cometa(diametro, temperatura, nombre){
this.diametro = diametro;
this.temperatura = temperatura;
this.nombre = nombre;
this.radio=0;
this.farenheit=0;
}

function prototipoCometa(){
this.definicionSegunDiccionario = 'Astro formado por un núcleo poco denso y una cola o cabellera luminosa, que describe una órbita muy excéntrica, visible cuando se aproxima al Sol';
this.obtenerRadio = function (){
this.radio = this.diametro/2;
};
this.obtenertemperaturaFarenheit = function(){
this.farenheit = (this.temperatura * 9/5) + 32;
};
}
cometa.prototype = new prototipoCometa();
function cargarCometa(){
var cometa1 = new cometa(50000,32,'Halley');
var cometa2 = new cometa(30000,22,'Shoemaker-Levy 9');
var cometa3 = new cometa(10000,12,'Hale-Bopp');
cometa1.obtenerRadio();
cometa1.obtenertemperaturaFarenheit();
alert('La definición segun diccionario es '+cometa1.definicionSegunDiccionario+' El cometa1 su nombre es '+cometa1.nombre+' tiene un diametro de '+cometa1.diametro+
' con una temperatura de '+cometa1.temperatura+'°C'+' con un radio de '+ cometa1.radio+' y temperatura farenheit '+cometa1.farenheit+' °F');
cometa2.obtenerRadio();
cometa2.obtenertemperaturaFarenheit();
alert('La definición segun diccionario es '+cometa2.definicionSegunDiccionario+' El cometa2 su nombre es '+cometa2.nombre+' tiene un diametro de '+cometa2.diametro+
' con una temperatura de '+cometa2.temperatura+'°C'+' con un radio de '+ cometa2.radio+' y temperatura farenheit '+cometa2.farenheit+' °F');
cometa3.obtenerRadio();
cometa3.obtenertemperaturaFarenheit();
alert('La definición segun diccionario es '+cometa3.definicionSegunDiccionario+' El cometa3 su nombre es '+cometa3.nombre+' tiene un diametro de '+cometa3.diametro+
' con una temperatura de '+cometa3.temperatura+'°C'+' con un radio de '+ cometa3.radio+' y temperatura farenheit '+cometa3.farenheit+' °F');

}
</script>
</head>
<body>
<section>
<div>
<button onclick="cargarCometa();">Cargar valor de las cometas</button>
</div>
</section>
<script type="text/javascript" src="js/funciones47.js"></script>
</body>
</html>


Buenas, añado algunas cuestiones: tienes mal el planteamiento del constructor de la clase Casa, ya que no estás incluyendo los atributos SalonCasa y CocinaCasa. Todos los atributos son atributos, independientemente de que sean tipos primitivos u objeto. También tienes mal los métodos set y get de la clase Casa. En este hilo tienes una explicación (verla con calma): https://aprenderaprogramar.com/foros/index.php?topic=1220.0

Nota: siguiendo a https://aprenderaprogramar.com/foros/index.php?topic=1460.0 cuando pongas título a un tema por el código (p.ej. CU00641B) al final del mensaje

Salu2

Buenas, para quien revise este hilo, overrides alude a sobreescribir un método para que no haya conflicto de nombres cuando se trata de un método que existe definido en una superclase.

Ejemplo superclase donde se define el método GetInfo()

' Superclase o clase base
Public Class Users
    Public Overridable Sub GetInfo()
        Console.WriteLine("Base Class")
    End Sub
End Class


Ejemplo subclase donde se define el método GetInfo() sobreescrito

'Subclase o clase derivada

Public Class Details
    Inherits Users
    Public Overrides Sub GetInfo()
        Console.WriteLine("Derived Class")
    End Sub
End Class

Si no lo declaramos overrides nos salta un aviso para indicarnos que puede haber un conflicto de nombres.

Buenas, para quien revise este ejercicio, está correcto. Salu2