Forum PHP.pl > przeniesienie kodu do klasy

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: przeniesienie kodu do klasy

Forum PHP.pl > Forum > PHP > Object-oriented programming

MitS

26.03.2007, 15:26:38

Witam!

od 2 dni mecze sie z przeniesieniem pewnej czesci kodu do klasy.
Moja strona:

index.php
http://rafb.net/p/bdcEHo79.html

class.Auth.php
http://rafb.net/p/xEAIMk34.html

i zadaniem tej strony jest rejestracja nowcyh userow do bazy danych...
To elegancko dziala!!

ale ... ten kod z index.php:

[PHP] pobierz, plaintext 
<?php
$login = htmlspecialchars(stripslashes(trim($_POST['login'])), ENT_QUOTES);
$haslo = htmlspecialchars(stripslashes(trim($_POST['haslo'])), ENT_QUOTES);
$rehaslo = htmlspecialchars(stripslashes(trim($_POST['rehaslo'])), ENT_QUOTES);
$email = htmlspecialchars(stripslashes(trim($_POST['email'])), ENT_QUOTES);
$imie = htmlspecialchars(stripslashes(trim($_POST['imie'])), ENT_QUOTES);
 
$opcja = trim($_REQUEST['opcja']);
$blad = 0;
if ($opcja == 'register'){
	if ((strlen($login)<4) || (strlen($login)>20) || (!eregi('^[a-zA-Z0-9_.]+$',$login))) { 
		  $blad++;
		echo 'Login musi mieć od 4 do 20 znaków bez polskich liter i spacji!<br />';
	} 
	else {
		if ($baza = mysql_connect($db_host, $db_user, $db_password)) {
			if (mysql_select_db($db_name)) {
				  $wynik=mysql_query("SELECT * FROM $d_section WHERE login='$login'");
			} 
			else 
				  echo "Nie można połączyć się z bazą";
		 mysql_close($baza);
		} 
		else 
			echo "Nie można połączyć się z serwerem MySQL";
 
		if (mysql_num_rows($wynik) != 0) { 
			   $blad++;
			   echo "Login już został przez kogoś użyty!. Zaproponuj inny!<br />";
		}
	}
 
	if ((strlen($haslo)<6) || (strlen($haslo)>20) || (!eregi("^[a-zA-Z0-9]+$",$haslo))) { 
		  $blad++;
		echo "Hasło musi mieć od 6 do 20 znaków bez polskich liter i spacji!<br />";
	}
 
	if ($haslo != $rehaslo)	{
		$blad++;
		echo 'Złe powtórzone hasło!!!';
	}
 
	if (!eregi("^[0-9a-z_.-]+@([0-9a-z-]+.)+[a-z]{2,4}$", $email)) { 
		  $blad++;
		echo "E-mail nie został podany prawidłowo!<br />";
	}
	else{
		if ($baza = mysql_connect($db_host, $db_user, $db_password)) {
			if (mysql_select_db($db_name))  {
				  $wynik=mysql_query("SELECT * FROM $d_section WHERE email='$email'");
			} 
			else 
				  echo "Nie można połączyć się z bazą";
 
			 mysql_close($baza);
		} 
		else 
			   echo "Nie można połączyć się z serwerem MySQL";
			   
		if (mysql_num_rows($wynik) != 0) { 
				   $blad++;
				   echo "E-mail już został przez kogoś użyty!. Zaproponuj inny!<br />";
		}
	}	
}
?>
[PHP] pobierz, plaintext

... chcialbym przenieśc do funkcji w klasie Auth
Więc probowalem tak:

index.php

[PHP] pobierz, plaintext 
<?php
session_start();
define('SITE_PATH", 'http://localhost/piaskownica/');
define('MAIL_PATH', 'mail@mail.com');
 
$db_host   = 'localhost';					
$db_user	 = 'root';								
$db_password	 = '';											
$db_name   = 'piaskownica';							
$d_section = 'logowanie';		
 
include("class.Auth.php");
echo '
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">	
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2" />
<script src="./js/scripts.js" type="text/javascript"></script>
<style type="text/css" title="currentStyle" media="screen">
	@import "./css/site.css";
</style>
<title>Piaskownica</title>
</head>
<body xml:lang="pl">';
$signin = new Auth($db_host, $db_user, $db_password, $db_name, $d_section, $login, $haslo, $email, $imie, $blad);
 
$authorize = $signin->init();
echo '
<form action="index.php" method="post" autocomplete="OFF">
<input type="hidden" name="opcja" value="register" />
login:*<input type="text" name="login" value="'.$login.'" /><br />
hasło:*<input type="password" name="haslo" value="" /><br />
powtorz hasło:*<input type="password" name="rehaslo" value="" /><br />
e-mail:*<input type="text" name="email" value="'.$email.'" /><br />
imie i nazwisko:<input type="text" name="imie" value="'.$imie.'" /><br />
<input type="submit" value=" OK, rejestruję się!" />
</form>';
 
echo '
</body>
</html>';
?>
[PHP] pobierz, plaintext

class.Auth.php
http://rafb.net/p/9YczBS71.html

Czemu to co ja robie nie dziala wlasciwie

?
pomozecie mi to poprawnie zrobić (na tyle by dzialalo)

Pozdrawiam

ps. kurde czemu po edycji topicu wszystkie " " zamieniaja mi sie na \" \"

Strzałek

27.03.2007, 10:24:49

Nie, nie, nie, nie i jeszcze raz ... NIE!

Nie o to w tym wszystkim chodzi.
Programowanie strukturalnie, a programowanie obiektowe to dwie różne bajki i nie można w 5 minut przerzucić kodu strukturalnego do klasy. Zrobić z f-cji metody, nazwać to jakoś i siup! Huraaa mamy kod obiektowy! Bo zamiast mojaFunkcja() wywołujemy, $SuperKlasa -> mojaFunkcja();

Programowanie obiektowe (OOP) wymaga innego myślenia i całkowicie innego spojrzenia na programowanie.

Trudno jest to wytłumaczyć (przynajmniej mi). Jednak po pewnym czasie to zrozumiesz. Po przeglądaj klasy, frameworki (wszystkie są pisane obiektowo) i w pewnym momencie rozumiesz. Dla mnie to też kiedyś była czarna magia i w sumie nikt mi tego nie tłumaczył. Samo jakoś wyszło w praniu, o co w tym biega.

MitS

27.03.2007, 13:51:18

yhy rozumiem ... :/
no spoko... mam nadzieje ze szybko mi wejdzie obiektowe pisanie w krew

DeyV

15.04.2007, 22:22:31

Taki kod mógłby wyglądać np. tak:

[PHP] pobierz, plaintext 
<?php
class DB{
 protected $sLogin ='';
 protected $sHaslo = '';
 
 static function newDb(){
  if( empty( self::$Db ) ){
	 self::$Db = new DB();
	 self::$Db->connect();
  }
  return self::$Db;
 }
 
 private function connect(){ }
 private function __construct(){ }
}
 
class UserModel{
  private $Db;
 
  function __construct(){
	  $this->Db = DB::newDb();
  }
 
  function register( $sLogin, $sHaslo, $sMail ){
	  if( empty( $sLogin ) ){
		  throw new InvalidArgumentException( 'Podaj login' );
	  }
 
	  $this->Db->Insert( /*... */);  
	  return true;
  }
}
?>
[PHP] pobierz, plaintext

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.