PDO connectie error

Door Izildo Pimentel op 30-12-2017 23:30 gewijzigd op 30-12-2017 23:32

2.855 views

Kan iemand mij helpen om verbinding te maken met de server aan de hand van PDO en een database uitlezen.

Mij config.php


<?php
	// Database info
	$host = 'localhost';
	$user = 'xxxxx';
	$pass = 'xxxxx';
	$database = 'db75076';

	// Make connection
	$conn = new PDO( "mysql:" . "host=$host;" . "dbname=$database", '$user', '$pass');
?>

Mij index.php


<!DOCTYPE html>
<html>
	<head>
		<title></title>
	</head>
	<body>

	<?php
		// Database + connection
		require_once('config.php');
		
		// Check connection
		if (!conn) {
			die('Connection failed: ' . PDO::erroCode());
		}

		// Create query
		$result = "SELECT * FROM Students";

		// Check query
		if (PDO::query($conn, $result)) {
			# code...
		}

		// Insert query into table

		// Close connection
		$conn = null;
	?>
	</body>
</html>

Ariën

30-12-2017 23:32 gewijzigd op 30-12-2017 23:33

Waar loop je nu op vast?

Izildo Pimentel

31-12-2017 00:12

Ik krijg een fatal error, in config.php op lijn 9 zegt die

'Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[28000] [1045] Access denied for user '$user'@'localhost'

Link naar mijn project: http://75076.ict-lab.nl/Php%20Project/index.php

Ariën

31-12-2017 00:17

Variabelen horen buiten quotes.

Ozzie PHP

31-12-2017 01:05

Probeer zo eens:

<?php

$conn = new PDO("mysql:host=$host;dbname=$database", $user, $pass);

?>

Ben van Velzen

31-12-2017 01:57

Kijk voor de grap eens naar hoe PDO::query werkt. Zoals je het nu probeert kan het echt niet. Ook het opbouwen van de connectie werkt niet zo. Voor referentie:
http://www.php.net/pdo_construct
http://www.php.net/pdo_query

Thomas van den Heuvel

31-12-2017 03:02

Enne, een constructie met een try-catch blok is min of meer verplicht, omdat PDO (meestal) werkt met exceptions.

Een niet-gevangen exception levert altijd een Fatal Error op.

En laat de exception bij het connecten nu alle connectie-info uitspugen wanneer deze mislukt...

En vergeet je character encoding niet.

Jan Koehoorn

31-12-2017 10:00 gewijzigd op 31-12-2017 10:01

Nog een aanvullende tip: zet nooit connectie info hard in je code. De manier waarop Laravel het doet, met .env bestanden, is al een stuk veiliger.

Izildo Pimentel

31-12-2017 12:01

Jan Koehoorn op 31/12/2017 10:00:57

Nog een aanvullende tip: zet nooit connectie info hard in je code. De manier waarop Laravel het doet, met .env bestanden, is al een stuk veiliger.

Heb er van gehoord maar weet niet precies hoe het werkt. Dus een config file met daarin mijn credentials is verkeerd?

Ariën

31-12-2017 12:53 gewijzigd op 31-12-2017 12:55

Nee...
Wat jij doet is je connectie in je config-bestand laden. En dat is niet echt de juiste bedoeling.
De config is bedoeld voor configuratie-instellingen, en jij doet meer dan dat in dat bestand. ;-)

Het beste is zelfs als je jouw config hoger dan jouw publieke map van je website zet. Mocht je server opeens geen PHP-meer willen parsen door een fout, dan liggen je codes niet zomaar op straat.

Izildo Pimentel

31-12-2017 12:58

Oh oke dan weet ik dat, heb het zo via school aangeleerd en sommige code snippets zie je het ook zo vandaar.

Reageren

Inloggen om te reageren