Fout connection.php

FrankY · 13 Januari 2015

Hallo,

ik heb het volgende script van het internet gehaald om een connectie met het database te leggen. Echter krijg ik alleen een wit scherm, ik heb Error_reporting aangezet en krijg niks.

Ik weet zeker dat het aan dit stukje code ligt aangezien ik gewoon mijn form krijg als ik de include weg haal.

het script:

PHP

<?php
class Connection {


    public $connection;
    public $query;


    public $host;
    public $user;
    public $pass;
    public $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);
        $this->connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    }
    catch (PDOException $e) {
        $e->getMessage();
        die();
    }
}
?>

Toon Meer

Zou iemand hier mij mee kunnen helpen?

met vriendelijke groet,

Frank sijbom

WHMCSAddons · 13 Januari 2015

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

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

FrankY · 13 Januari 2015

Hallo.

Heb het aangepast en nog steeds het zelfde. Alleen een wit beeld.

Met vriendelijke groet,

Frank sijbom

J.Rijdes · 13 Januari 2015

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

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

Hoop dat je er wat mee kan.

FrankY · 13 Januari 2015

Hallo.

Ook dit heb ik aangepast, het script ziet er nu als volgt uit:

PHP

<?php
error_reporting(E_ALL);
class Connection {


    public $connection;
    public $query;


    public $host;
    public $user;
    public $pass;
    public $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);
            $this->connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }
        catch (PDOException $e) {
            return $e->getMessage();
            die();
        }
    
    }
    


}
?>

Toon Meer

Met vriendelijke groet,

Frank sijbom

Luc · 13 Januari 2015

die die() is nergens voor nodig.

PHP

<?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();
    	}
}
}
?>

Toon Meer

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.

FrankY · 13 Januari 2015

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>

Toon Meer

Met vriendelijke groet,

Frank sijbom

Luc · 13 Januari 2015

Maybe deze?

PHP

<?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();
        }
    }    
}
?>

Toon Meer

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

FrankY · 13 Januari 2015

Hallo,

nee werkt nog steeds niet. Ik zal even kijken:)

Alvast bedankt,

Frank sijbom

A.Ruter · 13 Januari 2015

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>

Toon Meer

Met vriendelijke groet,

Frank sijbom

Toon Meer

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.

SC-Webmedia · 14 Januari 2015

Verander:

Code

ini_set("display_errors","on");

naar:

Code

ini_set("display_errors",1);

Vertel dan eens welke fouten er worden weergeven.

Patrick · 14 Januari 2015

PHP

<?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();
        }
    }
}

Toon Meer

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

Code

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

FrankY · 14 Januari 2015

Hallo.

Ik ben het lijstje even afgegaan, en tot nu toe heb ik nog steeds een wit scherm. Ik heb error_reporting weggehaalt en het zo gedaan zo als SC-scripting het aangaf. Heb echter weer geen fouten.
Heb het script van Patrick ook geprobeerd en ook aangeroepen in index.php. Wederom een wit scherm.

Zou het probleem niet aan de host kunnen liggen? Ik heb een webhosting bij one.com.

Ik heb ook even phpinfo() online gezet misschien kunnen jullie hier wat vinden..

phpinfo()

Bij voorbaat dank,

Frank sijbom

Ferhat.Remory · 14 Januari 2015

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

FrankY · 14 Januari 2015

@Ferhat :

PDO

PDO supportenabled
PDO drivers mysql, sqlite

pdo_mysql

PDO Driver for MySQLenabled
Client API version mysqlnd 5.0.8-dev - 20102224 - $Id: 731e5b87ba42146a687c29995d2dfd8b4e40b325 $

DirectiveLocal ValueMaster Value
pdo_mysql.default_socket/var/run/sockproxy.sock/var/run/sockproxy.sock

Volgens mij wel.

Ik het ook even gekeken via PDO:getAvailableDrivers(); en daar kwam dit uit:
Array ( [0] => mysql [1] => sqlite )

Naar mijn idee hoort dit goed te zijn?

Edit:
Ik heb het eens anders geprobeerd, heb nu geen classes meer en nu werkt het wel. Het is niet hoe ik het wou maar goed het moet dan maar zo.

Iedereen bedankt!

Met vriendelijke groet,

Frank sijbom

FangorN · 14 Januari 2015

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.

A.Ruter · 17 Januari 2015

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.

Toon Meer

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

Smarty Update

een eigen cloud sever

DNS update Directadmin

Graag advies voor mijn website

Ik wil op afstand studeren, kennen jullie Tech?

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

214 Nieuwe domeinnamen Augustus 2024

SoftwareSpecialist.net (50euro)

200 Nieuwe domeinnamen Juli 2024

Fout connection.php

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Delen