hi all, I have form in which people renew their agreements online. they can renew up to 5 agreements. I need to check if the expiration date is older than today, late fee will be added. if exp. date is equal or newer - no late fee. I have something like this so far and I can go further with this but my question is, if there is a way to simplify this code a little bit, or to make it shorter? $Today=date('m/d/y'); if ($selectedProdCode="agreem" and $errorArray['agr1expdate'] < $Today) { $selectedProdCode=// code with 1 agr and late fee } else { //1 agr if ( $selectedProdCode="agreem" and $errorArray['agr1expdate'] > $Today) { $selectedProdCode=// code with 1 agr and no late fee } else { if ($selectedProdCode="agreem" and $errorArray['agr1expdate'] < $Today) { $selectedProdCode=// code with 1 agr and late fee } } else { if ($selectedProdCode="agreem" and $errorArray['agr1expdate'] == $Today) { $selectedProdCode=// code with 1 agr and no late fee } } /////------------------------------------------------------------------------------------------------------ //2 agr else { if ( $selectedProdCode="agreem2" and $errorArray['agr1expdate'] == $Today and $errorArray['agr2expdate'] == $Today ) { $selectedProdCode=// code with 2 certs and no late fee } } else { if ($selectedProdCode="agreem2" and $errorArray['agr1expdate'] > $Today and $errorArray['agr2expdate'] > $Today { $selectedProdCode=// code with 2 agr and no late fee } } else { if ($selectedProdCode="agreem2" and $errorArray['agr1expdate'] == $Today and $errorArray['agr2expdate'] > $Today { $selectedProdCode=// code with 2 agr and no late fee } } else { if ($selectedProdCode="agreemn2" and $errorArray['agr1expdate'] > $Today and $errorArray['agr2expdate'] == $Today { $selectedProdCode=// code with 2 certs and no late fee } } else { if ($selectedProdCode="agreem2" and $errorArray['agr1expdate'] < $Today and $errorArray['agr2expdate'] == $Today { $selectedProdCode=// code with 2 certs and plus 1x late fee } } else { if ($selectedProdCode="agreem2" and $errorArray['agr1expdate'] == $Today and $errorArray['agr2expdate'] < $Today { $selectedProdCode=// code with 2 agr and plus 1x late fee } } else { if ($selectedProdCode="agreem2" and $errorArray['agr1expdate'] < $Today and $errorArray['agr2expdate'] > $Today { $selectedProdCode=// code with 2 certs and plus 1x late fee } } else { if ($selectedProdCode="agreem2" and $errorArray['agr1expdate'] > $Today and $errorArray['agr2expdate'] < $Today { $selectedProdCode=// code with 2 agr and plus 1x late fee } } Thanks -- View this message in context: http://www.nabble.com/how-to-simplify-my-code---I-will-have-tooo-many-lines---please-help-tp21585387p21585387.html Sent from the PHP - General mailing list archive at Nabble.com. -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php