JavaScript help please

Reply Subscribe

Thread Tools

Search this Thread

Jan 23, 2009 | 09:54 PM

spufus

Thread Starter

Scooby Regular

iTrader: (2)

Joined: Oct 2003

Posts: 661

Likes: 0

From: In the summerhouse

JavaScript help please

I've written this script to calculate the rough price to carpet a room.
Works fine in IE. It works if FF but instantly refreshes the form & clears the data.
Anyone any ideas?

Thanks in advance.

Code is below

<html>
<head>
<script type="text/javascript">

function roundTo4(raw)
{
strFloat = raw.toString();
var newFloat;

if(strFloat.indexOf(".") > 0 && strFloat.length - strFloat.indexOf(".") > 5)
{
strFloat = strFloat.substring(0,strFloat.indexOf(".") + 6);

var numFloat = parseFloat(strFloat);
var rndFloat = Math.round(numFloat * 10000);
var temp1Float = rndFloat / 10000;
var temp2Float = temp1Float.toString();

newFloat = temp2Float.substring(0,temp2Float.indexOf(".") + 5);
}
else
{
newFloat = strFloat;
}
return newFloat;
}

function roomCalc(form)
{
var p = form.p.value;
if(isNaN(p))
{
alert("Not a Number");
document.getElementById("p").value = "";
return false;
}

var w = form.w.value;
if(isNaN(w))
{
alert("Not a Number");
document.getElementById("w").value = "";
return false;
}

var l = form.l.value;
if(isNaN(l))
{
alert("Not a Number");
document.getElementById("l").value = "";
return false;
}

var volume = p * w * l;
var volumeOut = roundTo4(volume)
document.getElementById("bv").value = volumeOut;
}

</script>
</head>

<body>
<form>
<b>Room calculator</b><br />
<select id="p">
<option value="2">Karndean £2.00 sq metre</option>
<option value="2.5">Karndean £2.50 sq metre</option>
</select><br>

Room width (metres) approx: <input id="w" style="text-align:right"></input><br />
Room length (metres) approx: <input id="l" style="text-align:right"></input><br />
<button onclick="roomCalc(this.form)">Calculate</button><br>
Price: <input id="bv" style="text-align:right"></input>

</form>
</body>
</html>

Last edited by spufus; Jan 23, 2009 at 10:08 PM.

Jan 23, 2009 | 10:13 PM

Dedrater

Scooby Regular

Joined: May 2008

Posts: 3,957

Likes: 0

It looks fine to me. Have you got the Runtime Environment plugin for FF? It will flash off if you havn't

Jan 23, 2009 | 10:15 PM

Dedrater

Scooby Regular

Joined: May 2008

Posts: 3,957

Likes: 0

Could you post the calculation you are trying to achieve in base 10

Jan 23, 2009 | 10:47 PM

spufus

Thread Starter

Scooby Regular

iTrader: (2)

Joined: Oct 2003

Posts: 661

Likes: 0

From: In the summerhouse

Here it is with the rounding function gone:

Simpler, but still upsetting FF

<html>
<head>
<script type="text/javascript">

function roomCalc(form)
{
var p = form.p.value;
if(isNaN(p))
{
alert("Not a Number");
document.getElementById("p").value = "";
return false;
}

var w = form.w.value;
if(isNaN(w))
{
alert("Not a Number");
document.getElementById("w").value = "";
return false;
}

var l = form.l.value;
if(isNaN(l))
{
alert("Not a Number");
document.getElementById("l").value = "";
return false;
}

var volume = p * w * l;
document.getElementById("bv").value = volume;
}

</script>
</head>

<body>
<form>
<b>Room calculator</b><br />
<select id="p">
<option value="2.00">Karndean £2.00 sq metre</option>
<option value="2.50">Karndean £2.50 sq metre</option>
</select><br>

Room width (metres) approx: <input id="w" style="text-align:right"></input><br />
Room length (metres) approx: <input id="l" style="text-align:right"></input><br />
<button onclick="roomCalc(this.form)">Calculate</button><br>
Price: <input id="bv" style="text-align:right"></input>

</form>
</body>
</html>

Jan 23, 2009 | 10:52 PM

Dedrater

Scooby Regular

Joined: May 2008

Posts: 3,957

Likes: 0

Makes no sense to me unless you post off the math mate

I need to know what you are trying to achieve, base 10 to me would mean the actual function.

Last edited by Dedrater; Jan 23, 2009 at 10:54 PM.

Jan 23, 2009 | 10:55 PM

Dedrater

Scooby Regular

Joined: May 2008

Posts: 3,957

Likes: 0

Sorry, I don't know what you want to achieve?

Is that the whole program?

Last edited by Dedrater; Jan 23, 2009 at 10:57 PM.

Jan 23, 2009 | 10:59 PM

Dedrater

Scooby Regular

Joined: May 2008

Posts: 3,957

Likes: 0

[QUOTE=Dedrater;8450451]
I need to know what you are trying to achieve,

Trending Topics

ABS Delete Kit Fitting Questions Before I Take the Plunge

5

286
Flatspot on acceleration causes.

7

535
Engine starting issues

4

318
piston ring gap help

6

12.5k
You raced an E39 M5 on the A14

4

648

Jan 23, 2009 | 11:03 PM

spufus

Thread Starter

Scooby Regular

iTrader: (2)

Joined: Oct 2003

Posts: 661

Likes: 0

From: In the summerhouse

Quote:

Originally Posted by Dedrater

Sorry, I don't know what you want to achieve?

Is that the whole program?

Yes,

It calculates a rough cost to carpet a room. Carpet cost per metre in drop down * room width in text field * room length in text field.

The function works in IE & FF but in FF all fields are cleared for some reason.

Needs some more thought methinks....