Стиль написания нативных SQL-запросов?

Question

[Sorbing]

На новом проекте встретил такой стиль SQL-запросов. Коллега, привык писать такие запросы. Я же привык использовать форматированный стиль с двойными кавычками. Хочу услышать мнение сообщества по поводу одного и другого подхода. Спасибо.

Первый:

class SomeClass<br>
{<br>
	...<br>
	private function getErrorCodes() {<br>
		$query = <<<SQL<br>
SELECT<br>
	`code`<br>
FROM `table`<br>
...<br>
SQL;<br>
		return DB->fetch_column($query);<br>
	}<br>
	...<br>
}<br>

Второй:

class SomeClass<br>
{<br>
	...<br>
	private function getErrorCodes() {<br>
		$query = "SELECT<br>
					`code`<br>
				FROM<br>
					`table`<br>
				...";<br>
		return DB->fetch_column($query);<br>
	}<br>
	...<br>
}<br>

Answer 1

[Hint]

Если выбирать, то вариант №3:

return DB->fetch_column('SELECT code FROM table ...');

1. Не пойму, в чем может быть плюс here-строки.
2. Почему двойные кавычки вместо апострофов? Вы туда переменные вставляете? Если да, то ваш вариант заведомо ужасен. Используйте хотя бы pdo с плейсхолдерами.

Comments

[Sorbing]

1. Согласен.
2. Вставляем, т.к. некоторые скрипты «системные» — нужна максимальная производительность, строк ~ млн. поэтому (скорее всего) отказались от PDO. Считает строка запроса вида:

return DB->fetch("SELECT code FROM table WHERE id = '$id'");

действительно ужасная?
Но тут есть и второй момент, в проекте $DB — это глобальная переменная (я заведомо опустил это дабы не уходить от основного вопроса). Вот это я считаю ужасным. Да, проект требует рефакторинга, но как всегда акцент сделан на сроки. Так что увы, это больной вопрос. Но Ваши замечания верны.

[DevMan]

действительно ужасная?

Зависит от того как инициализируется $id, и что $id из себя представляет.

[DevMan]

Если $id число, то зачем там кавычки?

return DB->fetch( 'SELECT code FROM table WHERE id = ' . (int)$id );

[Sorbing]

А если строка? Чем чревато писать запросы в таком формате?

$sql = "SELECT code FROM table WHERE name = '$name'";

[Hint]

Вставляем, т.к. некоторые скрипты «системные» — нужна максимальная производительность, строк ~ млн. поэтому (скорее всего) отказались от PDO

PDO как раз будет эффективнее, если использовать prepared statements, а потом только подставлять разные id. Двойные кавычки плохо, потому что сейчас это id, а потом name, а потом кто-нибудь забудет сделать экранирование. Апострофы гарантируют, что внутри нет переменных (и ни у кого не будет соблазна их туда вставить).

[Hint]

А если строка? Чем чревато писать запросы в таком формате?
$sql = «SELECT code FROM table WHERE name = '$name'»;

ru.wikipedia.org/wiki/Внедрение_SQL-кода

[Sorbing]

Доступно. Спасибо.

[max_rip]

А если запрос большой и длинный?

[DevMan]

max_rip

А если запрос большой и длинный?

Этот вопрос к чему относится?

[max_rip]

DevManэто я к стилю написания запроса в одну строку)

[max_rip]

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

[Hint]

А если запрос большой и длинный?

Если запрос вылезает за границу виртуального листа (в phpstorm по умолчанию 120 символов), то разбиваю его на строки (просто переносом с выравниванием), но даже в этом случае я не переношу каждое ключевое слово (FROM, WHERE, ORDER, LIMIT и т. д.).

[DevMan]

max_rip, я любой запрос оформляю похоже на 2-ой вариант. Кроме запросов вида:

SELECT COUNT(*) FROM table;
SELECT MAX( date ) FROM table;

[DevMan]

Просто если вернет mysql ошибку он обычно пишет в какой строке она.

Я использую PDO + placeholders, там с ошибками все неоднозначно.

[Sorbing]

На счет переносов с конкатенацией — я не сторонник. По мне более читабельно переносить ключевые слова внутри строки — более читабельно.

Answer 2

[hlx]

$query = '
   SELECT
       code
   FROM
        table
   WHERE
        id = ?';

использует второй вариант и начинайте с новой строки (Будет почти тоже самое что и 1-ый вариант).

+ используйте одинарные кавычки
+ используйте плейсхолдеры

Comments

[DevMan]

Зачем

начинайте с новой строки

?

[Doktor_Gradus]

Видимо, чтобы не нарушать отступы:

$query = 'SELECT
       code
   FROM
        table
   WHERE
        id = ?';

выглядит не очень

[DevMan]

можно сделать так:

$query = 'SELECT
			   code
		   FROM
				table
		   WHERE
				id = ?';

хотя это все дело вкуса.

[patashnik]

+ используйте одинарные кавычки

Почему обязательно одинарные?

[DevMan]

Потому что в одинарные кавычки нельзя воткнуть переменную.

[hlx]

Почему обязательно одинарные?

обработка быстрее + куча других мелких плюсов.

[DevMan]

обработка быстрее

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

[Павло Пономаренко]

обработка быстрее

Глупость.

Если считаете иначе — пишите на ассемблере

Answer 3

[ixSci]

Я не PHPист, но первый вариант выглядит убого. По крайней мере с подсветкой хабра. Да и приклеенный запрос к стенке, тогда как всё остальное сделано с отступами, выглядит не логично.

Answer 4

[Анатолий]

Какая вообще разница? И там и там текст, и там и там на выходе получается валидный sql запрос который выполняется. Вообще не вижу никакой разницы.

Comments

[Sorbing]

Вы, наверное, не используете форматирование? Вам не принципиальны отступы и переносы строк?
Для командной разработки обычно оговаривают стиль кодирования. На скорость это не влияет, но «читать» легко и приятно.

[Анатолий]

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

[juneuniversum]

Придерживаюсь примерно второго варианта:
Скриншот кода

Answer 5

[Владимир Чернышев]

Если бы PHP умел бы закрывать here/newdoc не с начала строки, то выбрал бы первый вариант для себя, а так нарушается логика отступов, которую я очень люблю в том числе и при смешивании языков, без чего редкое приложение на PHP обходится. Сам использую вариант с конкатенацией.

Answer 6

[Sorbing]

Основная проблема в том, что 2 разработчика с разными восприятием красоты кода работают над одним проектом. И нужно найти общий «знаменатель». В Большей части проекта (если не весь проект) написан по 2-му варианту с запросом в HERE строке — мне режет глаз, а коллега так привык. Не тяну одеяло на себя, но нужно оговорить единый стиль. Хотя, как заметил, Hint — лучше отрефакторить проект с использованием PDO.

Answer 7

[patashnik]

Любые sql-запросы — это строки. Поэтому легче было бы форматировать их так, как форматируются строки у вас в проекте. Один из популярных вариантов такой

class SomeClass
{
    ...
    private function getErrorCodes() {
        $query = "SELECT"
            . "`code` "
            . "FROM "
            . "`table` "
            . "WHERE ...";
        return DB->fetch_column($query);
    }
    ...
}

Comments

[Sorbing]

>… как форматируются строки у вас в проекте…
т.е. по первому варианту?
Находите предложенный Вами вариант лучше (удобней), чем вариант №2 из первого поста? Если да, то чем?

[DevMan]

ИМХО, этот вариант неудобен: нет визуального оформления запроса.

[patashnik]

>… как форматируются строки у вас в проекте…
т.е. по первому варианту?

Вряд ли у вас все строки форматируются в heredoc, так ведь? Я имел ввиду форматировать запросы так, если бы это были просто многострочные тексты, которые оформляются в таком случае через конкатенацию.

Находите предложенный Вами вариант лучше (удобней), чем вариант №2 из первого поста? Если да, то чем?

Этот вариант ни чем не лучше всех остальных, а вопросы удобства — это Ваше личное мнение. Лично для меня это как минимум удобно тем, что отсутствуют паразитные пробелы между операторами sql запроса. Если завтра вы захотите прикрутить профайлер запросов, то в строке будет очень много лишнего.

нет визуального оформления запроса.

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

[juneuniversum]

Отвратительнейший запрос. На работе своих коллег я бы убил за такое.
Вы только что, вместо одной переменной типа string создали в аккурат 5 таких переменных. Ява бы вас не простила, ибо на это ей еще пришлось создавать аж целых 5 объектов. Плюс двойные кавычки на ровном месте. Плюс совершенно не понятно смысл переноса каждого слова на новую строку если вы даже не форматируете их. Осталось только все в нижний регистр перевести, и можно отправлять на конкурс «Худший SQL-запрос года»