WItam w jednym z poprzednich tematow byl link do faq z wikibooks i jako zadanie dla siebie postanowilem rozbudowac sobie tamten przyklad zrobilem jak narazie 2 klasy tworze 3.

1 klasa sluzy do logowania dla mysql jak i dla plikow *.txt -> http://3paste.com/s/374
2 klasa do rejestracji w 2 wyborach jak wyzej. -> http://3paste.com/s/378

Ogolnie plik mysqlDriver.php zawiera prosta klase z debilnym funkcjami tak tylko zeby pocwiczyc

Chodzi mi o to czy dobrze wszystko zaimplementowalem, co byscie poprawili, co jest do dupy a co jest good.

Czy np logowanie lepiej by wygladalo gdybym w klasie Login mial 2 publiczne pola np $nick i $pass i logowanie np tworzyl tak zamiast przekazywac zmienne z posta do funkcji login():

[PHP] pobierz, plaintext 
<?php
session_start();
error_reporting(E_ALL);
include('form.php');
include('classAuth.php');
 
 if(isset($_POST['login']) && isset($_POST['pass'])) {
 
  if(TYPE == 'SQL') $user = new Login(new MySqlAuth(new MySqlDriver('***', '***', '***', '***')));
 
  else $user = new Login(new FileAuth('users.txt'));
   
   $user -> nick = $_POST['login'];
   $user -> password = $_POST['pass'];
   echo($user -> AuthLogin());
 
 }
?>
[PHP] pobierz, plaintext 

I tak samo jesli chodzi o rejestracje jak i 3 klase ktora pisze edytowanie/usuwanie user'ow lepiej przekazywac wszystko do funckji lub ustawiac pola po instancji klasy lub to zalezy co sie robi?

Jak wy robicie co polecacie??

Ogolnie czekam na szczery komentarz ewentualne zmiany w kodzie i przedstawienie mi go bym mogl sie w koncu czegos pozytywnego nauczyc

P.S kod dalem na inny srv by nie robic burdelu w watku.

P.S2 tak wyglada np podstrona dla zalogowanych user'ow: http://3paste.com/s/376

Jakies sugestie??

EDIT:Link zostal poprawiony poprostu przegladalem ostatnio dodane kody na 3paste i przez przypadek skopiowalem nie ten link.

Witam !

Na wstepie musze powiedziec ze drugi link ktory podales to nie jest niestety " klasa do rejestracji " tylko implementacja scrollowanej textArea w Swingu . Ale to szczegol .
Co do rozwiazania logowania to moim zdaniem jest ok.
Podoba mi sie mozliwosc podania w klasie login zrodla danych , jak i zdefiniowanie ewentualnego drivera.
Jedyna sugestia z mojej strony :
Klasa logowania powinna realizowac mechanizm autentyfikacji usera ale niekoniecznie musi posiadac pola takie jak nick czy password zwiazane bezposrednio z userem.
Generalnie jej zadaniem jest proba zalogowania i stwierdzenie czy logowanie sie powiodlo czy nie.

Do implementacji danych usera uzylbym jakiegos modelu dziedziny z polami zwiazanymi typowo z userem a samego sprawdzenie przy probie logowania
dokonalbym raczej jakos

[PHP] pobierz, plaintext 
<?php
$login->doAuth($user->getNick,$user->getPassword)
?>
[PHP] pobierz, plaintext 

A propos przekazywania zmiennych do funkcji/constuctora vs ustawianiu pol w instancji to uwazam ze dobrze jest dac mozliwosc wyboru uzywajacemu twoja klase. Czyli generalnie fajnie jest jezeli klasa posiada obie mozliwosci.

Pozdrawiam

Dodalem EDIT do pi


Czy moglbys rozwinac twoja odpowiedz bo nie bardzo rozumiem.


Tego tez nie bardzo rozumiem.


To kumam chodz ci np o taka implementacje klasy Login i jej funkcji AuthLogin() i robienie nowej instancji:

[PHP] pobierz, plaintext 
<?php
include_once('mysqlDriver.php');
 
interface TAuth {
 
 public function login($login, $password);
 
 }
 
//tu taka sama implementacja klas MySqlAuth i FileAuth
 
class Login {
 
// zostawiam pola co byly wczesniej i dodaje 2 pola publiczne:
 
 public $name;
 public $pwd;
 private $nick;
 private $auth;
 
 public function __construct($auth) {
 
 $this -> auth = $auth;
 
  }
 
//metody statyczne LoginAuth i LogOut zostaja takie same.
 
 public function AuthLogin() {
 
 $this -> name = '';
 $this -> pwd = '';
 $usr = $_POST['login'];
 $haslo = $_POST['pass']
 
  if($this -> name == '' && $this -> pwd == '') $user = $this -> auth -> login($usr, $haslo);
 
  else if($this -> != '' && $this -> pwd != '') $user = $this -> auth -> login($this -> name, $this -> pwd);
 
 if($user) {
 
   $this -> nick = $user['login'];
   $_SESSION['login'] = true;
   $_SESSION['name'] = $this -> nick;
   header("Location: user.php");
 
      }
 
  else return '<div style="margin:auto;padding:5px;width:25%;border:solid 1px #FF0000;background-color:#FFFFFF;color:#FF0000;text-align:center;">Podano zle dane.</div>';
    
   }
 
 public function __destruct() {
 
 unset($this -> nick);
 unset($this -> auth);
 
   }
 
 }
 
//Instancja klasy Login wygladala by tak jesli nie ustawiam pola name i pola pwd:
 
session_start();
include('form.php');
include('classAuth.php');
 
 if(isset($_POST['login']) && isset($_POST['pass'])) {
 
  if(TYPE == 'SQL') $user = new Login(new MySqlAuth(new MySqlDriver('85.17.1.175', 'marcios', 'marciniak15', 'marcios')));
 
  else $user = new Login(new FileAuth('users.txt'));
 
   echo($user -> AuthLogin());
 
 }
 
 
//Z ustawieniem pola name i pola pwd:
 
session_start();
include('form.php');
include('classAuth.php');
 
 if(isset($_POST['login']) && isset($_POST['pass'])) {
 
  if(TYPE == 'SQL') $user = new Login(new MySqlAuth(new MySqlDriver('85.17.1.175', 'marcios', 'marciniak15', 'marcios')));
 
  else $user = new Login(new FileAuth('users.txt'));
 
   $user -> name = $_POST['login'];
   $user -> pwd = $_POST['pass'];
 
   echo($user -> AuthLogin());
 
 }
?>
[PHP] pobierz, plaintext 

Tylko nie wiem czy to zadziala bo nie sprawdzalem jak sprawdze to dopiero wieczorem bo glownie chodzi o to czy te warubki zadzialaja:

[PHP] pobierz, plaintext 
<?php
  if($this -> name == '' && $this -> pwd == '') $user = $this -> auth -> login($usr, $haslo);
 
   else if($this -> != '' && $this -> pwd != '') $user = $this -> auth -> login($this -> name, $this -> pwd);
?>
[PHP] pobierz, plaintext 

Poniewaz nie jestem znawca to jesli nie ustawie pol namei pwd czyli 1 przyklad tworzenia instacnji to powinno wywolac sie to:

Jesli ustawie:

Tylko nie wiem czy jak nie ustawie pol name i pwd to beda one empty czyli == '' lub beda null i nie wiem czy dobry jest warunek i czy na tej zasadzie ma to dzialac lub macie jakies inne sposoby na to bo w teorii powinno to dzialac.