Лучше ли делать меньше обращений к базе данных и выводить результаты в массиве?

Question

Лучше ли делать меньше обращений к базе данных и выводить результаты в массиве?

0

Я пытаюсь создать более сжатый способ сделать сотни вызовов db. Вместо того, чтобы писать весь запрос каждый раз, когда я хотел вывести одно поле, я попытался перенести код в класс, который выполнял весь запрос. Это класс, который у меня есть до сих пор:

class Listing {

/* Connect to the database */
private $mysql;

function __construct() {
    $this->mysql = new mysqli(DB_LOC, DB_USER, DB_PASS, DB) or die('Could not connect');
}
function getListingInfo($l_id = "", $category = "", $subcategory = "", $username = "", $status = "active") {
    $condition = "`status` = '$status'";
    if (!empty($l_id)) $condition .= "AND `L_ID` = '$l_id'";
    if (!empty($category)) $condition .= "AND `category` = '$category'";
    if (!empty($subcategory)) $condition .= "AND `subcategory` = '$subcategory'";
    if (!empty($username)) $condition .= "AND `username` = '$username'";
    $result = $this->mysql->query("SELECT * FROM listing WHERE $condition") or die('Error fetching values');
    $info = $result->fetch_object() or die('Could not create object');
    return $info;
}
}

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

$listing = new Listing;
echo $listing->getListingInfo('','Books')->title;

Это выводит заголовок первого листинга в категории "Книги". Но если я хочу вывести цену этого листинга, я должен сделать еще один вызов getListingInfo(). Это делает другой запрос на db и снова возвращает только первую строку.

Это намного проще, чем писать весь запрос каждый раз, но я чувствую, что часто звоню в db. Есть ли лучший способ вывода данных из моего класса и по-прежнему быть кратким в доступе к нему (возможно, вывести все строки в массив и вернуть массив)? Если да, то как?

codescribblr 14 июнь 2010, в 19:07

Источник

0

какое количество строк в этой таблице?
Your Common Sense 14 июнь 2010, в 16:54

Теги:

php

mysql

oop

resultset

4 ответа

7

У вас действительно проблема с производительностью?

Если ваша текущая настройка работает и не страдает от проблем с производительностью, я бы не стал ее трогать.

Такой вид абстракции доступа к БД, вероятно, станет проблемой обслуживания и, вероятно, не поможет производительности.

Кроме того, вы подвержены SQL-инъекции.

Ben S 14 июнь 2010, в 14:56

0

+1 за указание на уязвимость SQL-инъекции.
ZoogieZork 14 июнь 2010, в 16:57
0

Я думаю, что загрузка точно такой же снова, чтобы получить столбец, который вы уже получили в запросе, неэффективна. Сохранение результата в переменной - очень простой способ избежать этого.
2ndkauboy 14 июнь 2010, в 16:58
0

@ Кау-Бой: Хранение значений в переменных внутри PHP не масштабируется. Если требуется механизм кэширования, используйте реальное решение, такое как memcache.
Ben S 14 июнь 2010, в 17:17
0

Это может быть не масштабируемым, но получить значение из объекта или массива дешевле, чем получить его из базы данных, если вы уже получили его. В противном случае он не должен использовать «SELECT *», а только выбрать столбец. И если вы не используете транзакции, безопаснее получить все столбцы в одном запросе, чем получать их в разных запросах, поскольку они могли бы измениться.
2ndkauboy 14 июнь 2010, в 17:46
0

Ваша база данных должна кэшировать частые или недавние запросы в любом случае. Это просто не обязательно. Затраты на обслуживание этого типа кэширования не стоят минутного прироста производительности.
Ben S 14 июнь 2010, в 18:03
0

@Ben S: То есть вы говорите, что если я запрашиваю базу данных mysql 10000 раз за обновление одной страницы, mysql фактически кэширует запрос и использует кеш для вывода данных? Означает ли это, что мне никогда не нужно беспокоиться о том, чтобы слишком часто запрашивать базу данных, потому что MySQL все равно кэширует общие запросы?
codescribblr 14 июнь 2010, в 18:13
0

Как правило, да: dev.mysql.com/doc/refman/5.1/en/query-cache.html
Ben S 14 июнь 2010, в 18:19

Показать ещё 5 комментариев

0

Либо кешируйте результат во внутреннем var

Или Прокомментируйте это с предупреждением и объясните, как пользователи могут копировать результат в var вместо вызова его снова и снова с теми же параметрами

redben 14 июнь 2010, в 14:37

0

Да, это слишком часто вызывает db.

Несколько решений

1) поместите информацию о листинге в переменную

2) кэширует результаты в хэш-карте или словаре (будьте осторожны при утечке памяти)

Peter Tillemans 14 июнь 2010, в 14:23

Ещё вопросы

какое количество строк в этой таблице?
+1 за указание на уязвимость SQL-инъекции.
Я думаю, что загрузка точно такой же снова, чтобы получить столбец, который вы уже получили в запросе, неэффективна. Сохранение результата в переменной - очень простой способ избежать этого.
@ Кау-Бой: Хранение значений в переменных внутри PHP не масштабируется. Если требуется механизм кэширования, используйте реальное решение, такое как memcache.
Это может быть не масштабируемым, но получить значение из объекта или массива дешевле, чем получить его из базы данных, если вы уже получили его. В противном случае он не должен использовать «SELECT *», а только выбрать столбец. И если вы не используете транзакции, безопаснее получить все столбцы в одном запросе, чем получать их в разных запросах, поскольку они могли бы измениться.
Ваша база данных должна кэшировать частые или недавние запросы в любом случае. Это просто не обязательно. Затраты на обслуживание этого типа кэширования не стоят минутного прироста производительности.
@Ben S: То есть вы говорите, что если я запрашиваю базу данных mysql 10000 раз за обновление одной страницы, mysql фактически кэширует запрос и использует кеш для вывода данных? Означает ли это, что мне никогда не нужно беспокоиться о том, чтобы слишком часто запрашивать базу данных, потому что MySQL все равно кэширует общие запросы?
Как правило, да: dev.mysql.com/doc/refman/5.1/en/query-cache.html

2ndkauboy · Accepted Answer · 2010-06-14T14-09-00.000Z

Вы должны иметь возможность хранить весь объект из запроса в переменной и затем получать доступ к отдельным значениям из этого объекта:

$object = $listing->getListingInfo('','Books');
$title = $object->title;
$price= $object->price;

Но вы также можете использовать fetch_assoc() и вернуть весь Assiciative массив:

$array = $listing->getListingInfo('','Books');
$title = $object['title'];
$price= $object['price'];

Это даст вам те же результаты, а также только один запрос к БД.

EDIT. Если функция getListingInfo() является единственной функцией, вы должны подумать о следующем:

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

Используя эту новую функцию, вы можете получить каждую строку, соответствующую вашему запросу.