Меня давно мучает этот паттерн: Код (Text): Запрос на базу В случае библиотеки часто делается массив Массив обрабатывается в пользовательском коде (htmlspecialchars и прочее, когда простое, когда сложное) Скармливаем шаблону и он это всё раскручивает в цикле опять для вывода. В итоге имеем 2 или 3 цикла. А что если заюзать лямбда-функции PHP 5.3? При составлении запроса составляем массив вида array('field' => function ($field) {}, 'field2' => function ($field) {}, ....). Метод выборки генерирует объект, который мы передаём в шаблон и там его раскручиваем как обычный массив (т.е. это должен быть итератор я так понимаю). Все escape & co выполняются прямо на лету. Правда работает не всегда, т.е. это должен быть отдельный метод, типа $db->queryIterator() (имя надо придумать, пока в голову не пришло). Discuss?!
Подожди-подожди. Я не очень понял Давай, для примера, возьмем смарти. PHP: <?php $DB->select(array('select' => 'data', 'from' => 'table')); $OUT->assign('someData', array()); while($d = $DB->fetch()) $OUT->append('someData', $d); Раз крутим, так? Потом в шаблоне {foreach key=key item=d from=$someData} {/foreach} Два крутим. А как ты предлагаешь, объясни больше, пожалуйста.
Либо так же через foreach (если это Iterator + ArrayObject) либо while ($v = $dataObject->fetch()): .... endwhile;
В порядке бреда: Передавать в шаблон некий объект инкапсулирующий в себе ресурс. PHP: <?php $res=$db->query($sql); $res->setFetchMethod(MyResourse::ASSOC); $myTpl->arrData=$res; PHP: <?php class MyResourse implements Iterator{ ... protected function getRes(){ if (!this->res){ $this->res=$this->db->query($this->query); } return $this->res; } public function current (){ return $this->row; } public function key (){ return $this->key; } public function next ( void ){ $res=$this->getRes(); $fetchMethod=$this->fetchMethod; $this->row=$this->db->$fetchMethod($res); $this->key++; } public function rewind (){ $this->res=null; } public function valid ( void ){ return (bool)$this->row; } } Тогда и запрос можно посылать лениво (хотя вряд ли оно особо надо).
Некоторые шаблонизаторы так и делают - работают с объектом. Ну, в контексте какой-то системы, конечно. Вроде в http://wiki.limb-project.com/ так. Давно его копал, точно не помню.
1. Зачем каждому полю по лямбде? Можно одну лямбду, которая обрабатывает одну запись. 2. Чем плохи 2-3 цикла? Программисту придётся реализовывать каждый раз только от 0 до 1, остальные два внутри библиотеки БД и шаблонизатора. Производительностью? Пользовательский итератор и лямбды будут намного медленнее даже 3-х циклов по встроенным типам данных.
я делал и без лямда функций через обычные магические методы, тобишь как бы в шаблонизаторе происходил фетч. Отказался потом, что мне надо было делать фетч всего до того, как шаблонизатор вообще начнет присутствовать ибо prepare -> stmt механизм не позволяет не последовательно пользоваться stmt... А делалось всё просто - итератор надо mysqli_result, над ним коллекция, тоже итератор, с ленивым __get
admyx не на прямую, шаблонизатор имеет дело с коллекцией, фактически внутри него эта коллекция массив, а данные проходят через преобразования в каком-то методе, типа _onGetLogin в самом __get... Да и вообще тут
Костян Пыщь, дык мы и дискутируем =) Хз. конечно. Что у IPB, что у меня (ну ты помнишь) шаблонизатор играет только свою роль
прикол в том, что когда я не пробежал в коллекции по всем элементам, и попользовался второй коллекцией, то в первой на фечнутые данные я потерял. Так как я хочу прозрачно обращаться к коллекциям в разные моменты времени, то надо делать фечь всем результатам и класть в буфер, а это способствует появлению нового массива и нового цикла. Последнее удобнее, потому что оно намного проще и понятнее и не надо тут ничего придумывать.
Что плохого в 2-3-10 циклах ? Разница между PHP: <?php for($i=0; $<100; $i++) { $a = $b; $b = $a; } ?> и PHP: <?php for($i=0; $<100; $i++) { $a = $b; } for($i=0; $<100; $i++) { $b = $a; } ?> совершенно несущественная (не проверял, но теоретически)
у меня тоже он играл такую же роль, но он оперировал коллекциями так, что надо было 2 записи из одной коллекции 2 из другой, следующие 2 из одной, потом опять из другой.... И всё, вся малина при этом уходила, и выводилось всего лишь 4 записи, потому что коллекции не рассчитаны были на такое обращение... От такие дела!
Vladson ну чисто теоретически там будет: PHP: $i++ $i < 100 ? $a = $b $b = $a против PHP: $i++ $i < 100 ? $a = $b $i++ $i < 100? $b = $a
Vladson В данном случае не очень хороший пример, потому что когда мы вытаскиваем данные из ресурса, они копируются в массив. Ну и у тебя просто переменные содержащие 1 элемент. Смысл в снижении накладных расходов на циклы + там всё же логика по сложнее будет. Обычно выглядет так (схематично) PHP: <?php $res = $db->query($sql); $array = array(); while ($row = $res->fetch_assoc()) { $array[] = $row; } // ..... $data = array(); foreach ($array as $row) { $row['title'] = htmlspecialchars($row['title']); $row['user'] = getUserById($row['user']); } $tpl->set('data', $data); // ...... foreach ($data as $v):?> <div class="item"> <h3><?php echo $v['title']?></h3> <span class="user"> <?php echo $v['user']?> </span> </div> <?php echo endforeach;?> А мысль в том, что бы свести это к (схематично) PHP: <?php $obj = $db->query($sql, array( 'title' => function ($field) { return htmlspecialchars($field); }, 'user' => function ($field) { return getUserById($field); } )); $tmp->set('obj', $obj); // .... foreach ($obj as $v):?> <div class="item"> <h3><?php echo $v->title?></h3> <span class="user"> <?php echo $v->user?> </span> </div> <?php echo endforeach;?> В момент вывода в шаблоне вызываются лямбды и возращают данные. Да, я понимаю что это не везде применимо, это дополнительный функционал к стандартным. Просто часто бывает вывод ограничивается тем, что выбрали из базы, эскейпнули и вывели в шаблон. В этих случаях будет убыстрение и упрощение кода даже.
Я продолжаю настаивать, что итераторами и лямбдами снижения наклодных расходов по сравнению с циклами не добиться. Хотя в соседней теме меня обвиняли в байтодрочестве.
А я продолжаю настаивать, что смысла в этом нет, это можно сделать просто типизировав объект $obj и написав нужное в событиях при чтении свойств, хотя задумка тоже ничего...
vasa_c Дело не тока в накладных расходах, есть ещё уменьшение и упрощение кода Мну лично конкретно достало строчить шаблон вида "Запрос -> Оработка запроса -> Эскейп данных -> Отдать в шаблон -> В шаблоне развернуть в цикл". В идеале хочу " Запрос -> В шаблон -> Вывел" Как это будет выглядеть и работать я вам не говорю, тут каждый думает сам
Костян А я психа поддерживаю. Меня тоже заебало. Правда, я решил введением дополнительного метода в ядро. PHP: <?php $this->e->DB->select(array('select' => 'name, url', 'from' => 'core_pages', 'where' => $where )); $this->e->toOUT('childrens');
Psih бесспорно инъекции лямбда функций это круто, и может быть достаточно эффективнми во многих местах... Только вот у меня эскейп данных в основном происходит в шаблоне... Всё рано ты получишь то же самое, ну может меньше писать будешь...
А я вот, ради разнообразия, буду настаивать, что в случаях когда замена нескольких циклов на один принесет хоть чуточку заметный профит - эту часть приложения нужно переписывать на С. А написанное на PHP должно в первую очередь быть приятным для чтения и модификации, и в последнюю - оптимизировать быстродействие. Вот такой холивар.
Лямбда-функция на каждое поле, не уверен, что упростит код, но не буду упорствовать. Вот это как раз то, что делает моя библиотека в соседней теме и против чего лично ты высказался отрицательно. У меня программист не должен обрабатывать запрос отдельно. имхо, это дело шаблона. Ну и по всякому эскейп нужно прописывать в лямбде. Отдавать придётся по-всякому Ну и разворачивать в цикле итератор также придётся, как и массив.