|
|
Posted by "M. Sokolewicz" on 09/28/05 00:23
Charles Kline wrote:
>
> On Sep 27, 2005, at 3:42 PM, Andy Pieters wrote:
>
>
>> Hi
>>
>> Without you actually showing us these class files we can only guess
>> but a
>> common mistake is this:
>>
>> mysql_open(connection details)
>> mysql_query(query)
>>
>> In those cases the last opened handle is used. To prevent this, use
>> this
>> syntax
>>
>> $db1=mysql_open(connection for db1);
>> $db2=mysql_open(connection for db2);
>>
>> mysql_query($db1,$query_db_1);
>> mysql_query($db2,$query_db_2);
>>
>> If you have used this syntax then check your class if it is using a
>> global
>> variable to hold the database handle and if it does make it a class
>> variable
>> instead
>>
>> Instead of
>>
>> $db=null
>>
>> class db
>> {function db()
>> {$GLOBALS['db']=mysql_open(...
>>
>> do instead
>>
>> class db
>> {var $db=null;
>> function db()
>> {$this->db=mysql_open
>>
>> That way you can instanciate as many instances of the class as you
>> like and
>> each will have its own database handle.
>>
>> HTH
>>
>>
>> Andy
>>
>>
>
> What I have is more like this:
>
> class db {
> var $dbpath = "localhost";
> var $dbname = "testdb";
> var $dblogin = "test";
> var $dbpass = "test";
>
> function db() {
> $link = mysql_connect($this->dbpath, $this->dblogin, $this- >dbpass)
> or die ('Not Connected: ' . mysql_error());
> mysql_select_db($this->dbname, $link) or die ('Can\'t use this
> database: ' . mysql_error());
> }
>
> function retrieveData( $sql ) {
> $rs = mysql_query( $sql ) or die("Invalid query: " . mysql_error ());
> // if no result, return null
> if (($rs == null) || (mysql_num_rows($rs) == 0)) {
> return null;
> } else {
> return ( $rs );
> }
> }
>
> function insertData( $sql ) {
> mysql_query( $sql );
> // return new id if insert is successful
> if ( mysql_affected_rows() > 0 ) {
> return ( mysql_insert_id() );
> } else {
> return null;
> }
> }
>
> function updateData( $sql ) {
> $rs = mysql_query( $sql );
> if (mysql_affected_rows() > 0) {
> return ( $rs );
> } else {
> return null; // no changes were made
> }
> }
> }
>
>
> I then have another class with a different name and I changed the names
> of the functions as well. I have another set of class files that
> contain my various queries etc.
>
> Thanks for any help.
> Charles
that's where it's going wrong. You're not linking db::db()'s link in the
db::insertData() mysql_query() function. Thus, the mysql_query function
defaults to the "last opened mysql connection" regardless of object. A
way to fix this is to store the link in $this->link instead, and
changing your mysql_query()'s to mysql_query($sql, $this->link);
same goes for your mysql_affected_rows() and mysql_insert_id() functions
-tul
Navigation:
[Reply to this message]
|