efficient code

[Maverick25r]

Is there a more efficient or cleaner way to code this php mail form?

PHP: Select all

<?php

    //PHP Mail Form Code//



    //Get vaules from contact form//

    $companyname = $_POST['companyname'];

    $contactname = $_POST['contactname'];

    $contactphone = $_POST['contactphone'];

    $contactemail = $_POST['contactemail'];

    $contactaddress = $_POST['contactaddress'];

    $contactcity = $_POST['contactcity'];

    $contactstate = $_POST['contactstate'];

    $contactzip = $_POST['contactzip'];

    $streetaddress = $_POST['streetaddress'];

    $county = $_POST['county'];

    $city = $_POST['city'];

    $zip = $_POST['zip'];

    $singlefamily = $_POST['singlefamily'];

    $multifamily = $_POST['multifamily'];

    $condo = $_POST['condo'];

    $other = $_POST['other'];

    $sellingprice = $_POST['sellingprice'];

    $mortgageamount = $_POST['mortgageamount'];

    $valueestimate = $_POST['valueestimate'];

    $owner = $_POST['owner'];

    $homephone = $_POST['homephone'];

    $alternatephone = $_POST['alternatephone'];

    $borrowerclient = $_POST['borrowerclient'];

    $homephone2 = $_POST['homephone2'];

    $alternatephone2 = $_POST['alternatephone2'];

    $homephone3 = $_POST['homephone3'];

    $alternatephone3 = $_POST['alternatephone3'];

    $comments = $_POST['comments'];

    

    //Assign variables//

    $companynameX = "Company Name: $companyname \n";

    $contactnameX = "Contact Name: $contactname \n";

    $contactphoneX = "Contact Phone: $contactphone \n";

    $contactemailX = "Contact E-mail: $contactemail \n";

    $contactaddressX = "Contact Address: $contactaddress \n";

    $contactcityX = "Contact City: $contactcity \n";

    $contactstateX = "Contact State: $contactstate \n";

    $contactzipX = "Contact City: $contactzip \n\n";

    $streetaddressX = "Property Information\nStreet Address: $streetaddress \n";

    $countyX = "County: $county \n";

    $cityX = "City: $city \n";

    $zipX = "Zip Code: $zip \n";

    $classX = "Class: $singlefamily.$multifamily.$condo.$other \n";

    $sellingpriceX = "Selling Price: $sellingprice \n";

    $mortgageamountX = "Mortgage Amount: $mortgageamount \n";

    $valueestimateX = "Value Estimate if Refinancing: $valueestimate \n\n";

    $ownerX = "Owner Information\nOwner: $owner \n";

    $homephoneX = "Home Phone: $homephone \n";

    $alternatephoneX = "Alternate Phone: $alternatephone \n\n";

    $borrowerclientX = "Borrower/Client\nBorrower/Client: $borrowerclient \n";

    $homephone2x = "Home Phone: $homephone2 \n";

    $alternatephone2X = "Alternate Phone: $alternatephone2 \n\n";

    $homephone3x = "Real Estate Broker Information\nHome Phone: $homephone3 \n";

    $alternatephone3X = "Alternate Phone: $alternatephone3 \n";

    $commentsX = "Comments: $comments";

    

 

    

    

    

    

    //Extend variables for email//

    $totalmessage = $companynameX.$contactnameX.$contactphoneX.$contactemailX.$contactaddressX.$contactcityX.$contactstateX.$contactzipX.$streetaddressX.$countyX.$cityX.$zipX.$classX.$sellingpriceX.$mortgageamountX.$valueestimateX.$ownerX.$homephoneX.$alternatephoneX.$borrowerclientX.$homephone2x.$alternatephone2X.$homephone3x.$alternatephone3X.$commentsX;

    



    mail ("eric@sebagowebdesign.com", "Appraisal Request Email", $totalmessage);

    echo "Thank you for choosing Jordan Bay Appraisal $contactname.";

    echo " A response will be sent to $contactemail as soon as possible.";

    

?>

[jimz]

Where you have 'get values from contact form', you can just use a foreach to do the same thing...

foreach($_POST as $var => $value) {}