RE: My Java Experience... latest code - critique (last part of response)

["Stone, Brad V (TC)" <bvstone@xxxxxxxxxxxxxx>]

> 
> More nitpicky:
> For HTML format, you're always creating two FieldFormatter objects.
> First the default XML one, then the new HTML one. Object creation can
> get really expensive in Java code. I'd recommend the XML formatter
> instantiation getting put down the else leg.
>     FieldListFormatter listFormatter = new XMLFieldListFormatter();
>     [... snip ...]
>     if (output.equals("HTML"))
>     {
>       response.setContentType("text/html");
>       listFormatter = new HTMLFieldListFormatter();
>       header = "table";
> 
>     } else {
>       response.setContentType("text/xml");
>     }
> 

I assumed I was creating one instance of the object, then "reassigning" it
if the output requested was HTML.   Sort of like saying:

String myString = "XML"

if (blah)
  myString = "HTML";

Yes, this does make one needless assignment in some cases, but its still
just one object (myString), not two, right?  Or is it because I'm saying
"new FieldFormatter" that I'm creating two object?  Once instance, but
creating it twice?

Brad
+---
| This is the JAVA/400 Mailing List!
| To submit a new message, send your mail to JAVA400-L@midrange.com.
| To subscribe to this list send email to JAVA400-L-SUB@midrange.com.
| To unsubscribe from this list send email to JAVA400-L-UNSUB@midrange.com.
| Questions should be directed to the list owner: joe@zappie.net
+---