[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[ale] More JavaScript



On 04/30/2013 07:09 PM, Alex Carver wrote:
> On 4/30/2013 15:58, Jim Lynch wrote:
>> On 04/30/2013 03:26 PM, Geoffrey Myers wrote:
>>> InvertView('o', LinkDiv)
>>>
>>> Will fail.
>>>
>>> InvertView() loops through the arg list doing this:
>>>
>>> arguments[i+1].style.display =
>>> (Arguments[i] == 'o') ? "block" : "none"
>>>
>>> Fails with: Arguments[i+1].style is undefined.
>> Maybe test?
>>
>> if(arguments[i+1].style != undefined) {
>>      arguments[i+1].style.display =
>>      Arguments[i] == 'o') ? "block" : "none"
>> }
>>
>
> if ( arguments[i+1] != null && argments[i+1].style != undefined)
> {
> ...
> }
>
> Short circuit conditional just to be sure the object exists first 
> before attempting to test a property.
>
I agree.