[OOP] Schemat kalsy, i jej działanie Opcje

[piraciq]

Witam srdecznie,

Wygląda na to,iż będę tu dosyć często zaglądał.

Tak jak w temacie chodzi mi o schemat klasy i jej działanie.

[PHP] pobierz, plaintext 
<?php
class userRegist
{
     konstruktor - używam go do przeniesienia połączenia z baza danych
 
     funkcja sprawdzająca poprawność email zwraza true
    
     funkcja sprawdzająca czy podany email jest w bazie zwraca false jesli adres istnieje
   
     funkcja sprawdzająca długość podanego hasła zwraca true jeśli hało ma więcej niż x znaków
 
    funkcja sprawdzająca czy hasło i powtórzenie hasła pasują do siebie zwraca true
 
      funkcja do sprawdzania tokena przepisanego i wygenerowanego zwraca true     
}
?>
[PHP] pobierz, plaintext 

i teraz tak jezeli email jest prawidłowy przechodzi do sprawdzania w bazie, jeżeli go nie ma w bazie zwraca true, sprawdza hasla zwraca true, sprawdza tokeny i zwraca true w wyniku czego następuje wsadzenie rekordu do bazy i wysłanie emaila pod wskazany adres. i jest ok.


ale chodzi mi o sprawdzanie wartosci (true, false) z funkcji normalnie rozwiazal bym to tak ale chodzi mi o jakieś może lepsze rozwiazanie

[PHP] pobierz, plaintext 
<?php
//funkcje i cała reszta z kasy
 
if(poprawnyEmail)
{
   if(!emailJestWBazie)// tu akurat oczekuje false
     {
        if(dlugoscHaslaJestOdpowiednia)
          {
              if(podaneHaslaSaTakieSame)
               {
                    if(tokenSaTakieSame)
                    {
                           $this->rejejstrracja;
                      }
                 }
           }
      }
}
?>
[PHP] pobierz, plaintext 

mam nadzieję, że jasno to napisze!! Czy można by było to zapisać w jakiś inny sposób niż taki "blok" if?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

[marcok]

~mike mówisz, że wyjątki służą do sytuacji wyjątkowych jednak idąc dalej tym tropem gdy wstawiasz w kod try ... catch przewidujesz wystąpienie anomalii więc po co catch lepiej upakować to w if'y. Mówisz, że kod taki wykonuje się dłużej, szczerze mówiąc nie wiem nie sprawdzałem jednak dla klasy obsługującej jedynie rejestracje - czynność zdecydowanie rzadszą od np. wyświetlania strony głównej etc. Moim zdaniem jest to sposób korzystniejszy abstrahując od wydajności kodu, która czasami nie jest tak ważna.

Ten post edytował marcok 16.10.2008, 06:08:12

[mike]

Mówisz, że kod taki wykonuje się dłużej, szczerze mówiąc nie wiem nie sprawdzałem jednak dla klasy obsługującej jedynie rejestracje - czynność zdecydowanie rzadszą od np. wyświetlania strony głównej etc.

Dobre nawyki programowania powinno sosować się zawsze. Jeśli czasem odpuszczasz "bo to tylko jedna klasa" to po pewnym czasie pisząc dużą aplikację połowa to będzie jakiś "spaghetti code".
Tu dopuszczasz sprawdzaine parametrów przy rejestracji, tak? A podczas wykonywania dowolnej innej akcji jak sprawdzasz parametry żądania? Inaczej?
Po co dwa różne sposoby? COś mam wrażenie, że gdzieś indziej też byś tak zrobił a "dowolna inna akcja" wykonuje się już "trochę" częściej niż rejestracja.

Moim zdaniem jest to sposób korzystniejszy abstrahując od wydajności kodu, która czasami nie jest tak ważna.

W takim razie mam proste pytanie. Co zyskujesz, skoro tak jest korzystniej?

~mike mówisz, że wyjątki służą do sytuacji wyjątkowych jednak idąc dalej tym tropem gdy wstawiasz w kod try ... catch przewidujesz wystąpienie anomalii więc po co catch lepiej upakować to w if'y.

Spłaszczasz problem. Oczywiście, że można pokazać sytuacje kiedy wiesz, że coś może wystąpią ale nadal będzie to dla Ciebie niespodziewane. ~Crozin pokazał przykład.
Kolejny przykład - łączysz się z bazą, to połączenie zawsze powinno być ale dopuszczasz możliwość, że baza padła. Sieci nie ma. Dane dostępowe się zmieniły. Dajesz to w try ... catch bo niby połączenie zawsze powinno istnieć ale co jeśli?