[RESOLVED] validating price?

Velausanakha

I'm using the ereg function to check the validity of a price that can be entered.

I want it so no more than 999.99 can be added.

That is I can have 1-3 digits before the decimal point and only 2 after it. There should always be 2 after it.

The digits after the decimal point seem to work, if I put 2.999 in, rather than having entered 2.99 into the database it'll enter $3.00 (that'll do I suppose)

BUT

if I entere 10000.00 that still enteres the database that is too much.

This is the statement I currently have

if(!ereg("([0-9]{1,3}).([0-9]{2})", $_POST[Price], $regs)){
	echo "Invalid Price, Book Not Added";
	pageRedirect('sellabook.php');
}

Can someone correct my code for me, please

I suppose I can always write an IF statement preventing values over $999.99

bradgrafelman

The decimal point is a special character in regular expressions: it means match any character.

Why not just round the number entered off to two decimal places, and make sure that number is not greather than or equal 1000?

$price = number_format($_POST['Price'], 2);

if($price >= 1000) {
     echo 'Invalid Price, Book Not Added';
     pageRedirect('sellabook.php');
}

Also, see your other post for my comments about array indexes.

Velausanakha

oversight on arrays and see my comment in response to how I feel about ereg so far

bradgrafelman

I mean, if you really really wanted to use some regular expression syntax here for some reason, you can probably fix your code by escaping the period so you have a literal period matched:

if(!ereg("([0-9]{1,3})\.([0-9]{2})", $_POST['Price'], $regs)){

though as I said in other post, I prefer the faster, better preg_ functions:

if(!preg_metch('/([0-9]{1,3})\.([0-9]{2})/', $_POST['Price'], $matches)){

Velausanakha

I had to use

$price = number_format($_POST['Price'], 2, '.' ,'');

for it to work after checking the manual, but thank you, it works now

actually still got to add an ereg statement in case the user enters a non-numeral but my previous ereg statement inside an IF statement should do it, I'd imagine

bradgrafelman

Or just use [man]is_numeric/man

if(!is_numeric($_POST['Price'])) {
 // non-numeric ammount!
}

Velausanakha

if only I knew such a function existed previously.
I guess I'm probably obsessed with ereg and eregi because of email validation and having toyed with it.

This way is so much simpler and a lot more powerful too

Thanks again