[PHP]captcha Opcje

[b4rt3kk]

Witam, napisałem sobie system captcha, oto klasa:

[PHP] pobierz, plaintext 
class captcha{
 
	public $width;
	public $height;
	public $characters;
	public $lines;
 
	function __construct($width, $height, $characters, $lines){
		$this -> width = $width;
		$this -> height = $height;
		$this -> characters = $characters;
		$this -> lines = $lines;
	}
 
	function drawCaptcha(){
 
		// set up margin as 10% of image height
 
		$margin = ceil($this -> height * 0.1);
 
		// randomize background RGB color
 
		$backgroundR = mt_rand(100,150);
		$backgroundG = mt_rand(100,150);
		$backgroundB = mt_rand(100,150);
 
		// font color
 
		$fontR = mt_rand(0,20);
		$fontG = mt_rand(0,20);
		$fontB = mt_rand(0,20);
 
		// lines color
 
		$linesR = mt_rand(200,255);
		$linesG = mt_rand(200,255);
		$linesB = mt_rand(200,255);
 
		// create image
 
		$image = imagecreatetruecolor($this -> width, $this -> height);
 
		// colors
 
		$backgroundColor = imagecolorallocate($image, $backgroundR, $backgroundG, $backgroundB);
		$fontColor = imagecolorallocate($image, $fontR, $fontG, $fontB);
		$linesColor = imagecolorallocate($image, $linesR, $linesG, $linesB);
		imagefill($image, 0, 0, $backgroundColor);
 
		// draw lines on background
 
		for ($i=0; $i<$this -> lines; $i++) {
			imageline($image, rand(0, $this -> width), 0, rand(0, $this -> width), $this -> height, $linesColor);
			imageline($image, 0, rand(0, $this -> height), $this -> width, rand(0, $this -> height), $linesColor);
		}
 
		$font = '../font/Delicious-Roman.otf';
		$charTable = '0123456789abcdefghijklmnoprstuwyzABCDEFGHIJKLMNOPRSTUWYZ';
 
		// code generation
 
		$character = array();
		$angle = array();
 
		for ($i=1; $i<=$this -> characters; $i++) {
			$character[] = $charTable[mt_rand(0, strlen($charTable)-1)];
			$angle[] = mt_rand(-20,20);
		}
 
		$captchaCode = implode($character);
		$_SESSION['captchaCode'] = $captchaCode;
 
		// count max font size
 
		$fontSizeMax = floor($this -> height - $margin * 2);
 
		do {
 
			$fontSizeMax--;
			$captchaWidth = 0;
			$charHeight = array();
 
			for ($i=0; $i<sizeof($character); $i++) {
 
				$bbox = imageftbbox($fontSizeMax, $angle[$i], $font, $character[$i]);
				$captchaWidth += $charWidth = max(abs($bbox[0])+abs($bbox[2]), abs($bbox[4])+abs($bbox[6]))+$margin;
				$charHeight[] = max(abs($bbox[1])+abs($bbox[3]), abs($bbox[5])+abs($bbox[7]));
 
			}
 
			} while (max($charHeight) >= $this -> height OR $captchaWidth >= $this -> width - 2*$margin);
 
		// draw text to image
 
		$x = ($this -> width - $captchaWidth) / 2;
		$fontSpacing=$margin;
 
		for ($i=0; $i<sizeof($character); $i++) {
 
			$bbox = imageftbbox($fontSizeMax, $angle[$i], $font, $character[$i]);
			$charWidth = max(abs($bbox[0])+abs($bbox[2]), abs($bbox[4])+abs($bbox[6]))+$margin;
			imagettftext($image, $fontSizeMax, $angle[$i], $x+$fontSpacing, $this -> height/2+$margin, $fontColor, $font, $character[$i]);
			$fontSpacing += $charWidth;
 
		}
 
		Header('Content-type: image/png');
		imagepng($image);
 
	}
 
}
[PHP] pobierz, plaintext 

Plik index wygląda tak:

[PHP] pobierz, plaintext 
session_start();
 
echo '<img src="img/captcha.php"/>';
echo $_SESSION['captchaCode'];
[PHP] pobierz, plaintext 

oraz plik obrazka:

[PHP] pobierz, plaintext 
session_start();
 
function __autoload($class){
	require_once('../class/'.$class.'.class.php');
}
 
$captcha = new captcha(150, 50, 6, 10);
$captcha -> drawCaptcha();
[PHP] pobierz, plaintext 

No i po wyświetleniu pliku index.php generuje mi elegancko obrazek, natomiast $_SESSION['captchaCode'] na początku zwraca notice, że zmienna jest pusta, a po odświeżeniu jest nowo wygenerowany obrazek a $_SESSION zawiera kod poprzedniego, jak to przekazać, nie musi być w sesji, bym miał jednocześnie kod w postaci stringa i obrazek z tym samym kodem...

Ten post edytował b4rt3kk 17.07.2012, 00:50:19

[irmidjusz]

No tak na prosto, to skoro klasa captcha generuje kod i zapisuje go w sesji, to trzeba ten kod wygenerować zanim spróbujesz z niego skorzystać w jakimkolwiek innym miejscu programu. Nie trzeba generować od razu obrazka, wystarczy wygenerować tylko nowy 'captchaCode' i zapisać go np. w sesji, a klasa przy tworzeniu obrazka mogła by pobrać ten kod (a utworzy nowy, jeśli go jeszcze nie ma). Tylko musisz się upewnić, że za każdym odświeżeniem strony generowany jest nowy kod i nowy obrazek.

[d3ut3r]

Sama klasa captcha, nie powinna nic zapisywać w sesji jest to sprzeczne z paradygmatem obiektowości (IMG:style_emoticons/default/smile.gif) , powinna za to mieć metodę np. getCurrentCode, która zwraca ciąg jaki jest dostępny na obrazku, na zewnątrz klasy zapisujesz ten kod do sesji czy gdzie tam sobie chcesz.

Co do twojego problemu przyczyną jest sposób w jaki to robisz zobacz pierwsze wywołanie:

[PHP] pobierz, plaintext 
echo '<img src="img/captcha.php"/>';  //dopiero gdy user to odbierze, przeglądarka wywoła żądanie obrazka i dopiero wtedy ustawi captchaCode
echo $_SESSION['captchaCode']; // w pierwszym wywołaniu to nie istnieje
[PHP] pobierz, plaintext 

drugie

[PHP] pobierz, plaintext 
echo '<img src="img/captcha.php"/>';  //dopiero gdy user to odbierze, przeglądarka wywoła żądanie obrazka i dopiero wtedy ustawi captchaCode
echo $_SESSION['captchaCode']; // w drugim wywołaniu to ma wartość pierwszego wywołania (z tego samego powodu z którego w pierwszym wywołaniu jest puste
[PHP] pobierz, plaintext 

Jaśniej nie potrafię tego napisać (IMG:style_emoticons/default/biggrin.gif)

[b4rt3kk]

Dzięki, mała modyfikacja klasy:

[PHP] pobierz, plaintext 
	function generateCode(){
 
		$charTable = '0123456789abcdefghijklmnoprstuwyzABCDEFGHIJKLMNOPRSTUWYZ';
 
		$character = array();
		$angle = array();
 
		for ($i=1; $i<=$this -> characters; $i++) {
			$character[] = $charTable[mt_rand(0, strlen($charTable)-1)];
			$angle[] = mt_rand(-20,20);
		}
 
		$code = array($character, $angle);
 
		return $code;
 
	}
 
	function drawCaptcha($character, $angle){
[PHP] pobierz, plaintext 

indexu:

[PHP] pobierz, plaintext 
$width = 150;
$height = 50;
$characters = 6;
$lines = 10;
 
$captcha = new captcha($width, $height, $characters, $lines);
 
$_SESSION['code'] = $captcha -> generateCode();
$_SESSION['captchaCode'] = implode($_SESSION['code'][0]);
 
echo '<img src="img/captcha.php?w='.$width.'&h='.$height.'&c='.$characters.'&l='.$lines.'"/>';
echo $_SESSION['captchaCode'];
[PHP] pobierz, plaintext 

i samego obrazka:

[PHP] pobierz, plaintext 
$width = $_GET['w'];
$height = $_GET['h'];
$characters = $_GET['c'];
$lines = $_GET['l'];
 
$captcha = new captcha($width, $height, $characters, $lines);
$captcha -> drawCaptcha($_SESSION['code'][0], $_SESSION['code'][1]);
[PHP] pobierz, plaintext 

pomogły. (IMG:style_emoticons/default/smile.gif)