[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.

[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