Forum PHP.pl

Witam

Zastanawiam się, czemu wyświetlają mi się 2 razy rekordy. Chcę też stworzyć system templatek wg tego linku:
http://www.programuj.com/artykuly/www/template.php
Więc co mam:

[PHP] pobierz, plaintext 
class Template {
    var $tmpl;
    var $dane;
 
    function Template ($name)
    {
        $this->tmpl = implode('', file($name));
        $this->dane = Array();
    }
 
    function add($name, $value = '')
    {
        if (is_array($name)) {
            $this->dane = array_merge($this->dane, $name);
        } else if (!empty($value)) {
            $this->dane[$name] = $value;
        }
    }
 
    function execute() {
        return preg_replace('/{([^}]+)}/e', '$this->dane["\\1"]',
                $this->tmpl);
    }
 
}
class Bazowa
{
    public function PobierzDane()
    {
        $db = new mysqli("localhost", "root", "haselko", "test");
        if($db->connect_errno){
            die('Błąd połączenia: ' . $db->connect_errno);
            exit;
        }
        $sql = "select * from tabela1";
        $zapytanie = $db->query($sql);
        while($wynik=$zapytanie->fetch_assoc())
        {
            $dane[]= $wynik;
        }
        $db->close();
        return $dane;
    }
    function Wyswietl($tablica)
    {
        foreach($tablica as $wyn)
        { 
            foreach($wyn as $pokaz => $wart)
            {
                               echo $pokaz." = ".$wart."<br />";
            }
 
        }
 
        echo "</table>";
 
    }
    function Dodaj($id,$nazwa)
    {
        $db = new mysqli("localhost", "root", "qweasd", "test");
        $sql = "insert into `tabela1` values ($id , '$nazwa')";
        $db->query($sql);
        $db->close();
        echo "Dodano nowy rekord<br>";
        $dane = self::PobierzDane();
        self::Wyswietl($dane);
    }
    function __destruct()
    {
        echo "Obiekt został zniszczony!";
    }
}
$wiz = new Bazowa();
$dane = $wiz->PobierzDane();
print_r($dane);
echo "<br><br>";
$wiz->Wyswietl($dane);
[PHP] pobierz, plaintext 

Baza wygląda tak, że mam 2 tabele: ID i nazwa i tam są przykładowe dane (6 wierszy). Jeśli odpalę powyższy kod to jest ok - raz się wyświetla. Jednak jeśli przerobię funkcję wyświetlającą na taką:

[PHP] pobierz, plaintext 
    function Wyswietl($tablica)
    {
        echo "<table border=\"1\">";
        foreach($tablica as $wyn)
        { 
            foreach($wyn as $pokaz => $wart)
            {
               $tmpl = new Template('temp.tmp.php');
                $tmpl->add($wyn);
                echo $tmpl->execute();
            }
 
        }
[PHP] pobierz, plaintext 

To otrzymuję podwójne wyniki tzn ID 1 1 2 2 3 3 4 4.
Jeśli chodzi o plik z templatką to:

Gdzie tkwi błąd?
Jeśli dopiszę do tej 2 funkcji wyświetlającej print_r($wyn); nad $tmpl->add($wyn); to dostaję:
Array ( [ID] => 1 [nazwa] => php ) Array ( [ID] => 1 [nazwa] => php ) Array ( [ID] => 9 [nazwa] => c++ ) Array ( [ID] => 9 [nazwa] => c++ ) Array ( [ID] => 7 [nazwa] => kurs ) Array ( [ID] => 7 [nazwa] => kurs ) Array ( [ID] => 4 [nazwa] => mysql ) Array ( [ID] => 4 [nazwa] => mysql ) Array ( [ID] => 5 [nazwa] => oracle ) Array ( [ID] => 5 [nazwa] => oracle ) Array ( [ID] => 6 [nazwa] => js ) Array ( [ID] => 6 [nazwa] => js ) Array ( [ID] => 10 [nazwa] => C# ) Array ( [ID] => 10 [nazwa] => C# )
Czyli widać, że pobiera za dużo razy, ale dlaczego?

A może jest jakiś inny sposób na pobranie danych z bazy, przypisanie do jakiejś zmiennej, aby potem przekazać i wykorzystać w templatce?

Ten post edytował IceManSpy 20.08.2010, 21:31:29

Klasa "bazowa" jak rozumiem to Twoja inwencja? Nie ma sensu pisanie metod, które:
1. Zawierają kod html (html to powiedzmy szablon/widok, a szablon to dane a nie algorytm aplikacji)
2. Wyświetlają coś na ekranie (nawet obsługa błędów wysyła je do odpowiednich szablonów a nie echuje na ekranie)
3. Łącza wszystko w jeden kociokwik: łączą się z bazą, wykonują zapytania, przerywają działanie skryptu (utniesz stronę w połowie, to bez sensu przecież), echują coś na ekran (a gdy potem błąd to co? Ma browser zwrócić to co mu wysłałeś?) czy obsługują błędy, takich rzeczy nie robi się nawet pisząc kod strukturalny (oczywiście w miarę porządny kod strukturalny)

Poza tym przemyśl, co próbujesz zrobić (no i masz błąd logiczny w tej pętli) bo odnoszę wrażenie, że kompletnie nie rozumiesz jak to ma działać... właśnie po to tworzymy system szablonów, by nie musieć w pętli xxxxx razy tworzyć obiektu $tmpl, dodawać do niego po jednej zmiennej i o zgrozo - po każdej takiej akcji wykonywać metodę execute. Tak to ma działać:

1. Tworzysz obiekt $tmpl - RAZ!
2. Dodajesz do niego TABLICĘ zmiennych w układzie nazwa zmiennej => kod html - RAZ! Jeśli potrzebujesz dodać zmienną/zmienne to dodajesz je do tej tablicy (zalecane) lub używasz metody $tmpl->add (mniej wydajne, lepiej użyć metody add tylko raz), nie musisz ponownie tworzyć obiektu $tmpl
3. Łączysz zmienne z szablonem wykonując metodę execute - RAZ!

Jeśli jeden szablon nie wystarczy, możesz użyć ich wiele, np.:

[PHP] pobierz, plaintext 
$glowna['title'] = 'Tytul strony pobierany z bazy etc';
$glowna['lewa'] = $tmpl_lewa->execute($dane_lewej_manki);
$glowna['prawa'] = $tmpl_prawa->execute($dane_prawej_manki);
echo $tmpl_glowny->execute($glowna);
[PHP] pobierz, plaintext 

- pominąłem tworzenie obiektów tmpl, tak tworzysz szablon, który składa się z pod-szablonów, w tym wypadku szablon strony + dwa szablony na każdą kolumnę, w ten sposób budujemy drzewo szablonów.

I naucz się, że najpierw się przygotowuje wszystkie zmienne, potem łączy je z html a dopiero na samym końcu wysyła to do przeglądarki, taka jest elementarna logika tworzenia stron w PHP niezależnie od tego, czy zamykasz ten kod w klasy czy nie.

Czyli np pobieramy dane z bazy, mamy jakiś szablonik i wtedy podmieniamy te dane i wyświetlamy? (tak odbiegając od tematu właściwego, ale też ważny poruszyłeś (IMG:style_emoticons/default/smile.gif) ) W sumie czytałem o Smarty, ale podobno jeśli są jakieś proste strony to nie warto używać Smarty, bo biblioteka zajmuje ok 300 KB.