如何防止这些范围在 Java 中重叠？

Question

I am writing some code in java which checks through an array of 51 objects called beatArr which are circles which move from right to left towards a player positioned at x:200px. I want to make it so that the player is awarded points or deducted points based on the timing of their button presses compared to the position of a beat, and so far I have the following:

 public static void hit(){
        //Method for player to hit the beat
        for (Beat beat : Game.beatArr) {
            if (beat.getX() <= 220 && beat.getX() >=150){
                score+=100;
                System.out.println("Perfect+ "+score);
            }

            else if (beat.getX() <= 250 && beat.getX() >=100) {
                score += 100;
                System.out.println("Good "+score);
            }

            else if(beat.getX() <=99 && beat.getX()>=0 | beat.getX() >250){
                score -=50;
                System.out.println("Miss " + score);
            }
        }

This method causes several problems as the ranges overlap and so on a button press something like this usually gets outputted:

Miss 3000
Good 3100
Good 3200

Whereas I want it to only check a single position at a time really such as the one closest to the player for example or an iterator for checking the object at a given position in the array which I am not sure about the syntax for. So far I've tried changing the order of the checks to see if that affected anything which it unfortunately did not.

Testing out replacing beat.getX() with Game.beatArr[counter].getX() Where counter = 0 for now.

It seems to check the position of the 1 beat 51 times, the full length of the array for some reason. Output:

Good 100
Good 200
Good 300
Good 400
Good 500
Good 600
Good 700
Good 800
Good 900
Good 1000
Good 1100
Good 1200
Good 1300
Good 1400
Good 1500
Good 1600
Good 1700
Good 1800
Good 1900
Good 2000
Good 2100
Good 2200
Good 2300
Good 2400
Good 2500
Good 2600
Good 2700
Good 2800
Good 2900
Good 3000
Good 3100
Good 3200
Good 3300
Good 3400
Good 3500
Good 3600
Good 3700
Good 3800
Good 3900
Good 4000
Good 4100
Good 4200
Good 4300
Good 4400
Good 4500
Good 4600
Good 4700
Good 4800
Good 4900
Good 5000
Good 5100

Creation of beatArr:

 public static Beat [] beatArr = new Beat [51];

  int startingPoint = 800;
        //For loop 51 times
        for(int i=0;i<=50;i++){
            //modifier to start position to create differing gaps between beats
            int startModifier = random.nextInt(50);
            // instantiates objects into an array for each iteration of the loop
            handler.addObject(beatArr[i] = new Beat(startingPoint,300,ID.Beat));
            //redefines the starting point for each beat
            startingPoint = startingPoint+50+startModifier;

        }

Answer 1

array of 51 objects called beatArr

I believe a queue would serve your purpose much better.

    public static void hit(){
        //Method for player to hit the beat
        if (Game.beatArr.size()>0) {
            Beat beat = beatArr.remove();//use add() when inserting Beats into beatArr
            if (beat.getX() <= 220 && beat.getX() >=150) {
                score += 100;
                System.out.println("Perfect+ " + score);
            }
        
            else if (beat.getX() <= 250 && beat.getX() >=100) {
                score += 75;
                System.out.println("Good " + score);
            }
        
            else if(beat.getX() <=99 && beat.getX()>=0 || beat.getX() >250){
                score -= 50;
                System.out.println("Miss " + score);
            }
        }
        else {//No beats hence "Miss!"
            score -= 50;
            System.out.println("Miss " + score);
        }
    }

This function does the following:

1. Checks overlap of the earliest beat with the white circle.
2. Every time the user attempts to hit, the earliest circle is removed.
3. If there are no beats, it is a "Miss" .