Volgens mij zou de try { } binnen de __construct moeten staan.

Verder kan je host de error_reporting() uit hebben gezet, een kleine fix is.

<?php ini_set("display_errors","on"); error_reporting(E_ALL);  ?>

De try moet inderdaad in de construct staan. Want nu zeg je nergens waar die uitgevoerd moet worden, staat namelijk niet in een function.

Of maak er een aparte functie van of je moet hem in de construct zetten.

Daarnaast moet de catch als volgt zijn:

<?php
catch (PDOException $e) {
return $e->getMessage();
		die();
	}
?>

Hoop dat je er wat mee kan.

die die() is nergens voor nodig.

<?php
class Connection {


	private $connection;
	private $query;


	private $host; 
	private $user;
	private $pass;
	private $database;


	public function __construct($host, $user, $pass, $database){
   		$this->host = $host; 
   		$this->user = $user;
   		$this->pass = $pass;
   		$this->database = $database;
	}


public function Connect()  {
	
    	try {
       		$this->connection = new PDO("mysql:host=". $this->host .";dbname=". $this->database ."", $this->user, $this->pass);
       		$this->connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    	}
    	catch (PDOException $e) {
          		return $e->getMessage();
    	}
}
}
?>

aanroepen:

$db = new Connection();
$db->Connect();

Waarom wil je host, user etc public hebben? Dat betekend dat andere instanties/classes die mogen/kunnen uitlezen iets wat je om veiligheidsredenen niet wil. Heb ze nu op private gezet.

Maybe deze?

<?php
class Connection {
    private $connection;
    private $query;
    private $host;
    private $user;
    private $pass;
    private $database;
    
    public function __construct($host, $user, $pass, $database) {
        $this->host = $host;
        $this->user = $user;
        $this->pass = $pass;
        $this->database = $database;


        try {
            $this->connection = new PDO('mysql:host='.$this->host.';dbname='.$this->database.','.$this->user.','.$this->pass, 
            array(
                PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION,
                PDO::ATTR_PERSISTENT => true,
                PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"
            ));
        }
        catch (PDOException $e) {
            return $e->getMessage();
        }
    }    
}
?>

Anders staat er op de concurrent een andere pdo classe die minimalistisch is.

Citaat van FrankY

Hallo.
Heb het wederom even aagepast, bedankt voor het aangeven dat het private had moeten zijn. Ik heb dit script van het internet en heb dus nog niks aangepast.

Heb nog steeds een wit scherm en ik roep nog niks aan. Dit is mijn index.php:

PHP

<?php
ini_set("display_errors","on"); 
error_reporting(E_ALL);
include_once "connection.php";
?>


<html>
    <head></head>


    <body>
        <?php
        if(isset($_POST['submit']))    {
            


        }
        ?>
        <form method="POST">
            Gebruikersnaam: <input type="text" name="login" /><br />
            Wachtwoord: <input type="password" name="pass" /><br />
            <input type="submit" name="submit" value="login" />
        </form>
    </body>
</html>

Meer tonen

Met vriendelijke groet,

Frank sijbom

Citaat

ini_set("display_errors","on");
error_reporting(E_ALL);

Je roept hier ook tot 2x error reporting. Tegenwoordig kan je alleen al door je ini aante roepen de fouten laten weergeven.

Verander:

ini_set("display_errors","on");

naar:

ini_set("display_errors",1);

Vertel dan eens welke fouten er worden weergeven.

<?php
class database {
    
    public static $dbObj = null;
    protected $dbData = array();
    protected static $dbCon = null;
    
    // Because of singleton structure we do not allow direct acces to class
    private function __construct() { }
    private function __clone() { }
    private function __sleep() { }
    
    public static function getInstance() {
        if (!self::$dbObj instanceof self) {
            self::$dbObj = new self();
        } 
        return self::$dbObj;
    }
    
    public function init($hostname, $username, $password = null, $database, $options = null) {
        $this->dbData['hostname'] = $hostname;
        $this->dbData['username'] = $username;
        $this->dbData['password'] = (is_null($password) ? '' : $password);
        $this->dbData['database'] = $database;
        
        if (!is_null($options)) {
            if (!is_array($options)) {
                trigger_error('Options value is not an array');
                
                return;
            } else {
                $this->dbData['options'] = $options;
            }
        }
        
        try {
            self::$dbCon = new PDO('mysql:host='.$this->dbData['hostname'].';dbname='.$this->dbData['database'].','.$this->dbData['username'].','.$this->dbData['password'], $this->dbData['options']);
        } catch (PDOException $e) {
            return $e->getMessage();
        }
    }
}

en dan mijn versie van de database class zoals weergegeven :-)!
aanroepen doe je door:

<php
$db = database::getInstance();
$db->init(hostname, username, password, database, array(options));

Staat je pdo extentie op de host aan? Zit nu op gsm dus kan je phpinfo niet zien.

Vergeet ook vooral niet een character encoding in te stellen. Dat voorkomt echt een hele hoop problemen met in het ergste geval corrupte data in je database.

Vanaf PHP versie 5.3.6 kun je deze meegeven aan je Data Source Name (DSN - je "connectie string") via de charset-parameter. Oudere PHP-versies negeren deze parameter. In die gevallen zou je de character encoding via de (connectie)optie PDO::MYSQL_ATTR_INIT_COMMAND moeten instellen.

Tevens:
* Persistente connecties... Ik weet niet of dat enige meerwaarde heeft.
* Daarnaast kan het iets sneller zijn om voor de host altijd een IP op te geven, dat scheelt je weer een lookup.

Citaat

Je roept hier ook tot 2x error reporting. Tegenwoordig kan je alleen al door je ini aante roepen de fouten laten weergeven.

Nope. Met display_errors geef je aan of je fouten wilt weergeven. Dit is een PHP_INI_ALL instelling en mag je dus overal neerzetten. Met error_reporting() geef je aan welke fouten worden weergegeven. Dit zijn twee verschillende dingen.

Citaat van FangorN

Vergeet ook vooral niet een character encoding in te stellen. Dat voorkomt echt een hele hoop problemen met in het ergste geval corrupte data in je database.

Vanaf PHP versie 5.3.6 kun je deze meegeven aan je Data Source Name (DSN - je "connectie string") via de charset-parameter. Oudere PHP-versies negeren deze parameter. In die gevallen zou je de character encoding via de (connectie)optie PDO::MYSQL_ATTR_INIT_COMMAND moeten instellen.

Tevens:
* Persistente connecties... Ik weet niet of dat enige meerwaarde heeft.
* Daarnaast kan het iets sneller zijn om voor de host altijd een IP op te geven, dat scheelt je weer een lookup.

Nope. Met display_errors geef je aan of je fouten wilt weergeven. Dit is een PHP_INI_ALL instelling en mag je dus overal neerzetten. Met error_reporting() geef je aan welke fouten worden weergegeven. Dit zijn twee verschillende dingen.

Oh, ja daar heb ik inderdaad niet aan gedacht. Ik ging er vanuit dat zodra je je .ini aanroept hij alle geselecteerde foutmeldingen oppakte.