Just to avoid confusion and the NullPointerException, I would suggest this:
{
if (this.dueDateType() == null) {
return "";
}
else if (this.dueDateType().equals("N")) {
return "";
}
return this.dueDateType();
}
or this:
{
if (this.dueDateType() != null) {
if (!this.dueDateType().equals("N")) {
return this.dueDateType();
}
}
return "";
}
Steve
On Jul 31, 2010, at 8:13 AM, Jérémy DE ROYER wrote:
> So the answer is :
> - "if the object is null, as, in the second test, your use the method
> 'equals' on this object (null) > NullPointerException"
>
> On the first test, as you first test if the object is null, you can't have a
> NullPointerException
>
> Hope this help,
>
> Jérémy
>
> Le 31 juil. 2010 à 14:09, Theodore Petrosky a écrit :
>
>> Yes... I was asking 'Why?' is there a NullPointerException with number 2...
>>
>> Ted
>>
>>
>> --- On Sat, 7/31/10, Jérémy DE ROYER <[email protected]> wrote:
>>
>>> From: Jérémy DE ROYER <[email protected]>
>>> Subject: Re: checking for null Noob question
>>> To: "Theodore Petrosky" <[email protected]>
>>> Cc: [email protected]
>>> Date: Saturday, July 31, 2010, 8:07 AM
>>> I won't do the second one because you
>>> could see a NullPointerException. Is it your question ?
>>>
>>> Jérémy
>>>
>>> Le 31 juil. 2010 à 14:05, Theodore Petrosky a écrit :
>>>
>>>>
>>>> Why does this work:
>>>>
>>>> public String dueDateTypeLetter() {
>>>>
>>>> if (this.dueDateType() == null ||
>>> this.dueDateType().equals("N")) {
>>>> return "";
>>>> }
>>>> return this.dueDateType();
>>>> }
>>>>
>>>> and this doesn't....
>>>>
>>>> public String dueDateTypeLetter() {
>>>>
>>>> if (this.dueDateType().equals("N") ||
>>> this.dueDateType() == null) {
>>>> return "";
>>>> }
>>>> return this.dueDateType();
>>>> }
>>>>
>>>> obviously the difference is the order of checking in
>>> the 'if' statement...
>>>>
>>>> I don't really care that I must check in a specific
>>> order... It just took me time to understand that the order
>>> was important.
>>>>
>>>> Ted
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Do not post admin requests to the list. They will be
>>> ignored.
>>>> Webobjects-dev mailing list ([email protected])
>>>> Help/Unsubscribe/Update your Subscription:
>>>> http://lists.apple.com/mailman/options/webobjects-dev/jeremy.deroyer%40ingencys.net
>>>>
>>>> This email sent to [email protected]
>>>
>>>
>>
>>
>>
>
>
>
> _______________________________________________
> Do not post admin requests to the list. They will be ignored.
> Webobjects-dev mailing list ([email protected])
> Help/Unsubscribe/Update your Subscription:
> http://lists.apple.com/mailman/options/webobjects-dev/speery%40me.com
>
> This email sent to [email protected]
_______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list ([email protected])
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/webobjects-dev/archive%40mail-archive.com
This email sent to [email protected]
