Mark Taylor

Schönen guten Abend,
wende mich verzweifelt an Euch, da ich eine Fehlermeldung nicht nachvollziehen kann. Ich habe hier eine Klasse für den Datenbankzugriff und eine Klasse User. Ich habe alle für dieses Problem nicht relevanten Teile entfernt. Um in der Klasse User eine Datenbankabfrage machen zu können, übergebe ich bei der Erzeugung eines User-Objektes, eine Referenz auf ein Objekt der Datenbank-Klasse an den Konstuktor der User-Klasse. Leider wird anscheinend in diesem Moment das Objekt gelöscht. An dieser Stelle erhalte ich die Fehlermeldung:

Notice: Undefined variable: sql in .....

Wenn ich dann in meiner User-Klasse die Funktion CheckUserName() aufrufe bekomme ich die Fehlermeldung:

Fatal error: Call to a member function select() on a non-object in ...

Was ja auch ganz logisch ist, wenn die Referenz auf das Datenbank-Objekt leer ist, kann ich auch nicht die Funktion select() aufrufen.

Wieso übergibt der Interpreter bei : $user = new User(new DBMySQL());
nicht die Referenz an den Konstruktor der User-Klasse ?

Ich habe schon Stunden in Foren nach der Fehlermeldung gesucht und auch gefunden, aber leider nicht die passende Antwort gefunden. Jetzt habe ich mich hier angemeldet in der Hoffnung jemand von Euch könnte mir helfen.

Hier der gekürzte Code:
[php]

error_reporting(E_ALL);

class DBMySQL{
private $connected = FALSE;
private $conn_id = NULL;
private $result = NULL;
private $sql_query;

public function __construct(){
global $mySQL_settings;
$this->host = $mySQL_settings['host'];
$this->user = $mySQL_settings['user'];
$this->pswd = $mySQL_settings['pswd'];
$this->dbname = $mySQL_settings['dbname'];

$this->conn_id = mysql_connect($this->host, $this->user, $this->pswd);
if ($this->conn_id === FALSE) {
$this->error("CONN.OPEN");
}
if (mysql_select_db($this->dbname, $this->conn_id) === FALSE) {
$this->error("DB.SELECT");
}
else {
$this->connected = TRUE;
}
}

public function __destruct() {
if (!$this->connected) {
$this->error("NO CONN");
}
else {
if (@mysql_close($this->conn_id) === FALSE) {
$this->error("CONN.CLOSE");
}
}
}

public function query($sql) {
if (!$this->connected) {
$this->error("NO CONN");
}
else {
$this->sql_query = $sql;
$result = mysql_query($this->sql_query, $this->conn_id);
if ($result === FALSE) {
$this->error("QUERY FAILED");
}
else {
return $result;
}
}
}

public function select($column, $table, $arg, $anzahl = NULL) {
if($arg == NULL) $sql= "SELECT ".$column." FROM ".$table;
else $sql= "SELECT ".$column." FROM ".$table." WHERE ".$arg;
if (!$this->connected) {
$this->error("NO CONN");
}
else {
$result = $this->query($sql);
if ($result !== FALSE) {
$anzahl1 = mysql_num_rows($result);
$anzahl = $anzahl1;
for($i = 0; $i < $anzahl; $i++) {
$daten[$i] = mysql_fetch_object($result);
}
return $daten;
}
}
}

}

class User
{
private $sql = NULL;
private $user_name;

public function __construct($sgl) {
$this->sql = $sql;
}

public function CheckUserName($user_name) {
$this->user_name = $user_name;
$column = "user_name";
$table = "user";
$arg = "'user_name=".$user_name."'";
$result = $this->sql->select($column, $table, $arg, $anzahl = NULL);
return $result;
}
}

$user = new User(new DBMySQL());
$user_name = "Mark";
$result = $user->CheckUserName($user_name);
if(array_count_values($result)>0) echo "Dieser Username ist noch nicht vergeben !";

[php]

Vielen Dank !

Mit freundlichen Grüßen,

Mark Taylor

Mister Ad

nikosch

Hallo, willkommen.

Bitte editiere Deinen Beitrag und nutze [php] Tags für den Code.


Hast Du das denn im Konstruktor überprüft?

__________________
--
One pixel is still too big. Please make it smaller. ASAP.

Initiative Mittelstand.
Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers.
--

nikosch

Schau Dir das mal an.
Fällt Dir was auf? Tipp: Parametername....

__________________
--
One pixel is still too big. Please make it smaller. ASAP.

Initiative Mittelstand.
Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers.
--

Mark Taylor

Vielen Dank für die schnellen Antworten. Habe mir schon gedacht, das es etwas ganz simples ein muss. Danke für den Hinweis auf diesen Flüchtigkeitsfehler. Ich habe mir das tausendmal durch gelesen und keine Fehler entdeckt. Wenn man zu lange daran sitzt wird man blind für so was.
Nochmals vielen Dank und schönen Abend noch !

Beim nächsten mal benutze ich [php] zum Kennzeichnen des Codefragmentes.

Lieben Gruß, Mark

David

--war falsch, gelöscht--