Quadratic

Path: csiph.com!fu-berlin.de!uni-berlin.de!individual.net!not-for-mail From: Arno Welzel Newsgroups: comp.lang.php Subject: Re: test prog inpu tpost Date: Fri, 1 Jan 2016 18:24:38 +0100 Lines: 127 Message-ID: <5686B656.3040603@arnowelzel.de> References:

Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit X-Trace: individual.net sw0mYLG8uqIIHN0Da1v3NQvpRvquaIGm+Fm5SodNCx9IYkh8aJ Cancel-Lock: sha1:OY8ZLlGTYCNd9WMWc9qiKZ++8yY= X-Enigmail-Draft-Status: N1110 In-Reply-To: Xref: csiph.com comp.lang.php:16051 Jerry Stuckle schrieb am 2016-01-01 um 15:32: > On 12/31/2015 9:22 PM, Lew Pitcher wrote: >> On Thursday December 31 2015 20:57, in >> comp.lang.php, "vjp2.at@at.BioStrategist.dot.dot.com" >> wrote: >> >>> >>> Should this work? >> >>> I'm stuck with lynx browser for a few days and can't tell >> >> That shouldn't stop you, assuming that you have a PHP-enabled webserver to >> host the page on. >> >> Lynx, like all browsers, just renders the html sent to it by the server. >> >> PHP code executes on the server. >> >> If the html with PHP is served from a PHP-enabled web server, lynx will render >> the results correctly. >> >>> >>> >>> Quadratic >>> >>> >>> >> $t; >> >> What is $t ? >> Why are you trying to execute it as a php statement? >> >>> $b=$_POST['b'];$a=$_POST['a'];$c=$_POST['c']; >> >> Here, you should test whether the page is invoked from a GET or a POST. None >> of the $_POST variables will be set on the initial GET, and your later >> computation will be invalid. >> > > In addition, you should check to see if the values are actually set. > Even if it came here from a POST request, there is no guarantee the > values have been set. > > $b = isset($_POST['b']) ? $_POST['b'] : ''; > > puts an empty string in $b $_POST['b'] is not set. You could also use 0 > instead of '' or any other default value you wish. > >>> $x;$y; >> >> Same question as above with $t. What is $x? What is $y? Why are you trying to >> execute two variables as php statements? >> >>> $t=(pow($b,2))-(4*$a*$c); >> >> Meaningless (and possibly errant) results on GET >> > > The advantage to using '' as a default value above is you can test here: > > if ($a == '' || $b == '' || $c = '') { Of course this should be the opposite as one would do the calculation only if *all* values are set and not if any of the value is empty: if ($a != '' && $b != '' && $c != '') { It would also be nice to see the values entered as default in the input elements and to have valid HTML. And finally it is not neccessary to use echo sprintf(...) as there is printf() as well which outputs the result directly. So the whole script could be like this: Quadratic Solution in Addition = %f in Subtraction = %f

", $x, $y); } ?> But next time you promise to do your homework on your own, yes? ;-) -- Arno Welzel http://arnowelzel.de http://de-rec-fahrrad.de http://fahrradzukunft.de