-
PHP
(
http://www.devprotalk.com/forumdisplay.php?f=9)
| salebab |
29. 09. 2012. 12:19 |
Potreban code review PDO wrappera
Cao svima,
ukoliko imate vremena pogledajte ovaj mali projekat PDO Database Wrapper. Zanima me misljenje ostalih PHP programera da li jedna ovakva biblioteka moze da vam koristi, da li bi ste nesto izmenili, dodali, itd.
Hvala unapred :) |
| Miroslav Ćurčić |
29. 09. 2012. 20:36 |
Nisam detaljno gledao, samo ukratko:
Mogao bi u konfiguraciji da ubaciš prefix za nazive tabela koji će posle metod from() sam da dodaje, a u običnom queriju bi nazive tabela mogao da okružiš vitičastim zagradama ({}) pa bi pri regex-om prepoznao da je to naziv tabele i ubacio mu prefix. Drupal radi upravo sa {}.
Uglavnom deluje solidno.
Kasnije možeš dodati podršku za transakcije, keširanje, Postgree.
|
| salebab |
29. 09. 2012. 20:53 |
S obzirom da se samo extenduje PDO klasa, kao i PDOStatement klasa, transakcije rade bez problema, isto vazi i za Postgree, nisam proveravao ali trebalo bi.
Hmm, nikada nisam do sada imao potrebe za table prefixima, pa nisam ni razmisljao o tome, ali thx imacu to u vidu.
|
| webarto |
30. 09. 2012. 10:53 |
Zašto query builder ako ćeš kucati kompletan plain query svejedno? Po meni taj fluent interface nema smisla.
getInstance() je Singleton, pogledaj malo Dependency Injection.
Koristiš include/require, ne valja... dodaj autoloader, ili ako neko hoće da koristi u svojoj nekoj aplikaciji koja već koristi autoloader, napravi namespace i drži se neke nomenklature, tipa new DB\Query a ne include DBQuery.php itd.
Stavio si SET NAMES UTF8, možda neko ne koristi UTF8.
Postavi PDO::ATTR_EMULATE_PREPARES na FALSE. |
| salebab |
30. 09. 2012. 21:09 |
Pre svega hvala za review, pokusacu da opravdam neke odluke:
Query builder nije ovde da bi se ubrzalo pisanje koda tako sto npr necu morati da kucam join("INNER JOIN table ON table.user_id = users.user_id") vec samo join("table", "table.user_id = users.user_id"), kao sto je kod CI ActiveRecord klase. Ideja je da query string prebacim u objekat, tako da taj jedan query mogu graditi iz vise metoda, evo ukratko primera nekog modela:
PHP kôd:
<?php
class PostModel
{
/**
* @var DBQuery
*/
private $query;
function __construct() {
$this->query = DB::getInstance()->createQuery();
$this->query->select("p,*")
->from("posts p")
->join("INNER JOIN users u ON u.user_id = p.user_id");
}
function getPosts($args = array()) {
$args = array_merge(array(
"posts_per_page" => 10,
"is_active" => 1
), $args
);
$this->query->limit($args["posts_per_page"]);
$this->query->where("p.is_active = ?", $args["is_active"]);
return $this->query->execute()->fetchCollection();
}
function getPostsByUserId($user_id, $args = array()) {
$this->query->where("p.user_id = ?", $user_id);
return $this->getPosts($args);
}
}
Nadam se da je sada jasnije i da mozes da shvatis smisao tako neke klase.
Da, DB je samo implementacija singleton paterna kako ne bi uvek prosledjivao db objekat. Singleton u ovakvim slucajevima i ima najvise smisla i tu ga je najpozeljnije i koristiti.
Sto se include/require, nisam hteo da koristim autoloadere da ne bih bezveze komplikovao kod, zelim da bude sto jednostavnije a i kompatibilno sa 5.2 verzijom.
UTF8 je default vrednost, ali se moze prepisati tako se DBWrapperu prosledi u poslednjem $options parametru druga vrednost za PDO::MYSQL_ATTR_INIT_COMMAND. Ali da, probacu da nadjem elegantnije resenje za ovo. |
| Vreme je GMT +2. Trenutno vreme je 22:52. |
|
Powered by vBulletin® Verzija 3.6.8
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
Copyright © DevProTalk. All Rights Reserved.
Mišljenja, saveti, izjave, ponude ili druge informacije ili sadržaji nastali na Sajtu su vlasništvo onoga ko ih je kreirao, a ne DevProTalk.com, tako da ne morate da se oslanjate na njih.
Autori poruka su jedini odgovorni za ovakve sadržaje. DevProTalk.com ne garantuje tačnost, kompletnost ili upotrebnu vrednost informacija, stavova, saveta ili datih izjava. Ne postoje uslovi pod kojima bi mi bili odgovorni za štetu ili gubitak koji je posledica bilo čijeg oslanjanja na nepouzdane informacije, ili bilo kakve informacije nastale kroz komunikaciju između registrovanih članova.Web sajt može sadržavati linkove na druge web sajtove na Internetu ili neke druge sadržaje. Ne kontrolišemo niti podržavamo te druge web sajtove, niti smo pregledali bilo kakve sadržaje na takvim sajtovima. Mi nećemo biti odgovorni za legalnost, tačnost ili prikladnost bilo kog sadržaja, oglasa, proizvoda, usluga ili informacije lociranim na ili distribuiranih kroz druge web sajtove, niti za bilo kakvu štetu nastalu kao posledica takvih informacija. DevProTalk.com drži i čuva druga prava vlasništva na web sajtu. Web sajt sadrže materijale zaštićene copyright-om, zaštitne znakove i druge informacije o pravu vlasništva ili softver. Članovi mogu poslatu informacije zaštićene pravima vlasništva njihovih nosilaca i ona ostaju zaštićena bez obzira da li su oni koji prenose te informacije to naveli ili ne. Osim informacija koje su u javnom vlasništvu ili za koje dobijete dozvolu, nemate pravo da kopirate, modifikujete ili na bilo koji način menjate, objavljujete, prenosite, distribuirate, izvršavate, prikazujete ili prodajte bilo koju informaciju zaštićenu pravima vlasništva. Slanjem informacija ili sadržaja na bilo koji deo DevProTalk.com, Vi automatski dozvoljavate i predstavljate garanciju da imate pravo da dozvolite DevProTalk.com ili članovima DevProTalk.com bespovratnu, kontinualnu, neograničenu, globalnu dozvolu da koriste, kopiraju, izvršavaju, prikazuju i distribuiraju takve informacije i sadržaje i da iz takvih sadžaja koriste bilo koji deo u bilo koje svrhe, kao i pravo i dozvolu da koriste gore navedene sadržaje. Svi zaštitni znakovi (trademarks), logotipi, oznake usluga, firme ili imena proizvoda koji se pominju na ovom web sajtu su vlasništvo kojim raspolažu njihovi vlasnici.
