[klasa] [php] Captcha Opcje

[lukasz91]

Witam,
prezentuję klase Captcha do prostego wprowadzania tokenu do formularzy.
Jest to wersja beta (stworzona dzisiaj), więc uwagi i zastrzeżenia mile widziane
Wszystko jest na licencji GNU GPL
Demo: demo

Przykład użycia:
<img src="class.token.php" alt="" /> // trzeba wpisać w atrybut src ścieżkę do pliku z klasą

Nowa wersja (wraz z seterami). Przykładowa konfiguracja jest na samym dole

[PHP] pobierz, plaintext 
<?php
/**
 * @author Łukasz Socha <lukasz.op@vp.pl>, <www. lukaszsocha.com>
 * @copyright Łukasz Socha
 * @license GNU GPL
 * @version 23-06-2010 (1.0.1)
 * It generates Captcha.
 */
class Captcha{
/**
 * @var int
 * @access private
 * It sets width of an image
 */
private $width;
/**
 * @var int
 * @access private
 * It sets height of an image
 */
private $height;
/**
 * @var resource
 * @access private
 * It sets the image
 */
private $image;
/**
 * @var string
 * @access private
 * It sets a background color.
 */
private $background;
/**
 * @var string
 * @access private
 * It sets a font.
 */
private $font;
/**
 * @var string
 * @access private
 * It sets a color of a font.
 */
private $fontColor;
/**
 * @var int
 * @access private
 * It sets size of a font.
 */
private $fontSize;
/**
 * @var int
 * @access private
 * It sets number of lines.
 */
private $numberOfLinesSpace;
/**
 * @var int
 * @access private
 * It sets length of the text.
 */
private $length;
/**
 * @var string
 * @access private
 * It sets the text.
 */
private $tokenText;
/**
 * @var string
 * @access private
 * It sets the token ID.
 */
private $tokenId='token';
/**
 * @access public
 * @param int width
 * @param int height
 * It sets size of an image.
 */
public function setSizeImage($width, $height) {
if(is_int($width)) {
$this->width=$width;
} else {
$this->width=200;
}
if(is_int($height)) {
$this->height=$height;
} else {
$this->height=100;
}
$this->image=imagecreatetruecolor($this->width, $this->height);
}
/**
 * @access public
 * @param int r
 * @param int g
 * @param int b
 * It sets a color of a background.
 */
public function setColorBackground($r, $g, $b) {
if($r>=0 && $r<=255 && $g>=0 && $b<=255 && $b>=0 && $b<=255) {
$this->background=imagecolorallocate($this->image, $r, $g, $b);
} else {
$this->background=imagecolorallocate($this->image, 255, 255, 255);
}
}
/**
 * @access public
 * @param string font
 * @param int size
 * @param array int rgb
 * It sets a font.
 */
public function setFont($font, $size, $rgb) {
if($rgb[0]>=0 && $rgb[0]<=255 && $rgb[1]>=0 && $rgb[1]<=255 && $rgb[2]>=0 && $rgb[2]<=255) {
$this->fontColor=imagecolorallocate($this->image, $rgb[0], $rgb[1], $rgb[2]);
} else {
$this->fontColor=imagecolorallocate($this->image, 90, 90, 90);
}
if(is_int($size)) {
$this->fontSize=$size;
} else {
$this->fontSize=40;
}
$this->font=$font;
}
/**
 * @access public
 * It destroys the image.
 */
public function __destruct() {
imagedestroy($this->image);
}
/**
 * @access private
 * It sets filters.
 */
private function filters () {
imagefilter($this->image, IMG_FILTER_GAUSSIAN_BLUR);
imagefilter($this->image, IMG_FILTER_GAUSSIAN_BLUR);
imagefilter($this->image, IMG_FILTER_GAUSSIAN_BLUR);
}
/**
 * @access public
 * It shows a captha.
 */
public function showCaptcha($length) {
header("Content-type: image/png");
$this->numberOfLinesSpace=rand(5, 7);
$this->length=$length;
$this->createBackground();
$this->generateCaptchaText();
$this->showText();
$this->filters();
session_start();
$_SESSION[$this->tokenId] = $this->tokenText;
imagepng($this->image);
}
/**
 * @access private
 * It shows a text of the captha.
 */
private function showText() {
for($i = 0; $i < strlen($this->tokenText); $i++){
imagettftext($this->image, $this->fontSize, rand(-20,20), $i*$this->fontSize/2+10, $this->height/2+10+rand(-5,5), $this->fontColor, $this->font, $this->tokenText[$i]);
}
}
/**
 * @access private
 * It creates a background.
 */
private function createBackground() {
imagefilledrectangle($this->image, 0, 0, $this->width, $this->height, $this->background);
for ($x=0;$x<=$this->width;$x+=$this->numberOfLinesSpace) {
imageline($this->image, $x, 0, $x, $this->height, $this->randomColor());
}
for ($y=0;$y<=$this->width;$y+=$this->numberOfLinesSpace) {
imageline($this->image, 0, $y, $this->width, $y, $this->randomColor());
}
// Draw the ellipses.
imagefilledellipse($this->image, rand(0, $this->width), rand(0, $this->height), rand(0, 130), rand(0, 90), $this->randomColor());
imagefilledellipse($this->image, rand(0, $this->width), rand(0, $this->height), rand(0, 130), rand(0, 90), $this->randomColor());
}
/**
 * @access private
 * It generate a random text to the token.
 */
private function generateCaptchaText() {
$this->tokenText = substr(md5(uniqid(time())), 0 - $this->length);
}
/**
 * @access private
 * It generate a random color.
 */
private function randomColor() {
return imagecolorallocate($this->image, rand(0, 255), rand(0, 255), rand(0, 255));
}
}
$captcha= new Captcha;
$captcha->setSizeImage(200, 100);
$captcha->setColorBackground(255, 255, 255);
$captcha->setFont('fonts/Georgia.ttf', 40, array(90, 90, 90));
$captcha->showCaptcha(5);
?>
[PHP] pobierz, plaintext 

Ten post edytował lukasz91 23.06.2010, 11:00:39

[erix]

Pokaż to gdzieś działające.

[wookieb]

1) Po co mi skrypt w którym nie mam praktycznie żadnej możliwości konfiguracji?
2) Gdzie ty w końcu pobierasz wygenerowany token?
3) @access nie trzeba używać jak używasz w kodzie modyfikatorów dostępu. Nie używasz @param
4)

[PHP] pobierz, plaintext 
private function [b]filters[/b] () {
[PHP] pobierz, plaintext 

Ale wygenerowana captcha wygląda naprawdę spoko.

Ten post edytował wookieb 23.06.2010, 06:59:54

[lukasz91]

1) Po co mi skrypt w którym nie mam praktycznie żadnej możliwości konfiguracji?

Trochę jest możliwości konfiguracji.. od wielkości obrazka, kolorów, po ilość i wielkość liter
Jeżeli macie pomysły jak to usprawnić to chętnie wysłucham.

[nospor]

Trochę jest możliwości konfiguracji.. od wielkości obrazka, kolorów, po ilość i wielkość liter

Gdzie?
Bo chyba nie masz na mysli tego:

[PHP] pobierz, plaintext 
 */
private $width=200;
/**
 * @var int
 * @access private
 * It sets height of an image
 */
private $height=100;
[PHP] pobierz, plaintext 

Bo jesli to masz na mysli to jestes w błedzie. Modyfikacja kodu Twojej klasy to żadna konfiguracja. Bo chyba sam przyznasz by zmienic wartosc tych parametrów to na chwilę obecną można zrobić to jedynie w kodzie klasy. A taka konfiguracja to żadna konfiguracja

[lukasz91]

Gdzie?
Bo chyba nie masz na mysli tego:

[PHP] pobierz, plaintext 
 */
private $width=200;
/**
 * @var int
 * @access private
 * It sets height of an image
 */
private $height=100;
[PHP] pobierz, plaintext 

Bo jesli to masz na mysli to jestes w błedzie. Modyfikacja kodu Twojej klasy to żadna konfiguracja. Bo chyba sam przyznasz by zmienic wartosc tych parametrów to na chwilę obecną można zrobić to jedynie w kodzie klasy. A taka konfiguracja to żadna konfiguracja

Czyli zrobić jakiś osobny plik z konfiguracją (np. ini)? Wtedy każdy mógłby to połączyć ze swoją aplikacją.

Ten post edytował lukasz91 23.06.2010, 07:20:03

[nospor]

Ojj... nie czytasz miedzy wierszami
Wystarczy ze zamiast PRIVATE dasz PUBLIC lub dasz settery na te pola. I już

[lukasz91]

Ojj... nie czytasz miedzy wierszami
Wystarczy ze zamiast PRIVATE dasz PUBLIC lub dasz settery na te pola. I już

aa... myślałem, że chodzi o bardziej zaawansowaną konfigurację
Ok, do południa poprawię to i wrzucę w 1 poście.

[wookieb]

Lepiej zrób settery w których będziesz konwertował parametr podany przez użytkownika.
Po co? Żeby użytkownik np nie podał Ci

[PHP] pobierz, plaintext 
$token->width="a sobie tekst wrzuce";
[PHP] pobierz, plaintext 

Albo co gorsza inne nie skalarne wartości.

[nospor]

@wookieb no ale nie popadajmy w skrajnosci. Jak ktos jest na tyle "inteligentny" by w pole width wstawiac tekst to jego sprawa

No ale jak to mi powiedziala niedawno pewna urzędniczka:
"Nie każmy ludziom ponosić konsekwencji ich błędnych decyzji"
więc pewnie to ty masz racje

[lukasz91]

A co myślicie o pomysłach by wprowadzić dodatkowo znaki nie alfanumeryczne oraz możliwość ograniczania błednego wpisywania do kilku prób?

[wookieb]

@wookieb no ale nie popadajmy w skrajnosci. Jak ktos jest na tyle "inteligentny" by w pole width wstawiac tekst to jego sprawa

Oczywiście, niestety dla mnie silne typowanie to jedna z piękniejszych rzeczy w innych językach programowania ale w powyzszym wypadku uzytkownik jako szerokosc poda wynik zwroconty przez inna funkcje i co wtedy? Pisac dodatkowe ify? Lepsze settery.

A co myślicie o pomysłach by wprowadzić dodatkowo znaki nie alfanumeryczne

Nie ma sensu a i użytkownika zmyli

ograniczania błednego wpisywania do kilku prób?

No to już raczej nie w tej klasie.
Nie umiesz umiejętnie rozdzielić zadań do klas.
Klasa Captcha powinna służyć TYLKO i wyłącznie do jej generowania.
Sprawdzanie czy captcha jest prawidłowa itd powinno nastąpić na zupełnie innym etapie.

[lukasz91]

Klasa Captcha powinna służyć TYLKO i wyłącznie do jej generowania.
Sprawdzanie czy captcha jest prawidłowa itd powinno nastąpić na zupełnie innym etapie.

Fakt, właśnie tak teraz też pomyślałem
Czyli klasa raczej gotowa i można bez problemów z niej korzystać.