android NullPointerException on checking null value

Question

I am getting some data from server and setting the values in a getter and setter class. When the values seems to be null i am getting following values in my getter setter class

Tag [seatNo=null, Value=null, player1=null, player2=null, player3=null, player4=null, player5=null ]

The above value is been printed in Logcat

after getting above values i am checking an if condition because of which the app crashes

if (bet.getSeatNo() != null || !bet.getSeatNo().isEmpty() || !bet.getSeatNo().equals("null"))
{

}

how to check it is null or not ?

you are getting a NPE because your bet object is null. What you are trying to do here is ,retrieving a value on a null object resulting in a NPE — Abhishek
– Abhishek, Commented Apr 17, 2015 at 10:52

Brian · Accepted Answer · 2015-04-17 10:57:53Z

1

First you need to check if the value is not empty.

if(!TextUtils.isEmpty(bet.getSeatNo())){
//Check your condition here
}

edited Apr 17, 2015 at 10:57

Brian

7,1629 gold badges49 silver badges86 bronze badges

answered Apr 17, 2015 at 10:53

Samir Bhatt

363 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Prashant Patel · Accepted Answer · 2015-04-17 12:29:41Z

1

Try doing this way:

    if (bet.getSeatNo() != null)
    {
        if(!bet.getSeatNo().isEmpty() && !bet.getSeatNo().equals("null"))
        {

        }
    }

answered Apr 17, 2015 at 12:29

Prashant Patel

1,09711 silver badges18 bronze badges

Comments

Ajinkya Pote · Accepted Answer · 2015-04-17 10:52:15Z

0

Before checking the field empty/null, check whether "bet" object is null or not. NullPointerException occurs while accessing fields of null object.

answered Apr 17, 2015 at 10:52

Ajinkya Pote

113 bronze badges

Comments

laalto · Accepted Answer · 2015-04-17 13:03:53Z

0

if (bet.getSeatNo() != null || !bet.getSeatNo().isEmpty() || !bet.getSeatNo().equals("null"))

If bet.getSeatNo() != null is false, only then the second !bet.getSeatNo().isEmpty() is evaluated. But since the first only evaluates to false if getSeatNo() returns null, the second conditional must NPE. Notice the problem?

Use && instead of || so that the short-circuiting stops on first false and not on the first true.

answered Apr 17, 2015 at 13:03

laalto

153k68 gold badges293 silver badges306 bronze badges

Collectives™ on Stack Overflow

android NullPointerException on checking null value

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related