[MSSQL][PHP][SMARTY]Wyświetlenie danych z BD Opcje

[DerekDX]

Witam, zaczełem przerabiać swoją stronę na szablony Smart. I pojawił się pierwszy problem. Otóż przy wykonaniu kodu pojawia się błąd o treści:
Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 36 bytes) in E:\Dropbox\xampp-portable\htdocs\xampp\PJWSTK\users.php on line 16

Jako, że dopiero zaczynam naukę ze Smarty PHP kożystam z manulala zamieszczonego tu

Poniżej zamieszczam funkcję z której korzystam w skrypcie, szablon .tpl oraz skrypt PHP.

[PHP] pobierz, plaintext 
function db_czytaj($zapytanie){
    require('mysql_config.php');
    $mysql=mysqli_connect($mysql_server,$mysql_user,$mysql_password) or die('nie polaczylo z baza danych');
   	$wybrana=mysqli_select_db($mysql,$db_name) or die('nie wybrano bazy danych');
    $wynik=mysqli_query($mysql,$zapytanie) or die('Nie poprawne zapytanie');
    $wynik2=mysqli_fetch_assoc($wynik);
    return $wynik2;
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
    require('include/include.php');
    require('header.php');
    require('menu.php');
 
    $tpl = new Smarty;
    $tpl -> template_dir = 'smarty/templates/';
    $tpl -> compile_dir = 'smarty/templates_c/';
 
    $zapytanie="select `user_id`,`login`,`name`,`surname`,`email`,`telephone` FROM `users`;";
    $wynik2 = array();
    $wynik=db_czytaj($zapytanie);
 
 
    while($wynik){
        $wynik2[]=array(
            'user_id'=>$wynik['user_id'],
            'login'=>$wynik['login'],
            'name'=>$wynik['name'],
            'surname'=>$wynik['surname'],
            'email'=>$wynik['telephone'],
        );
    }
 
 
    $tpl -> assign('wynik', $wynik2);
    $tpl -> display('users.tpl');
 
?>
 
[PHP] pobierz, plaintext 

[XML] pobierz, plaintext 
<fieldset>
    <legend><p>Użytkownicy</p></legend>
    <a href="add_user.php"><button type="button"><p>Dodaj użytkownika</p></button></a>
<table>
        {section name=i loop=$wynik}
        <tr>
            <td>
                {$wynik[i].login}
            </td>
            <td>
                {$wynik[i].name}
            </td>
            <td>
                {$wynik[i].surname}
            </td>
            <td>
                {$wynik[i.email]}
            </td>
            <td>
                {$wynik[i].telephone}
            </td>
        </tr>
        {/section}
    </table>
</fieldset>
[XML] pobierz, plaintext 

[nospor]

while($wynik){
Przecież ty się tutaj zapętlasz i nigdy nie kończysz..... Problem ze smarty nie ma żadnego związku. To są podstawy php
Warunek w WHILE musi się zmieniać, a u ciebie ciągle jest taki sam i prawdziwy

Proszę, tu masz napisane jak pobierać dane z bazy przy pomocy WHILE
http://pl1.php.net/manual/pl/mysqli-result.fetch-assoc.php
Skorzytaj z tego bo jeszcze nie jesteś na etapie by kombinować.

[DerekDX]

wyrażeniem w pętli while jest jest wynikiem funkcji db_czytaj() która zwraca wynik funkcji mysqli_fetch_assoc() tak jak w manualu podanym przez Ciebie

[PHP] pobierz, plaintext 
$query = "SELECT Name, CountryCode FROM City ORDER by ID DESC LIMIT 50,5";
 
if ($result = mysqli_query($link, $query)) {
 
    /* fetch associative array */
    while ($row = mysqli_fetch_assoc($result)) {
        printf ("%s (%s)\n", $row["Name"], $row["CountryCode"]);
    }
 
    /* free result set */
    mysqli_free_result($result);
}
[PHP] pobierz, plaintext 

[nospor]

A ty nadal nie rozumiesz o czym mówie....

U ciebie $wynik w pętli się nie zmienia. Przec co generujesz niekończącą się pętle.
Zaś w manuala warunek w WHILE się zmienia, gdyż masz tam na bięząco wynik mysql_fetch_assoc() który jest zmienny a na końcu jest FALSE i konczy WHILE.

[DerekDX]

Dobrze myśle ze oczyszciłem kod php z błedów:

Funkcja realizujaca połączenie z baza danych:

[PHP] pobierz, plaintext 
function db_czytaj($zapytanie){
    require('mysql_config.php');
    $mysql=mysqli_connect($mysql_server,$mysql_user,$mysql_password) or die('nie polaczylo z baza danych');
    $wybrana=mysqli_select_db($mysql,$db_name) or die('nie wybrano bazy danych');
    $wynik=mysqli_query($mysql,$zapytanie) or die('Nie poprawne zapytanie');
    return $wynik;
[PHP] pobierz, plaintext 

Skrypt php

[PHP] pobierz, plaintext 
<?php
    require('include/include.php');
    require('header.php');
    require('menu.php');
 
    $tpl = new Smarty;
    $tpl -> template_dir = 'smarty/templates/';
    $tpl -> compile_dir = 'smarty/templates_c/';
 
    $zapytanie="select `user_id`,`login`,`name`,`surname`,`email`,`telephone` FROM `users`;";
    $wynik2 = array();
    $wynik=db_czytaj($zapytanie);
 
    while($wynik2=mysqli_fetch_assoc($wynik)){
        $wynik2=array(
            'user_id'=>$wynik2['user_id'],
            'login'=>$wynik2['login'],
            'name'=>$wynik2['name'],
            'surname'=>$wynik2['surname'],
            'email'=>$wynik2['telephone'],
        );
    }
 
 
    $tpl -> assign('wynik', $wynik2);
    $tpl -> display('users.tpl');
 
?>
 
[PHP] pobierz, plaintext 

i plik z szablonem

[XML] pobierz, plaintext 
<fieldset>
    <legend><p>Użytkownicy</p></legend>
    <a href="add_user.php"><button type="button"><p>Dodaj użytkownika</p></button></a>
<table>
        {section name=i loop=$wynik}
        <tr>
            <td>
                {$wynik[i].login}
            </td>
            <td>
                {$wynik[i].name}
            </td>
            <td>
                {$wynik[i].surname}
            </td>
            <td>
                {$wynik[i.email]}
            </td>
            <td>
                {$wynik[i].telephone}
            </td>
        </tr>
        {/section}
    </table>
</fieldset>
[XML] pobierz, plaintext 

Teraz niestety nic mi nie wyświetla tzn nie wyświetla mi tych rekordów z BD w szablonie

[abort]

[PHP] pobierz, plaintext 
 while($wynik2=mysqli_fetch_assoc($wynik)){
$wynik2=array(
'user_id'=>$wynik2['user_id'],
'login'=>$wynik2['login'],
[PHP] pobierz, plaintext 

Najpierw wczytujemy z bazy wiersz do zmiennej $wynik2.
Potem tworzymy (znowu!) zmienną $wynik2, przypisując jej fragment TEJ SAMEJ zmiennej $wynik2...
EDIT: I wszystko to robimy TYLKO PO TO, by podczas odczytywania następnego wiersza utracić te wartości przez nadpisanie je wartościami z właśnie odczytanego nowego wiersza...

Ech...

Tak czy inaczej, pisanie kluczowej pętli programu z użyciem jednej zmiennej to dość ciekawe podejście programistyczne.
Swoją drogą, nazewnictwo zmiennych ($wynik, $wynik2) znudzi Ci się w okolicy $wynik10, o ile wcześniej nie dostaniesz szału z debugowaniem ("Kruca bomba, który numerek po $wynik dodać, aby znaleźć to, czego szukam?!").

Ten post edytował abort 19.10.2012, 19:44:01

[DerekDX]

tak faktycznie bardzo poważny błąd logiczny
poprawiony skrypt

[PHP] pobierz, plaintext 
<?php
    require('include/include.php');
    require('header.php');
    require('menu.php');
 
    $tpl = new Smarty;
    $tpl -> template_dir = 'smarty/templates/';
    $tpl -> compile_dir = 'smarty/templates_c/';
 
    $zapytanie="select `user_id`,`login`,`name`,`surname`,`email`,`telephone` FROM `users`;";
    $users = array();
    $wynik=db_czytaj($zapytanie);
 
    while($wynik2=mysqli_fetch_assoc($wynik)){
        $users=array(
            'user_id'=>$wynik2['user_id'],
            'login'=>$wynik2['login'],
            'name'=>$wynik2['name'],
            'surname'=>$wynik2['surname'],
            'email'=>$wynik2['telephone'],
        );
    }
 
 
    $tpl -> assign('users', $users);
    $tpl -> display('users.tpl');
 
?>
 
[PHP] pobierz, plaintext 

teraz dla odmiany sypie błędami z Smartów

Notice: Undefined offset: 0 in E:\Dropbox\xampp-portable\htdocs\xampp\PJWSTK\smarty\templates_c\5f14d12ab6ac7e7a6ed3fab9fca28161a609a98c.file.users.tpl.php on line 56

[XML] pobierz, plaintext 
<fieldset>
    <legend><p>Użytkownicy</p></legend>
    <button type="button" onClick="window.open('add_user.php','Dodaj użytkownika','resizable=0,height=300,width=550')"><p>Dodaj użytkownika</p></button></a>
<table>
        {section name=i loop=$users}
        <tr>
            <td>
                {$users[i].user_id}
            </td>
            <td>
                <a href="#" onClick="window.open('view_user.php?id={$users[i].user_id}')">{$users[i].login}</a>
            </td>
            <td>
                {$users[i].name}
            </td>
            <td>
                {$users[i].surname}
            </td>
            <td>
                {$users[i].email}
            </td>
            <td>
                {$users[i].telephone}
            </td>
        </tr>
        {/section}
    </table>
</fieldset>
[XML] pobierz, plaintext 

[abort]

Dawno nie używałem smarty, ale: http://www.smarty.net/docsv2/en/language.f....html.table.tpl - krócej i efektywniej.
Co prawda oznaczona w rozdziale "Custom Functions", ale po wpisaniu w googlach po "smarty display array as table" dostaniesz więcej wyników (to był pierwszy link).

[nospor]

while($wynik2=mysqli_fetch_assoc($wynik)){

$users=array(

'user_id'=>$wynik2['user_id'],

'login'=>$wynik2['login'],

'name'=>$wynik2['name'],

'surname'=>$wynik2['surname'],

'email'=>$wynik2['telephone'],

);

}

Przecież ty tutaj w petli ciągle nadpisujesz $users.... w wyniku czego po pętli w $users masz tylko ostani rekord....

Tak ma być
$users = array();
while($wynik2=mysqli_fetch_assoc($wynik)){
$users[]=array(
'user_id'=>$wynik2['user_id'],
'login'=>$wynik2['login'],
'name'=>$wynik2['name'],
'surname'=>$wynik2['surname'],
'email'=>$wynik2['telephone'],
);
}

I zamiast section używaj FOREACH - jest o niebo czytelniejsze i oniebo trudniej się pomylić.