FUDforum
Fast Uncompromising Discussions. FUDforum will get your users talking.

Home » Imported messages » comp.lang.php » database access
Show: Today's Messages :: Polls :: Message Navigator
Return to the default flat view Create a new topic Submit Reply
Re: database access [message #174190 is a reply to message #174189] Wed, 25 May 2011 21:37 Go to previous messageGo to previous message
Jerry Stuckle is currently offline  Jerry Stuckle
Messages: 2598
Registered: September 2010
Karma:
Senior Member
On 5/25/2011 1:17 PM, Jivanmukta wrote:
> I new relatively new to PHP. I need your opinion if the following set
> of database functions make sense:
>
> function openDatabase() {
> global $db;
> try {
> $db = new PDO('mysql:host=' . MYSQL_SERVER . ';dbname=' .
> MYSQL_DATABASE, MYSQL_USER, MYSQL_PASSWORD);
> $db->exec('SET NAMES latin2');
> $db->exec('SET CHARACTER SET latin2 COLLATE latin2_general_ci');
> $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
> register_shutdown_function('closeDatabase');
> } catch (PDOException $e) {
> return false;
> }
> return true;
> }
>
> function closeDatabase() {
> global $db, $inTransaction;
> if ($inTransaction) {
> $db->rollBack();
> $inTransaction = false;
> }
> if (!is_null($db)) {
> $db = null;
> }
> }
>
> function queryDatabase($sql) {
> global $db, $inTransaction;
> $result = array();
> try {
> $qry = $db->query($sql);
> if (!empty($qry)) {
> $result = $qry->fetchAll();
> }
> } catch (PDOException $e) {
> if ($inTransaction) {
> $db->rollBack();
> }
> error(ERRMSG_DATABASE_OPERATION_FAILED);
> }
> return $result;
> }
>
> function execDatabase($sql) {
> global $db, $inTransaction;
> $result = array();
> try {
> $result = $db->exec($sql);
> } catch (PDOException $e) {
> if ($inTransaction) {
> $db->rollBack();
> }
> error(ERRMSG_DATABASE_OPERATION_FAILED);
> }
> return $result;
> }
>
> function startTransaction() {
> global $db, $inTransaction;
> $db->beginTransaction();
> $inTransaction = true;
> }
>
> function commitTransaction() {
> global $db, $inTransaction;
> $inTransaction = false;
> if ($db->commit()) {
> return;
> }
> $db->rollBack();
> error(ERRMSG_DATABASE_OPERATION_FAILED);
> }
>
> function rollbackTransaction() {
> global $db, $inTransaction;
> $inTransaction = false;
> if ($db->rollBack()) {
> return;
> }
> error(ERRMSG_DATABASE_OPERATION_FAILED);
> }
>
> function lockTables($tables) {
> return execDatabase('LOCK TABLES ' . join(' WRITE, ', $tables) . '
> WRITE');
> }
>
> function unlockTables() {
> return execDatabase('UNLOCK TABLES');
> }

Kind of, and maybe a bit overkill. But personally I'd put it all in a
Database class. The code would be more flexible and less prone to errors.

--
==================
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
jstucklex(at)attglobal(dot)net
==================
[Message index]
 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: Stats comp.lang.php (last 7 days)
Next Topic: a more elegant way?
Goto Forum:
  

-=] Back to Top [=-
[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Sat Nov 23 04:45:43 GMT 2024

Total time taken to generate the page: 0.04474 seconds