[AJAX][PHP] Formularz z AJAX - zgodnie z MVC. Opcje

[Damonsson]

Mam taką zagwozdkę: zrobiłem formularz sprawdzający czy dany login już istnieje w bazie danych i wyświetlający informację na bieżąco po wprowadzeniu znaku (keyup).


AJAXem wysyłam posta do kontrolera url: "register/checkLogin",

[JAVASCRIPT] pobierz, plaintext 
$(function()
{
    $('#username').keyup(function() {
        $("#status").html('<img src="public/img/ajax-loader.gif">&nbsp;Sprawdzanie dostępności.');
        clearTimeout($.data(this, 'timer'));
        var wait = setTimeout(search, 600);
        $(this).data('timer', wait);
    });
 
    function search() {
 
        var username = $("#username").val();
        var msgbox = $("#status");
 
        if(username.length > 4)
        {
            $("#status").html('Sprawdzanie dostępności.');
            $.ajax({
                type: "POST",
                url: "register/checkLogin",
                data: "username="+ username,
                success: function(msg){
                    $("#status").ajaxComplete(function(event, request){
                        if(msg == 'OK')
                        {
                            $("#username").removeClass("red");
                            $("#username").addClass("green");
                            msgbox.html('<img src="yes.png"> <font color="Green"> Dostępny</font>');
                        }
                        else
                        {
                            $("#username").removeClass("green"); 
                            $("#username").addClass("red"); 
                            msgbox.html(msg);
                        }
                    });
                }
            });
 
        }
        else
        {
            $("#username").addClass("red"); 
            $("#status").html('<font color="#cc0000">Za mało znaków</font>');
        }
        return false;
    }
});
[JAVASCRIPT] pobierz, plaintext 

W kontrolerze mam funkcję:

[PHP] pobierz, plaintext 
public function checkLogin() {
        $data = array(
            'login' => $_POST['username']
        );
        $this->model->checkLogin($data);
 
    }
[PHP] pobierz, plaintext 

W modelu:

[PHP] pobierz, plaintext 
public function checkLogin($data) {
        if (isset($data['login'])) {
            $sth = $this->db->selectOne('SELECT userId FROM user WHERE login = :login', array(
                ':login' => $data['login']
                    ));
            if ($sth) {
                echo '<span style="color: #cc0000;"><strong>' . $data['login'] . '</strong> jest już zajęty.</span>';
            } else {
                echo 'OK';
            }
        }
    }
[PHP] pobierz, plaintext 

W modelu wywołując print_r($sth) daje taki efekt dla zajętego loginu: Array ( [userId] => 6 ). Dla wolnego nie zwraca nic.

Nie podoba mi się, że w modelu mam umieszczony ten fragment:

[PHP] pobierz, plaintext 
if ($sth) {
                echo '<span style="color: #cc0000;"><strong>' . $data['login'] . '</strong> jest już zajęty.</span>';
            } else {
                echo 'OK';
            }
[PHP] pobierz, plaintext 

Tylko nie wiem gdzie mam go wrzucić i jak przekazać odpowiednio? Do widoku? Do kontrolera? Czy może, wcale to nie jest złe i powinno być to w modelu?

W widoku mam jedynie

[HTML] pobierz, plaintext 
<input id="username" name="username" type="text" /> <span id="status"> </span>
[HTML] pobierz, plaintext 

Gdyby mógł ktoś rzucić okiem i fachowo mnie oświecić jak to powinno się zrobić należycie, byłbym niezmiernie wdzięczny.

[Spawnm]

zwracaj 0/1 i w js generuj html.

[Sebek87]

Ja proponuję string wyświetlać w kontrolerze. W modelu zwróć sobie boolean, a w kontrolerze np. echo 'true/false' i gitara.

[abort]

Pomiń zwracanie tego HTMLa (w zamian dodawaj go w jQ), zostaw tylko zwracanie "OK". Lub tak jak napisał Spawnm, zwracaj bool 0/1 - ale tak czy inaczej kod HTML dodałbym do jQuery.

[markonix]

Model koniecznie powinien zwracać true / false, aby zachować jego uniwersalność - przecież możemy go wykorzystywać w różnych kontekstach, nie tylko ajaxowym.
Potem gdybyśmy chcieli zachować taką uniwersalność dla kontrolera to można się pokusić o sprawdzenie czy request jest ajaxowy.
Zwykle w FW jest coś w stylu $this->request->isAjax().
Jeśli ajax dajesz echo albo json_encode(array('status' => $status)).
Jeśli nie ajax to po prostu return $status.

Ten post edytował markonix 24.10.2012, 23:31:25

[Damonsson]

Dzięki wszystkim, ale w szczególności @markonix, o to mi chodziło, teraz jest git.

w .js dałem

[JAVASCRIPT] pobierz, plaintext 
 if(check == 'false')
                        {
                            bla bla HTML
                        }
                        else
                        {
                            bla bla 2. HTML
                        }
[JAVASCRIPT] pobierz, plaintext 

a w modelu

[PHP] pobierz, plaintext 
public function checkLogin($data) {
        if (isset($data['login'])) {
            $check = $this->db->selectOne('SELECT userId FROM user WHERE login = :login', array(
                ':login' => $data['login']
                    ));
            echo json_encode($check);
        }
    }
[PHP] pobierz, plaintext 

[Spawnm]

Ale czemu model zwraca jsona i czemu dajesz echo w modelu!? Niech zwraca boola, będziesz mógł wykorzystać w różnych przypadkach.

[Damonsson]

Chodzi Ci o coś takiego?

[PHP] pobierz, plaintext 
public function checkLogin($data) {
 
        if (isset($data['login'])) {
 
            $check = $this->db->selectOne('SELECT userId FROM user WHERE login = :login', array(
 
                ':login' => $data['login']
 
                    ));
 
          if ($check) {
                $status = FALSE;
            } else {
                $status = TRUE;
            }
 
        }
 
    }
[PHP] pobierz, plaintext 

Tylko jak mam później to przekazać do skryptu .js w widoku? Muszę to jakoś "wyeksportować" do widoku przecież. A nie mogę, bo nie generuję oddzielnego widoku dla checkLogin, bo i po co, Gubię się z tym AJAXem ;/



Jak mam normalnie np. pobieranie tam wszystkich rekordów to robię tak:

w kontrolerze generuję widok:

[PHP] pobierz, plaintext 
public function index() {
        $this->view->noteList = $this->model->noteSelectAll();
        $this->view->render('note/index');
    }
[PHP] pobierz, plaintext 

W modelu zwracam:

[PHP] pobierz, plaintext 
public function noteSelectAll() {
        return $this->db->selectAll("SELECT * FROM note WHERE id = :id", array('id' => $_SESSION['userId']));
    }
[PHP] pobierz, plaintext 

no i w widoku przykładowo wyświetlam tak

[PHP] pobierz, plaintext 
foreach ($this->noteList as $key => $value) {
        echo '<tr>';
        echo '<td>' . $value['title'] . '</td>';
        echo '<td>' . $value['content'] . '</td>';
        echo '<td>' . $value['date_add'] . '</td>';
        echo '</tr>';
    }
    ?>
[PHP] pobierz, plaintext 

[markonix]

[PHP] pobierz, plaintext 
 if ($check) {
$status = FALSE;
} else {
$status = TRUE;
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
if ($check)
   return false;
return true;
[PHP] pobierz, plaintext 

Albo jeszcze lepiej to się pewnie da zapisać zwracając wynik metody selectOne rzutowanej na boolean (tylko nie mam pewności co ta metoda normalnie zwraca).

Gdy masz już model - czyli metodę to przechodzimy do ajax.
W ajax wywołujesz URL twoja-strona.pl/klasa/metoda/ (w zależności od FW różnie to wygląda) przesyłając login.
Kontroler wywołuje metodę z modelu z odpowiednim argumentem zawierającym login.
Model zwraca true / false.
Kontroler odbiera informacje i zwraca JSON w przypadku gdy jest to request ajaxowy.
jQuery odbiera JSON z informacją i aktualizuje odpowiednio informacje przy formularzu.

Ten post edytował markonix 26.10.2012, 13:31:10