Czym zastąpić "eval" ? Opcje

[MESSIAH :)]

Czytałem w manualu aby nie stosować tej funkcji. Więc jak powinien wyglądać poprawnie funkcja renderowania html?

[PHP] pobierz, plaintext 
if (file_exists($path)) {
            $content = file_get_contents($path);
 
            eval(' ?>' . $content . '<?php ');
        }
[PHP] pobierz, plaintext 

Co takiego złego jest w tym eval?

Ten post edytował MESSIAH :) 4.08.2015, 20:43:20

[nospor]

A po grzyba ci tu eval? O include nie slyszal?

[Comandeer]

Wszystko… Już lepiej po prostu

Kod

echo file_get_contents($path);

Chyba że w środku jest PHP, więc można pokombinować z buforem i include (albo po prostu zaincludować).

[MESSIAH :)]

Słyszałem o include i stosuje to. Mam mały system renderowania html w którym używam preg_replace i np mam kod html:

[HTML] pobierz, plaintext 
Witaj [user] ostatni byłeś [online] masz [messages] nieprzeczytanych.
[HTML] pobierz, plaintext 

Dlatego użyłem eval

[PHP] pobierz, plaintext 
eval(' ?>' . $content . '<?php ');
[PHP] pobierz, plaintext 

aby oddzielić php od html.

[nospor]

No i po co ci tu eval? Przeciez to zwykly tekst.... eval nie ma tu nic do roboty.

[MESSIAH :)]

eval — Evaluate a string as PHP code

Poprawny kod wygląda następująco:

[PHP] pobierz, plaintext 
    if (file_exists($path)) {
                $content = file_get_contents($path);
                foreach (this->vars as $key => $value) {
                      $content = preg_replace('/\[' . $key . '\]/', $value, $content)
 
                eval(' ?>' . $content . '<?php ');
            }
[PHP] pobierz, plaintext 

[nospor]

No i dobrze... nadal sie pytam po grzyba ci tu eval?? Czemu preg_replacem nie zamieniasz od razu na wartosc jaka ma byc? Bo teraz rozumiem, ze zamieniasz to na zmienna a potem evalem parsujesz... bez sensu totalnie.

ps: i nie cytuj mi tu prosze definicji. Ja wiem co robi eval. To ty bez sensu kombinujesz a nie ja (IMG:style_emoticons/default/tongue.gif)

[MESSIAH :)]

Bez eval wyświetla mi się pusta strona. Więc jest on potrzebny.

[Comandeer]

Może dlatego, że nie ma nigdzie echo?

[nospor]

@Comandeer o echo juz pisales w pierwszym poscie. Chyba nie podejrzewasz tegoż tu autora o nieczytanie tego co się do niego pisze (IMG:style_emoticons/default/wink.gif)

Bez eval wyświetla mi się pusta strona. Więc jest on potrzebny.

No tak, nie ma to jak wyciągnac logiczny wniosek (IMG:style_emoticons/default/wink.gif)
Nie dam rady poleciec do sklepu oddalonego o 200 metrow - musze kupic helikopter.... Ale zeby pomyslec zeby uzyc nóg, to już ciezko na to wpaść.
Tak jak juz wspominano wczesniej: by sie cos wyswietlilo na ekranie, to trzeba uzyc echo. Wiesz, takie tam podstawy, ktore powinno sie znac od samego początku.

[Comandeer]

@nospor im dłuższy staż na forach, tym bardziej podejrzliwym człowiekiem się staję (IMG:style_emoticons/default/wink.gif)

[Tuminure]

Nie wiem jaki efekt chcesz uzyskać ale z pewnością go uzyskasz bez używania eval (IMG:style_emoticons/default/wink.gif) .

[nospor]

@Comandeer to sobie pomysl jaką ja mam fazę (IMG:style_emoticons/default/biggrin.gif)

[Comandeer]

@nospor jestem adminem na (konkurencyjnym?) forum, więc też już mam odpowiednią…

Co do tematu: nie łatwiej już wziąć jakiś super mega prosty system szablonów, skoro i tak ten kod go ma przypominać?

[MESSIAH :)]

Więc sprawa wygląda następująco. Nie mogę używać echo lub tym podobne gdyż moje if, else i endif będą miały postać komentarza html czyli:

[HTML] pobierz, plaintext 
<!-- if 1 == 1 -->
[HTML] pobierz, plaintext 

I tutaj będę podmieniał tylko wartość dla if. Więc eval jest tutaj jak najbardziej na miejscu. Echo wyświetla wszystko i psuje output preg_replace.

[Comandeer]

Jeśli masz tam if to najlepszy znak tego, że potrzebujesz systemu szablonów, a nie rzeźbienia czegoś takiego na szybko.

Najprostszy system szablonów można uzyskać używając PHP (tzw. PHP Views), np. przy pomocy output bufferingu + include jak wspominałem.

[PHP] pobierz, plaintext 
<?php
function generateView($file, array $vars = [])
{
	ob_start();
 
	extract($vars);
 
	include $file;
 
	return ob_get_clean();
}
 
echo generateView('view.php', [
	'a' => 1
]);
[PHP] pobierz, plaintext 

I plik view.php:

[PHP] pobierz, plaintext 
<?php if($a > 0): ?>
	<p>Super! $a wynosi aż <?=$a;?>.</p>
<?php endif; ?>
[PHP] pobierz, plaintext 

Zero eval.

Oczywiście PHP Views są najmniej przyjemne w pracy (ale za to masz dostęp do całego PHP, co w zależności od punktu widzenia jest plusem lub minusem) i warto zaznajomić się choćby z Twigiem czy jakąś implementacją wąsów dla PHP

[MESSIAH :)]

W moim pliku z View używam tylko html - php odpada. Chcę mieć w nim czysty html

[HTML] pobierz, plaintext 
Witaj [użytkownik] <br>
 
<!-- Jeśli 1 == 1 -->
Jest równe
<!-- Inaczej -->
Nie jest równe
<!-- KoniecJeśli -->
[HTML] pobierz, plaintext 

Moim celem jest przetworzenie

[HTML] pobierz, plaintext 
<!-- Jeśli 1 == 1 -->
[HTML] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
<?php if ($1) : ?>
[PHP] pobierz, plaintext 

i ponownie na html. Czy bez eval się nie obędzie?

[nospor]

W moim pliku z View używam tylko html - php odpada. Chcę mieć w nim czysty html

Jest ku temu jakiś konkretny powód czy zwykły kaprys?

[MESSIAH :)]

Chcę po prostu sprawdzić zachowanie takiego renderowania. Muszę przetestować kilka opcji i wybrać tą najlepszą dla mnie.

[nospor]

Aaaaa, to ty..... ten co nie slucha innych tylko robi po swojemu... juz sobie przypomnialem (IMG:style_emoticons/default/smile.gif)
Dobra, powodzenia (IMG:style_emoticons/default/smile.gif)