RE: Serialization

[Jeroen Frijters]

Guilhem Lavaux wrote:
>+            try
>+              {
>+                Class field_type = field.getType();
>+                
>+                if (type == field_type ||
>+                    (type == null && !field_type.isPrimitive()))
>+                  {
>+                    /* See defaulted */
>+                    return field;
>+                  }
>+       
>+                illegal = true;
>+                throw new IllegalArgumentException
>+                  ("Field requested is of type "
>+                   + field_type.getName()
>+                   + ", but requested type was "
>+                   + (type == null ?  "Object" : type.getName()));
>+              }
>+            catch (NullPointerException _)
>+              {
>+                /* Here we catch NullPointerException, because it may
>+                   only come from the call 'field.getType()'. If field
>+                   is null, we have to return null and classpath ethic
>+                   say we must try to avoid 'if (xxx == null)'.
>+                */
>+              } 

*Please* don't use exception handling instead of an explicit null check.

Regards,
Jeroen