Solved Scoreboard Null Pointer

I'm testing things with scoreboard and when I added a point every time a player broke a block It works fine but when I tried adding a point when a projectile hits something It doesn't work It give me a NPE and I'm not sure how to fix it

Code:

Code:java

    Scoreboard board;
    Score score;
 
    public void onEnable() {
        makeScoreboard();
    }
 
    public void makeScoreboard() {
        ScoreboardManager manager = Bukkit.getScoreboardManager();
        board = manager.getNewScoreboard();
 
        Objective objective = board.registerNewObjective("Test", "Test2");
        objective.setDisplayName(ChatColor.AQUA + "Points");
        objective.setDisplaySlot(DisplaySlot.SIDEBAR);
 
        score = objective.getScore(Bukkit.getOfflinePlayer(ChatColor.GOLD + "Points"));
    }
 
    public void onJoin(PlayerJoinEvent e) {
        e.getPlayer().setScoreboard(board);
    }
 
    @SuppressWarnings("deprecation")
    @EventHandler
    public void onShoot(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        if (e.getAction() != Action.RIGHT_CLICK_AIR) {
            return;
        } else if (p.getItemInHand().getType() == Material.IRON_AXE) {
                p.throwSnowball();
            }
    }
 
    @SuppressWarnings("deprecation")
    @EventHandler
    public void onHit(ProjectileHitEvent event) {
        BlockIterator iterator = new BlockIterator(event.getEntity().getWorld(), event.getEntity().getLocation().toVector(), event.getEntity().getVelocity().normalize(), 0.0D, 4);
        Block hitBlock = null;
        while (iterator.hasNext()) {
            hitBlock = iterator.next();
            if (hitBlock.getType() != null) {
                Location breakBlocks = hitBlock.getLocation();
                breakBlocks.getWorld().playEffect(hitBlock.getLocation(), Effect.STEP_SOUND, hitBlock.getTypeId());
                breakBlocks.getBlock().setType(Material.AIR);
                score.setScore(score.getScore() + 1);
            }
        }
    }

Line I'm getting the NPE:

Code:java

score.setScore(score.getScore() + 1);

 

Buizelfan2 which means score is null. onEnable isn't being called, and I assume the cause for that is because this isn't your main class. In order to get it working, call onEnable after registering the Listener.

 

Okay now the scoreboard works but now it's not adding one it's adding anywhere from 4-6
Edit:
I tested some things out to see what the problem is.
When I hit a block that is not wool I get this
Not Wool
But when I hit a block that is wool I get this
Hit Wool
So It's like I'm hitting more then one block but I'm not sure what could be causing that
Code:

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler
    public void onHit(ProjectileHitEvent event) {
        Player p = (Player) event.getEntity().getShooter();
        BlockIterator iterator = new BlockIterator(event.getEntity().getWorld(), event.getEntity().getLocation().toVector(), event.getEntity().getVelocity().normalize(), 0.0D, 4);
        Block hitBlock = null;
        while (iterator.hasNext()) {
            hitBlock = iterator.next();
            if (hitBlock.getType() != null) {
                if (hitBlock.getType() == Material.WOOL) {
                    p.sendMessage("Hit");
                } else {
                    p.sendMessage("Not Wool");
                }
            }
        }
    }

 

Buizelfan2 you loop through all blocks that could be your target within 4 distance. If you just want to hit the first wool block you encounter, add a break; statement after p.sendMessage("hit")

 

Now it comes up with
"Not Wool"
"Hit"

 

Buizelfan2 which is what you want, no? If you want it to ignore AIR, you should check for getType() != Material.AIR instead of getType() != null ;3

 

What ever kind of block I hit I get this
Block Hit
Code:

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler
    public void onHit(ProjectileHitEvent event) {
        Player p = (Player) event.getEntity().getShooter();
        BlockIterator iterator = new BlockIterator(event.getEntity().getWorld(), event.getEntity().getLocation().toVector(), event.getEntity().getVelocity().normalize(), 0.0D, 4);
        Block hitBlock = null;
        while (iterator.hasNext()) {
            hitBlock = iterator.next();
            if (hitBlock.getType() == Material.AIR) {
                if (hitBlock.getType() == Material.WOOL) {
                    p.sendMessage("Hit");
                    break;
                } else {
                    p.sendMessage("Not Wool");
                    break;
                }
            }
        }
    }

 

Buizelfan2 Just noticed a typo in my post. It is supposed to be != Material.AIR

 

Okay it works now thanks for the help