Chatrapati Posted August 17, 2013 Share Posted August 17, 2013 <?php include "config.php"; $id=$_POST['id']; $name=htmlspecialchars(addslashes($_POST['name'])); $sms=htmlspecialchars(addslashes($_POST['sms'])); $time=date("d-m-y"); if(isset($_POST['submitbutton'])) { if(empty($name)) { echo "Kindly enter your name<br/>"; } if(empty($sms)) { echo "May be you forgot to Add YOUR SMS <br/>"; } $count=strlen($name); if($count < 2) { echo "Your name must contain atleast Two characters<br/>"; } if($count > 12) { echo "Your name Must contain less than Twelve characters<br/>"; } $countsms=strlen($sms); if($countsms < 20) { echo "Your Sms Must contain atleast Twenty characters<br/>"; } if($countsms > 5000) { echo "Your Sms Must contain below Five Thousand Characters<br/>"; } else { $sql="INSERT INTO form1(id,name,sms,time) VALUES('$id','$name','$sms','$time')"; $result=mysql_query($sql) or die(mysql_error()); } if ($result) { echo "SMS added successfully"; } else { echo "could'nt add your SMS"; } } else { header("Location:form1.php"); } ?> what's the error here, if a user enter's incorrect details, it should not be inserted into the database, it should show error, but in this code, if statements are not working. even if user enters incorrect details, it's inserted into database, pls specify the error here, and take a look at http://wapfull.comoj.com/form2.php yu'll know the Error's ! ! Quote Link to comment Share on other sites More sharing options...
hakimserwa Posted August 17, 2013 Share Posted August 17, 2013 try using the code tag so we can able to read your code clearly Quote Link to comment Share on other sites More sharing options...
jcbones Posted August 17, 2013 Share Posted August 17, 2013 That code will submit all request to the database if the strlen of the sms message is less than 5000. Quote Link to comment Share on other sites More sharing options...
gristoi Posted August 17, 2013 Share Posted August 17, 2013 You are only echoing an error or. Not killing script. You need to do return anywhere you want the script to stop Quote Link to comment Share on other sites More sharing options...
jcbones Posted August 17, 2013 Share Posted August 17, 2013 OR, change the if, if, if, if, else statement to an if,elseif,elseif,elseif,else statement. Quote Link to comment Share on other sites More sharing options...
Chatrapati Posted August 17, 2013 Author Share Posted August 17, 2013 @gristoi How to do that @Jcbones its not wrking Quote Link to comment Share on other sites More sharing options...
Chatrapati Posted August 17, 2013 Author Share Posted August 17, 2013 THANK YOU @Jcbones @gristoi Quote Link to comment Share on other sites More sharing options...
jazzman1 Posted August 17, 2013 Share Posted August 17, 2013 (edited) What do you want to be count here? $count=strlen($name); The "$name" is already a string not an array. Edited August 17, 2013 by jazzman1 Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.