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

Buenas, quizás te sea útil consultar el curso de JavaScript en https://www.aprenderaprogramar.com/index.php?option=com_content&view=category&id=78&Itemid=206

Faltaría que concretaras cuál es la consulta o dónde encuentras el error o problema, ver https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0 de cara a plantear consultas

Salu2

Buenas, veo el código que has creado bastante interesante, aunque no sea exactamente lo que pedía el ejercicio. El ejercicio te pedía que la primera columna fuera lunes y en tu código sale el domingo como primera columna.

Otra pequeña mejora que habría que hacer es que en la casilla de fecha aparece AA-MM-DD cuando realmente lo que se espera que se introduzca es AAAA-MM-DD

Al probar el código me he encontrado con que si introduzco como fecha 2017-10-21 en el calendario me aparece marcado el 2017-10-22 (es decir, el 22 de octubre), es decir, que parece que se desfasa un día

Corrigiendo esos detalles creo que quedaría todo perfecto

Salu2

Buenas, el ejercicio pedía

d) Usando CSS establece como color de fondo para los párrafos dentro de elementos cuyo atributo class sea <<secundario>> el amarillo (yellow).

Tú has establecido color de fondo amarillo para párrafos dentro de clase principal en lugar de secundario, eso sería lo que habría que corregir.

Salu2

Buenas, código correcto, ejercicio bien resuelto. Recomendamos que <meta charset="utf-8"> se escriba con el cierre /> para cumplir el estándar xhtml, es decir, sería:

<meta charset="utf-8"/>

Salu2

Buenas PatAs el ejercicio 1 funciona correctamente y cumple con lo que se pedía en el ejercicio, posiblemente yo no hubiera hecho el diseño de clases igual, pero se ve bien trabajado y que has estructurado el código en distintas clases con responsabilidades divididas

El ejercicio 2 también funciona bien. Una posible mejora sería evitar que el usuario pueda elegir dos números repetidos, por ejemplo ahora permite elegir 1, 22, 33, 23, 34, 33 donde como se ve el 33 está repetido, pero esto no debería ser legal (ya que se han de elegir 6 números distintos).

Salu2

Buenas david82, ejercicio correcto

Salu2

Buenas, el código se ve bien planteado, bien organizadas las clases y métodos, etc.

Algunas observaciones: un método get se usa para recuperar algo (con un return), por eso este método no estaría bien nombrado

public void getNumerosSorteo(){
for(i=0;i<numerosSorteo.length;i++)
System.out.print(numerosSorteo[i]+" ");
System.out.println();
}

Este método sería preferible que se nombrara como mostrarNumerosSorteo o de forma similar. get y set son palabras reservadas para recuperar y establecer (normalmente atributos de la clase). No deben usarse para otra cosa estos nombres porque contravienen los convenios que usan todos los programadores.

Otra cosa a cambiar sería en System.out.println("Introduzca 6 números del 1 al 50 sin repetir ninguno: "); debe decir del 1 al 49 en lugar del 1 al 50

Esto hace que para ajustarse a lo que se pedía habría que corregir otras partes del código por ejemplo while(apuesta<1 || apuesta>50)

Con tombola.nextInt(49)+1; si no me equivoco estás generando números del 1 al 49 (ya que serían entre 0 y 48, y al sumarle 1 te queda entre 1 y 49) con lo cual no cuadra con que se permita elegir el 50

Salu2

Buenas, el ejercicio está bien resuelto, usando una de las estrategias planteadas en el curso. Otra forma de resolverlo sería usando la clase BigDecimal, como se hace en este hilo: https://www.aprenderaprogramar.com/foros/index.php?topic=5672.0

Salu2

Buenas, hago los mismos comentarios que para un ejercicio de otro compañero que acabo de revisar

El programa funciona correctamente pero creo que el planteamiento podría mejorarse.

Ahora mismo estás almacenando en el mismo array el nombre de los países y de las ciudades. Eso sería como tener en una misma caja peras y manzanas, cuando quizás es preferible que ambas cosas estén por separado.

Por ello una posible propuesta de solución alternativa se basaría en lo siguiente:

- Declarar un array pais[4] que almacene los cuatro países. De esta forma este array almacena solo ciudades.

- Declarar un array ciudad[4][3] donde almacene para cada país las tres ciudades correspondientes. De esta forma este array almacena sólo ciudades (para saber el país usaríamos el primer índice, y para saber la ciudad el segundo índice)

El ejercicio no pide que países y ciudades se almacenen en el mismo array, sólo pide que las ciudades se almacenen en un array multidimensional cuyo primer índice vaya asociado al número de país y el segundo índice vaya asociado al número de ciudad.

Salu2

Buenas PatAs. Ejercicio bien resuelto

Salu2