brauche schnell hilfe

[fringis]

Bei folgenden Code:

eval("$errorClass[plz] = 'inputField';
            $errorClass[ort] = 'inputField';
            $errorClass[mail] = 'inputField';"); 

erscheint folgende Fehlermeldung:


Parse error: syntax error, unexpected '=' in meinem Script(38) : eval()'d code on line 1

Könnt ihr mir bitte schnell helfen, ist dringend

Gruß
fringis

[SquallCW]

eval("\$errorClass[plz] = 'inputField';
\$errorClass[ort] = 'inputField';
\$errorClass[mail] = 'inputField';"); 

Damit sollte es funktionieren.

[PHP-Desaster]

Warum machst du so kriminellen Krams mit eval??

[fringis]

Warum kriminell ?

Ich werde den Code ma kurz testen.
Danke

edit:// Hat funktioniert, danke.
Aber warum die Schrägstriche ?

Mfg
Fringis

[PHP-Desaster]

eval ist evil und für eine einfache Zuweisung totaler Blödsinn!

Aber warum die Schrägstriche ?

Schaust du hier

[SquallCW]

Weil er es sonst sofort mit dem Inhalt tauschen würde. Für den Parser würde

<?php
$str = "Meine Variable";
eval("$str = \"Meine Variable verändert\";");
echo $str;
?>

so aussehen:

<?php
$str = "Meine Variable";
eval("Meine Variable = \"Meine Variable verändert\";");
echo $str;
?>

Es verhindert einfach, dass die Variable ersetzt wird beim Parsen.

[tontechniker]

Was soll der Unsinn? Warum nicht einfach so:

$errorClass [ 'plz' ] = 'inputField';
$errorClass [ 'ort' ] = 'inputField';
$errorClass [ 'mail' ] = 'inputField'; 

[fringis]

Hallo,
erstmal tausend dank für eure Antworten.

Es ist mir auch klar, wie man Anweisungen sonst schreibt, aber da ich eine Klasse erstellen will, in der Man über einen Parameter halt zusätzlichen Code einfügen kann, brauch ich das.

function($code=null)
{
echo "Hallo";
eval($code);

}

Ist jetzt nur ein Beispiel dafür.

Gruß

fringis

[SquallCW]

Ich denke mal, das Verfahren ist sehr Fehleranfällig oder auch sehr unsicher. Ich bin mir aber gerade nicht genau, wie man es anders lösen könnte, da ich ja auch nicht weiß, was für Daten übertragen werden sollen.

[tontechniker]

Es ist mir auch klar, wie man Anweisungen sonst schreibt, aber da ich eine Klasse erstellen will, in der Man über einen Parameter halt zusätzlichen Code einfügen kann, brauch ich das.

Das ist bestimmt Unsinn.

[PHP-Desaster]

Wenn du eigenen Code in einer Klasse ausführen willst, entwirfst du dein System vernünftig, leitest von der Klasse ab und überschreibst die entsprechende Methode.

[fringis]

Das Problem ist einfach, dass die Klasse zum Fehlermarkieren von Formularen ist.
Es geht alle übergebenen Felder durch und wenn sie leer sind, wird halt die Klasse des Feldes geändert.

Das Problem ist aber, wenn man nun z.B. überprüfen will, ob zwei Felder den gleichen Wert haben, muss man extra Code in das Objekt einführen und das soll halt bei jeder instanzierung eines objektes unterschiedlich sein.

Wenn ihr eine andere Lösung wisst, ich würde mich über alle Vorschläge freuen :-)

Gruß fringis

[SquallCW]

Ich verstehe dein Anliegen nicht wirklich.
Du kannst doch einfach deine Variablen überprüfen und gut ist?
Oder du erstellst dir eine "Fehler-Variable" ->

<?php
$fehler = "";
if(!empty($_POST["Mein Feld"])) $fehler .= "Mein Feld wurde nicht ausgefüllt";
// [...]
?>

Und am Ende überprüfst du dann, ob $fehler leer ist; falls nicht, Scriptfehlermeldung ausgeben. Und selbst wenn du die Variablen ändern wolltest, bräuchtest du immer noch kein eval().

[onemorenerd]

Original geschrieben von fringis
Das Problem ist einfach, dass die Klasse zum Fehlermarkieren von Formularen ist.
Es geht alle übergebenen Felder durch und wenn sie leer sind, wird halt die Klasse des Feldes geändert.

Das Problem ist aber, wenn man nun z.B. überprüfen will, ob zwei Felder den gleichen Wert haben, muss man extra Code in das Objekt einführen und das soll halt bei jeder instanzierung eines objektes unterschiedlich sein.

Darf man den Code dazu mal sehen?

[fringis]

class CheckForm
{

    // TRUE, wenn Fehler aufgetreten ist
    public $error = FALSE;
    public $los = FALSE;
    public $method = "POST";
    // Alle Pflichtfelder
    public $errorFields = null;
    public $additionCode = null;
    

    public function __construct($errorFields,$additionCode=null, $method="POST")
    {

        //Error-Fields in das Klassen-Funktion übernehmen
        
        $this->errorFields=$errorFields;
        
        $this->method=$method;
        $this->additionCode =$additionCode;
        $this->smarty = $GLOBALS['smarty']; 
        for($x=0;$x<count($errorFields);$x++)
            {
                $errorClass[$errorFields[$x]] = 'inputField';
                
            }
        if(additionCode!==null)
        {
            
            eval($additionCode);
            
        }
        $this->smarty->assign('errorClass', $errorClass);
        
    }
    
    public function checkIt($additionCode=null)
    {
        
        
        $errormessage="<span style='color:#990000;'>";                                             
        foreach($this->errorFields as $key => $value)
                {
                            
                    if(empty($_POST[$this->errorFields[$key]]))
                    {
                        
                        $this->error = TRUE;
                        $errorPicture[$value]='<img src="/images/achtung_klein.gif">&nbsp;';
                        $errorClass[$value]='errorField';
                    
                        
                        
                    }
                }
//Beispiel für möglichen zusätzlichen Code zum Prüfen, ob zwei Felder die gleichen Angaben besitzen

        if($_POST['password']!==$_POST['password2'] && !empty($_POST['password']))
                {
                    $this->error = TRUE;
                        $errorClass['passowrd'] = 'errorField';
                        $errorPicture['password'] = '<img src="images/achtung_klein.gif">&nbsp;';
                        $errorClass['password2'] = 'errorField';
                        $errorPicture['password2'] = '<img src="images/achtung_klein.gif">&nbsp;';
                    $errormessage.='<p> - Passw&ouml;rter stimmen nicht &uuml;berein</p>';
                }
        
        
        if($this->additionCode!==null)
        {
            
            eval($additionCode);
            
        }
        
        $errormessage.="</span>";
        $this->smarty->assign('errorPicture', $errorPicture);
        $this->smarty->assign('errorClass', $errorClass);
        $this->smarty->assign('errorMessage', $errormessage);
            
        return $this->error;
    }
}