Forum PHP.pl

witam
mam system logowania oparty na MySQL który na moje potrzeby działa dobrze, ale niechce go przepisywac za każdym razem gdy robię nowy projekt.Chciałbym to upakowac w klase tylko nie bardzo wiem jak, moje OOP jest narazie na poziomie podstawowym.
Chodzi mi o pokazanie przykładu takiej klasy.

wstaw wszystkie funkcje i zmienne do klasy, dodaj na poczatku public/private/..., utworz deklaracje zmiennych i ewentualnie konstruktor, w ktorym przypiszesz do tych zmiennych wartosci.

mniej wiecej wiem jak to zrobić , ale jednak juz tak namieszałem ze nie moge sie z tego wyplątać, tak jak mówiłem moje OOP jest narazie nędzne a klasy które pisze są na poziomie ala ma kota (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) , a tym bardziej ze mój system logowania jest dość złożony to jeszcze nieporadze sobie z tym

Rozpisz sobie (na kartce) jakich metod do obslugi bazy bedziesz potrzebowal - polaczenie, wykonanie zapytania, pobranie ilosci wierszy zwroconych, itd., zapisz ich parametry, zwracane wartosci, wrzuc do klasy i zaimplementuj te metody.

po wielu bólach zrobiłem coś takiego i nawet to chodzi, ale strasznie mi sie to niepodoba.

[PHP] pobierz, plaintext 
<?php
session_start();
class system_autoryzacji{
		private $login;
		private $haslo;
		private $link;
		private $dbname;
		private $query;
		private $result;
		private $dane;
		private $wynik;
 
		public function logowanie(){
 
				$login = $_POST["login"];
				$haslo = $_POST["haslo"];
				if ($_GET["login"]=="koniec") { // wylogowanie
				session_unset();
				session_destroy();
				} else if ($login<>"" and $haslo<>"") {
				$haslo = md5($haslo);
				$dbname ="moja_baza";
				if ($link = mysql_connect('moj_host','moj_user','moj_pass')){
				if (mysql_select_db($dbname) or die (mysql_error())) {
				$query = "SELECT * FROM user WHERE login='$login' and haslo='$haslo' and status=5";
				$result = mysql_query ($query ,$link) or die (mysql_error());
				if (mysql_num_rows($result)==1) {
				$dane=mysql_fetch_array($result);
				$_SESSION["zalogowany"]="tak";
				$_SESSION["login"]=$dane["login"];
				$_SESSION["imie"]=$dane["imie"];
				$_SESSION["email"]=$dane["email"];
				}
				} else echo "Nie można połaczyc sie z baza";
				mysql_close();
				}
				}
		}
 
		public function formularz_logowanie (){
				if ($_SESSION["zalogowany"]=="tak") {
				echo "<p>Zalogowany : <b>".$_SESSION["login"]."</b></p>";
				echo "<br><a href="klasa_logowanie.php?login=koniec">wyloguj sie</a>";
				echo "<br><a href="zmiany.php">zmien hasło</a>";
				echo "<br><a href="index2.php">index2</a>";
				} else {
				echo <<<KONIEC
				<center>
				<form action="klasa_logowanie.php" method="post">
				<table>
				<tr>
				<td align="right">login:&nbsp;</td>
				<td><input type="text" name="login" /></td>
				</tr>
				<tr>
				<td align="right">hasło:&nbsp;</td>
				<td><input type="password" name="haslo" /></td>
				</tr>
				<tr>
				<td colspan="2" align="right">
				<input type="submit" value="zaloguj" /></td>
				</tr>
				</table>
				</form>
				<a href="rejestracja.php?opcja=przypomnij">PRZYPOMNIJ HASŁO</a></br />
				<a href="rejestracja.php">ZAREJESTRUJ SI!</a>
				</center>
KONIEC;
				}
		}
}
 
$logo = new system_autoryzacji();
$logo -> logowanie();
$logo -> formularz_logowanie();
?>
[PHP] pobierz, plaintext 

jakie macie propozycje zeby to poprawić i jak?

Ja napisalem takie cos, tez nie wiem czy jest poprawne (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
 
	class User
	{
  public $uLogin;
  public $uPass;
 
 
  public function __construct ( $userLogin, $userPass )
  {
	  $this->uLogin = $userLogin;
	  $this->uPass = md5($userPass);
  }	
 
  public function Login ( )
  {
	  global $db, $tb_users;
 
	  $qTl = mysql_fetch_array(mysql_query('SELECT COUNT(*) FROM ' . $tb_users . ' WHERE login = '' . $this->uLogin . '' AND password = '' . $this->uPass  . '' '));
 
 
	  if ( $qTl[0] !== 0 )
	  {
	$_SESSION['aUser'] = $this->uLogin;
	  }
  }
 
  public function Logout ( )
  {
	  unset($_SESSION['aUser']);
  }
 
 
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował Diablos 4.12.2005, 13:04:56

ma ktos jeszcze jakieś propozycje co do mojego tworu?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

----edit----
Proszę nie nabijać postów.
~mike_mech

wolferin:
tez sie dopiero ucze ale wydaje mi sie ze wyswietlanie formularza jako jedna z metod klasy to chyba cos nie tak - nie o to chodzi w obiektowce. Moim zdaniem, choc mylic sie moze, wiec moze ktos mnie poprawi?

Pozdro

Access_user Class

osobiscie uwazam ze cos jest oprawne jesli dziala (bez bledow skladniowych i logicznych) i jest zgodne z jasno sprecyzowanymi poczatkowymi zalozeniami i oczekiwaniami. tak wiec wedlug tego sam ocen czy jest poprawne... ;-)

[PHP] pobierz, plaintext 
<?php
final class MemberLog {
private $user;
 
public function __construct( $u ) {
$this->user = $u;
}
 
public function in( $pass ) {
$q = mysql_query("SELECT pass FROM ". TBL_NAME ." WHERE name='".$this->user."'");
return (mysql_fetch_row($q) === $pass) ? true : false;
}
}
?>
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
$member = new MemberLog($_POST['nick']);
if($member->in($_POST['password'])) setcookie('in', 'in', time()+3600);
?>
[PHP] pobierz, plaintext 

pozdrawiam

Ten post edytował Ociu 16.12.2005, 16:41:08