0

Есть два куска php кода. Первый с кавычками.

echo "<option value='".$key['id']."'>".$key['name']." (".$key['width']." x ".$key['height'].")"."</option>";

И второй без кавычек.

echo "<option value='$key[id]'>$key[name] ($key[width] x $key[height])</option>";

Оба работают. Вопрос такой: Является ли второй вариант приемлемым с точки зрения правильности кода? Можно ли использовать такие конструкции повсеместно? Спасибо заранее за консультацию.

7
  • @Cactus на самом деле те тесты сделаны очень давно. Сейчас надо бы заново замерить и причем на разных версиях 25 авг 2017 в 12:03
  • @АлексейШиманский, да, возможно. Но спасибо их автора и за то, что есть!)
    – user242433
    25 авг 2017 в 12:03
  • @МенеджерТипографии странно, а где одинарные кавычки? ведь вся соль была раньше между применением одинарных кавычек и конкатинации VS двойных кавычек.... а всё потому, что в них была разница, о которой вы видимо не знаете 25 авг 2017 в 12:04
  • @Cactus неактуальные данные - бессмысленны) 25 авг 2017 в 12:06
  • Кроме того нет варианта "{$test}". В общем да, тесты хорошие, но морально устаревшие, а что касается сабжа, то как по мне, второй вариант ужасен (аргументов не будет). 25 авг 2017 в 12:07

2 ответа 2

5

Вижу, что вы не присваиваете результат переменной, а выводите его, поэтому оба «неверно». PHP не для этого изначально был создан, чтобы разметку выводить.

Правильно и удобно для поддержки проекта в дальнейшем будет:

<option value="<?= $key['id'] ?>">
    <?= $key['name'] ?> (<?= $key['width'] ?> x <?= $key['height'] ?>)
</option>

Старайтесь изначально разделять логику от представления.

Так же когда начнете шаблонизаторы использовать — будет проще под них разметку изменить.

Изучите основы синтаксиса языка PHP, в частности:

5
  • <?= $key['name'] ?> (<?= $key['width'] ?> x <?= $key['height'] ?>) - вот это нихрена не правильно и не удобно. Такой код сложно как писать (руки отваливаются от постоянных <?=?>) так и читать. Лучше все-таки написать что-то вроде <?= "$key[name] ($key[width] x $key[height])" ?>. 25 авг 2017 в 12:34
  • @PavelMayorov правильнее вообще писать {{ $key['name'] }} ({{ $key['width'] }} x {{ $key['height'] }}), но в любом случае объединять встроки не стоит. разметка дело такое - всегда понадобится втюхать пару тегов между. Так что пусть будет одна переменная - один вывод 25 авг 2017 в 12:36
  • @PavelMayorov ахахах, ну да, ну да. Я только сокращенным синтаксисом даже не пользуюсь (ответ подкорректировали), чтобы поддерживаеть единообразие. Бывает, что нужно что-то в несколько строчек вывести, например, <?php $num = 123; $str = "String"; ?>. Но против них ничего особо не имею. 25 авг 2017 в 12:38
  • С Павлом согласен, когда дело дойдет до сложной разметки — по другому на это взгляните. Разметка должна быть «чистой». А что если у тегов классы будут, несколько классов, а с id? А если именование будет по БЭМ методологии? Дело конечно ваше. Пуская ответ для других будет — новички прочтут, может для других будет полезно. 25 авг 2017 в 12:42
  • Еще вопрос в команде: если в команде принято писать так <?=, значит либо я пишу так, либо... ну вообщем понятно. И когда человек будет ваш код читать — очень важно, чтобы максимально с первого взгяда было понятно что и как... 25 авг 2017 в 12:45
0

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

По мне первый вариант предпочтительный. Я по крайне мере так пишу.

Плюс первого варианта: Все редакторы правильно подкрашивает переменные. Плюс второго варианта: Код короче получается. Минус второго варианта: Если вы захотите вставить символ - " то необходимо писать \"

2
  • Технически верным тоже считаю первый вариант. И пишу так же первым. А тут пошли ну уж больше длинные вставки в одно ECHO перемнных, объектов и решил задаться вопросом насколько это "легально" в среде разработки. 25 авг 2017 в 12:07
  • @МенеджерТипографии писать нужно так, чтобы потом было просто понимать и поддерживать код. В этом разрезе наиболее правильным является вариант из ответа AlexanderBragin. Если же сравнивать два ваших варианта, то я бы предпочел вариант 2, так как он банально человекочитаемей. А подсвечивать переменные внутри текста сейчас умеет любая среда разработки.
    – rjhdby
    25 авг 2017 в 12:27

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.