Простой алгоритм оценки игры

Вам не нужна часть "successPercent < 100" каждый раз, потому что вы переходите к следующему else, только если текущее if неверно.

   if (successPercent == 100)
        successMessage = "OUTSTANDING";
    else if (successPercent > 80)
        successMessage = "EXCELENT";
    else if (successPercent > 60)
        successMessage = "VERY GOOD";
    else if (successPercent > 40)
        successMessage = "GOOD";
    else if (successPercent > 10)
        successMessage = "AVERAGE";
    else if (successPercent!=0 && successPercent < 10)
        successMessage = "BAD";
    else if (successPercent == 0)
        successMessage = "TERRIBLE";

Я бы создал перечисление со всеми возможными оценками. Затем вы можете вызвать статический метод getGameScore, который просматривает все игровые результаты, пока не найдет тот, который соответствует заданному игровому счету. Это позволяет избежать большого количества повторяющегося кода и всех остальных, если они не облегчают чтение кода.

В зависимости от того, что вы хотите делать с оценками, которые не попадают в правильный диапазон, вам нужно будет сделать что-то еще, чем я написал, в настоящее время он возвращает нулевую ссылку, если вы вводите оценку, например -1 или 101.

/**
 * Enum representing all possible game scores.
 * Each game score has a lower limit and upper limit.
 */
public enum GameScore {
  OUTSTANCING(100, 100),
  EXCELLENT(80, 99),
  VERY_GOOD(60, 79),
  GOOD(40, 59),
  AVERAGE(10, 39),
  BAD(1, 9),
  TERRIBLE(0, 0)
  ;

  // The max of the range for this GameScore
  private final Integer min;
  // The min of the range for this GameScore
  private final Integer max;

  /**
   * Constructs a gameScore type by specifying the range.
   * @param min
   * The minimum of the range.
   * @param max
   * The maximum of the range.
   */
  private GameScore(final Integer min, final Integer max) {
    this.min = min;
    this.max = max;
  }

  /**
   * Searches a game score based on a given score.
   * @param score
   * The score for which a GameScore is searched for.
   * @return
   * Returns the first GameScore that matches the given score. This means that
   * the given score falls in the range of the GameScore.
   * Will return null if no matching GameScore is found.
   */
  public static GameScore getGameScore(final Integer score) {
    for(GameScore gameScore : values()) {
      if(score >= min && score <= max) {
        return gameScore;
      }
    }
    return null;
  }

Вы можете попробовать этот способ

    if (successPercent == 100)
        successMessage = "OUTSTANDING";
    else if (successPercent > 80)
        successMessage = "EXCELENT";
    else if (successPercent > 60)
        successMessage = "VERY GOOD";
    else if (successPercent > 40)
        successMessage = "GOOD";
    else if (successPercent > 10)
        successMessage = "AVERAGE";
    else if (successPercent!=0&&successPercent <= 10) 
   //needs both condition since successPercent==0 is also
                                     valid successPercent <=10
        successMessage = "BAD";
    else if (successPercent == 0)
        successMessage = "TERRIBLE";

Вы можете использовать хэш-таблицу:

Объявите переменную класса:

Map<Integer,String> map;

Инициализируйте его один раз:

map = new HashMap<Integer,String>();
map.put( 0,"TERRIBLE");
map.put( 1,"BAD");
map.put( 2,"AVERAGE");
map.put( 3,"AVERAGE");
map.put( 4,"GOOD");
map.put( 5,"GOOD");
map.put( 6,"VERY GOOD");
map.put( 7,"VERY GOOD");
map.put( 8,"EXCELLENT");
map.put( 9,"EXCELLENT");
map.put(10,"OUTSTANDING");

Получите строку счета, когда это необходимо:

successMessage = map.get((successPercent-10)/10+1);