I am writing a simple javascript function that grabs information out of three text areas. Pretty basic, but for some reason javascript is telling me that my variables are not defined and I am confused as to why.
The html is as followed:
<input type = "text" id = "month" name = "month"/>
<input type = "text" id = "day" name = "day"/>
<input type = "text" id = "year" name = "year"/>
<input type = "button" id = "submit" value = "Go!" onClick = "calculate(month, day, year)"/>
The javascript is simply
function calculate(month, day, year){
var setMonth = document.getElementById(month).value;
var setDay = document.getElementById(day).value;
var setYear = document.getElementById(year).value;}
And when I run this and click my button, it says "month is not defined" in Firebug. So I am a little confused, since ... they are defined!
I am writing a simple javascript function that grabs information out of three text areas. Pretty basic, but for some reason javascript is telling me that my variables are not defined and I am confused as to why.
The html is as followed:
<input type = "text" id = "month" name = "month"/>
<input type = "text" id = "day" name = "day"/>
<input type = "text" id = "year" name = "year"/>
<input type = "button" id = "submit" value = "Go!" onClick = "calculate(month, day, year)"/>
The javascript is simply
function calculate(month, day, year){
var setMonth = document.getElementById(month).value;
var setDay = document.getElementById(day).value;
var setYear = document.getElementById(year).value;}
And when I run this and click my button, it says "month is not defined" in Firebug. So I am a little confused, since ... they are defined!
Share edited Oct 11, 2024 at 6:50 dumbass 27.3k4 gold badges38 silver badges74 bronze badges asked Mar 31, 2011 at 18:26 QuintinQuintin 3091 gold badge5 silver badges12 bronze badges6 Answers
Reset to default 9month, day, and year are interpreted as variable names, which don't exist. You want to pass strings:
onClick = "calculate('month', 'day', 'year')"/>
you did not define them: to do that you would do:
calculate('a', 'b', 'c');
in your case you need to change you code alot or change what your function does:
i would say do option 2:
onclick ="calculate(document.getElementById('month'), document.getElementById('day'), document.getElementById('year'));"
and the function to:
function calculate(month, day, year){
var setMonth = month.value;
var setDay = day.value;
var setYear = year.value;
}
Notice that the calculate function does not have any arguments and that the document is queried based on the name of the fields.
HTML
<input type = "text" id = "month" name = "month"/>
<input type = "text" id = "day" name = "day"/>
<input type = "text" id = "year" name = "year"/>
<input type = "button" id = "submit" value = "Go!" onClick = "calculate()"/>
Javascript
function calculate(){
var setMonth = document.getElementById('month').value;
var setDay = document.getElementById('day').value;
var setYear = document.getElementById('year').value;}
In the "onClick" portion of your html, you're attempting to send "month", "day", and "year" variables that have yet to be defined.
Instead of passing the variables, try just running the "calculate();" function, and getting the data from the form from within the function.
When you call a function you must pass parameters initiates
<input type = "text" id = "month" name = "month"/>
<input type = "text" id = "day" name = "day"/>
<input type = "text" id = "year" name = "year"/>
<input type = "button" id = "submit" value = "Go!" onClick = "calculate(document.getElementById(month), document.getElementById(day), document.getElementById(year))"/>
Javascript code:
function calculate(month, day, year){
var setMonth = month.value;
var setDay = day.value;
var setYear = year.value;}
if you want tu use the function $(document).ready(...)
you need to add in the <HEAD></HEAD> tag the folowing line:
<script src="https://ajax.googleapis./ajax/libs/jquery/1.3.2/jquery.min.js"></script>