Does this PHP code look OK?

[Steen2]

    function mysqlconn() {
         $user = "DB_DB";
         $pass = "pass";
         $db = "DB_DB";
         $newconnect = mysql_pconnect(localhost, $user,$pass) or die(mysql_error());
         mysql_select_db($db) or die(mysql_error());
         if(mysql_error()) { echo mysql_error(); }
    } 

It's causing this error in my error logs and looks to be hurting the server: PHP Warning: mysql_pconnect(): Link to server lost, unable to reconnect in ...

The line the error references is
$user = "DB_DB";

But I included the lines around it for better insight.

[ssp]

Have you tried using all lower cases in your username?

[Steen2]

Have you tried using all lower cases in your username?

Sorry?

"DB_DB" is to edit out information that's "sensitive".

[ssp]

Perhaps it's because there is a space in this line:

mysql_pconnect(localhost, $user,$pass)

[Calvinguy]

Try to use single qoutes ' rather then doubleqoutes "


or put qoutes around localhost also...

[Steen2]

Perhaps it's because there is a space in this line:

mysql_pconnect(localhost, $user,$pass)

Try to use single qoutes ' rather then doubleqoutes "


or put qoutes around localhost also...

Thanks, but would this crash the server's mysql?

[Serge Litehead]

put localhost in single qoutes
whitespace such as spaces, tabs, etc in code is ignored

seems there s a prob with connection rather than anything else at this point
i've never used mysql_pconnect, see if mysql_connect works better for you.

[Nysus]

Hope you're putting that function in an include file as opposed to on a page, or have multiple DBs at different locations to use... you only need to connect to a mySQL server once each load.

Matt

[who]

Is your username the same as the database name?

[Serge Litehead]

here is a good sample for you from php manual:

$link = mysql_connect('localhost', 'mysql_user', 'mysql_password') or die (mysql_error());
db_selected = mysql_select_db('db_name', $link) or die (mysql_error());

or die() is not recommended for use in live production.

[Serge Litehead]

err in above post

db_selected = $db_selected