[MYSQL / PHP] Probleem met database updaten

[A.Kebab]

Ik heb een probleempje met de volgende code:

PHP:

<?php
error_reporting(E_ALL);
$con = mysql_connect("localhost","xx","xx");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }

mysql_select_db("xx", $con);

$string = file_get_contents('etext_nl2.txt');

// begin loop

$token = strtok($string, "-");
while ($token != false)
  {
    
    $myFile = "ebooks/".$token.".txt";
    $fh = fopen($myFile, 'r');
    $theData = fread($fh, filesize($myFile));
    fclose($fh);
    
    mysql_query("UPDATE books_nl SET contents = '$theData'
    WHERE etext = '$token'");

    echo $token."<br>";
    $token = strtok("-");
  }


mysql_close($con);
?>

De database wil de rijen niet updaten. De query vult content netjes in wanneer ik '$theData' vervang met 'blabla' en als ik $theData echo krijg ik ook netjes de gewilde output.

Het probleem is misschien dat ik ebooks van max 1.5 mb in de database wil zetten. Kan het hem zitten in de grootte van die bestanden? Ik gebruik een longtext veld.

Alvast bedankt!

[RobIII]

Je probleem zit 'm geheid in het feit dat er quotes in de tekst voorkomen; je zult daarom moeten escapen. Maar beter gebruik je parameterized queries; dan heb je er geen omkijken naar.

(En idd, zie hieronder, je query zal wel te groot worden).

[ Voor 36% gewijzigd door RobIII op 14-09-2009 21:34 ]

[TheBorg]

Er gaan een aantal dingen fout:
1. Het kan zijn dat de mysql instellingen geen queries van meer dan 1MB toestaan.
2. $theData is niet geescaped. mysql_real_escape_string($theData)
3. Die hele fopen/fread kun je beter vervangen voor file_get_contents();

[ Voor 8% gewijzigd door TheBorg op 14-09-2009 21:33 ]

[A.Kebab]

TheBorg schreef op maandag 14 september 2009 @ 21:32:
Er gaan een aantal dingen fout:
1. Het kan zijn dat de mysql instellingen geen queries van meer dan 1MB toestaan.
2. $theData is niet geescaped. mysql_real_escape_string($theData)
3. Die hele fopen/fread kun je beter vervangen voor file_get_contents();

Ja het was het escapen! Super bedankt

[Cartman!]

Daar had je makkelijk zelf achter kunnen komen als je de error van MySQL had bekeken.

PHP:

mysql_query('UPDATE...') or die(mysql_error());

[Confusion]

RobIII schreef op maandag 14 september 2009 @ 21:32:
Je probleem zit 'm geheid in het feit dat er quotes in de tekst voorkomen; je zult daarom moeten escapen. Maar beter gebruik je parameterized queries; dan heb je er geen omkijken naar.

Alleen jammer dat PHP daar geen fatsoenlijke ondersteuning voor heeft. Voor MySql inmiddels wel, middels mysqli_stmt_prepare, maar iets gelijksoortigs voor MS Sql bestaat niet.

[RobIII]

Confusion schreef op dinsdag 15 september 2009 @ 09:15:
Alleen jammer dat PHP daar geen fatsoenlijke ondersteuning voor heeft. Voor MySql inmiddels wel, middels mysqli_stmt_prepare, maar iets gelijksoortigs voor MS Sql bestaat niet.

Ik volg 't effe niet Wie heeft het over MSSQL? En je bent, aan je link te zien, bekend met MySQLi...dus je weet dat er wel ondersteuning is voor parameterized queries...

[ Voor 42% gewijzigd door RobIII op 15-09-2009 09:32 ]

[Cartman!]

Er bestaat ook gewoon pdo_mssql in PHP Confusion

[Confusion]

RobIII schreef op dinsdag 15 september 2009 @ 09:29:
[...]
Ik volg 't effe niet Wie heeft het over MSSQL?

Ik, aangezien ik gisteren een PHP applicatie ging uitbreiden, daarbij SQL iInjection mogelijkheden zag, dat wilde fixen door de queries naar geparametriseerde varianten om te zetten en me vervolgens zonder succes wezenloos heb gezocht naar een manier om dat op een fatsoenlijke manier te doen met MS Sql. Een subtiele aanpassing van je Google link helpt daarbij niet.

Cartman! schreef op dinsdag 15 september 2009 @ 09:30:
Er bestaat ook gewoon pdo_mssql in PHP Confusion

Dank je, maar zo 'gewoon' is die niet, want die heb ik gisteren niet gevonden. Ik heb ook niet onder PDO gezocht, aangezien ik niet dacht dat ik daar database-specifieke querymogelijkheden zou vinden. Dat is een totaal onlogische plaats om ze te verbergen.