PHP eval unter PHP 7

Bigberle

Hallo
nach dem update vom Server auf PHP 7 geht mein Programm nicht mehr.

PHP

$output = "";
        foreach($this->gamedata as $category){
            $games = "";
            $gamedata = "";
            if(count($category['games']) > 0){
                $count = 0;
                foreach($category['games'] as $game){
                    if(($count % 2) == 1 || count($category['games']) == 1){

                        eval("\$games .= "".$this->template['game']."";");
                        eval("\$gamedata .= "".$this->template['gdoublewrapper']."";");
                        $games = "";

                    } else {
                        eval("\$games .= "".$this->template['game']."";");
                    }
                    $count++;


                    if(count($category['games']) != 1 && (count($category['games']) % 2) == 1 && ($count == count($category['games']))){
                        eval("\$gamedata .= "".$this->template['gdoublewrapper']."";");
                    }
                }

                eval("\$output .= "".$this->template['category']."";");
            }
        }

Alles anzeigen

finde nicht was falsch ist.
Bitte um Hilfe!

Arne Drews

Wenn ich eval() sehe, kräuseln mir ja schon die Nackenhaare. Das solltest Du anders lösen.

Was Dein Problem angeht, würde ich mal tippen, dass Du das

PHP

eval("\$games .= "".$this->template['game']."";");

in etwa so abändern solltest

PHP

eval("\$games .= \"" . $this->template['game'] . "\";");

Das sollte dann für alle eval()s gemacht werden.

basti1012

Über eval() sagt man das

Zitat

[FONT=&quot]Achtung[/FONT][FONT=&quot]Das eval()-Sprachkonstrukt ist sehr gefährlich, weil es die Ausführung von beliebigem PHP-Code erlaubt. Seine Verwendung wird daher nicht empfohlen. Wenn sorgfältig überprüft wurde, dass es keine andere Möglichkeit gibt als dieses Konstrukt zu verwenden, ist besonders darauf zu achten keine von Nutzern bereit gestellten Daten zu übergeben ohne diese zuvor ordnungsgemäß zu validieren.[/FONT]

Aber das nur nebenbei.
Was sagt den überhaupt deine Error report

Bigberle

Keine Fehlermeldung die Seite bleibt weiß.
Wie würdes du das ändern oder lösen ?

Arne Drews

Keine Fehlermeldung bei aktivierte ErrorReport?

PHP

error_reporting ( -1 );
ini_set ( 'display_errros', true );

Bigberle

Leider kein Fehler
unter PHP 5 war alles gut jetzt hat Server PHP 7
und wenn ich eval jetzt ändere kommt Fehler :
eval("\$games .= "".$this->template['game']."";");
kommt der Fehler
Parse error: syntax error, unexpected '"'

basti1012

Das liegt woh an dein "" .Kuck doch mal in post 2# wie @Arne Drews das da gemacht hat .So ist bei mir der Fehler nicht mehr da

Bigberle

habe es so geändert
$output = "";
foreach($this->gamedata as $category){
$games = "";
$gamedata = "";
if(count($category['games']) > 0){
$count = 0;
foreach($category['games'] as $game){
if(($count % 2) == 1 count($category['games']) == 1){
eval("\$games .= "".$this->template['game']."";");
eval("\$gamedata .= "".$this->template['gdoublewrapper']."";");
$games = "";
} else {
eval("\$games .= "".$this->template['game']."";");
}
$count++;

if(count($category['games']) != 1 && (count($category['games']) % 2) == 1 && ($count == count($category['games']))){
eval("\$gamedata .= "".$this->template['gdoublewrapper']."";");
}
}
eval("\$output .= "".$this->template['category']."";");
}
}
return $output;
}
kopiert den flash nicht mit ist aber alles so geändert.
eval("\$games .=\ "" . $this->template['game'] . "";");
kein Fehler aber immer noch weißer Bildschirm

basti1012

Ich weiß gerade nicht was ich sagen soll.

Zitat von basti1012

Das liegt woh an dein "" .Kuck doch mal in post 2# wie @Arne Drews das da gemacht hat .So ist bei mir der Fehler nicht mehr da

Den Fehler hat du jetzt in 5 Zeilen ( Hat mir sogar der ErrorReport angezegt nach und nach )
Wenn die Fehler weg sind kommt noch ein Fehler das eine Klammer zu viel ist.
Vieleicht hat sich die Klammer beim reinkopieren hier rein geschlichen oder so. Auf jedenfall ist die bei deinen geposteten Code jetzt zu viel.

Wieso kopiert er den \ nicht mit ? Ist doch sonst überall zu sehen.
Bist du sicher das dein ErrorReport aktiviert ist ? Kommt mir irgendwie nicht so vor

Bigberle

Ja denke das der ErrorReport aktiv ist
error_reporting ( E_ALL );
der \ ist weg sobald ich auf antworten klicke.

Arne Drews

Zitat von Bigberle

Ja denke das der ErrorReport aktiv ist
error_reporting ( E_ALL );

Nimm mal -1 und display_errors setzen, siehe #5

Zitat von Bigberle

der \ ist weg sobald ich auf antworten klicke.

Dann verwende die PHP-Tags hier im Editor, dann verschwinden die nicht.

Bigberle

danke, hab ich jetzt hinbekommen

PHP

function makeGamesList(){
        if(!$this->isLoaded()){
            $this->Load();

        }
        $output = "";
        foreach($this->gamedata as $category){
            $games = "";
            $gamedata = "";
            if(count($category['games']) > 0){
                $count = 0;
                foreach($category['games'] as $game){
                    if(($count % 2) == 1 || count($category['games']) == 1){

                        eval("\$games .= \"".$this->template['game']."\";");
                        eval("\$gamedata .= \"".$this->template['gdoublewrapper']."\";");
                        $games = "";


                    } else {
                        eval("\$games .= \"".$this->template['game']."\";");
                    }
                    $count++;


                    if(count($category['games']) != 1 && (count($category['games']) % 2) == 1 && ($count == count($category['games']))){
                        eval("\$gamedata .= \"".$this->template['gdoublewrapper']."\";");
                    }
                }

                eval("\$output .= \"".$this->template['category']."\";");
            }
        }

        return $output;
    }

Alles anzeigen

Bigberle

das mit eval hab ich hinbekommen, danke
was ändert sich bei $var in PHP 7 das geht leider auch nicht

PHP

class Spiele {





    var $gamedata;
    var $imgdir;
    var $swfdir;
    var $loaded;
    var $template;




    function GamesSystem($admin_mode = 0){
        if($admin_mode != 1){
            global $templates;
            $this->template = $templates;
        }
        $this->gamedata = array();
        $this->imgdir = './images';
        $this->loaded = false;
        $this->swfdir = './swf';
        $this->am = $admin_mode;
    }

Alles anzeigen

Arne Drews

Wenn Du mit Klassen arbeitest, mache es bitte gleich richtig. Die Syntax oben bei Dir stammt aus PHP4 Zeiten, das ist seit August 2008, also bereits über 10 Jahre(!) tot.
Als Beispiel für Dein Snippet oben:

PHP

class Spiele {




    public $gamedata;
    public $imgdir;
    public $swfdir;
    public $loaded;
    public $template;




    public function __construct( $template, $admin_mode = 0 ) {


        if( $admin_mode != 1 )
            $this->template = $template;


        $this->gamedata = [];
        $this->imgdir = './images';
        $this->swfdir = './swf';
        $this->loaded = false;
        $this->am = $admin_mode;


    }




}

Alles anzeigen

Auch global sollte man nicht verwenden, übergebe die Daten, die Du benötigst als Parameter an die Methoden, in diesem Fall den Konstruktor.

Aufruf für obiges Beispiel:

PHP

// AdminMode:
$Spiele = new Spiele( $templates, 1 );


// default mode:
$Spiele = new Spiele( $templates );

Bandit

Meinst du nicht eher

PHP

public function __construct( $template, $admin_mode = 0 )

Arne Drews

Jo, stimmt, verdreht... korrigiert.

Bigberle

Danke Arne und Bandit
ich mache das nicht so oft, nur wenn ich etwas bestimmte brauche suche ich mir das zusammen was die möchten.
Mache das Ehrenamtliche für die Homepage der Freiwilligen Feuerwehr.

Bigberle

Habe leider jetzt den Fehler :
Warning: Missing argument 1 for Spiele::__construct()

Fehler behoben !
Alles gut vielen, vielen DANK

Tags

Benutzer online in diesem Thema