$_SESSION und Captcha funktioniert nicht richtig - Seite 2

fg__killer · 18.12.2011, 10:49

Ich habe es mal ein bisschen umgeschrieben, sollte so funktionieren.

PHP-Code:

  <?php
session_start();
header('Content-type: image/png');
header("Cache-Control: no-cache, must-revalidate");
header("Expires: Sat, 26 Jul 1997 05:00:00 GMT");

if($SESSION['captcha']<time()-2) {
    $bild = imagecreatefrompng("2.png");
    function srand($a, $b) { return rand(-1, 1); }
    $chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
    $chars .= strtolower($chars);
    $chars .= "0123456789";
    $chars = str_split($chars);
    usort($chars, 'srand');
    $chars = join("", $chars);
    $length = 5;

    function rnd() {
        global $chars;
        $a = strlen($chars);
        $b = (int)(rand(0, $a-1));
        return substr($chars, $b, 1);
    }
    
    $str = "";
    
    for($i=0;$i<$length;$i++) {
        $c = rnd();
        $str .= $c;
        $angle = rand(-30, 30);
        $size = rand(24, 32);
        $j = rand(-10, 10);
        $farbe_b = imagecolorallocate($bild, 0, 0, 0);
        imagettftext($bild, $size, $angle, ($i*30+50), (92+$j), $farbe_b, dirname(__FILE__).DIRECTORY_SEPARATOR.'arial.ttf', $c);
    }
    global $SESSION;
    @unlink('captcha/cap'.$SESSION['captcha'].'.png'); // ordner captcha sollte chmod 777 haben
    $SESSION['captcha_str'] = $str;
    $SESSION['captcha'] = time();
    $a->__destruct();
    imagepng($bild, 'captcha/cap'.$SESSION['captcha'].'.png');
}
echo file_get_contents('captcha/cap'.$SESSION['captcha'].'.png');

lstegelitz · 18.12.2011, 13:48

Da dürfte so ziemlich garnichts funktionieren...

Zitat:

PHP-Code:

  global $SESSION;

1. Ist die Session superglobal
2. Ist sie falsch geschrieben: $_SESSION

global ist generell schlechter Stil. Funktionsparameter sind guter Stil.

"conditional function declaration" ist ebenfalls schlechter Stil. (D.h. du hast eine Funktion innerhalb eines if-Blocks deklariert)

srand() ist eine Funktion aus der PHP API. Redeklaration gibt einen fatalen Fehler.

Zitat:

PHP-Code:

  $a->__destruct();

Magische Methoden sollte man niemals direkt aufrufen (sonst ist die Magie dahin

), die werden automagisch aufgerufen, wenn die Umstände passen.
Es gibt Ausnahmen, z.B. Aufruf des Parent-Destruktors. Das passiert aber im Destruktor der abgeleiteten Klasse.
Besser wäre in diesem Fall:

PHP-Code:

  unset($a);

Der Code ist weit entfernt von dem, was man von einem Fortgeschrittenen erwarten darf.

K313 · 18.12.2011, 14:39

PHP-Code:

  <?php

session_start();

header('Content-type: image/png');

header("Cache-Control: no-cache, must-revalidate");

header("Expires: Sat, 26 Jul 1997 05:00:00 GMT");

 
if($_SESSION['captcha']<($time=time())-2) { // 1. $_SESSION    2. wenn du 2 mal oder mehr die functionsergebnis benutzst- speicher die ergebnis einfach in eine variable (eine function auszurufen braucht php viel meher zeit und memory)

    $bild = imagecreatefrompng("2.png"); // wenn der hintergrund nicht zu wichtig ist, dann lieber "$bild=imagecreatetruecolor(width,height)".  bei dir muss php viele schritte schaffen: 1. die file finden; 2. ablesen; 3. von png-format in gd2-format umwandeln.

    function srand($a, $b) { return rand(-1, 1); }// ????? gibd diese function  etwas anderes als "rand(-1,1);" aus?

    $chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; // wenn du macht's so   $chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789"; - dann werden ALLE deine php-scripts im variable $chars ein link an diese string behalten was bedeutet etwa 6-10 bytes im memory

    $chars .= strtolower($chars); // aber wenn du die variable verenderst, dann muss php neues object im memory erzeugen...

    

    // alles folgendes was um eine zufälige code zu bauen geht ist Überschuss...

    $chars .= "0123456789";

    $chars = str_split($chars);

    usort($chars, 'srand');

    $chars = join("", $chars);

    $length = 5;

 
    function rnd() { // zu viel arbeit für gleiches geld.....

        global $chars;

        $a = strlen($chars);

        $b = (int)(rand(0, $a-1));

        return substr($chars, $b, 1); //wenn du nur ein simbol brauchst so siet es besser aus " return $chars{$b}; "

    }

    

     // bis hier

    

     

   $str = "";

    

 

    for($i=0;$i<$length;$i++) { // $length bei dir is fest und glecht 5 " for($i=0;$i<5;$i++) { "

        $c = rnd(); // du brauchst diese function nicht immer wieder auszurufen

        $str .= $c; // man kann einfach so " $str.=$chars{rand(0, 65)} ; " du hast stabile lenge des strings $chars

 
        // und hier alles im gegenteil dies " $time=time() ",  du verwenderst folgende variables nur ein mal, wofür dann erzeugst du neue variables? fur php bedeutet es meher arbeit und memoryverwendung

        $angle = rand(-30, 30);

        $size = rand(24, 32);

        $j = rand(-10, 10);

 
        $farbe_b = imagecolorallocate($bild, 0, 0, 0);

        imagettftext($bild, $size, $angle, ($i*30+50), (92+$j), $farbe_b, dirname(__FILE__).DIRECTORY_SEPARATOR.'arial.ttf', $c);

        //  nach gnädige weise "  imagettftext($bild, rand(24, 32), rand(-30, 30), ($i*30+50), (92+rand(-10, 10)), $farbe_b, dirname(__FILE__).DIRECTORY_SEPARATOR.'arial.ttf', $c); "

    }

    global $SESSION; // $_SESSION ist immer globalvariable denke ich.... :roll:

    @unlink('captcha/cap'.$SESSION['captcha'].'.png'); // ordner captcha sollte chmod 777 haben -------  nein die benutzern dürfen in diesem ordner kein php-script ausrufen und auch nicht lesen und writen, dass bedeutet mindestens 0770 (vorstehende 0 nicht vergessen). Aber du hast dort vielleich auch kein php-script, du benotigst dies ord nur zum lesen und writen, und dann chmod = 0760

    $SESSION['captcha_str'] = $str;

    $SESSION['captcha'] = time();

    $a->__destruct();

    imagepng($bild, 'captcha/cap'.$SESSION['captcha'].'.png'); // :-| und jetzt uberhaupt ganz toll... du speicherst das bild in file und danach wirdst wieder dies file ablesen... mit welchem zweck?.....

}

echo file_get_contents('captcha/cap'.$SESSION['captcha'].'.png');

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Variablenübergabe mit $_Session funktioniert nicht richtig	salerno	PHP Einsteiger	5	25.09.2011 15:59
[Erledigt] function funktioniert ohne alert() nicht richtig	Tholi	JavaScript, Ajax und mehr	1	19.02.2011 02:40
[Erledigt] Captcha funktioniert nicht	DJ_RhoxxZ	PHP Tipps 2010	12	22.07.2010 14:20
[Erledigt] FTP und MySQL funktioniert nicht mehr richtig	JackSparrow	PHP Tipps 2010	2	06.06.2010 20:54
Verschlüsselung funktioniert nicht richtig	netmario	PHP Tipps 2010	2	19.05.2010 22:57
[Erledigt] Checkuser und/oder Logout funktioniert nicht richtig	p300d	PHP Tipps 2009	7	28.06.2009 13:33
Select funktioniert nicht richtig	JooiPHP	Datenbanken	7	18.06.2009 15:09
[Erledigt] Apache funktioniert nicht richtig	YYA	Server, Hosting und Workstations	5	16.06.2008 15:24
Warum funktioniert meine Statistik nicht richtig?	Calli	PHP Tipps 2007	1	27.11.2005 08:49
Script funktioniert nicht richtig -> fehlermeldungen		PHP Tipps 2005-2	2	12.09.2005 11:29
Karte funktioniert nicht ganz richtig ( koardinaten system )		PHP Tipps 2005-2	7	15.06.2005 16:48
PHP funktioniert nicht richtig?!		PHP Tipps 2005-2	26	11.06.2005 09:05
datenübergabe funktioniert nicht richtig	Fraylman	PHP Tipps 2004	6	25.08.2004 13:49
Emailversendung funktioniert nicht richtig		PHP Tipps 2004	1	11.08.2004 23:16
php funktioniert nicht richtig		PHP Tipps 2004	3	14.06.2004 08:04

18.12.2011, 10:49
fg__killer Neuer Benutzer Registriert seit: 17.12.2011 Beiträge: 7 PHP-Kenntnisse: Fortgeschritten	Ich habe es mal ein bisschen umgeschrieben, sollte so funktionieren. PHP-Code: <?php session_start(); header('Content-type: image/png'); header("Cache-Control: no-cache, must-revalidate"); header("Expires: Sat, 26 Jul 1997 05:00:00 GMT"); if($SESSION['captcha']<time()-2) { $bild = imagecreatefrompng("2.png"); function srand($a, $b) { return rand(-1, 1); } $chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; $chars .= strtolower($chars); $chars .= "0123456789"; $chars = str_split($chars); usort($chars, 'srand'); $chars = join("", $chars); $length = 5; function rnd() { global $chars; $a = strlen($chars); $b = (int)(rand(0, $a-1)); return substr($chars, $b, 1); } $str = ""; for($i=0;$i<$length;$i++) { $c = rnd(); $str .= $c; $angle = rand(-30, 30); $size = rand(24, 32); $j = rand(-10, 10); $farbe_b = imagecolorallocate($bild, 0, 0, 0); imagettftext($bild, $size, $angle, ($i30+50), (92+$j), $farbe_b, dirname(__FILE__).DIRECTORY_SEPARATOR.'arial.ttf', $c); } global $SESSION; @unlink('captcha/cap'.$SESSION['captcha'].'.png'); // ordner captcha sollte chmod 777 haben $SESSION['captcha_str'] = $str; $SESSION['captcha'] = time(); $a->__destruct(); imagepng($bild, 'captcha/cap'.$SESSION['captcha'].'.png'); } echo file_get_contents('captcha/cap'.$SESSION['captcha'].'.png'); Geändert von fg__killer (18.12.2011 um 11:02 Uhr).*


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

18.12.2011, 14:39
K313 Benutzer Registriert seit: 18.12.2011 Beiträge: 37 PHP-Kenntnisse: Fortgeschritten	PHP-Code: <?php session_start(); header('Content-type: image/png'); header("Cache-Control: no-cache, must-revalidate"); header("Expires: Sat, 26 Jul 1997 05:00:00 GMT"); if($_SESSION['captcha']<($time=time())-2) { // 1. $_SESSION 2. wenn du 2 mal oder mehr die functionsergebnis benutzst- speicher die ergebnis einfach in eine variable (eine function auszurufen braucht php viel meher zeit und memory) $bild = imagecreatefrompng("2.png"); // wenn der hintergrund nicht zu wichtig ist, dann lieber "$bild=imagecreatetruecolor(width,height)". bei dir muss php viele schritte schaffen: 1. die file finden; 2. ablesen; 3. von png-format in gd2-format umwandeln. function srand($a, $b) { return rand(-1, 1); }// ????? gibd diese function etwas anderes als "rand(-1,1);" aus? $chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; // wenn du macht's so $chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789"; - dann werden ALLE deine php-scripts im variable $chars ein link an diese string behalten was bedeutet etwa 6-10 bytes im memory $chars .= strtolower($chars); // aber wenn du die variable verenderst, dann muss php neues object im memory erzeugen... // alles folgendes was um eine zufälige code zu bauen geht ist Überschuss... $chars .= "0123456789"; $chars = str_split($chars); usort($chars, 'srand'); $chars = join("", $chars); $length = 5; function rnd() { // zu viel arbeit für gleiches geld..... global $chars; $a = strlen($chars); $b = (int)(rand(0, $a-1)); return substr($chars, $b, 1); //wenn du nur ein simbol brauchst so siet es besser aus " return $chars{$b}; " } // bis hier $str = ""; for($i=0;$i<$length;$i++) { // $length bei dir is fest und glecht 5 " for($i=0;$i<5;$i++) { " $c = rnd(); // du brauchst diese function nicht immer wieder auszurufen $str .= $c; // man kann einfach so " $str.=$chars{rand(0, 65)} ; " du hast stabile lenge des strings $chars // und hier alles im gegenteil dies " $time=time() ", du verwenderst folgende variables nur ein mal, wofür dann erzeugst du neue variables? fur php bedeutet es meher arbeit und memoryverwendung $angle = rand(-30, 30); $size = rand(24, 32); $j = rand(-10, 10); $farbe_b = imagecolorallocate($bild, 0, 0, 0); imagettftext($bild, $size, $angle, ($i30+50), (92+$j), $farbe_b, dirname(__FILE__).DIRECTORY_SEPARATOR.'arial.ttf', $c); // nach gnädige weise " imagettftext($bild, rand(24, 32), rand(-30, 30), ($i30+50), (92+rand(-10, 10)), $farbe_b, dirname(__FILE__).DIRECTORY_SEPARATOR.'arial.ttf', $c); " } global $SESSION; // $_SESSION ist immer globalvariable denke ich.... :roll: @unlink('captcha/cap'.$SESSION['captcha'].'.png'); // ordner captcha sollte chmod 777 haben ------- nein die benutzern dürfen in diesem ordner kein php-script ausrufen und auch nicht lesen und writen, dass bedeutet mindestens 0770 (vorstehende 0 nicht vergessen). Aber du hast dort vielleich auch kein php-script, du benotigst dies ord nur zum lesen und writen, und dann chmod = 0760 $SESSION['captcha_str'] = $str; $SESSION['captcha'] = time(); $a->__destruct(); imagepng($bild, 'captcha/cap'.$SESSION['captcha'].'.png'); // :-\| und jetzt uberhaupt ganz toll... du speicherst das bild in file und danach wirdst wieder dies file ablesen... mit welchem zweck?..... } echo file_get_contents('captcha/cap'.$SESSION['captcha'].'.png');