Programowanie Obiektowe..., czy tak nalezy postepowac Opcje

[mhs]

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

[Parti]

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.