Блок if внутри оператора echo?

Question

Я подозреваю, что это недопустимо, потому что я получаю ошибку "Parse error: синтаксическая ошибка, неожиданный T_IF in...". Но я не мог найти способ достичь своей цели. Вот мой код:

<?php 

  $countries = $myaddress->get_countries();

  foreach($countries as $value){
    echo '<option value="'.$value.'"'.if($value=='United States') echo 'selected="selected"';.'>'.$value.'</option>';
  }
  ?>

Что он делает, так это отображает список стран в выбранном элементе и устанавливает США по умолчанию. Я не работаю, к сожалению...

Answer 1

Вы хотите использовать троичный оператор, который действует как сокращенный оператор IF/Else:

echo '<option value="'.$value.'" '.(($value=='United States')?'selected="selected"':"").'>'.$value.'</option>';

Answer 2

Вы всегда можете использовать ( <condition> ? <value if true> : <value if false> ) синтаксис (он называется троичным оператором - спасибо Марку за напоминание:)).

Если <condition> верно, заявление будет оценено как <value if true>, Если нет, это будет оценено как <value if false>

Например:

$fourteen = 14;
$twelve = 12;
echo "Fourteen is ".($fourteen > $twelve ? "more than" : "not more than")." twelve";

Это так же, как:

$fourteen = 14;
$twelve = 12;
if($fourteen > 12) {
  echo "Fourteen is more than twelve";
}else{
  echo "Fourteen is not more than twelve";
}

Answer 3

Используйте троичный оператор:

echo '<option value="'.$value.'"'.($value=='United States' ? 'selected="selected"' : '').'>'.$value.'</option>';

И пока вы на это, вы могли бы использовать printf чтобы сделать ваш код более читабельным / управляемым:

printf('<option value="%s" %s>%s</option>',
    $value,
    $value == 'United States' ? 'selected="selected"' : ''
    $value);

Answer 4

Ради удобочитаемости это должно быть что-то вроде

<?php 
  $countries = $myaddress->get_countries();
  foreach($countries as $value) {
    $selected ='';
    if($value=='United States') $selected ='selected="selected"'; 
    echo '<option value="'.$value.'"'.$selected.'>'.$value.'</option>';
  }
?>

желание засунуть ВСЕ в одну строчку - это смерть, чувак. Пиши отчетливо.

Но есть другой способ, лучший. Нет необходимости использовать эхо вообще. Научитесь использовать шаблоны. Сначала подготовьте свои данные, и только потом отобразите их.

Часть бизнес-логики:

$countries = $myaddress->get_countries();
$selected_country = 1;    

Часть шаблона:

<? foreach($countries as $row): ?>
<option value="<?=$row['id']?>"<? if ($row['id']==$current_country):> "selected"><? endif ?>
  <?=$row['name']?>
</option>
<? endforeach ?>