Captcha

Ja dat ligt echt vet wel aan je code, om precies te zijn aan regel 43 en 45..

Oftewel laat even de relevante code zien: de code zien waarmee je code in de afbeelding en sessie zet...

<?php
// Voor de sessie...
session_start();

// Hier stoppen we alle letters en getallen in een string...
$sCode = implode(range('a', 'z'));
$sCode .= implode(range(0, 9));

// De eerste 6 tekens sellecteren van een string die random gemaakt is...
$sCode = substr(str_shuffle($sCode), 0, 6);

// De IMG zelf...
if ($image = imagecreatetruecolor(115, 25))
{
// De achtergrond kleur...
$fontcolor = imagecolorallocate ($image, rand(190, 255), rand(190, 255), rand(190, 255));

// Lusje maken...
for ($i = 0; $i < strlen($sCode); $i++)
{
// Ze zullen om-en-om in een negative of positieve rotatie staan...
$iRotation = ($i % 2 == 0) ? rand(-12, -6) : rand(6, 12);
$iMargin = ($i == 0) ? 14 : $i * 14 + 20;

// We gaan de text in de IMG zetten..
imagettftext($image, 11, $iRotation, $iMargin, rand(12, 22), $fontcolor, 'arial.ttf', $sCode{$i});
}

// De waarde in een sessie zetten...
$_SESSION['captcha'] = $sCode;

// Headertje verzenden...
header('Content-type: image/png');
// De IMG weergeven...
imagepng ($image);
// Het is klaar...
imagedestroy ($image);
}
else
{
// Error weergeven...
echo'Het plaatje kon niet worden aangemaakt.');
}
?>

Zoals je op de site kan zien zijn de codes het zelfde, alleen die sessie loopt 1 code achter.

klopt, hoe pas ik dat aan?

Laat code van je formulier eens zien

<img src="captcha.php" alt="captcha">
<input class="input" type="text" name="captcha" maxlength="7">

Ik heb dus een bestand captcha.php daar word de sessie en de afbeelding gemaakt die ik dan in mijn reactiescript oproep.

<?php
session_start();
header("Content-type: image/png");
$chars = Array_Merge(range('a', 'z'), range(0, 9));

$_SESSION['string'] = '';

for ($i = 0; $i < 6; $i++) {
    $_SESSION['string'] .= $chars[rand(0, count($chars)) - 1];
}
$img = imagecreate(6 * 18, 45);
imagefill($img, 0, 0, imagecolorallocate($img, 255, 255, 255));
imageline($img, 0, 0, 0, 44, imagecolorallocate($img, 0, 0, 0));
imageline($img, 0, 0, 6 * 18 - 1, 0, imagecolorallocate($img, 0, 0, 0));
imageline($img, 6 * 18 - 1, 0, 6 * 18 - 1, 45, imagecolorallocate($img, 0, 0, 0));
imageline($img, 0, 44, 6 * 18 - 1, 44, imagecolorallocate($img, 0, 0, 0));

for ($i = 0; $i < 10; $i++) {
    imageline($img, rand(2, 6 * 18 - 1), 1, rand(2, 6 * 18 - 1), 44, imagecolorallocate($img, rand(0, 255), rand(0, 255), rand(0, 255)));
}

for ($i = 0; $i < 6; $i++) {
    imagettftext($img, 19, rand(-20, 20), $i * 16 + 6, rand(20, 40), imagecolorallocate($img, 0, 0, 0), 'font.ttf', $_SESSION['string'][$i]);
}

imageline($img, 1, rand(2, 16), 6 * 18 - 1, rand(2, 16), imagecolorallocate($img, rand(0, 245), rand(0, 245), rand(0, 245)));
imageline($img, 1, rand(17, 33), 6 * 18 - 1, rand(17, 33), imagecolorallocate($img, rand(0, 245), rand(0, 245), rand(0, 245)));
imagepng($img);
imagedestroy($img);
?>

:-)

Misschien klopt het niet wat ik nu zeg, maar het is al laat...

Volgens mij is dit wat er gebeurt:

1) er wordt door het script een captcha code aangemaakt (die noemen we even "code A")
2) er wordt een plaatje gegenereerd dat onderaan het formulier wordt getoond
3) de gebruiker vult de code in in het invoerveld en drukt op de verzendknop
4) DE PAGINA WORDT OPNIEUW AANGEROEPEN en de post waardes worden meegestuurd
5) er wordt nu opnieuw een captcha code aangemaakt (die noemen we even "code B")
6) de door de gebruiker ingevoerde captcha code wordt vergeleken met de zojuist aangemaakte "code B"

Je vergelijkt dus de door de gebruiker ingevoerde code met een zojuist nieuw gegenereerde code. Ik denk dat je dit simpelweg kunt voorkomen door een if-statement te bouwen om de php-code die de captcha code genereert, dus zoiets:

<?php
if ($_SERVER['REQUEST_METHOD'] != 'POST') {
// genereer captcha code
}
?>

Jasper, (in je post van 27/02/2011 17:27:06) je hebt een parse error op lijn 42.

------------
(aanpassing)

Wat is het probleem eigenlijk?
Bij mij werkt dit perfect.
Een illustratie:

index.php
<?php
session_start();
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
echo '$_SESSION["captcha"]: '. $_SESSION["captcha"] .'<br/>';
echo '$_POST["captcha"]: '. $_POST["captcha"] .'<br/>';
}
echo '
<form action="" method="post">
<input class="input" type="text" name="captcha" maxlength="7">
<input type="submit" value="GO!"/>
<br/>
<img src="captcha.php" alt="captcha"> <br/>
</form>
';
?>

captcha.php: onveranderd (juist de parse error weggehaald)

(Edit: Leuke captcha afbeelding. Ik zou dit ook wel durven gebruiken)