I'm trying to pass two parameters to a JavaScript function. Unfortunately this give me error.
Here is the function
function load(do,div)
{
var do;
var div;
document.getElementById('spinner').innerHTML = '<img src=\'/images/spinner.gif\'>';
$('#'+div).load('rpc.php?'+do,
function()
{
document.getElementById('spinner').innerHTML = '';
$('#'+div).show('slow');
}
}
And I call it like this:
<a href="javascript:;" onclick="load('limit','mydiv');">Get Limit</a>
How can I fix that?
Peter Mortensen
31.3k22 gold badges110 silver badges134 bronze badges
asked Dec 25, 2009 at 20:24
streetparade
33.1k39 gold badges106 silver badges123 bronze badges
3 Answers 3
do is a reserved word in JavaScript. Change the variable name to something else. Additionally, don't re-declare the arguments in the function body. So remove the 2 var lines from the top of your function body.
If you're curious what the do keyword is for, it's for do...while loops where the condition is evaluated at the end, not the beginning of the loop. It's used something like this:
do {
// do stuff in loop at least once
} while (some_condition_is_true);
For more info check out W3Schools.
Peter Mortensen
31.3k22 gold badges110 silver badges134 bronze badges
answered Dec 25, 2009 at 20:26
Asaph
164k25 gold badges204 silver badges204 bronze badges
Sign up to request clarification or add additional context in comments.
5 Comments
BalusC
... and remove the two declarations which are the overriding the arguments.
Luca Matteis
re-declaring variables with same argument name, doesn't effect the arguments.
Asaph
@Luca Matteis: Interesting. I didn't know that. Even so, it's unconventional enough that I still advise removing those lines.
BalusC
Interesting. This is a-logical.
Asaph
@BalusC: What exactly is a-logical? The fact that re-declaring variables has no effect, or my decision to leave the recommendation to remove the re-declaration in my answer despite knowing the former?
do is a reserved word in JavaScript.
answered Dec 25, 2009 at 20:28
gahooa
138k14 gold badges101 silver badges104 bronze badges
Comments
Is it because you are redefining do div again in function and they are overriding the scope of passed parameters?
answered Dec 25, 2009 at 20:28
dhaval
2,3889 gold badges35 silver badges60 bronze badges
Comments
lang-js