Napisałem sobie prostą klasę do obsługi captcha i zastanawiam się czy jest to w miarę sensownie napisane, bo nie wzorowałem się na niczym innym.
Proszę o rady.

[PHP] pobierz, plaintext 
class captcha
{
    static $image;
    static $string;
    static $font = 'captcha.ttf';
    static $type = 'random'; /* random, words */
 
    static public function image ()
    {
        echo '<img src="captcha.render.php" alt="captcha" />';
    }
 
    static public function render ()
    {
        self::set();
        $_SESSION["captcha"] = self::$string;    
        self::create();    
    }
 
    static private function create()
    {
        $string = $_SESSION["captcha"];
        header('Content-type: image/png');
        self::$image = imagecreatefromjpeg ( 'captcha.jpg' );
        $text_color = imagecolorallocate(self::$image, 130, 130, 130);
        imagettftext(self::$image, 15, 0, 20, 27, $text_color, self::$font, $string);
        imagepng(self::$image);
        imagedestroy(self::$image);    
    }
 
    static private function set()
    {
        switch ( self::$type )
        {
            case 'random':
            for ( $i = 0; $i < 5; $i++ )
            {
                $string .= chr ( rand ( 97, 122 ) );
            }    
            self::$string = $string;
            break;
            case 'word':
                /* nie dziala */
            break;
        }    
    }
}
[PHP] pobierz, plaintext 

niby ok, tylko brakuje jeszcze najważniejszego czyli walidacji captchy, wydaje mi się też że pokazywanie tekstu z tlem w takiej wersji jak jest teraz może być łatwe do zgadnięcia przez roboty. zastanów się też czy ładowanie wszystkiego w static ma sens, bo chyba lepiej byłoby tworzyć nową instancję captchy przy danym użyciu jej w określonym momencie.

niby ok, tylko brakuje jeszcze najważniejszego czyli walidacji captchy, wydaje mi się też że pokazywanie tekstu z tlem w takiej wersji jak jest teraz może być łatwe do zgadnięcia przez roboty. zastanów się też czy ładowanie wszystkiego w static ma sens, bo chyba lepiej byłoby tworzyć nową instancję captchy przy danym użyciu jej w określonym momencie.