Re: See if this makes any sense

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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);

?>
------------------------------------------------------

#1. If $table and $sql are class variables, why are you passing them as params to the InsertDB method?
#2. You don't even us the $table param.

Otherwise, not sure what this buys you...

--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux