Form problem - Price break

E

Ed

Hoping someone an assist me urgently. I would truly appreciate and help.

I have a form and prices are based on 'price break'.

1.The price break fields work fine, but I cannot for the life of me add more
than one to a page.
2. Also need to add the two results to a 'total' field. (See code attached).

Many thanks in advance.

Sample code ...

<HEAD>

</HEAD>

<BODY onLoad="findPrice();">

<SCRIPT LANGUAGE="JavaScript">
<!-- Begin
var max_hsunits1 = 1000; // quantities in excess of max_hsunits1 all have
the same unit price
var currency = "$"; // currency sign used in 'formatMessage()'
// Edit this function to reflect discount prices!
function getDiscountPrice(hsunits1) {
// Note: Work way down from max to min amounts!
if (hsunits1 >= max_hsunits1) return 18;
if (hsunits1 >= 750) return 20;
if (hsunits1 >= 500) return 25;
if (hsunits1 >= 250) return 28;
if (hsunits1 >= 150) return 30;
if (hsunits1 >= 75) return 31;
if (hsunits1 >= 50) return 40;
if (hsunits1 >= 25) return 45;
if (hsunits1 >= 10) return 47;
if (hsunits1 >= 5) return 50;
if (hsunits1 >= 1) return 55;
if (hsunits1 <= 0) return 0;
}
function getNumberOfUnits() {
var hsunits1 = document.Hypersnap.hsunits1.value;
return (hsunits1 == "") ? 0 : hsunits1;
}
function showResult(result) {
// adjust the following line if result must popup somewhere else
document.Hypersnap.hsresult1.value = result;
}
function formatMessage(hsunits1, unit_price) {
return hsunits1 + " * " + currency + formatPrice(unit_price) + " = " +
currency + formatPrice(hsunits1 * unit_price);
}
// AltUnits (alternate hsunits1): add extra hsunits1 to reach minimum for
next discount price
function getAltUnits(hsunits1) {
var discount_price = getDiscountPrice(hsunits1);
if (hsunits1 < max_hsunits1) do { hsunits1++ } while (discount_price ==
getDiscountPrice(hsunits1));
return hsunits1;
}
function findPrice() {
var hsunits1 = getNumberOfUnits();
var unit_price = getDiscountPrice(hsunits1);
var alt_hsunits1 = getAltUnits(hsunits1);
var alt_unit_price = getDiscountPrice(alt_hsunits1);
var result;
if ((hsunits1 * unit_price) < (alt_hsunits1 * alt_unit_price))
result = formatMessage(hsunits1, unit_price);
else
result = formatMessage(alt_hsunits1, alt_unit_price);
showResult(result);
}
function formatPrice(value) {
var result= Math.floor(value) + ".";
var cents = 100 * (value-Math.floor(value)) + 0.5;
result += Math.floor(cents / 10);
result += Math.floor(cents % 10);
return result;
}
function filterNonNumeric(field) {
var result = new String();
var numbers = "0123456789";
var chars = field.value.split(""); // create array
for (i = 0; i < chars.length; i++) {
if (numbers.indexOf(chars) != -1) result += chars;
}
if (field.value != result) field.value = result;
}
// End -->
</script>
<SCRIPT LANGUAGE="JavaScript">
<!-- Begin
var max_hsunits2 = 1000; // quantities in excess of max_hsunits2 all have
the same unit price
var currency = "$"; // currency sign used in 'formatMessage2()'
// Edit this function to reflect your discount prices!
function getDiscountPrice2(hsunits2) {
// Note: It is important to work your way down from max to min amounts!
if (hsunits2 >= max_hsunits2) return 18;
if (hsunits2 >= 750) return 20;
if (hsunits2 >= 500) return 25;
if (hsunits2 >= 250) return 28;
if (hsunits2 >= 150) return 30;
if (hsunits2 >= 75) return 31;
if (hsunits2 >= 50) return 40;
if (hsunits2 >= 25) return 45;
if (hsunits2 >= 10) return 47;
if (hsunits2 >= 5) return 50;
if (hsunits2 >= 1) return 55;
if (hsunits2 <= 0) return 0;
}
function getNumberOfUnits2() {
var hsunits2 = document.Hypersnap.hsunits2.value2;
return (hsunits2 == "") ? 0 : hsunits2;
}
function showResult2(result2) {
// adjust the following line if result2 must popup somewhere else
document.Hypersnap.hsresult22.value2 = result2;
}
function formatMessage2(hsunits2, unit_price2) {
return hsunits2 + " * " + currency + formatPrice2(unit_price2) + " = " +
currency + formatPrice2(hsunits2 * unit_price2);
}
// AltUnits2 (alternate hsunits2): add extra hsunits2 to reach minimum for
next discount price
function getAltUnits2(hsunits2) {
var discount_price2 = getDiscountPrice2(hsunits2);
if (hsunits2 < max_hsunits2) do { hsunits2++ } while (discount_price2 ==
getDiscountPrice2(hsunits2));
return hsunits2;
}
function findPrice2() {
var hsunits2 = getNumberOfUnits2();
var unit_price2 = getDiscountPrice2(hsunits2);
var alt_hsunits2 = getAltUnits2(hsunits2);
var alt_unit_price2 = getDiscountPrice2(alt_hsunits2);
var result2;
if ((hsunits2 * unit_price2) < (alt_hsunits2 * alt_unit_price2))
result2 = formatMessage2(hsunits2, unit_price2);
else
result2 = formatMessage2(alt_hsunits2, alt_unit_price2);
showResult2(result2);
}
function formatPrice2(value2) {
var result2= Math.floor(value2) + ".";
var cents = 100 * (value-Math.floor(value2)) + 0.5;
result2 += Math.floor(cents / 10);
result2 += Math.floor(cents % 10);
return result2;
}
function filterNonNumeric(field) {
var result2 = new String();
var numbers = "0123456789";
var chars = field.value.split(""); // create array
for (i = 0; i < chars.length; i++) {
if (numbers.indexOf(chars) != -1) result2 += chars;
}
if (field.value2 != result2) field.value = result2;
}
// End -->
</script>
<form name="Hypersnap" method="post">

&nbsp;
<p><font face="Arial" size="2"><b>Number of Users Required :</b></font>
<input type=text value="1" name="hsunits1" onkeydown="findPrice()"
onKeyUp="filterNonNumeric(this); findPrice()" onkeypress="findPrice()"
size="4" style="font-weight:bold; text-align:center;">
<input type=text onfocus="this.blur()" name="hsresult1" size="24"
style="border:0; font-weight:bold;">
<br><font face="Arial" size="2"><b>Number of Users Required :</b></font>
<input type=text value="1" name="hsunits2" onkeydown="findPrice()"
onKeyUp="filterNonNumeric(this); findPrice()" onkeypress="findPrice()"
size="4" style="font-weight:bold; text-align:center;">
<input type=text onfocus="this.blur()" name="hsresult2" size="24"
style="border:0; font-weight:bold;">
&nbsp;</form>
<form name="form1">
<p align="left"><font
color="black"><strong>Total Sales
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbs
p;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input
type="text" name="hstotal" value="0.00" size="16"
onchange="ComputeForm()"
style="font-family:Arial; font-style:normal;
font-weight:bolder; font-size:11pt; color:white; background-color:black;"
language="JavaScript"></strong></font></p>
</form>
</html>
 
M

Michael Winter

Ed wrote on 14 Dec 2003 at Sun, 14 Dec 2003 15:35:44 GMT:
Hoping someone an assist me urgently. I would truly appreciate
and help.

I have a form and prices are based on 'price break'.

1.The price break fields work fine, but I cannot for the life of
me add more than one to a page.
2. Also need to add the two results to a 'total' field. (See
code attached).
Click to expand...

Rather than hard-code the form elements that the script checks, pass
the form elements as objects to the functions. For example:

<FORM name="someForm" id="someForm">
<INPUT name="someTextBox" onkeypress="checkKey()">

function checkKey() {
var txtBox = document.forms['someForm'].elements['someTextBox'];

// Do something with the text box
txtBox.value
}

What I've presented above is similar to how you access the form in
your code below. It would work fine, but it can only access the hard-
coded forms and controls. One could change it to this:

<FORM name="someForm" id="someForm">
<INPUT name="someTextBox" onkeypress="checkKey(this)">

function checkKey(control) {
var txtBox = control;

// Do something with the text box
txtBox.value
}

Notice the function call and how the control is referenced. The
snippet above is effectively the same: the variable 'txtBox' refers
to the same control, someTextBox. When the keyword, this, is used in
an intrinsic event, it creates an object reference that represents
the control. You can then use that reference in the same way as you
would a fully-qualified one. This way, you can use the same function
on multiple controls.

That will stop you from having to duplicate the same script, and
allow you to use as many price breaks (whatever they are) as you
like. What follows are some comments on your HTML.
<HEAD>
Click to expand...

You are using intrinsic events, but you have not specified the
default scripting language. Include:

<META http-equiv="Content-Script-Type" content="text/javascript">

here. Also, HTML documents required a TITLE element (and a DOCTYPE).
</HEAD>

<BODY onLoad="findPrice();">

<SCRIPT LANGUAGE="JavaScript">
Click to expand...

The type attribute is /required/ in HTML. When used, it renders the
language attribute (which is deprecated, anyway) redundant.

<!-- Begin
Click to expand...

Don't use SGML comments to hide SCRIPT contents - there's no need as
the archaic reasons for doing it are no longer valid.

function getNumberOfUnits() {
var hsunits1 = document.Hypersnap.hsunits1.value;
return (hsunits1 == "") ? 0 : hsunits1;
}
Click to expand...

To make your script work across more browsers, you should use the
'collections syntax' when accessing form elements:

var hsunits1 =
document.forms['Hypersnap'].elements['hsunits1'].value;

// End -->
Click to expand...

This closing comment should be removed.
</script>
Click to expand...

<form name="Hypersnap" method="post">

&nbsp;
<p><font face="Arial" size="2"><b>Number of Users Required
:</b></font>
Click to expand...

Why use style sheets, but also include deprecated FONT, B, and bad
layout techniques (non-breaking spaces)?

<input type="text" name="hstotal" value="0.00" size="16"
Click to expand...
language="JavaScript">
Click to expand...

INPUT elements do not have a language attribute.

Hope that helps,

Mike
 
E

Ed

Thank you so much Michael. I will go away and digest your advice.

regards.

Michael Winter said:
Ed wrote on 14 Dec 2003 at Sun, 14 Dec 2003 15:35:44 GMT:
Hoping someone an assist me urgently. I would truly appreciate
and help.

I have a form and prices are based on 'price break'.

1.The price break fields work fine, but I cannot for the life of
me add more than one to a page.
2. Also need to add the two results to a 'total' field. (See
code attached).
Click to expand...

Rather than hard-code the form elements that the script checks, pass
the form elements as objects to the functions. For example:

<FORM name="someForm" id="someForm">
<INPUT name="someTextBox" onkeypress="checkKey()">

function checkKey() {
var txtBox = document.forms['someForm'].elements['someTextBox'];

// Do something with the text box
txtBox.value
}

What I've presented above is similar to how you access the form in
your code below. It would work fine, but it can only access the hard-
coded forms and controls. One could change it to this:

<FORM name="someForm" id="someForm">
<INPUT name="someTextBox" onkeypress="checkKey(this)">

function checkKey(control) {
var txtBox = control;

// Do something with the text box
txtBox.value
}

Notice the function call and how the control is referenced. The
snippet above is effectively the same: the variable 'txtBox' refers
to the same control, someTextBox. When the keyword, this, is used in
an intrinsic event, it creates an object reference that represents
the control. You can then use that reference in the same way as you
would a fully-qualified one. This way, you can use the same function
on multiple controls.

That will stop you from having to duplicate the same script, and
allow you to use as many price breaks (whatever they are) as you
like. What follows are some comments on your HTML.
<HEAD>
Click to expand...

You are using intrinsic events, but you have not specified the
default scripting language. Include:

<META http-equiv="Content-Script-Type" content="text/javascript">

here. Also, HTML documents required a TITLE element (and a DOCTYPE).
</HEAD>

<BODY onLoad="findPrice();">

<SCRIPT LANGUAGE="JavaScript">
Click to expand...

The type attribute is /required/ in HTML. When used, it renders the
language attribute (which is deprecated, anyway) redundant.

<!-- Begin
Click to expand...

Don't use SGML comments to hide SCRIPT contents - there's no need as
the archaic reasons for doing it are no longer valid.

function getNumberOfUnits() {
var hsunits1 = document.Hypersnap.hsunits1.value;
return (hsunits1 == "") ? 0 : hsunits1;
}
Click to expand...

To make your script work across more browsers, you should use the
'collections syntax' when accessing form elements:

var hsunits1 =
document.forms['Hypersnap'].elements['hsunits1'].value;

// End -->
Click to expand...

This closing comment should be removed.
</script>
Click to expand...

<form name="Hypersnap" method="post">

&nbsp;
<p><font face="Arial" size="2"><b>Number of Users Required
:</b></font>
Click to expand...

Why use style sheets, but also include deprecated FONT, B, and bad
layout techniques (non-breaking spaces)?

<input type="text" name="hstotal" value="0.00" size="16"
Click to expand...
language="JavaScript">
Click to expand...

INPUT elements do not have a language attribute.

Hope that helps,

Mike
Click to expand...
 
C

Chris

Ed said:
Hoping someone an assist me urgently. I would truly appreciate and help.

I have a form and prices are based on 'price break'.

1.The price break fields work fine, but I cannot for the life of me add more
than one to a page.
2. Also need to add the two results to a 'total' field. (See code attached).

Many thanks in advance.

... snip ...
Click to expand...

Ed. your code was very difficult to follow, so I'm not sure exactly what it
is supposed to do.
From your text above I think the following example is likely to provide you
with a reasonable starting point.
I've tested it on IE and suspect it would need quite some changes to support
Netscape.
Security Note. Prices calculated on a client should never be used by a
server as this would expose you to huge security risks.

Regards,
Chris.

<HTML>

<table>
<tr>
<td></td>
<td>Quantity</td>
<td>Unit Price</td>
<td>Total Price</td>
</tr>
<tr>
<td>Number of Widgets: </td>
<td><input value = 0

onKeyUp="findPrice(this.value,1,Product1Bands,Product1Price,Product1Total)">
</input>
</td>
<td id="Product1Price"></td>
<td id="Product1Total"></td>
</tr>
<tr>
<td>Number of Wodgers: </td>
<td><input value = 0

onKeyUp="findPrice(this.value,2,Product2Bands,Product2Price,Product2Total)">
</input>
</td>
<td id="Product2Price"></td>
<td id="Product2Total"></td>
</tr>
<tr>
<td></td>
<td></td>
<td></td>
<td id="TotalPrice" style="font-weight:bold"></td>
</tr>
</table>


<script>

var Product1Bands = [ // Array of Thresholds and Price Bands for
Product 1.
"1000:18",
"500:25",
"250:28",
"1:30",
"0:0"
];

var Product2Bands = [ // Array of Thresholds and Price Bands for
Product 1.
"1000:9",
"500:15",
"1:20",
"0:0"
];

// Initialise the calculated fields
findPrice(0,1,Product1Bands,Product1Price,Product1Total);
findPrice(0,2,Product2Bands,Product2Price,Product2Total);

function
findPrice(Quantity,ProductNumber,ProductBands,ProductPriceField,ProductTotal
Field) {
for (i=0;i<ProductBands.length;i++) {
Thresholds = ProductBands.split(":");
if (1*Quantity >= 1*Thresholds[0]) { // Force Numeric
comparison
ProductPriceField.innerText = Thresholds[1];
ProductTotalField.innerText = Thresholds[1]*Quantity;
TotalPrice.innerText = 1*Product1Total.innerText +
1*Product2Total.innerText;
return;
}
}
}
</script>
</HTML>
 
E

Ed

The good Doctor,
Please read, understand, and honour c.l.j FAQ 2.3 para 5 sentence 6,
Click to expand...

Really Doc, you should play more golf.

Still, my gratitude to Mike and Chris for their helpful and sensible
responses.

Dr John Stockton said:
JRS: In article <[email protected]>, seen



Please read, understand, and honour c.l.j FAQ 2.3 para 5 sentence 6, etc.
Click to expand...
links.
 
D

Dr John Stockton

JRS: In article <[email protected]>, seen
in news:comp.lang.javascript said:
Really Doc, you should play more golf.

Still, my gratitude to Mike and Chris for their helpful and sensible
responses.
Click to expand...


If you cannot learn to behave yourself in public, you will find yourself
ostracised by those who consider such things significant. Perhaps you
do not expect to need further informed assistance here; though, from
your original question, that seems unlikely.
 
E

Ed

Dr Stockton,
You should read the reply to 'Lee'.
Yes, I do need assistance - and it's a shame I have to feed your ego to get
that help.
Behave myself? Could you please show me one instance where I have not simply
asked a question as clearly as my (very) limited JS knowledge allows?
Again, where are your contributions Doc? Apart from being the groups 'How To
Ask a Question Policeman' and your impolite links that border on Spam.

Ed
 
L

Lasse Reichstein Nielsen

Ed said:
Dr Stockton,
You should read the reply to 'Lee'.
Yes, I do need assistance - and it's a shame I have to feed your ego to get
that help.
Click to expand...

Don't think for a second that Dr. Stockton is the only person not
liking top posting. He pointed you to the *FAQ* for *this* newsgroup,
in particular section 2.3, paragraph 5, sentence 6. It tells you how
you should quote in this newsgroup. You have given no evidence that
you have even read it.

While Dr. Stockton might be the one to tell you that you shoudn't top
post, he's far from being the only one of that opinion. The hint, to
go Matrix on you, was for your protection. And since you repeatedly
ignore it, I too will ignore you until you change your mind. You can
assume that other people have already made the same decission without
even bothering to inform you ... after all, you won't bother to please
them, why should they care about you.

Good luck.
/L
 
I

Ivan Marsh

Don't think for a second that Dr. Stockton is the only person not liking
top posting. He pointed you to the *FAQ* for *this* newsgroup, in
particular section 2.3, paragraph 5, sentence 6. It tells you how you
should quote in this newsgroup. You have given no evidence that you have
even read it.

While Dr. Stockton might be the one to tell you that you shoudn't top
post, he's far from being the only one of that opinion. The hint, to go
Matrix on you, was for your protection. And since you repeatedly ignore
it, I too will ignore you until you change your mind. You can assume
that other people have already made the same decission without even
bothering to inform you ... after all, you won't bother to please them,
why should they care about you.
Click to expand...

Indeed.

Follow the LONG established and (almost) globally followed rules of usenet
and you will get many more responses to your requests.

The rules are there for a VERY specific reason. The majority of news
readers in existance depend on these rules to properly format threads in
messages.

If someone has to work to figure out what's been said, or who said it, in
a thread they are much less likely to want to be part of it.
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Similar Threads

Order 'total' problem. 0
Can someone help me correct this code? It should be a two line countdown timer. 3
Issue with passing fetched data to POST form. How can I? 0
Collect Excel Data from Website 5
Cash register challenge 2
I Need Help with making a function that draws in a canvas using location data. 1
using refresh to reload page, problem is it loads header, footer andeverything, can I avoind those t 8
? Trouble with: passing dragElement(e.target.id); 8

Members online

No members online now.
Total: 37 (members: 3, guests: 34)
Robots: 73

Forum statistics

Threads
473,780
Messages
2,569,611
Members
45,280
Latest member
BGBBrock56

Latest Threads

Top