Syntax Error in Class

coolerj · 07.03.2010, 17:21

Hallo,

Ich habe ein Syntax Error in meiner Klasse verstehe diesen aber kein bisschen.
Error:
Parse error: syntax error, unexpected T_VARIABLE, expecting T_FUNCTION in C:\xampp\htdocs\encarnium\includes\classes\captcha .php on line 42

Code:

PHP-Code:

  <?php

 
session_start(); 

unset($_SESSION['captcha_spam']); 

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

 
/**

  * Bitte am Anfang des Verwender Code´s header('Content-type: image/png'); einfügen.

  * Oder bevor der ersten Ausgabe

  */

 
 
class createCaptcha {

 
    private $usec;

    private $sec;

    private $possible;

    private $str;

    private $text;

    private $img;

    private $color;

    private $ttf;

    private $ttfsize;

    private $angle;

    private $t_x;

    private $t_y;

    

       function randomString($len) { 

          function make_seed(){ 

               list($this -> usec , $this -> sec) = explode (' ', microtime()); 

                return (float) $this -> sec + ((float) $this -> usec * 100000); 

           } 

        srand(make_seed());  

           //Der String $possible enthält alle Zeichen, die verwendet werden sollen 

           $this -> possible="ABCDEFGHJKLMNPRSTUVWXYZabcdefghijkmnpqrstuvwxyz23456789"; 

           $this -> str=""; 

         while(strlen($this -> str)<$len) { 

               $this -> str.=substr($this -> possible,(rand()%(strlen($this -> possible))),1); 

           } 

        return($this -> str); 

       } 

       $this -> text = randomString(5);  //Die Zahl bestimmt die Anzahl stellen 

      $_SESSION['captcha_spam'] = $this -> text; 

           

       $this -> img = ImageCreateFromPNG('./captcha-Data/captcha.PNG'); //Backgroundimage 

       $this -> color = ImageColorAllocate($img, 255, 255, 255); //Farbe 

       $this -> ttf = "./captcha-Data/XFILES.TTF"; //Schriftart 

       $this -> ttfsize = 25; //Schriftgrösse 

       $this -> angle = rand(0,5); 

       $this -> t_x = rand(5,30); 

       $this -> t_y = 35; 

       imagettftext($this -> img, $this -> ttfsize, $this -> angle, $this -> t_x, $this -> t_y, $this -> color, $this -> ttf, $this -> text); 

       imagepng($this -> img); 

       imagedestroy($this -> img); 

}

 
?>

Arne Drews · 07.03.2010, 17:28

Es fehlt einfach nur $this!

PHP-Code:

  $this->text = $this->randomString(5);

Chriz · 07.03.2010, 17:33

Rück deinen Code ein, dann siehst dus.

Arne Drews · 07.03.2010, 17:37

Was mich noch irretiert ist die function innerhalb der function...

coolerj · 07.03.2010, 17:48

Immer noch

PHP-Code:

  <?php

session_start(); 
unset($_SESSION['captcha_spam']); 
header('Content-type: image/png');

/**
  * Bitte am Anfang des Verwender Code´s header('Content-type: image/png'); einfügen.
  * Oder bevor der ersten Ausgabe
  */


class createCaptcha {

    private $usec;
    private $sec;
    private $possible;
    private $str;
    private $text;
    private $img;
    private $color;
    private $ttf;
    private $ttfsize;
    private $angle;
    private $t_x;
    private $t_y;
    
       function randomString($len) { 
          function make_seed(){ 
               list($this -> usec , $this -> sec) = explode (' ', microtime()); 
                return (float) $this -> sec + ((float) $this -> usec * 100000); 
           } 
        srand(make_seed());  
           //Der String $possible enthält alle Zeichen, die verwendet werden sollen 
           $this -> possible="ABCDEFGHJKLMNPRSTUVWXYZabcdefghijkmnpqrstuvwxyz23456789"; 
           $this -> str=""; 
         while(strlen($this -> str)<$len) { 
               $this -> str.=substr($this -> possible,(rand()%(strlen($this -> possible))),1); 
           } 
        return($this -> str); 
       } 
       $this -> text = $this -> randomString(5);  //Die Zahl bestimmt die Anzahl stellen 
      $_SESSION['captcha_spam'] = $this -> text; 
           
       $this -> img = ImageCreateFromPNG('./././Data/captcha/captcha.PNG'); //Backgroundimage 
       $this -> color = ImageColorAllocate($img, 255, 255, 255); //Farbe 
       $this -> ttf = "./././Data/captcha/XFILES.TTF"; //Schriftart 
       $this -> ttfsize = 25; //Schriftgrösse 
       $this -> angle = rand(0,5); 
       $this -> t_x = rand(5,30); 
       $this -> t_y = 35; 
       imagettftext($this -> img, $this -> ttfsize, $this -> angle, $this -> t_x, $this -> t_y, $this -> color, $this -> ttf, $this -> text); 
       imagepng($this -> img); 
       imagedestroy($this -> img); 
}

?>

ragtek · 07.03.2010, 18:12

Verwende eine vernünftige IDE / (sinnvolles einrücken hätte es vermutlich auch aufgezeigt) dann ersparst du dir das nervige Posten hier.

Deine Funktion/Methode endet bei:

PHP-Code:

          return($this -> str); 
       } 
       $this -> text = $this -> randomString(5);  //Die Zahl bestimmt die Anzahl stellen

das bedeutet das $this->text nicht mehr in der Methode ist und damit weiß php nicht was es damit anfangen soll.

nikosch · 07.03.2010, 19:58

Was soll nun make_seed() da? Damit hast Du randomString($len) zur Einmal-Methode entwertet!

Chriz · 07.03.2010, 21:00

Rück deinen Code und überleg dir was alles in einen Klassenrumpf gehört. Eigenschaften und Methoden, sicherlich keine Implementierung.

Das sind aber Grundlagen, eingerückt hast du auch noch nichts. Daher
[MOD: Thread geschlossen]

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
Kleine Themplate Class	Deltachaos	PHP-Fortgeschrittene	8	22.01.2010 22:29
Class Projekt verstehen?	reeky	PHP Tipps 2009	23	09.09.2009 20:32
[Erledigt] Datenbank Class	Shuyio88	PHP Tipps 2009	2	02.03.2009 19:47
[Erledigt] syntax Fehler ?	fulltilt	PHP Tipps 2009	2	19.02.2009 16:34
PHP Syntax Error (Zeilenangabe)	marquies	PHP Tipps 2008	7	21.12.2008 13:19
wie müsste die korrekte Syntax sein?	dh1sbg	Datenbanken	2	06.05.2008 21:45
Class Vars	fmueller	PHP Tipps 2006	10	14.02.2006 19:54
Mehrere Div-Layer per Class schliessen	Diego1978	HTML, Usability und Barrierefreiheit	4	02.12.2005 12:10
komm bei der Syntax nicht mehr weiter		PHP Tipps 2005-2	3	03.08.2005 18:09
PHP Class Datei Basirende Datenbank mit MySQL Syntax	atom-dragon	Beitragsarchiv	4	01.08.2005 06:57
Thema: class; Problem: Grundlegende :)		PHP Tipps 2005-2	4	13.07.2005 18:40
Class -- Funktion	Creativ	PHP Tipps 2005	2	30.03.2005 23:10
a mouseover class ändern	022.9	HTML, Usability und Barrierefreiheit	4	18.02.2005 15:51
[Erledigt] Probleme mit Delete Syntax		Datenbanken	6	27.12.2004 12:07

07.03.2010, 17:21
coolerj Erfahrener Benutzer Registriert seit: 11.12.2008 Beiträge: 157	Syntax Error in Class Hallo, Ich habe ein Syntax Error in meiner Klasse verstehe diesen aber kein bisschen. Error: Parse error: syntax error, unexpected T_VARIABLE, expecting T_FUNCTION in C:\xampp\htdocs\encarnium\includes\classes\captcha .php on line 42 Code: PHP-Code: <?php session_start(); unset($_SESSION['captcha_spam']); header('Content-type: image/png'); /** * Bitte am Anfang des Verwender Code´s header('Content-type: image/png'); einfügen. * Oder bevor der ersten Ausgabe / class createCaptcha { private $usec; private $sec; private $possible; private $str; private $text; private $img; private $color; private $ttf; private $ttfsize; private $angle; private $t_x; private $t_y; function randomString($len) { function make_seed(){ list($this -> usec , $this -> sec) = explode (' ', microtime()); return (float) $this -> sec + ((float) $this -> usec 100000); } srand(make_seed()); //Der String $possible enthält alle Zeichen, die verwendet werden sollen $this -> possible="ABCDEFGHJKLMNPRSTUVWXYZabcdefghijkmnpqrstuvwxyz23456789"; $this -> str=""; while(strlen($this -> str)<$len) { $this -> str.=substr($this -> possible,(rand()%(strlen($this -> possible))),1); } return($this -> str); } $this -> text = randomString(5); //Die Zahl bestimmt die Anzahl stellen $_SESSION['captcha_spam'] = $this -> text; $this -> img = ImageCreateFromPNG('./captcha-Data/captcha.PNG'); //Backgroundimage $this -> color = ImageColorAllocate($img, 255, 255, 255); //Farbe $this -> ttf = "./captcha-Data/XFILES.TTF"; //Schriftart $this -> ttfsize = 25; //Schriftgrösse $this -> angle = rand(0,5); $this -> t_x = rand(5,30); $this -> t_y = 35; imagettftext($this -> img, $this -> ttfsize, $this -> angle, $this -> t_x, $this -> t_y, $this -> color, $this -> ttf, $this -> text); imagepng($this -> img); imagedestroy($this -> img); } ?> __________________ Bettlergame is coming ...


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

07.03.2010, 17:28
Arne Drews Erfahrener Benutzer Registriert seit: 22.04.2009 Beiträge: 3.341 PHP-Kenntnisse: Anfänger	Es fehlt einfach nur $this! PHP-Code: `$this->text = $this->randomString(5);`

07.03.2010, 17:33
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.069	Rück deinen Code ein, dann siehst dus. __________________ "Nuschel ich?" - "Was?"

07.03.2010, 17:37
Arne Drews Erfahrener Benutzer Registriert seit: 22.04.2009 Beiträge: 3.341 PHP-Kenntnisse: Anfänger	Was mich noch irretiert ist die function innerhalb der function...

07.03.2010, 17:48
coolerj Erfahrener Benutzer Registriert seit: 11.12.2008 Beiträge: 157	Immer noch PHP-Code: <?php session_start(); unset($_SESSION['captcha_spam']); header('Content-type: image/png'); /** * Bitte am Anfang des Verwender Code´s header('Content-type: image/png'); einfügen. * Oder bevor der ersten Ausgabe / class createCaptcha { private $usec; private $sec; private $possible; private $str; private $text; private $img; private $color; private $ttf; private $ttfsize; private $angle; private $t_x; private $t_y; function randomString($len) { function make_seed(){ list($this -> usec , $this -> sec) = explode (' ', microtime()); return (float) $this -> sec + ((float) $this -> usec 100000); } srand(make_seed()); //Der String $possible enthält alle Zeichen, die verwendet werden sollen $this -> possible="ABCDEFGHJKLMNPRSTUVWXYZabcdefghijkmnpqrstuvwxyz23456789"; $this -> str=""; while(strlen($this -> str)<$len) { $this -> str.=substr($this -> possible,(rand()%(strlen($this -> possible))),1); } return($this -> str); } $this -> text = $this -> randomString(5); //Die Zahl bestimmt die Anzahl stellen $_SESSION['captcha_spam'] = $this -> text; $this -> img = ImageCreateFromPNG('./././Data/captcha/captcha.PNG'); //Backgroundimage $this -> color = ImageColorAllocate($img, 255, 255, 255); //Farbe $this -> ttf = "./././Data/captcha/XFILES.TTF"; //Schriftart $this -> ttfsize = 25; //Schriftgrösse $this -> angle = rand(0,5); $this -> t_x = rand(5,30); $this -> t_y = 35; imagettftext($this -> img, $this -> ttfsize, $this -> angle, $this -> t_x, $this -> t_y, $this -> color, $this -> ttf, $this -> text); imagepng($this -> img); imagedestroy($this -> img); } ?> __________________ Bettlergame is coming ... Geändert von coolerj (07.03.2010 um 17:51 Uhr).

07.03.2010, 18:12
ragtek Erfahrener Benutzer Registriert seit: 10.02.2009 Beiträge: 920	Verwende eine vernünftige IDE / (sinnvolles einrücken hätte es vermutlich auch aufgezeigt) dann ersparst du dir das nervige Posten hier. Deine Funktion/Methode endet bei: PHP-Code: `return($this -> str); } $this -> text = $this -> randomString(5); //Die Zahl bestimmt die Anzahl stellen` das bedeutet das $this->text nicht mehr in der Methode ist und damit weiß php nicht was es damit anfangen soll.

07.03.2010, 19:58
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 34.241 PHP-Kenntnisse: Fortgeschritten	Was soll nun make_seed() da? Damit hast Du randomString($len) zur Einmal-Methode entwertet! __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

07.03.2010, 21:00
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.069	Rück deinen Code und überleg dir was alles in einen Klassenrumpf gehört. Eigenschaften und Methoden, sicherlich keine Implementierung. Das sind aber Grundlagen, eingerückt hast du auch noch nichts. Daher [MOD: Thread geschlossen] __________________ "Nuschel ich?" - "Was?"