Re: [PHP] See if this makes any sense
- From: ceo@xxxxxxxxx ("Richard Lynch")
- Date: Thu, 5 Oct 2006 14:56:18 -0500 (CDT)
You do *NOT* want to re-connect to the database on every InsertDB!
That's very expensive in resources/time.
Do that once in the constructor and be done with it for as many
Inserts as you need.
I'm baffled why you'd be rolling your own instead of using an existing
abstraction layer, and even more baffled why you wouldn't just skip
all this and just do:
$query = "insert ...";
$insert = mysql_query($query, $connection);
It's not like you can take your MySQL user-creating queries and
directly map them to ANY other database on the planet...
So what does the DB abstraction and class OOP stuff gain you?
Nada.
On Thu, October 5, 2006 2:18 pm, Deckard wrote:
Hi,
I've burned my brain, checked other sites and come to a code that
works.
I ask you, please, to see if this makes any sense and/or can be
improved.
I'd really appreciate.
Warm Regads,
Deckard
dbInsert.php:
-------------------------------------------------------------------
<?php
/*
* class to make inserts
*
*/
// includes
include_once('/var/www/html/config.inc.php');
include_once('adodb/adodb.inc.php');
class dBInsert
{
// global variables
var $table;
var $sql;
// constructor
function dBInsert($table, $sql)
{
$this->table = $table;
$this->sql = $sql;
return(TRUE);
}
// function that constructs the sql and inserts it into the database
function InsertDB($table, $sql)
{
print($sql);
// connect to MySQL
$conn = &ADONewConnection('mysql');
$conn->debug=1;
$conn->PConnect('localhost', 'gamito', 'ble', 'wordlife');
// execute the insert
if ($conn->Execute($sql) === false)
print 'error inserting: '.$conn->ErrorMsg().'<BR>';
return (TRUE);
}
}
?>
------------------------------------------------------
testedb.php
------------------------------------------------------
<?php
include_once("classes/dBInsert.php");
$sql = "INSERT INTO wl_admins VALUES ('',3)";
$dBInsert = new dBInsert('wl_admins', $sql);
$dBInsert->InsertDB('wl_admins', $sql);
?>
------------------------------------------------------
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
--
Some people have a "gift" link here.
Know what I want?
I want you to buy a CD from some starving artist.
http://cdbaby.com/browse/from/lynch
Yeah, I get a buck. So?
.
- Follow-Ups:
- References:
- See if this makes any sense
- From: Deckard
- See if this makes any sense
- Prev by Date: Re: See if this makes any sense
- Next by Date: Re: [PHP] Help on objects
- Previous by thread: Re: See if this makes any sense
- Next by thread: Re: [PHP] See if this makes any sense / Att. Richard Lynch
- Index(es):
Relevant Pages
|
