Path: csiph.com!newsfeed.hal-mli.net!feeder3.hal-mli.net!newsfeed.hal-mli.net!feeder1.hal-mli.net!feeder.erje.net!fu-berlin.de!uni-berlin.de!individual.net!not-for-mail From: "J.O. Aho" Newsgroups: comp.lang.php Subject: Re: Verifying time in php Date: Sun, 14 Oct 2012 20:41:36 +0200 Lines: 37 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Trace: individual.net 4fVOSDN3YZdskijFQn7HngOQYMr+WvmqVE3QWtFWUi1cs+Vl2uCIZDiwx+C8ZiVbKQ Cancel-Lock: sha1:mhFG5fgdmtqkZC44MdD8QmQZP6k= User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120923 Thunderbird/15.0.1 In-Reply-To: Xref: csiph.com comp.lang.php:7321 On 14/10/12 19:20, houghi wrote: > Found finaly the solution. Instead of looking at the seperate values and > then trying to combine them, it is better to combine them and then > evaluate them with > $testtime=$_POST['hours'].$_POST['minutes'].$_POST['seconds']; > > $hours $min $sec $testtime Valid? > - - - - No > 1 - - 1 Yes > 1 - 1 11 Yes > - 1 - 1 Yes > 1 1 1 111 Yes > A - - A No > 1 A - 1A No > 1A - - 1A No > > That way I can just do a 'is_numeric($testtime)' and no need to use > is_set or anything else. an alternative is: if you have in your form then you can in your script $ok = true; foreach($_POST['variable'] as $value) { if(!is_numeric($value)) $ok = false; } even if your method works now, I wouldn't bet on it in the future. -- //Aho