How to add a simple IF statement in this PHP file?

Question

Question by a novice, here:

I have a simple form returning a person's name, company, and address among other things.

Right now, these are returned on separate lines. If there is no entry in the "company" field, there is a blank line in the email that is sent.

I'd like to make an IF statement so that if there is no entry in the "company" field, that "address" will come back right after "name" without an extra space, but will include the "company" info if that field is filled-in.

The relevant part of the PHP code looks like this:

$name = $_POST['name'];
$company = $_POST['company'];
$address = $_POST['address'];
$city = $_POST['city'];
$state = $_POST['state'];
$zip = $_POST['zip'];
$email = $_POST['email'];
$optin = $_POST['optin'];
$comments = ($_POST['comments']);



$body = <<<EOD

Please send samples to:

$name
$company
$address
$city, $state $zip

Email: $email

Opt-In to occasional email list?: $optin

Comments: $comments

EOD;
$headers = "From: $email\r\n";
$success = mail($webMaster, $emailSubject, $body,
$headers);

I will really appreciate your help!

deceze · Accepted Answer · 2010-06-28 03:41:45Z

2

You'll have to break out of HEREDOC for this, or prepare the value beforehand, but this is nice and short:

echo join("\n", array_filter(array($name, $company, $address)));

answered Jun 28, 2010 at 3:41

deceze♦

525k89 gold badges807 silver badges954 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

middus · Accepted Answer · 2010-06-28 03:35:39Z

0

What about this?

$name = $_POST['name'];
//...

if(empty($company)){
  $nameAndCompany = $name;
}else{
  $nameAndCompany = $name."\n".$company;
}
// or using the ternary operator, because it's a rather simple if
$nameAndCompany = (empty($company))?$name:$name."\n".$company;
$body = <<<EOD
...

$nameAndCompany
$address
$city, $state $zip
...

\n in double quotes means newline. See the php.net documentation for more info on the ternary operator.

answered Jun 28, 2010 at 3:35

middus

9,1291 gold badge34 silver badges33 bronze badges

2 Comments

Yanick Rochon Over a year ago

the array_filter trick is still the cleanest solution (no IF...)

middus Over a year ago

I'm glad I could help. @Yanick: Why is an "IF" not clean?

Yanick Rochon · Accepted Answer · 2010-06-28 03:47:26Z

0

You are using heredoc syntax in your PHP script. You cannot use IF statements in there. But you can use other methods :

close your PHP tag, and use <?= $var ?> in your code
use templates and replace keywords with preg_replace(), etc.
put all your strings into an array and use implode("\n", $arr) to construct a nice string out of it
concatenate your string directly

The easiest way is the first one (EDITED with array_filter from another answer):

$name = $_POST['name'];
$company = $_POST['company'];
$address = $_POST['address'];
$city = $_POST['city'];
$state = $_POST['state'];
$zip = $_POST['zip'];
$email = $_POST['email'];
$optin = $_POST['optin'];
$comments = ($_POST['comments']);



ob_start();
?>
Please send samples to:

<?= implode("\n", array_filter($name, $company, $address)) ?>
<?= $city ?>, <?= $state ?> <?= $zip ?>

Email: <?= $email ?>

Opt-In to occasional email list?: <?= $optin ?>

Comments: <?= $comments ?>

?>
$body = ob_get_clean();  // return the output since ob_start()

$headers = "From: $email\r\n";
$success = mail($webMaster, $emailSubject, $body, $headers);

edited Jun 28, 2010 at 3:47

answered Jun 28, 2010 at 3:40

Yanick Rochon

53.8k30 gold badges147 silver badges232 bronze badges

3 Comments

middus Over a year ago

Actually, the OP is already using kind of a template: a heredoc string. I'd say that using the output buffer is a rather quirky way to achieve what the OP wants to do. I like the array way you suggested, though.

Yanick Rochon Over a year ago

yes, I use arrays personally. Pretty useful when you got many embedded IF's and need to construct output with them...

JustBobF Over a year ago

But, I tried the IF statements and they did work. Both of Middus' solutions worked. Thank you all for so much help!!!!!

superUntitled · Accepted Answer · 2010-06-28 03:54:06Z

0

I edited this to use \n instead of <br />

$optin = $_POST['optin'];
$comments = ($_POST['comments']);



$body = "Please send samples to:";
if($_POST['name'] != '') $body .= "\n".$_POST['name'];
if($_POST['company'] != '') $body .= "\n".$_POST['company'];
if($_POST['address'] != '') $body .= "\n".$_POST['address'];
if($_POST['city'] != '') $body .= "\n".$_POST['city'];
if($_POST['state'] != '') $body .= ", ".$_POST['state'];
if($_POST['zip'] != '') $body .= " ".$_POST['zip'];
if($_POST['email'] != '') $body .= "\n\nEmail: ".$_POST['email'];
$body .= "Opt-In to occasional email list?: $optin";
$body .= "\n\nComments: $comments";


$headers = "From: $email\r\n";
$success = mail($webMaster, $emailSubject, $body,
$headers);

edited Jun 28, 2010 at 3:54

answered Jun 28, 2010 at 3:24

superUntitled

22.6k30 gold badges86 silver badges113 bronze badges

3 Comments

Mala Over a year ago

you'll want to use "\n"s instead of "<br />"s: his script uses carriage returns not HTML linebreaks

middus Over a year ago

He does not seem to want HTML breaks.

JustBobF Over a year ago

"\n"s sound great! I had HTML, but was talked out of it earlier today. Thank you!

Collectives™ on Stack Overflow

How to add a simple IF statement in this PHP file?

4 Answers 4

Comments

2 Comments

3 Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

3 Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related