Goeieavond,

Ik ben nu een 2-tal weken bezig met php & mysql. Ik werk aan de hand van een boek die pas uitgekomen is en van de recentste Lynda.com videos. Ik focus me op Mysqli ipv PDO omdat ik zelf toch uitsluitend werk met Mysql. Het bevalt me enorm! Het enige dat er natuurlijk gebeurd is dat je na een tijdje begint te experimenteren. Het werkt en je bent blij...maar is het wel goed?...of correct?

Ik heb bijvoorbeeld iets simpels in elkaar geflanst, vlug vlug...

Doordat men enorm waarschuwt voor sql injection werk ik met prepared statements....

Kan iemand mij vertellen of mijn spaghetticode op "iets" trekt en veilig is? Ben ik op de goede weg?


<!DOCTYPE html>
<html lang="en">
  <head>
    <meta charset="utf-8">
    <title>title</title>
    <link rel="stylesheet" href="style.css">
    <script src="script.js"></script>
  </head>
  <body>
  	<form action="homes.php" method="POST">
  		<p>Zoek: <input type="text" name="search" /></p>
  		<p>Voornaam: <input type="text" name="Voornaam" /></p>
  		<p>Achternaam: <input type="text" name="Achternaam" /></p>
  		<p>Adres: <input type="text" name="Adres" /></p>
  		<p>Discipline: <input type="text" name="Discipline" value="1 tot 5" /></p>
  		<p>Bevestig zoeken: <input type="submit" name="Submit" /></p>
  		<p>Voeg toe aan databank <input type="submit" name="Add_DB" value="Add to Database" /></p>
  	</form>
  <?php
	require_once 'isset.php';
	?>
  </body>
</html>



This is the PHP

<?php
	require_once 'login.php';
	$dbcon =  mysqli_connect($db_host, $db_username, $db_password, $db_database);
	if(mysqli_connect_errno()) die ("Error during connection");

	if(isset($_POST['Submit'])){
		

		$Naam = $_POST['search'];

		$Result = mysqli_query($dbcon,"SELECT * FROM customers WHERE Voornaam = '$Naam'");
		if(!$Result) die ("Nothing to show");

		$Rows = $Result->num_rows;

		for($i=0; $i < $Rows; $i++){
			$Row = mysqli_fetch_array($Result, MYSQLI_ASSOC);
			echo "Voornaam: " . $Row['Voornaam'] . "<br>";
			echo "Achternaam: " . $Row['Achternaam'] . "<br>";
			echo "Adres: " . $Row['Adres'] . "<br>";
			echo "<hr>";
			
		}
	}

		if(isset($_POST['Add_DB'])){
			$sql="INSERT INTO customers(Voornaam, Achternaam, Adres, Actief, Discipline)
				VALUES(?,?,?,NOW(),?)";

			if($stmt = $dbcon->prepare($sql)){

					$stmt->bind_param('sssi', $Voornaam, $Achternaam, $Adres, $Discipline);
					
					$Voornaam = $_POST['Voornaam'];
					$Achternaam = $_POST['Voornaam'];
					$Adres = $_POST['Adres'];
					$Discipline = $_POST['Discipline'];
					$stmt->execute();
					

					echo "New records created successfully";
}
		}

		mysqli_close($dbcon);
?>



Groeten,
Michaël
1) procedureel en oop door elkaar gebruiken kan prima aangezien de procedurele functies wrappers zijn voor de oop classes en methoden.
Het is natuurlijk wel zo lelijk als het maar kan...

2) op regel 39 van het eerste script kan in SQL Injecteren aangezien je $Naam klakeloos in de SQL statement plakt.

3) leer jezelf of ALTIJD prepared statements te gebruiken.

PS. prepared statements zijn dus ook mogelijk met procedureel gebruik van de MySQLi extensie.
@Aar - Thx voor uitleg!

@ Dos -> op puntje 2) -> Hoe moet ik daar mezelf beveiligen? escape_string?

Er bestaan nog niet zoveel boeken over Mysqli...mag ik nog boeken gebruiken over mysql om dan zelf te converten naar Mysqli of is dit bad practice?

Groeten!
>> Hoe moet ik daar mezelf beveiligen? escape_string?

Nee met prepared statements:
<?php
$naam= "Frank";

/* prepared statement */
if ($stmt = mysqli_prepare($dbcon, "SELECT voornaam, achternaam, adres FROM customers WHERE voornaam=?")) {

/* bind parameters */
mysqli_stmt_bind_param($stmt, 's', $naam);

/* execute query */
mysqli_stmt_execute($stmt);

/* bind result variables */
mysqli_stmt_bind_result($stmt, $voornaam, $achternaam, $adres);

/* fetch value */
mysqli_stmt_fetch($stmt);

printf("%s %s %s\n", $voornaam, $achternaam, $adres);

/* close statement */
mysqli_stmt_close($stmt);
}
?>
>> Er bestaan nog niet zoveel boeken over Mysqli.
Heb je niet nodig joh. Koop dan een boek over (My)Sql in het algemeen. Dan leer je tenminste over vele mogelijkheden die Mysql je biedt.
Ik kwam trouwens onlangs een heel aardig engels ebook tegen die je zo kan downloaden: http://it-ebooks.info/book/3818/


[size=xsmall]Toevoeging op 16/12/2014 09:31:21:[/size]

Nog even een paar kleine aanmerkingen:

1) Algemene richtlijn is om variabelen zonder hoofdletter te starten. Bijv $voornaam maar ook $contentArray
2) tabelnamen en kolomnamen zie je meestal ook in uitsluitend kleine letters alhoewel dit zeker geen wet is.
Daarnaast valt me op dat je met engelse en nederlandse termen door elkaar werkt. Hou gewoon één taal aan en bij voorkeur engels.

Nogmaals het zijn geen wetten maar wel goede gebruiken.
Yep, goeie opmerkingen...de reden van engels/nederlands is gewoon dat ik begin te experimenteren en dan zeg ik, hup, dit erbij, hup dat erbij....testen....

Wanneer ik echt aan het werk ga, dan hou ik me uiteraard aan die regels! Goeie opmerking wel.

Ik heb dit boek die je voorgesteld hebt nu besteld bij Bol.com nadat ik het ingekeken heb op de pdf. Ik werk liever met iets voor me. Kwestie van beter te onthouden.



[size=xsmall]Toevoeging op 16/12/2014 11:58:00:[/size]

Voila...

Trekt dit op iets meer?


HTML
<!DOCTYPE html>
<html lang="en">
  <head>
    <meta charset="utf-8">
    <title>title</title>
    <link rel="stylesheet" href="style.css">
    <link href='http://fonts.googleapis.com/css?family=Raleway:200' rel='stylesheet' type='text/css'>
    <script src="script.js"></script>
  </head>
  <body>
      <table class="table_form">
        <form method="POST" action="homes.php">
          <tr>
            <td>Voornaam: </td><td><input type="text" name="Voornaam"></td>
          </tr>
          <tr>
            <td>Achternaam: </td><td><input type="text" name="Achternaam"></td>
          </tr>
          <tr>
            <td>Adres: </td><td><input type="text" name="Adres"></td>
          </tr>
          <tr>
            <td>Discipline: </td><td><input type="text" name="Discipline"></td>
          </tr>
          <tr>
            <td>Voeg toe aan databank: </td><td><input type="submit" name="Adddb" value="Bevestigen"></td>
          </tr>
         </form>
      </table>
<?php
  require_once 'isset.php';
?>



  </body>
</html>


php/sql
<?php
	require_once 'login.php';
	$db_con= new mysqli($db_host, $db_username, $db_password, $db_database);
	if($db_con->connect_error) die ("(" . $db_con->connect_error . " Error during connection");


	if(isset($_POST['Adddb'])){
		$stmt = $db_con->prepare("INSERT INTO customers (Voornaam, Achternaam, Adres, Actief, Discipline) VALUES(?,?,?,NOW(),?)");


		$stmt->bind_param("sssi",$voornaam, $achternaam, $adres, $discipline);

		$voornaam = $_POST['Voornaam'];
		$achternaam = $_POST['Achternaam'];
		$adres = $_POST['Adres'];
		$discipline = $_POST['Discipline'];
		$stmt->execute();

		echo "New records created successfully";

		$stmt->close();
		$db_con->close();

	}




?>
Dit ziet er goed uit. Neem aan dat het ook werkt?

- Voeg ook even dit toe nadat je connectie gemaakt is:

<?php
$db_con->set_charset("utf8");
?>
Het werkt...


Ik ga nu onder mijn form (gewoon ter test) proberen een sectie te voorzien om de data in te lezen

Zoeken op naam.

Is het mogelijk om alles uit te lezen en dan bij de resultaten een delete knop te laten verschijnen? Is dit mogelijk? Dat je bijvoorbeeld zoekt op iedereen die "Peter" noemt en vanaf dat er resultaten zijn, dat ernaast een button staat "Delete record" bijvoorbeeld. Geen resultaten -> geen knop. Of is dit eerder iets js?

Is mogelijk.
Waar plaats ik dan mijn <input type="submit" name="delete" value="delete record" />?
Gewoon naast het record.
Bij value is het makkelijker om daar het id in te zetten.
Hoe bedoel je San? Welk id moet ik zetten bij value?

Reageren