Заключение переменных в двойные кавычки, нормально или грех?

Question

[Роман Говоров]

Возник спор с программистом на работе..

$a = 55;
$b = 65;
$res = $a + $b;

echo "Ответ $res";

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

Кто прав?
Стоит ли писать так как пишу я или же придерживаться такому синтаксису

$a = 55;
$b = 65;
$res = $a + $b;

echo 'Ответ '.$res;

Comments

[alex-1917]

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

[Роман Говоров]

alex-1917, как положено это как?) Я же сказал, в документации написано что именно так делать можно, и не написано что НЕ НУЖНО)

[alex-1917]

Роман Вопроскин, как положено - это как лично для меня положено. с одинарными кавычками и с точками...
реально удобно.
например, когда отсылаешь начинашке для корреткировки, чтоб они глаза не выпучивали, так как у них банальный нотпадплюсплюс. и там подсветка обычно сходит с ума вот оттакой хрени

echo "Ответ {$res}";

спецом прошерстил движки (12шт), НИГДЕ хрень

echo "Ответ {$res}";

не используется.
ну и представьте, как там будут выглядеть такие вкрапления?

Answer 1

[Abdula Magomedov]

Нормально. Если я не ошибаюсь в PHP7 этот синтаксис специально оптимизировали. Так что он даже быстрее склеивает строки.
php.net/manual/ru/language.types.string.php#langua...

UPD. Еще добавлю.
В PHP эта возможность "всегда" была. А вот в JavaScript только "недавно" добавили. Не будь это опция такой полезной, вряд ли его стали добавлять в "современный" JS.
https://habr.com/post/252323/#template-strings

Answer 2

[JhaoDa]

Можно. Я, например, использую и не вижу проблем. И лучше использовать синтаксис с усами — как по мне, так повышает читабельность помимо прочих плюшек:
echo "Ответ {$res}";

Comments

[galliard]

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

[JhaoDa]

Роман Вопроскин если не секрет, почему этот ответ перестал быть решением?

Answer 3

[Сергей Соколов]

В некоторых IDE подсветка синтаксиса не работает для случая переменной-в-строке. Для меня лично это основная причина избегать такого стиля.

А вообще это «нормально».

Comments

[JhaoDa]

Что это за IDE, если не секрет?

[DevMan]

JhaoDa, зачем сразу IDE?
код не всегда открывается в настроенном редакторе.
да тот же тостер не подсвечивает переменные в строках.

[JhaoDa]

DevMan, автор сказал про IDE.

[DevMan]

JhaoDa, я дополнил его ответ.

[Сергей Соколов]

JhaoDa, Coda под макось. Но сведения устарели, уже подсвечивается, сейчас проверил ))

[galliard]

В некоторых IDE подсветка синтаксиса не работает для случая переменной-в-строке. Для меня лично это основная причина избегать такого стиля.

А по моему это скорее причина сменить IDE, разве нет?

Answer 4

[Олег]

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

А спорить можно или нельзя можно долго. Только в правильном проекте такого вообще не должно встречаться.

А если нужно, собирать строки например для смс уведомления

$mes =  sprintf("Уважаемый, %s! Стоимость вашего заказа %d руб.",$username,$cost);

просто сравните на сколько проще потом будет менять тексты, его можно даже вынести в таблицу

Comments

[PendalF89]

ИМХО "швейцарский нож" не всегда оправдан, все от проекта зависит.

[Олег]

php.net/manual/ru/function.sprintf.php
это для чистого PHP

// вывести 
printf('Record %d created ', $id_ins);
// вернуть строчку
$a = sprintf('Record %d created ', $id_ins);

такое потом легче перевести.

[galliard]

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

А твоя основная проблема в том, что ты отвечаешь не по теме и что-то там подозреваешь (совершенно не обоснованно, кстати).

Человек написал сферический код в вакууме, дабы описать суть проблемы, а ты из этого какие-то глобальные выводы сделал.

[Олег]

galliard, сферического кода не бывает. Есть конкретные задачи, когда нужно вставить значение переменной
в строку. функция printf подходит для этого лучше, чем echo "{$var} ";
так как маску "%s" можно смело делать редактируемой пользователем , а вот "{$var}" - чревато проблемами. Также маски легко вынести в отдельные файлы и менять только их не переписывая код
каждый раз.

А если код какая-то времянка, то обсуждать правильность ее реализации смысла нет.

Если же речь идет о шаблонах, то twig, smarty (устарел), да хоть тот же нативный php
<div><?= $var ?></div>
читается лучше чем стиль "HTML в PHP"

[galliard]

Олег Муравейко, мое негодование вызвали первые 3 строчки твоего сообщения, которые были соовсем уж не по теме. Что касается функции printf - то это сугубо твое мнение, с которым я, конечно, тоже не согласен, но оно хотя бы по теме.

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

Твой вариант красив потому, что в нем 5 слов и 2 переменные. Если переменных станет в разы больше, ну например 10, то начинаешь путаться во множестве %s и %d. А если еще и текста много, то их нужно будет еще найти, и IDE тебе тут вряд ли поможет. Вот если бы им можно было имена задать... Но это уже за рамками стандартной библиотеки php.

О шаблонизаторах типа twig и smarty речи не идет, поэтому их не обсуждаем.

[Олег]

galliard, я правильно понимаю, речь идет о написание программ без фреймворков и классов ?
Я бы понял этот спор во времена до PHP 4.

Хорошо, 10 переменных, это уже шаблон.
Пусть его нужно собирать программно.
Но в шаблонах требуется фильтровать данные.
В результате строка приобретает вид " что-то".int($val)."чтото"; int()-просто для примера

[galliard]

Олег Муравейко, речь идет безотносительно фреймворков и классов. Просто задача по формированию строки. Все, что вокруг - не важно.

Программная сборка шаблона - опять же другая задача, поэтому тоже не обсуждаем)))