Question about Javascript Variables

[newspost2000]

Hi...

I was just reviewing some javascript pop up window functions on my
website and I descovered something... that I set the exact same
variable name twice for both functions... which is bookWindow. I
tested this for pages that use both functions and both types of pop
windows work no problem. Is there a problem in that I use the same
variable name twice? Is this considered bad form? Or maybe it makes
not difference at all because the variable only gets used when it is
called upon and then it leave memory.

All help and insite is appreciated.
<script language="JavaScript" type="text/javascript">
<!--
function newWindow(webpage) {
bookWindow = window.open(webpage, "newWin",
"menubar,status,scrollbars,resizable,toolbar,locat ion,height=400,width=500,top=26,left=280")
bookWindow.focus()
}
function msiWindow(webpage) {
bookWindow = window.open(webpage, "msiWin",
"menubar,status,scrollbars,resizable,toolbar,locat ion,height=600,width=754,top=0,left=0")
bookWindow.focus()
}
// -->

[Jonas Raoni]

newspost2000 wrote:

I was just reviewing some javascript pop up window functions on my
website and I descovered something... that I set the exact same
variable name twice for both functions...
[...]
Is there a problem in that I use the same
variable name twice? Is this considered bad form?

There's no problem, but the variable value will be replaced.

You call function A:
X = A
After calling function B:
X = B

If you're not going to use that variable, I suggest you to declare it
using "var", so it won't be acessible out of function.

function getWindow(bla, bla, bla){
var w = window.open(....
:
return w;
}
--
Jonas Raoni Soares Silva
http://www.jsfromhell.com

[web.dev]

newspost2000 wrote:

<script language="JavaScript" type="text/javascript">
The language attribute is deprecated, just stick with the type
attribute:

<script type = "text/javascript">
<!--
HTML comment delimiters are unnecessary.
function newWindow(webpage) {
bookWindow = window.open(webpage, "newWin",
"menubar,status,scrollbars,resizable,toolbar,locat ion,height=400,width=500,top=26,left=280")
bookWindow.focus()
}
function msiWindow(webpage) {
bookWindow = window.open(webpage, "msiWin",
"menubar,status,scrollbars,resizable,toolbar,locat ion,height=600,width=754,top=0,left=0")
bookWindow.focus()
}

You could _potentially_ have a problem. Since you have not declared
bookWindow with a "var", the variable is not local to the function.
You are using bookWindow as a global variable, thus you might overwrite
a value that you have assigned earlier. It is generally good form to
declare variables using the var keyword.