NullPointerException : Cannot assign field "x" because "this.activeP" is null

[near flicker]

Hello, so I was following a youtube chess game tutorial and I got stuck with this problem (on the first screenshot). I did everything on the video (https://www.youtube.com/watch?v=jzCxywhTAUI&t=1783s) yet I cant solve the issue, I tried giving activeP parameters by creating a new piece object like " Piece activeP = new Piece(WHITE,1,2);". The error disappears by doing so but I still can't move the chess piece. It feels like the issue is easy to solve but I just can't find the solution. I am not very good with Java so if anyone can help me with it I would very happy

[final torrent]

<@&987246399047479336> please have a look, thanks.

[steady cairn]

try to debug your code
add breakpoint at the line before the exception happened then from your IDE menu click debug
then keep move on step by step till you get the cause of the issue

from the error msg it say that at line 133 you tried to get access x field of activep which currently is null

[near flicker]

I debugged it and it says that activeP = null and I know it's the issue but I dont know why activeP is null, i feel like it should try to get the position of my mouse but nothing happens when I try to grab a chess piece

[steady cairn]

i think the else part that call simulate should be inside the if(mouse.pressed)
i.e it should be else for the if(active==null)

@near flicker it just a guess as i do not know the full code and how things run

[near flicker]

I followed what the guy showed from 24:36 to 31:00 on the video if you wanna take a look

[near flicker]

As for this, I didn't understand what you tried to explain sorry, are you suggesting to move simulate() inside if(mouse.pressed) instead of putting in else ?

[steady cairn]

share the code in the image so i can modify it and resend it to you

and it's ok do not be sorry

@near flicker check 29:14 of the video and you will see the issue that i mentioned

[near flicker]

    public void update(){
        //Mouse button pressed
        if(mouse.pressed){
            if(activeP == null){
                //If the activeP is null, check if you can pick up a piece
                for(Piece piece : simPieces){
                    //If the mouse is on an ally piece, pick it up as the activeP
                    if(piece.color == currentColor && piece.col == mouse.x/Board.SQUARE_SIZE &&
                            piece.row == mouse.y/Board.SQUARE_SIZE){
                        activeP = piece;
                    }
                }
            }
        }
        else{
            //If the player is holding a piece, simulate the move
            simulate();
        }
    }

[final torrent]

Detected code, here are some useful tools:

[near flicker]

Of course! let me check quickly

@steady cairn Thank you so much, i understood what you tried to say

[near flicker]

The issue was this, I can't believe I didnt see it lol

I put the else statement at the end of the wrong if statement, very epic mistake

Really thank you a lot!

[steady cairn]

@near flicker you very welcome and it's ok we all fail for those kind of issue
that why debug and sometime different eye would help
and happy coding