mwh wrote:
Hi. If you remember, I posted Expressons Help. Now I am making a
calculator with javascript. I can't get this to work:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<title>Calculat or</title>
<script language="Javas cript">
The language attribute is depreciated, type is required:
<script type="text/javascript">
<!-- Begin Hiding
Completely unnecessary.
var total = 0
You should end statements with a semi-colon, though it isn't strictly
required and isn't causing your issues here.
var total = 0;
var operator = 0
function calculate(numbe r){
firm = document.form.v alue1
firm.value = firm.value + number
}
function operator(opvalu e){
You have a global variable called 'operator' and a function called
'operator'. Firefox assigns 'operator' to the variable, the function
is not defined. Change the name of either the variable or the
function.
Seems you really wanted the global variable to be 'theoperator', so I'd
change the declaration of the variable (and tweak the name) to:
var theOperator = 0
theoperator = opvalue
total = document.form.v alue1.value
document.form.s up.value = ""
There is no element in the form 'form' with a name of 'sup', did you
mean 'value1'?
}
function equals(){
You have defined equals as a form element name, that conflicts with
your equals function name - change one of them (I'll change the
element name since you don't use it for anything anyway).
currentDspvalue = eval(document.f orm.value1.valu e)
Do not use 'eval', there is nearly always a better way. In this case,
it is totally redundant.
currentDspvalue = document.form.v alue1.value;
previousDspvalu e = eval(total)
And here too.
previousDspvalu e = total;
// add
if (theoperator == "+"){
answer = currentDspvalue + previousDspvalu e
The variables you are adding are likely strings, you need to ensure
they are numbers. The unary operator is simplest:
answer = +currentDspvalu e + +previousDspval ue;
}
// divide
else if (theoperator == "/"){
answer = currentDspvalue / previousDspvalu e
}
// multilply
multiply ?
else if (theoperator == "*"){
answer = currentDspvalue * previousDspvalu e
}
// subtract
else if (theoperator == "-"){
answer = currentDspvalue - previousDspvalu e
}
document.form.s up.value = answer
}
It may be suitable to use a switch statement rather than ifs, but
that's up to you.
// -->
Remove this too.
</script>
<body bgcolor="blue">
<form name="form">
Forms require an action attribute, even if it's empty:
<form name="form" action="">
I'd change the name of the form to make it more obvious that it is the
name of a form, not just a form.
<input name="value1" length=15 type="text"><br >
Inputs do not have a 'length' attribute. 'size' will set the width of
a text input in characters, 'maxlength' will set the maximum number of
characters the user may enter.
<input name="1" type="button" value="1" onClick="calcul ate(1)">
<input name="2" type="button" value="2" onClick="calcul ate(2)">
<input name="3" type="button" value="3" onClick="calcul ate(3)">
<input name="4" type="button" value="4" onClick="calcul ate(4)">
<input name="5" type="button" value="5" onClick="calcul ate(5)">
<input name="6" type="button" value="6" onClick="calcul ate(6)">
<input name="7" type="button" value="7" onClick="calcul ate(7)">
<input name="8" type="button" value="8" onClick="calcul ate(8)">
<input name="9" type="button" value="9" onClick="calcul ate(9)">
<input name="0" type="button" value="0" onClick="calcul ate(0)">
<input name="decimal" type="button" value=" . " onClick=calcula te(".")>
You must use quotes around javascript in the onclick attribute:
<input name="decimal" type="button" value=" . "
onClick="calcul ate('.')">
The same goes for all following onclick attributes:
<input name="plus" type="button" value="Plus" onClick =operator("+")>
<input name="minus" type="button" value="minus" onClick =operator("-")>
<input name="multiply" type="button" value="times" onClick
=operator("*")>
<input name="divide" type="button" value="divided by" onClick
=operator("/")>
<input name="equals" type="button" value=" = " onClick = "equals()">
Change the name of this element, say:
<input name="signEqual s" ... >
<input name="reset" type="reset" value="clear">
</form>
</body>
</html>
It works fine until I press the equal button.
I presume you are not using any debugging tools or you would not have
come to that conclusion.
A working version of your script with the above corrections applied is
below.
[...]
The above fixes just get your current code to work, there is a lot more
required before your calculator becomes a robust solution. For
example, you do not prevent users directly entering characters into
the text field and do not validate the input at all.
Once a sum is complete, the first entry of the next number is appended
to the last result - users have to clear the input manually.
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<title>Calculat or</title>
<meta http-equiv="Content-Type"
content="text/html; charset=ISO-8859-1">
<script type="text/javascript">
var total = 0
var theOperator = 0
function calculate(numbe r){
firm = document.form.v alue1;
firm.value = firm.value + number;
}
function operator(opvalu e){
theOperator = opvalue;
total = document.form.v alue1.value;
// document.form.s up.value = ""
document.form.v alue1.value = ""
}
function equals(){
// currentDspvalue = eval(document.f orm.value1.valu e);
currentDspvalue = document.form.v alue1.value;
previousDspvalu e = total;
// add
if (theOperator == "+"){
answer = +currentDspvalu e + +previousDspval ue;
// divide
} else if (theOperator == "/"){
answer = currentDspvalue / previousDspvalu e
// multiply
} else if (theOperator == "*"){
answer = currentDspvalue * previousDspvalu e
// subtract
} else if (theOperator == "-"){
answer = currentDspvalue - previousDspvalu e
}
// document.form.s up.value = answer
document.form.v alue1.value = answer;
}
</script>
<body bgcolor="blue">
<form name="form">
<input name="value1" size="15" type="text"><br >
<input name="1" type="button" value="1" onClick="calcul ate(1)">
<input name="2" type="button" value="2" onClick="calcul ate(2)">
<input name="3" type="button" value="3" onClick="calcul ate(3)">
<input name="4" type="button" value="4" onClick="calcul ate(4)">
<input name="5" type="button" value="5" onClick="calcul ate(5)">
<input name="6" type="button" value="6" onClick="calcul ate(6)">
<input name="7" type="button" value="7" onClick="calcul ate(7)">
<input name="8" type="button" value="8" onClick="calcul ate(8)">
<input name="9" type="button" value="9" onClick="calcul ate(9)">
<input name="0" type="button" value="0" onClick="calcul ate(0)">
<input name="decimal" type="button" value=" . "
onClick="calcul ate('.')">
<input name="plus" type="button" value="Plus"
onClick="operat or('+')">
<input name="minus" type="button" value="minus"
onClick="operat or('-')">
<input name="multiply" type="button" value="times"
onClick="operat or('*')">
<input name="divide" type="button" value="divided by"
onClick="operat or('/')">
<input name="signEqual s" type="button" value=" = "
onClick="equals ()">
<input name="reset" type="reset" value="clear">
</form>
</body>
</html>
--
Rob
