Forum PHP.pl

Witam serdecznie,
mam nastepujaca klase (wlasciwie jej poczatek), jest to klasa odpowiedzialna za prawdzanie danych...

wyglada ona w sposob nastepujacy:
SprawdzanieDanych.class.php

[PHP] pobierz, plaintext 
<?php
 
 class SprawdzanieDanych
 {
	 // sprawdzany ciag
	 var $ciag;
	 
	 
	 /**
	  * @desc Kontruktor klasy
	  */
	 function SprawdzanieDanych($ciag) {
		 $this->ciag = $ciag;
	 }
	 
	 
	 /**
	  * @desc Sprawdzenie poprawnosci adresu poczty elektronicznej
	  *
	  * @return bool
	  */
	 function sprawdzEmail() {
		return preg_match('/^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+.[a-zA-Z]{2,}$/', $this->ciag);
	 }
}
 
?>
[PHP] pobierz, plaintext 

a teraz wykorzystanie tej klasy w kodzie

index.php

[PHP] pobierz, plaintext 
<?php
 
require_once('SprawdzanieDanych.class.php');
 
$ciag = 'mhs_news@interia.pl';
 
$spr = new SprawdzanieDanych($ciag);
var_export($spr->sprawdzEmail());
 
$ciag = 'mhs_newsinteria.pl';
 
$spr->ciag = $ciag;
var_export($spr->sprawdzEmail());
 
?>
[PHP] pobierz, plaintext 

co do tego kodu to mam pytanie czy w prawidlowy sposob tworze konstruktor klasy oraz czy pozniej prawidlowo wykonuje czynnosci w pliku index.php zwlaszcza chodzi mi o linie 12 - czy w ten sposob powinno sie postepowac... gdyz moge przekazywac ciag poprzez argument w metodzie (czyli: $spr->email($ciag), oczywiscie trzeba zmienic troche kod klasy) ale wowczas z kolei nie mam co "wrzucic" do kontruktora klasy...

prosze o podpowiedzi...

mhs

Nie widze sensu tworzenia klasy do czegoś takiego. A jeśli już to wszystkie metody niech będa statyczne. Nie ma sensu tworzyć instancji takiej klasy.

Tak jak powiedział przedmówca.
Taka klasa jak teraz pokazales jest bezzensowan gdyz wystarczy do tego z powodzeniem 1 funkcja....

Jezeli to bardziej rozbudujesz i zaleznosci jakies miedzy kalsami nawiazesz to moze bedzie z tego jakis 'pozytek'.

jezeli chodzi o sama klase to mysle, ja rozbudowac o kolejne metody sprawdzajace nip, regon, pesel i troche innych rzeczy...

w chwili obecnej to ona ma nastepujaca postac:

[PHP] pobierz, plaintext 
<?php
 
class SprawdzanieDanych
{
	// sprawdzany ciag
	var $ciag;
	 
	// zmienna przechowujaca sume
	var $suma;
	 
	// liczba kontrolna
	var $liczba_kontrolna;
 
	// tablica przechowujaca wagi dla poszczegolnych pozycji w sprawdzanym numerze
	var $wagi;
 
	// licznik petli
	var $i;
	 
	 
	 
	 
	 
	 
	/**
	 * @desc Kontruktor klasy
	 */
	function SprawdzanieDanych($ciag) {
		$this->ciag = $ciag;
	}
 
 
 
	/**
	 * @desc Sprawdzenie poprawnosci adresu poczty elektronicznej
	 *
	 * @return bool
	 */
	function email() {
		return preg_match('/^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+.[a-zA-Z]{2,}$/', $this->ciag);
	}
	 
	 
	 
	/**
	 * @desc Sprawdzenie poprawnosci kodu pocztowego
	 *
	 * @return bool
	 */
	function kodPocztowy() {
		return preg_match('/^[0-9]{2}[-][0-9]{3}$/', $this->ciag);
	}
 
 
 
	/**
	 * @desc Sprawdzanie poprawnosci numeru NIP wg. wzorca: xxx-xxx-xx-xx
	 *
	 * @return bool
	 */
	function nip() {
		// sprawdzenie dopasowania do wzorca
		if (!ereg('^[0-9]{3}-[0-9]{3}-[0-9]{2}-[0-9]{2}$', $this->ciag))
			return 0;
 
		$this->ciag = str_replace('-', '', $this->ciag);
 
		$this->wagi = array(6, 5, 7, 2, 3, 4, 5, 6, 7);
 
		$this->suma = 0;
		for ($this->i = 0; $this->i < count($this->wagi); $this->i++)
			$this->suma += substr($this->ciag, $this->i, 1) * $this->wagi[$this->i];
 
		if ( ($this->suma % 11) == substr($this->ciag, 9, 1) )
			return 1;
		else
			return 0;
	}
 
 
 
	/**
	 * @desc Sprawdzanie poprawnosci numeru REGON
	 *
	 * @return bool
	 */
	function regon() {
		if (!ereg('^[0-9]{9}$', $this->ciag))
			return 0;
 
		$this->wagi = array(8, 9, 2, 3, 4, 5, 6, 7);
 
		$this->suma = 0;
		for ($this->i = 0; $this->i < count($this->wagi); $this->i++)
			$this->suma += substr($this->ciag, $this->i, 1) * $this->wagi[$this->i];
 
		if ( ($this->suma % 11) == substr($this->ciag, 8, 1))
			return 1;
		elseif ( (($this->suma % 11) == 10) && (substr($this->ciag, 8, 1) == 0) )
			return 1;
		else
			return 0;
	}
 
 
 
	/**
	 * @desc Sprawdzanie poprawnosci numeru PESEL
	 *
	 * @return bool
	 */
	function pesel() {
		if (!ereg('^[0-9]{11}$', $this->ciag))
			return 0;
 
		$this->wagi = array(1, 3, 7, 9, 1, 3, 7 , 9, 1, 3);
 
		$this->suma = 0;
		for ($this->i = 0; $this->i < count($this->wagi); $this->i++)
			$this->suma += substr($this->ciag, $this->i, 1) * $this->wagi[$this->i];
 
		$this->liczba_kontrolna = 10 - ($this->suma % 10);
 
		if ( ($this->liczba_kontrolna == 10) && (substr($this->ciag, 10, 1) == 0) )
			return 1;
		elseif ($this->liczba_kontrolna == substr($this->ciag, 10, 1))
			return 1;
		else
			return 0;		
	}
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował mhs 1.09.2004, 10:54:13

Nadal bezsensu.
Lepiej tak jeśli już koniecznie chcesz klasę:

[PHP] pobierz, plaintext 
<?php
class SprawdzanieDanych
{
	function SprawdzEmail($ciag)
	{
		return preg_match(&#092;"/lala/\", $ciag);
	}
 
	function SprawdzKodPocztowy($ciag)
	{
		return preg_match(&#092;"/bleble/\", $ciag);
	}
 
// itd.
}
 
if (SprawdzanieDanych::SprawdzEmail(&#092;"parti@poczta.com\"))
// cos tam..
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Parti 1.09.2004, 11:12:33

Kompletna katastrofa. W programowaniu obiektowym chodzi o obiekty, które mają odwzorowywać dziedzinę biznesową, blah, blah. To wcale nie jest to samo co programowanie "klasowe", lub raczej "statyczno-klasowo-proceduralne", które tak naprawdę jest gorsze niż programowanie proceduralne, bo wprowadza dodatkowy zamęt i bałagan.

Krótko, OOP != wrzucenie kilku funkcji to przypadkowej klasy. To co widzimy to raczej wymuszona emulacja namespaces przez mechanizm klas.

Zgadzam się w 100%. Jeszcze jeden kwiatek z tej klasy:

[PHP] pobierz, plaintext 
<?php
class SprawdzanieDanych
{
	// licznik petli
	var $i;
 
?>
[PHP] pobierz, plaintext 

no dobrze, ale spsob w jaki chce do tego podejsc (czyli napisanie klasy dzieki ktorej bede mogl sprawdzac poprawnosc danych jest zle) czy sposob w jaki to wykonuje..?

Napisanie takiej klasy nie jest złe, tylko nie nazywajmy tego programowaniem obiektowym.
Traktuj to raczej jako namespace pozwalający Ci uniknąć konfliktów nazw i używaj nie metody obiektu a funkcji klasy, bo obiekt jest tutaj wogóle nie potrzebny.