Witam ponownie po krótkiej przerwie...

Przez dwa ostatnie dni próbowałem napisać w pełni funkcjonalną klasę do obsługi logowania, zapamiętywania hasła i kontroli użytkownika, czy mi się udało ocenicie sami.

Kod:
http://wklej.org/hash/a408abdd6a/

Jest to jedna z pierwszych tak rozbudowanych klas którą napisałem, mam nadzieję że nie będziecie zbytnio narzekać. Proszę o ocenę kodu jak i funkcjonalności i o podpowiedzi co powinno być inaczej.

Jeden z sposobów użycia:

[PHP] pobierz, plaintext 
<?php
$form=new users();
    $form->conf(array('name'=>'admin','cookie'=>'TRUE'));
    
    $form->check();
    
    if(!$form->ifLogged())
      echo
        $form->form('In').
        'Nick: '.
        $form->inputNick().
        'Hasło: '.
        $form->inputPass().
        'Zapamietaj hasło: '.
        $form->inputRememberPass().
        $form->submitButton('Zaloguj').
        $form->endForm()
      ;
      
    if($form->ifLogged())
      echo
        $form->idSession().
        ' - Jesteś Zalogowany ;) <br />'.
        $form->form('Out').
        $form->submitButton('Wyloguj').
        $form->endForm()
      ;
  
    unset($form);
    
  ?>
[PHP] pobierz, plaintext 

Chciał bym również przypomnieć, że dopiero zaczynam zgłębiać tajniki programowania obiektowego, wiem że muszę do pisać do wszystkich funkcji tryb dostępu ale tym zajmę się na koniec.

Pozdrawiam i z góry dziękuję!

Formy i inputy powinny być osobną klasą. Nie rozumiem też sessionUser. Chcesz sprawdzać co dokładnie? Zalogowanych userów? Wtedy można by sie pokusić za zmianę obslugi sesji w php na własną.

Ja nie lubię też mieszać widoku z modelem, kontrolerem (jak kto nazywa). Teraz na sztywno masz takie inputy, a co jeśli chciałbym dodać fieldset albo inny input, czy wrzucić to do tabeli?

A więc wczoraj w nocy już zapomniałem napisać, że tak klasa będzie korzystała(dziedziczyła) z klasy form która będzie obsługiwać formularze, dodatkowo dołączę do tego kontroler mysql.

Co do kontroli użytkownika, chodzi głównie o ulepszenie kontroli sesji wbudowanej w php przez sprawdzanie ip (zawsze ktoś może pożyczyć sobie identyfikator sesji od innej osoby), możliwość wylogowania użytkownika za pomocą panelu admina (ktoś się zbytnio rozszalał więc blokuję mu konto i wylogowuję go).

Co do wstawienia tego do jakiegoś diva czy czegoś podobnego - nie ma żadnych problemów klasa jest tak skonstruowana by umożliwiała teo typu zabiegi, każdy input może być wyświetlany niezależnie od innego. A oto przykład:

[PHP] pobierz, plaintext 
<?php
    if(!$form->ifLogged())
      echo
        $form->form('In').
        '<table><tr></td><td>Nick: </td><td>'.
        $form->inputNick().
        '</td><td>Hasło: </td><td>'.
        $form->inputPass().
        '</td><td>Zapamietaj hasło: </td><td>'.
        $form->inputRememberPass().
       '</td><td>'.
        $form->submitButton('Zaloguj').
       '</td></tr></table>.'
        $form->endForm()
       ;
  ?>
[PHP] pobierz, plaintext 

Oczywiście przykład jest bardzo banalny można go znacznie powiększyć i rozbudować. A z fieldset sprawa wygląda identycznie. Co do zmiany inputa na np bottom to muszę się zastanowić i myślę o dodaniu identycznej funkcji jak i w przypadku input ale z bottomem, i faktycznie muszę dodać możliwość zmiany typu forma ale tym zajmę się po napisaniu klasy która będzie je konstruowała.

Dziękuję za opinię i z chęcią przeczytam nowe...

Ps Sądzicie że powinienem pozbyć się wszystkich formów i pozostawić samą kontrolę a formy tworzyć jako oddzielny obiekt (tym sposobem oddzielę prezentację od kontrolera)

Słowa klucze dla Ciebie, to:
- hermetyzacja
- zakres odpowiedzialności obiektu

Odpowiedz sobie na pytanie: Co Ci daje to, że ten kod jest ujęty w klasie.

Pozdrawiam

Hmmm, zakres odpowiedzialności obiektu - nie rozumiem ale zaraz poszperam coś na ten temat, co do tego co dostałem po napisaniu klasy to:
- Mogę w łatwy sposób stworzyć trzy logowania (nie martwiąc się o nazwy).
- Nie powtarzanie za każdym razem tego samego kodu.
- Łatwa implementacja w przyszłych projektach.

Chyba tyle ;P

Uważasz, że ta klasa jest bezsensowna i nie ma po co z niej korzystać??