Error bij escapen van een string

wat is het nut van die esc() functie?

Je kan toch ook direct $db->quote gebruiken?

Zo.. ik lees ik ben begonnen met PHP5 en ik zie al functies en zelfs een class :-)

Goed:
functies zijn min of meer geïsoleerde stukjes programmacode. Hetgeen wil zeggen dat als je buiten de functie een variabele declareert ($db in jouw geval) deze variabele nog niet bestaat binnen je functie.

Je zou dan bijvoorbeeld de variabele mee kunnen geven als een parameter net als $str
<?PHP
function esc($db, $str)
{
return $db->quote($str);
}
?>

vervolgens moet je dan ook wel een waarde meegeven bij het aanroepen zoals op regel 28
<?php
$this->voornaam = esc($db, $voornaam);
?>

- Aar - op 20/07/2014 20:41:19

wat is het nut van die esc() functie?

Je kan toch ook direct $db->quote gebruiken?

Klopt, maar dan zou ik elke keer opnieuw de database connectie moeten maken in elke class. Stel dat ik dan een keer de database settings wil wijzigen dan moet ik dat in elke class doen.

Het lijkt er op alsof de PDO-verbinding niet tot stand gebracht wordt.

Pipo Clown op 20/07/2014 20:53:25

Het lijkt er op alsof de PDO-verbinding niet tot stand gebracht wordt.

De PDO-verbinding werkt!

@ Frank:
Ja klopt, ben er enorm enthousiast over :)
Ik heb het script aangepast maar krijg alsnog de foutmeldingen:

Notice: Undefined variable: db in /*/toevoegen.php on line 28

Fatal error: Call to a member function quote() on a non-object in /*/toevoegen.php on line 8

Hier het aangepaste script:

[CODE]
<?PHP
error_reporting(E_ALL);

$db = new PDO('mysql:host=localhost;dbname=XXXX', 'XXXX', 'XXXXX');

function esc($db, $str)
{
return $db->quote($str);
}

class Ervaring
{

public $voornaam;
public $achternaam;
public $email;
public $ervaring;

public function __construct($voornaam, $achternaam, $email, $ervaring)
{


if (trim($voornaam == "") OR trim($achternaam == "") OR trim($email == "") OR trim($ervaring == ""))
{
throw new Exception ("<p>U heeft niet alle velden ingevuld.</p>");
}

$this->voornaam = esc($db, $voornaam);
$this->achternaam = esc($db, $achternaam);
$this->email = esc($db, $email);
$this->ervaring = esc($db, $ervaring);
$this->ip = $_SERVER['REMOTE_ADDR'];
$this->datum = time();

$results = $db->exec("INSERT INTO Reviews (Ip, Datum, Voornaam, Achternaam, Email, Ervaring)
VALUES ('$this->ip', $this->datum, $this->voornaam, $this->achternaam, $this->email, $this->ervaring)");

if($results != 1)
{
throw new Exception ("DATABASE INVUL FOUT");
}

}

}

try
{
$ervaring = new Ervaring('voornaam', 'achternaam', '[email protected]', 'het was goed');
}

catch (Exception $e)
{
echo $e->getMessage();
}

?>

Oh sorry vergeten maar ook je class is een geïsoleerde omgeving.
<?php

class Ervaring
{

public $voornaam;
public $achternaam;
public $email;
public $ervaring;
private $db;

public function __construct($db, $voornaam, $achternaam, $email, $ervaring)
{
$this->db = $db;

// vervolg bestaande code
?>

en regel 28:
<?php
$this->voornaam = esc($this->db, $voornaam);
?>

even over dit:
<?php
public $voornaam;
public $achternaam;
public $email;
public $ervaring;
?>

dit zijn properties. Leer jezelf aan om die direct private te declareren. Weet je het verschil tussen private en public?

Toevoeging op 20/07/2014 21:43:42:

En een constructor is enkel bedoeld voor initialisatie. de rest hoort er niet in thuis.
<?php
class Ervaring
{

private $voornaam;
private $achternaam;
private $email;
private $ervaring;
private $db;

// een constructor is ALTIJD public dus van mij mag je dat weghalen. Het is echter niet fout (in PHP).
/*public*/ function __construct($voornaam, $achternaam, $email, $ervaring)
{
$this->voornaam = esc($db, $voornaam);
$this->achternaam = esc($db, $achternaam);
$this->email = esc($db, $email);
$this->ervaring = esc($db, $ervaring);
$this->ip = $_SERVER['REMOTE_ADDR'];
$this->datum = time();
}

}
?>

Bedankt voor je reactie, ik leer er veel van :)

Ik heb het script aangepast maar krijg nu de foutmelding: Parse error: syntax error, unexpected T_VARIABLE, expecting T_FUNCTION in /*/toevoegen.php on line 39

Ik kan natuurlijk de $db database connectie in de function __construct() plaatsen maar dan zou ik dat in elke class moeten doen die ik ga maken. Wat ik wil doen is een config.php includen in al mijn bestanden. In dat bestand wordt dan de database connectie gemaakt. Als ik dan ooit de db gegevens moet aanpassen dan hoef ik dat maar in 1 bestand te doen ipv in alle classes.


[CODE]
<?PHP
error_reporting(E_ALL);

$db = new PDO('mysql:host=localhost;dbname=XXX', 'XXX', 'XXX');

function esc($db, $str)
{
return $db->quote($str);
}

class Ervaring
{

private $voornaam;
private $achternaam;
private $email;
private $ervaring;
private $db;

function __construct($db, $voornaam, $achternaam, $email, $ervaring)
{

$this->db = $db;

if (trim($voornaam == "") OR trim($achternaam == "") OR trim($email == "") OR trim($ervaring == ""))
{
throw new Exception ("<p>U heeft niet alle velden ingevuld.</p>");
}

$this->voornaam = esc($this->db, $voornaam);
$this->achternaam = esc($this->db, $achternaam);
$this->email = esc($this->db, $email);
$this->ervaring = esc($this->db, $ervaring);
$this->ip = $_SERVER['REMOTE_ADDR'];
$this->datum = time();

}

$results = $db->exec("INSERT INTO Reviews (Ip, Datum, Voornaam, Achternaam, Email, Ervaring)
VALUES ($this->ip, $this->datum, $this->voornaam, $this->achternaam, $this->email, $this->ervaring)");


if($results != 1)
{
throw new Exception ("DATABASE INVUL FOUT");
}

}


try
{
$ervaring = new Ervaring('voornaam', 'achternaam', '[email protected]', 'het was goed');
}

catch (Exception $e)
{
echo $e->getMessage();
}

?>

A van Gent op 21/07/2014 14:18:13

Ik kan natuurlijk de $db database connectie in de function __construct() plaatsen

Daar gaat het niet om. Je kan geen losse code in een class hebben, alles moet binnen methods (functies) staan. Regels 39 t/m 46 bij jou staan niet binnen een method, dat kan dus niet.