Как написать условие внутри переменной?

Question

[Дима Паутов]

Наверно не обычный вопрос.
У меня есть переменная, которая содержит такой вот код:

$message = '
			<table>
				<tr>
					<td>Имя: '.$_POST["name"].'</td>
				</tr>
				<tr>
					<td>Телефон '.$_POST["tel"].'</td>
				</tr>
				<tr>
					<td>Дата '.$_POST["date"].'</td>
				</tr>
			</table>
		';

Да, это для формы отправки сообщения.

Например, если я не заполню в форме поле с телефоном, то при отправке должны придти на почту только строка с именем и датой. Как можно решать данный вопрос?

Comments

[Дмитрий Энтелис]

Важно понимать: переменная не содержит код. Переменная содержит текстовую строку.

[FanatPHP]

Осталось только узнать, что такого ужасного случится, если придет строка с пустым телефоном.

Answer 1

[Max Payne]

???
if ???

Comments

[Дима Паутов]

да, но как его в переменную засунуть? У меня так и не вышло!

[Max Payne]

Дима Турков: А зачем его в переменную засовывать? О_О Самый простой вариант:

if(!$_POST['name']) {
$message = '
      <table>
        <tr>
          <td>Телефон '.$_POST["tel"].'</td>
        </tr>
        <tr>
          <td>Дата '.$_POST["date"].'</td>
        </tr>
      </table>
    ';
}

if(!$_POST['tel']) {
$message = '
      <table>
        <tr>
          <td>Имя: '.$_POST["name"].'</td>
        </tr>
        <tr>
          <td>Дата '.$_POST["date"].'</td>
        </tr>
      </table>
    ';
}
if(!$_POST['date']) {
$message = '
      <table>
        <tr>
          <td>Имя: '.$_POST["name"].'</td>
        </tr>
        <tr>
          <td>Телефон '.$_POST["tel"].'</td>
        </tr>
      </table>
    ';
}

[Max Payne]

Это жуткий говнокод и лучше его не использовать, но как пример -- сгодиться.

[Дима Паутов]

Arti Markelov: ну для простой отправки 2-3х полей, достаточно. Например тут php.net/manual/ru/function.mail.php, так же, текст сообщения лежит в переменной(руководствовался от туда). Насчет вашего примера, спс, пробовал, но потом подумал, а если полей будет с десяток? Что, на каждое поле свое условие?

[Max Payne]

$messages = '
      <table>';
if($_POST['name']) {
$messages. = '       <tr>
          <td>Имя: '.$_POST["name"].'</td>
        </tr>';
}
If($_POST['tel']) {
$messages .= '
        <tr>
          <td>Телефон '.$_POST["tel"].'</td>
        </tr>';
}
if($_POST['date']) {
$messages .= '
        <tr>
          <td>Дата '.$_POST["date"].'</td>
        </tr>
';
}
$messages .= '      </table>';

[Дима Паутов]

Arti Markelov: Большое спасибо за потраченное время!

[FanatPHP]

Arti Markelov: честно говоря, это было в основном насчет первого варианта, который был совсем уж откровенной халтурой

[Max Payne]

FanatPHP: Я с этим полностью согласен -- пример был предоставлен, чтобы продемонстрировать СЛОЖНЕЙШУЮ управляющую конструкцию If.

Answer 2

[FanatPHP]

Отвечу на вопросы, которые остались без ответа

1. На случай, если никто не умрет, получив сообщение с пустой ячейкой "Телефон", оставляем код как есть.
2. На случай, если умрет и полей будет десяток

$fields = array(
    'name'  => 'Имя',
    'phone' => 'Телефон',
    // ...
); 
$message = '<table>';
foreach ($fields as $key => $title)
{
    if ($value = trim($_POST[$key]))
    {
        $message .= "<tr><td>$title:</td><td>$value</td></tr>";
    }
}
$message .= '</table>';

Comments

[Дима Паутов]

А вот это уже интересней решение. Спс!

Answer 3

[acspro]

Правильно будет так думаю:

$message = '
      <table>
        '.(isset($_POST['name'])&&trim($_POST['name'])!=''?'
		<tr>
          <td>Имя: '.$_POST["name"].'</td>
        </tr>
		':'').'
         '.(isset($_POST['tel'])&&trim($_POST['tel'])!=''?'
		<tr>
          <td>Телефон '.$_POST["tel"].'</td>
        </tr>
		':'').'
		 '.(isset($_POST['date'])&&trim($_POST['date'])!=''?'
        <tr>
          <td>Дата '.$_POST["date"].'</td>
        </tr>
		':'').'
      </table>
    ';

Comments

[FanatPHP]

ужас, летящий на крыльях ночи

[acspro]

FanatPHP: Чего? Красиво ж вроде, и точки всякие есть и скобочки и вопросики.

[acspro]

у меня быстрее работает)))) чем отмечено правильным))))

[FanatPHP]

про быстроту ты бы лучше помолчал. такие заявления сразу выдают малолетних недоучек.

[FanatPHP]

а вот с красотой как раз-то все плохо. от этих скобочек глаза болят

[acspro]

FanatPHP: ты бы не грубил. Сделай профилирование да посмотри сколько памяти жрет твой код и сколько работает он и мой. А скобочки прекрасно смотрятся. И решение думаю более приемлемое для данного случая, хотя твое более универсальное,

[FanatPHP]

я не грублю, а констатирую факт. дураки, которые берутся рассуждать о скоростных преимуществах одной синтаксической формы над другой, разбираются в оптимизации как свинья в апельсинах.

[acspro]

FanatPHP: Заметь, ты не можешь аргументировано ответить. вместо этого ты меня 2 раза оскорбил. Если встретимся, я припомню тебе. В интернете не нужно много смелости. А в данном случае речь не идет о синтаксической форме. Мой вариант действительно работает быстрее и меньше памяти занимает. Это факт. Да оптимизация бывает разной. Зачем громоздить для конкретной задачи массив, затем проходить его? Докажи аргументированно что я не прав. Просто приведи факты или достойно промолчи.

[FanatPHP]

разговаривая на тему уровня сколько будет 2х2 не нужны факты. Если человек изначально не понимает смысла цифр, которые видит, если у него оптимизация головного мозга, то и говорить с ним не о чем. вас очень много таких бегает, виртуальных оптимизаторов. Вы не понимаете, как работает РНР, вы не умеете тестировать нагрузку, кроме как "миллион бессмысленных циклов", но увидев какую-нибудь абсолютно виртуальную и бессмысленную, вообще ни на что не влияющую цифру, вдруг проникаетесь к ней таким почтением, что это превращается в религию. Разговаривать с вами бесполезно. Я для себя решил, что это какое-то свойство мозга, человек или понимает относительность величин, или нет. Если понимает, то он оптимизирует то, что действительно играет роль. Если не понимает - то дрочит на одинарные кавычки. У каждого свой уровень.

[acspro]

FanatPHP: Как ни пиши в данном случае я прав и ты это знаешь.

На самом деле есть дебилы, которые везде все усложняют. Такие дебилы как правильно знают php и ООП, имеют практику достаточную чтобы писать более менее сложные решения но не понимают его архитектуру в целом. И когда такое дол..ебы превращают простейшее решение в монстра - их уже не исправить. Стремление к универсальности кода и применимости его в любой ситуации напрочь сносит у них все остальные моменты. Решать задачу нужно конкретно для каждой ситуации и оптимально. Вот приведи хоть 1 довод что ты прав? Ты же понимаешь что не прав! И что то берет под сомнение то что ты писал высоконагружаемые проекты, вероятно прочитал пар статей на хабре об оптимизации и архитектуре web приложений в целом.
-------------------------
Самое интересное что ты больше играешь на публику своими комментариями. Ни ума ни смелости на большее не хватает.

[FanatPHP]

если бы вы, дети, только знали, насколько однообразны в своей риторике. и с какой уморительной серьезностью относитесь к задаче "победить в споре" :)

[acspro]

FanatPHP: тут нету спора, ты же просто убегаешь от ответа. Тебе задали вопрос, ты на него не ответил. Вот посмотри весь диалог. Вместо ответов ты из себя корчишь крутого программиста, а признать свою ошибку не можешь. Вот и все.

[Дима Паутов]

acspro: FanatPHP: пожалуйста, перестаньте! Я очень рад что вы мне помогли! И благодаря этому я смог разглядеть разные подходы к моей проблеме!

Answer 4

[LittleFatNinja]

$message = '//' . 
if () {} 
. '//';