¿Es esta la forma correcta de usar una IllegalArgumentException?

Question

Estoy tratando de trabajar en una tarea de Java. Esto es lo que pregunta:

Escriba una clase llamada TestScores. El constructor de clase debe aceptar una matriz de puntajes de prueba como su argumento. La clase debe tener un método que devuelva el promedio de los puntajes de las pruebas. Si un puntaje de prueba en el conjunto es negativo o mayor que 100, la clase debe arrojar un IllegalArgumentException. Demostrar. Necesito un archivo llamado TestScores y TestScoresDemo.

Esto es lo que tengo hasta ahora. Sé que algunos de que es malo y necesito ayuda fijándola:

class TestScores { 
    public static void checkscore(int s) { 
     if (s<0) throw new IllegalArgumentException("Error: score is negative."); 
     else if (s>100) throw new IllegalArgumentException("Error Score is higher then 100"); 
     else if (s>89)throw new IllegalArgumentException("Your grade is an A"); 
     else if (s>79 && s<90)throw new IllegalArgumentException("Your grade is an B"); 
     else if (s>69 && s<80)throw new IllegalArgumentException("Your grade is an C"); 
     else if (s>59 && s<70)throw new IllegalArgumentException("Your grade is an D"); 
     else if (s<60)throw new IllegalArgumentException("Your grade is an F"); 

     { 
      int sum = 0; //all elements together 
      for (int i = 0; i < a.length; i++) 
       sum += a[i]; 
     } 
     return sum/a.length; 
    } 
} 

class TestScoresDemo { 
    public static void main(String[] args) { 
     int score = 0; 
     Scanner scanner = new Scanner(System.in); 
     System.out.print(" Enter a Grade number: "); 
     String input = scanner.nextLine(); 
     score = Integer.parseInt(input); 
     TestScores.checkscore(score); 
     System.out.print("Test score average is" + sum); 
    } 
} 

Conozco las llamadas de asignación para una declaración try porque en mi libro que es lo que veo con el IllegalArgumentException. ¿Alguien puede ayudarme? Estoy usando Eclipse como un IDE.

Answer 1

Su clase TestScores debe tener dos miembros: un constructor que acepta una matriz de puntajes y un método que devuelve el promedio de los puntajes. La asignación no es del todo clara en cuanto a cuál de estos debería arrojar un IllegalArgumentException si el puntaje de una prueba está fuera de rango, pero yo lo convertiría en el constructor (ya que ese es el argumento).

public class TestScores { 
    public TestScores(int[] scores) throws IllegalArgumentException { 
     // test the scores for validity and throw an exception if appropriate 
     // otherwise stash the scores in a field for later use 
    } 

    public float getAverageScore() { 
     // compute the average score and return it 
    } 
} 

estás en el camino correcto con su clase TestScoresDemo. Primero deberá recopilar un conjunto de puntajes en una matriz. Entonces debería construir un objeto TestScores. Esto es lo que debe estar dentro de un bloque try/catch porque puede lanzar una excepción. Entonces solo necesita llamar al getAverageScore() y hacer algo con el resultado.

Answer 2

public class TestScores { 
private final int[] scores; 

public TestScores(int[] scores) { 
    this.scores = scores; 
} 

public int getAverage() { 
    int sum = 0; 

    if(scores.length == 0) { 
     return 0; 
    } 

    for(int score: scores) { 
     if(score < 0 || score > 100) { 
      throw new IllegalArgumentException("Score is not valid!"); 
     } 
     sum += score; 
    } 
    return sum/scores.length; 
} 

}

Answer 3

Una excepción es algo que se usa para definir algo que no vaya a la derecha en el flujo normal de una aplicación. Debe lanzar IllegalArgumentException cuando se llama al método checkScore y encuentra cualquier argumento fuera del rango (entre 0 y 100).

Su clase debe tener la siguiente estructura:

clase

public class TestScore { 

    private int scores[]; //With setters and getters. 

    public TestScore(int scores[]) { 
     //Here, you set the scores array to the one on this class. 
    } 

    public int getAverage() { 
     //You do the average here, and since you need to iterate over the 
     //array to sum each value, you can check the value and if it's not 
     //ok you throw the IllegalArgumentException. No throws keyword 
     //required since this kind of exception (like NullPointerException 
     //and many others) are unchecked exceptions, meaning they can be 
     //thrown by a method and it does not need to specify them. 
    } 

} 

La prueba debe crear un objeto TestScore con una matriz int como un parámetro de su constructor. A continuación, crea un método testAverageScore que tiene la instrucción try-catch, ya que es necesario llamar al método getAverage.

Espero que ayude. ¡Buena suerte!.

EDITAR: IllegalArgumentException es una excepción no comprobada.