[MySQL][PHP] Klasy abstrakcyjne... Opcje

[sloenthran]

Otóż mam taki kod:

[PHP] pobierz, plaintext 
<?php
 
	class Buy
	{
 
		private $_id_serwera;
		private $_flaga;
		private $_flaga_dostepu;
		private $_dni;
		private $_time;
 
		public function __construct($id_serwera, $flaga, $flaga_dostepu, $dni, $time)
		{
 
			$this->_id_serwera    = $id_serwera;
			$this->_flaga         = $flaga;
			$this->_flaga_dostepu = $flaga_dostepu;
			$this->_dni           = $dni;
			$this->_time          = $time;
 
		}
 
		public function sprawdz($kod_sms, $numer)
		{
 
			$Zapytanie  = @mysql_query("SELECT `api_gamesol` FROM ustawienia");
			$Pobieranie = @mysql_fetch_array($Zapytanie);
 
			$API = $Pobieranie['api_gamesol'];
 
			if($kod_sms == 'test1' || $kod_sms == 'test2' || $kod_sms == 'test3' || $kod_sms == 'test4' || $kod_sms == 'test5' || $kod_sms == 'test6' || $kod_sms == 'test9' || $kod_sms == 'test17' || $kod_sms == 'test20' || $kod_sms == 'test25')
			{
 
				return false;
 
			}
 
			$wynik = file_get_contents("http://gamesol.pl/api/sms-pro.php?id=".$API."&taryfa=".$numer."&kod=".$kod_sms."&opis=sloenthran:shop");
 
			$wynik_tab = json_decode($wynik, true);
 
			if($wynik_tab['blad'] != 0) 
			{
 
				return false;
 
			}
 
			else
			{
 
				return true;
 
			}
 
			return false;
 
		}
 
		public function zakoncz($nick, $haslo)
		{		
 
			if(!empty($nick) and !empty($haslo))
			{
 
				$pass 	= md5($haslo);
				$upraw  = $this -> _flaga;
				$flagi  = $this -> CheckFree($this->_id_serwera, $nick, $upraw);
 
				if(!$flagi) 
				{  			
 
					$exp = $this->_time + ($this->_dni * 86400);
 
					mysql_query("INSERT INTO `amx_amxadmins` VALUES(NULL, '{$nick}', '{$pass}', '{$upraw}', '{$this->_flaga_dostepu}', '{$nick}', '{$nick}', 0, 0, '{$this->_time}', '{$exp}', {$this->_dni})");
 
					$id = mysql_insert_id();
 
					mysql_query("INSERT INTO `amx_admins_servers` VALUES({$id}, {$this->_id_serwera}, '', 'no')");
 
					return TRUE;
 
				}
 
				else
				{
 
					return TRUE;
 
				}
 
			}
 
			return FALSE;
 
		}
 
		private function CheckFree($sid = 0, $na_co = '', $upraw = '')
		{
 
			$query = @mysql_query("SELECT * FROM `amx_amxadmins` WHERE `steamid` = '{$na_co}'");
 
			if(mysql_num_rows( $query ) > 0)
			{
 
				$row      = mysql_fetch_assoc($query);
				$newFlags = $this->sumFlags($row['access'], $upraw);
				$ex       = ($this->_time + ((($row['expired'] - $this->_time) + ($this->dni * 86400)) / 2));
 
				mysql_query("UPDATE `amx_amxadmins` SET `access` = '{$newFlags}' WHERE `steamid` = '{$na_co}'");
 
				$query2 = mysql_query("SELECT * FROM `amx_admins_servers` WHERE `admin_id` = '{$row['id']}' AND `server_id` = '{$this->_id_serwera}'");
 
				if(mysql_num_rows( $query2 ) > 0)
				{
 
					return TRUE;
 
				}
 
				else
				{
 
					mysql_query("INSERT INTO `amx_admins_servers` VALUES ({$row['id']}, {$this->_id_serwera}, '', 'no')");
					return TRUE;
 
				}
 
				return FALSE;
 
			}
 
			return FALSE;
 
		}
 
		private function sumFlags($flagi_a = "", $flagi_b = "")
		{
 
			for($i = 0; $i < strlen($flagi_b); $i++) 
			{
 
				if(!(strlen(strstr($flagi_a, $flagi_b[$i]))))
				$out .= $flagi_b[$i];
 
			}
 
			$to = str_split($flagi_a . $out);
			sort($to);
			$ciag = implode('', $to);
 
			return $ciag;
 
		}
 
	}
 
?>
[PHP] pobierz, plaintext 

I ktoś powiedział mi że lepiej abym zamiast za każdym razem ciągle pisać (kilka tych samych klas tylko z zmienioną funkcją sprawdź) to samo to lepiej skorzystać z klas abstrakcyjnych bądź interfejsów... Próbowałem taką klasę samemu jednakże nic mi nie wychodzi... Czy mógłby ktoś z kodu wyżej napisać mi coś takiego?

[buliq]

interfejs odpada, bo nie zdefiniujesz tych powtarzających się metod.

W abstrakcji generalnie możesz połączyć interfejs z zwykłą klasą. Klasa abstrakcyjna może być zwykłą klasą (powiedzmy). Zatem, utworzenie klasy abstrakcyjnej:

[PHP] pobierz, plaintext 
abstract class Name {}
[PHP] pobierz, plaintext 

W tej klasie robisz normalne metody, oraz metody abstrakcyjne. Metody abstrakcyjne nie posiadają ciała (definicji) a jedynie ich deklarację (nazwę, parametry, itd.). Przykład:

[PHP] pobierz, plaintext 
abstract class Name {
 
public function cosTam($a) {
echo $a;
}
 
abstract public function innaMetoda($b, $c $d = null);
 
}
[PHP] pobierz, plaintext 

Następnie robisz drugą klasę która dziedziczy po tej klasie abstrakcyjnej, i tam definiujesz metodę innaMetoda

Ten post edytował buliq 29.08.2013, 08:06:35

[freemp3]

Podstawowe pytanie, czy wiesz na jakiej zasadzie działają klasy abstrakcyjne oraz interfejsy? Pomysł dobry, ale nie ma potrzeby korzystania z klas abstrakcyjnych ani interfejsów. Zamiast nich możesz bezpośrednio dziedziczyć z tej klasy i nadpisywać te metody, które chcesz zmienić.

Odnośnie klas abstrakcyjnych to różnią się od zwykłych tym, że metody nie muszą mieć kodu i nie można z nich tworzyć obiektów. Można z nich jedynie dziedziczyć. Dlatego też klasa abstrakcyjna nie może być jednocześnie finalną. Co oczywiscie nie wyklucza tego, że może mieć finalne metody, pod warunkiem, że metoda nie jest abstrakcyjna

Co do interfejsów to jest to spis metod publicznych, które nie mają "ciała". Nie mogą mieć one żadnego kodu oraz dostęp do nich musi być publiczny. Może wydawać się to dziwne, ale przydaje się np. do tworzenia API zewnętrznego. Każda nowa klasa w tedy musi zawierać pewien zestaw metod, które są zdefiniowane właśnie w interfejsie. Co one będą robić zależy już tylko i wyłącznie od klasy, ale wszystkie muszą być zaimplementowane.

[sloenthran]

Czyli rozumiem że jeżeli zapisałbym to tak:

[PHP] pobierz, plaintext 
    <?php
 
   abstract class Buy
    {
 
    private $_id_serwera;
    private $_flaga;
    private $_flaga_dostepu;
    private $_dni;
    private $_time;
 
    public function __construct($id_serwera, $flaga, $flaga_dostepu, $dni, $time)
    {
 
    $this->_id_serwera = $id_serwera;
    $this->_flaga = $flaga;
    $this->_flaga_dostepu = $flaga_dostepu;
    $this->_dni = $dni;
    $this->_time = $time;
 
    }
 
    public function zakoncz($nick, $haslo)
    {	
 
    if(!empty($nick) and !empty($haslo))
    {
 
    $pass = md5($haslo);
    $upraw = $this -> _flaga;
    $flagi = $this -> CheckFree($this->_id_serwera, $nick, $upraw);
 
    if(!$flagi)
    {
 
    $exp = $this->_time + ($this->_dni * 86400);
 
    mysql_query("INSERT INTO `amx_amxadmins` VALUES(NULL, '{$nick}', '{$pass}', '{$upraw}', '{$this->_flaga_dostepu}', '{$nick}', '{$nick}', 0, 0, '{$this->_time}', '{$exp}', {$this->_dni})");
 
    $id = mysql_insert_id();
 
    mysql_query("INSERT INTO `amx_admins_servers` VALUES({$id}, {$this->_id_serwera}, '', 'no')");
 
    return TRUE;
 
    }
 
    else
    {
 
    return TRUE;
 
    }
 
    }
 
    return FALSE;
 
    }
 
    private function CheckFree($sid = 0, $na_co = '', $upraw = '')
    {
 
    $query = @mysql_query("SELECT * FROM `amx_amxadmins` WHERE `steamid` = '{$na_co}'");
 
    if(mysql_num_rows( $query ) > 0)
    {
 
    $row = mysql_fetch_assoc($query);
    $newFlags = $this->sumFlags($row['access'], $upraw);
    $ex = ($this->_time + ((($row['expired'] - $this->_time) + ($this->dni * 86400)) / 2));
 
    mysql_query("UPDATE `amx_amxadmins` SET `access` = '{$newFlags}' WHERE `steamid` = '{$na_co}'");
 
    $query2 = mysql_query("SELECT * FROM `amx_admins_servers` WHERE `admin_id` = '{$row['id']}' AND `server_id` = '{$this->_id_serwera}'");
 
    if(mysql_num_rows( $query2 ) > 0)
    {
 
    return TRUE;
 
    }
 
    else
    {
 
    mysql_query("INSERT INTO `amx_admins_servers` VALUES ({$row['id']}, {$this->_id_serwera}, '', 'no')");
    return TRUE;
 
    }
 
    return FALSE;
 
    }
 
    return FALSE;
 
    }
 
    private function sumFlags($flagi_a = "", $flagi_b = "")
    {
 
    for($i = 0; $i < strlen($flagi_b); $i++)
    {
 
    if(!(strlen(strstr($flagi_a, $flagi_b[$i]))))
    $out .= $flagi_b[$i];
 
    }
 
    $to = str_split($flagi_a . $out);
    sort($to);
    $ciag = implode('', $to);
 
    return $ciag;
 
    }
 
    }
 
    ?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
class BuyTwo extends Buy
{
 
    public function sprawdz($kod_sms, $numer)
    {
 
    $Zapytanie = @mysql_query("SELECT `api_gamesol` FROM ustawienia");
    $Pobieranie = @mysql_fetch_array($Zapytanie);
 
    $API = $Pobieranie['api_gamesol'];
 
    if($kod_sms == 'test1' || $kod_sms == 'test2' || $kod_sms == 'test3' || $kod_sms == 'test4' || $kod_sms == 'test5' || $kod_sms == 'test6' || $kod_sms == 'test9' || $kod_sms == 'test17' || $kod_sms == 'test20' || $kod_sms == 'test25')
    {
 
    return false;
 
    }
 
    $wynik = file_get_contents("http://gamesol.pl/api/sms-pro.php?id=".$API."&taryfa=".$numer."&kod=".$kod_sms."&opis=sloenthran:shop");
 
    $wynik_tab = json_decode($wynik, true);
 
    if($wynik_tab['blad'] != 0)
    {
 
    return false;
 
    }
 
    else
    {
 
    return true;
 
    }
 
    return false;
 
    }
 
}
[PHP] pobierz, plaintext 

To będzie wtedy działać tak samo jakbym zapisał to razem czy czegoś nie rozumiem?

[buliq]

Tak, tylko. Jeżeli definiujesz ze klasa jest abstrakcyjna, warto z tej abstrakcji skorzystać. Nie zadeklarowałeś metody sprawdz w klasie Buy. Deklaracja wymusi na każdej klasie dziedziczącej definicję tej metody.

[freemp3]

Tak. Możesz też dodać w klasie Buy metodę:

Kod

abstract public function sprawdz();

W tedy jak zapomnisz jej dodać w klasie dziedziczącej to wyświetli się błąd, który Ci o tym przypomni

Ten post edytował freemp3 29.08.2013, 08:56:55

[sloenthran]

OK dzięki wam za pomoc