[Kohana] Tworzenie odnośników Opcje

[PawelC]

Wszystko ładnie pięknie idzie mi w kohana php, tylko teraz mam problem jak tworzyć odnośniki. Może wstawię model, widok i kontroler.

Model:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') or die('No direct script access.');
 
class Test_Model extends Model {
 
    public function __construct()
    {
        // load database library into $this->db (can be omitted if not required)
        parent::__construct();
    }
    
    public function kategorie()
    {
    $db=new Database();
    $result=$db->query('select * from slowa limit 5');
    foreach ($result as $row)
{
    echo '<a href="test/slowo/id/'.$row->id.'">'.$row->slowo.'<br />'; // to zapewne jest źle
 
}
}
 
}
?>
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') or die('No direct script access.');
 
$user = new Test_Model;
$name = $user->kategorie();
?>
[PHP] pobierz, plaintext 

Oraz kontroler:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') OR die('No direct access allowed.');
 
class Test_Controller extends Controller
{
    public function index()
    {
        $view = new View('test');
        $view->render(TRUE);
 
     }
   
        
    }
}
?>
[PHP] pobierz, plaintext 

I skrypt działa w taki sposób, że pobiera dane z bazy, nazwy programów i teraz chciałbym aby po kliknięciu w odnośnik(jak powinien on wyglądać?) wyświetliło wszystko z bazy o klikniętym programie.
Zdaję sobie sprawę że będę musiał utworzyć jeszcze jeden model, który będzie odpowiedzialny za pobranie danego programu.

Ten post edytował ExPlOiT 5.06.2009, 11:31:52

[kbsucha]

Hmm coś pomieszałeś. Czemu zamiast w kontrolerze pobierać dane z modelu i je przekazywać do widoku, uruchamiasz model w widoku?
Przekaż sobie tablice zawierająca id i słowo do widoku (to powinno być w kontrolerze). I tam poprzez jakąś pętle wyświetlaj sobie linki, albo ręcznie przez html, albo poprzez helper url::anchor.

Ten post edytował kbsucha 5.06.2009, 12:15:00

[phpion]

Poza tym nie musisz pobierać instacji połączenia z bazą za każdym razem:

[PHP] pobierz, plaintext 
<?php
$db=new Database();
$result=$db->query('select * from slowa limit 5');
?>
[PHP] pobierz, plaintext 

możesz z powodzeniem zastąpić:

[PHP] pobierz, plaintext 
<?php
$result=$this->db->query('select * from slowa limit 5');
?>
[PHP] pobierz, plaintext 

Ponadto osobiście polecałbym Ci korzystanie z query buildera - ma kilka błędów i ograniczeń ale ogólnie staje się bardzo przydatny szczególnie w sytuacji dynamicznego generowania zapytań. W Twoim przypadku nie jest to konieczne ale wg mnie dobrze jest korzystać ze wszystkich dobrodziejstw, jakie daje nam Kohana.

[PawelC]

Już sobie poradziłem, wystarczyło stworzyć w modelu kolejną funkcje, czyli w moim wypadku:

[PHP] pobierz, plaintext 
<?php
public function show($id){
        
        $db=new Database();
    $result=$db->query('select * from slowa where id="'.$id.'"');
    foreach ($result as $row)
{
    echo 'id: '.$row->id.'<br />';
    echo 'Słowo: '.$row->slowo;
 
}
}
?>
[PHP] pobierz, plaintext 

A w modelu kategorie, zmienić forme linku na

[PHP] pobierz, plaintext 
<?php
echo '<a href="test/show/'.$row->id.'">'.$row->slowo.'<br />';
?>
[PHP] pobierz, plaintext 

A później to już wiadomo co i jak.

Poza tym nie musisz pobierać instacji połączenia z bazą za każdym razem:

Tego to nie wiedziałem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Hmm coś pomieszałeś. Czemu zamiast w kontrolerze pobierać dane z modelu i je przekazywać do widoku, uruchamiasz model w widoku?

Dokładnie, pomieszałem trochę ale dałem rade (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował ExPlOiT 5.06.2009, 21:37:37

[AxZx]

łooooo
dlaczego tak to rozumujesz? poczytaj więcej o MVC.

html wyświetlaj w widoku
dane pobieraj w modelu
w kontrolerze pobieraj dane z modelu, wstawiaj je do widoku.

w modelu możesz mieć takie metody jak: get_all, get_one - czyli pobieranie wszystkich elementów i pobieranie tylko jednego elementu.
w kontrolerze w zależności od tego co masz w URL pobieraj albo tylko jeden element, albo wszystkie elementy. również w zależności od tego co masz w URL wyświetlaj odpowiedni widok - albo ze wszystkimi wpisami, albo tylko z jednym.

pozdrawiam

[PawelC]

Mam taki kontroler:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') OR die('No direct access allowed.');
 
class Test_Controller extends Controller
{
    public function index()
    {
        $view = new View('test');
        $view->render(True);
    }
    public function show($nazwa){
       
       $db=new Database();
   $result=$db->query('select * from gd_pliki where nazwa="'.$nazwa.'"');
   foreach ($result as $row)
{
   echo 'id: '.$row->id.'<br />';
   echo 'Słowo: '.$row->nazwa;
 
}
}
}
?>
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') or die('No direct script access.');
$test = new Test_Model;
$test->test();
?>
[PHP] pobierz, plaintext 

Model:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') or die('No direct script access.');
 
class Test_Model extends Model {
 
    public function __construct()
    {
        // load database library into $this->db (can be omitted if not required)
        parent::__construct();
    }
    public function test()
    {
    $db=new Database();
    $result=$db->query("select * from gd_pliki limit 10");
    foreach($result as $rows)
    {
    echo '<a href="test/show/'.$rows->nazwa.'">'.$rows->nazwa.'</a><br />';
    }
    }
    
}
?>
[PHP] pobierz, plaintext 

I chodzi mi o to, aby ktoś życzliwy pokazał mi jak to powinno wyglądać zgodnie z architekturą MVC, bo podejrzewam że mój kod nie jest z MVC zgodny.

[Villentre]

Witaj.
Podstawa: żadnych ech w modelach i kontrolerach (no, chyba, że jedynie w kontrolerze jedno echo za wyświetlenie zrenderowanego widoku odpowiedzialne (echo $view -> render(FALSE);)).

Przepisałem Twój kod po swojemu, mam nadzieję, że nic nie zgubiłem i nigdzie się nie pomyliłem. Wyszły mi 4 plik, wszystkie opisane poniżej:

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') OR die('No direct access allowed.');
//controllers/test.php
//Kontroler;
    class Test_Controller extends Controller
    {
        public function __construct()
        {
            $this -> test = new Test_Model;
        }
        //Wyświetlanie 10 wpisów z bazy;
        public function index()
        {
            $view = new View('test10');
            $view -> date = $this -> test -> getTen();
            $view -> render(TRUE);
        }
 
        //Wyświetlanie 1 wpisu;
        public function show($name)
        {
            $view = new View('test1');
            $view -> date = $this -> test -> getOne($name);
            $view -> render(TRUE);
        }
    }
 
 
<?php defined('SYSPATH') or die('No direct script access.');
//models/test.php
//Model
    class Test_Model extends Model
    {
        public function __construct()
        {
            parent::__construct(); //m.in. przypisanie new Database pod $this -> db;
        }
 
        //Pobranie 1 wpisu z bazy
        public function getOne($name)
        {
            $name = $this -> db -> escape($name); //"czyszczenie";
            return $this -> db -> query('select * from gd_pliki where nazwa='.$name.'');
        }
 
        //Pobranie 10 wpisów z bazy
        public function getTen()
        {
            return $this -> db -> query("select * from gd_pliki limit 10");
        }
    }
 
 
<?php defined('SYSPATH') or die('No direct script access.');
//views/test10.php
//Widok wyświetlający listę 10 wpisów;
    if($date -> count())
    {
        echo '<ul>';
        foreach($date as $dat)
        {
            echo '<li>'.html::anchor('test/show/'.$dat -> nazwa, $dat -> nazwa).'</li>';
        }
        echo '</ul>';
    }
    else
    {
        echo 'Brak danych';
    }
 
 
<?php defined('SYSPATH') or die('No direct script access.');
//views/test1.php
//Widok wyświetlający 1 wpis;
    if($date -> count())
    {
        echo '<b>ID<b>: '.$date[0] -> id;
        echo '<b>Nazwa</b>: '.$date[0] -> nazwa;
    }
    else
    {
        echo 'Brak danych';
    }
?>
[PHP] pobierz, plaintext 

Edit:
@down: Rzeczywiście, mój błąd (tak to jest jak coś się kopiuje (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ). Już poprawiłem.

Edit:
Wkurzające forum, zamienia html na bbc ;/

Ten post edytował Villentre 14.06.2009, 00:46:50

[AxZx]

to chyba tak powinno być:

[PHP] pobierz, plaintext 
<?php
$this->test = new Test_Model;
?>
[PHP] pobierz, plaintext 

[PawelC]

Dzięki, teraz już wiem o co chodzi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował ExPlOiT 14.06.2009, 11:40:34

[phpion]

Ja ze swojej strony miałbym jeszcze uwagi odnośnie widoków. Tworzenie ich w "czystym" PHP jest moim zdaniem nieeleganckie. Zdecydowanie lepiej użyć do tego "szablonów" czyli przejść z HTML do PHP i na odwót tak, jak to zostało pokazane w dokumentacji.

[PawelC]

Wstawiłem ten kod co Villentre podał i wywala mi błąd:

Kod

Fatal error:  Call to a member function _kohana_load_view() on a non-object in C:\wamp\www\kohana\system\libraries\View.php on line 272

Już sobie poradziłem, brakowało parent::contruct();

Teraz już wiem, jak projektować aplikacje zgodne z MVC (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował ExPlOiT 14.06.2009, 23:46:27