Feedback on: Multi-dialogue forms on one page

irt.org | About | Feedback | 1840 [ previous next ]

Feedback on:
Multi-dialogue forms on one page

Sent by
Rob Stewart on October 09, 2000 at 16:56:38:

Worth:
Very worth reading

Comments:
This article discusses a useful technique. However, I find it falls short in several regards.

First, the code fails to cache properties and instead repeats lookups. For example, within the show() function, the expression, document.hiddenForm, appears repeatedly. Accessing the document object itself can be expensive. Accessing a property on the same object repeatedly is a waste of time. Instead, a local variable, perhaps hiddenForm should be assigned that property (var hiddenForm = document.hiddenForm;), then reused everywhere else. (A similar issue arises near the bottom of the same function in accessing document.layers['myTab'].document. Whew!)

Second, the show() function puts the HTML for all the tabs into a single function. The code will be easier to maintain if show() calls showTab1(), showTab2(), etc. Those functions can return an HTML string that show() can use for the rest of its work.

Third, this technique is not truly reusable. It is reusable, in the sense that you can copy and paste the code, then change the HTML, etc., to create a new form. However, it is not truly reusable because there is no framework that can be parameterized with the custom form content.

I haven't created a better version at this time, and I don't know that I'd be able to share it if I did, but I'm certain a reusable framework is possible.

Other feedback on 'Multi-dialogue forms on one page' - show all

• Multi-dialogue forms on one page - glitch September 25, 2002 at 07:15:46:
• Multi-dialogue forms on one page - Narasimha December 12, 2001 at 01:23:58:
• Multi-dialogue forms on one page - Jonathan Chard 06:56:44 10/12/00
• Multi-dialogue forms on one page - Jonathan Chard 06:38:16 10/12/00
• Multi-dialogue forms on one page - David Morris 19:06:14 10/10/00
• Multi-dialogue forms on one page - Rob Stewart 16:56:38 10/09/00
• Multi-dialogue forms on one page - Andy Boddington 07:07:19 8/01/00
• Multi-dialogue forms on one page - Kimberly Albrecht 18:02:26 6/22/00
• Multi-dialogue forms on one page - Basil McDonnell 16:41:42 5/11/00
• Multi-dialogue forms on one page - Basil McDonnell 16:36:36 5/11/00
• Multi-dialogue forms on one page - Kimberly Albrecht 15:34:14 4/18/00