tarea java

Iniciado por valera22, 23 Enero 2016, 01:08 AM

0 Miembros y 1 Visitante están viendo este tema.

valera22

Buenas tardes!!
Necesito que me echeis un vistazo a una tarea entregada y que está calificada con un 3 por el profesor....más que nada por ver si a vuestro juicio la veis tan mal...no creo que sea de 10..pero al menos un 5..


https://www.dropbox.com/s/241c2emon6rum0o/PRO_Tarea.rar?dl=0

saludos

0xFer

#1
Lo que veo a simple vista es que tu código no está indentado, por lo tanto es difícil de leer, na mentira... por suerte no es un programa grande porque de lo contrario yo no sería capas de leerlo.

Código (java) [Seleccionar]
UsoParquimetro UsoParquimetro = new UsoParquimetro();

Los objetos que crees no deben tener el mismo nombre que su clase.

Código (java) [Seleccionar]
public  void excesoImporte(){
     
   try {

   }catch (Exception ex){}

    System.err.println("ERROR: Tiempo máximo sobrepasado. Recoja monedas.");
 }


¿Qué hace ese try y catch allí? Lo pones en todos los métodos pero, ¿Realmente son necesarios?,¿Cuando ocurre un Exception por qué no haces que se imprima el error? Sólo imprimes mensajes del tipo "Error en el sistema",pero eso no dice nada.

Código (java) [Seleccionar]
private  int saldoMax=180;

usa la palabra reservada final para declarar constantes.

Código (java) [Seleccionar]
Scanner teclado = new Scanner (System.in);

¿Qué tipo de acceso tiene ese objeto?

Hay otros detalles pero creo que te puso esa calificación porque tu código no está indentado, es más, creo que ni siquiera lo intentó leer por ese motivo, la indentación es algo muy muy básico. Tómate tu tiempo y acomoda tu código no sólo es para verse más bonito.

Pídele a tu maestro que te diga el porqué de esa calificación, para que sepas qué cosas es lo que él toma en cuenta...

Saludos.
 
Código (java) [Seleccionar]
int getRandomNumber(){
    return 4; //chosen by fair dice roll
              //guaranteed to be random
}

Uzumaki_Naruto

En la clase parquimetro, el if debería mostrar error al  (saldo > saldoMax) ya que se permite tener de saldo 1.8€.
Código (java) [Seleccionar]
// si saldo es igual o mayor al saldo maximo saltamos el exceso de importe
            if (saldo > saldoMax) {...}

* En el método excesoImporte muestras el mensaje de error fuera del catch.
* No te piden mostrar la hora actual en el reporte (algunas veces cuando te dan una captura de la salida deseada es por que lo quieren igual, no sé si este sea el caso).
* Las clases siempre con la primera letra mayúscula.
Lo de la identación se soluciona facilmente al estar en netbeans (Alt + shift + F  o anticlick y Format), saludos.