DevProTalk

Forumi IT profesionalaca
web development, web design, e-business, SEO


Idite nazad   DevProTalk > Web development i web aplikacije > PHP
Beach Wedding Dresses - Looking for the Wedding Dress? Here, 1dress.co.uk stunning collection of beach wedding dresses is just what you are looking for.

PHP PHP aplikacije, Smarty, PEAR

Odgovori
 
Alati teme Način prikaza
Staro 29. 09. 2012.   #1
salebab
profesionalac
Qualified
 
Avatar salebab
 
Datum učlanjenja: 10.11.2006
Lokacija: Beograd
Poruke: 102
Hvala: 19
21 "Hvala" u 18 poruka
salebab is on a distinguished road
Pošaljite ICQ poruku za salebab Pošaljite poruku preko MSN za salebab
Default 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
__________________
www.salebab.net
salebab je offline   Odgovorite uz citat
Staro 29. 09. 2012.   #2
Miroslav Ćurčić
mV
Certified
 
Avatar Miroslav Ćurčić
 
Datum učlanjenja: 22.08.2009
Lokacija: Novi Sad
Poruke: 67
Hvala: 0
16 "Hvala" u 13 poruka
Miroslav Ćurčić is on a distinguished road
Default

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.
__________________
Quis custodiet ipsos custodies ?
Miroslav Ćurčić je offline   Odgovorite uz citat
"Hvala" Miroslav Ćurčić za poruku:
Staro 29. 09. 2012.   #3
salebab
profesionalac
Qualified
 
Avatar salebab
 
Datum učlanjenja: 10.11.2006
Lokacija: Beograd
Poruke: 102
Hvala: 19
21 "Hvala" u 18 poruka
salebab is on a distinguished road
Pošaljite ICQ poruku za salebab Pošaljite poruku preko MSN za salebab
Default

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.
__________________
www.salebab.net
salebab je offline   Odgovorite uz citat
Staro 30. 09. 2012.   #4
webarto
expert
Grand Master
 
Avatar webarto
 
Datum učlanjenja: 11.04.2010
Poruke: 998
Hvala: 141
890 "Hvala" u 153 poruka
webarto is on a distinguished roadwebarto is on a distinguished roadwebarto is on a distinguished roadwebarto is on a distinguished roadwebarto is on a distinguished roadwebarto is on a distinguished roadwebarto is on a distinguished roadwebarto is on a distinguished road
Default

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.
__________________
Github // LinkedIn // PHP // ZCE // Stackoverflow PHP // Site5 Web Hosting
webarto je offline   Odgovorite uz citat
Staro 30. 09. 2012.   #5
salebab
profesionalac
Qualified
 
Avatar salebab
 
Datum učlanjenja: 10.11.2006
Lokacija: Beograd
Poruke: 102
Hvala: 19
21 "Hvala" u 18 poruka
salebab is on a distinguished road
Pošaljite ICQ poruku za salebab Pošaljite poruku preko MSN za salebab
Default

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.
__________________
www.salebab.net
salebab je offline   Odgovorite uz citat
Odgovori


Alati teme
Način prikaza

Pravila pisanja
Možete ne započinjati nove teme
Možete ne slati odgovore
Možete ne slati priloge
Možete ne izmeniti svoje poruke
vB kôd je Uključen
Smajliji su Uključen
[IMG] kod je Uključen
HTML kôd je Isključen
Pogledajte forum


Vreme je GMT +2. Trenutno vreme je 02:39.


Blogodak - Domaci blogovi na jednom mestu Caught in a web - web dev blog
Powered by vBulletin® Verzija 3.6.8
Copyright ©2000 - 2017, 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.