obsluga formularza Opcje

[skowron-line]

[PHP] pobierz, plaintext 
<?php
 
class obsluga_formularza {
 
	public $nazwa;
	public $adres;
	private $wzorzec = '/^[a-zA-Z0-9.-_]+@[a-zA-Z0-9-.]+.[a-zA-Z]{2,4}$/';
 
 
	function __construct($nazwa, $adres) {
 
		$this -> nazwa = $nazwa;
		$this -> adres = $adres;
 
	}
 
	function policz_znaki() {
 
		$ile_znakow = strlen($this -> nazwa);
		return $ile_znakow;
 
	}
	function sprawdz_adres() {
 
		if(!preg_match($this ->wzorzec, $this -> adres)){
			return "bledny adres e-mail";
		}else{
			return "adres e-mail jest poprawny";
		}
 
	}
	function pokaz() {
		return $this -> policz_znaki().
			$this -> sprawdz_adres();
	}
}
 
 
echo'<form action="" method="post">
		nazwa:<input type="text" name="nazwa">
		adres:<input type="text" name="adres">
	<input type="submit" name="submit">';
 
if(isset($_POST['submit'])){
 
		$parm = new obsluga_formularza($_POST['nazwa'], $_POST['adres']);
 
		echo $parm -> pokaz();
 
}
?>
[PHP] pobierz, plaintext 

od jakiegos czasu usiluje sie nauczyc programowac obiektowo i napisalem cos takiego.
nie wiem czy jest to napisane poprawnie ale dziala jak ja to sobie zalorzylem.
lecz moje pytanie brzmi czy tak powinno to wyglada czy moze sprawdzanie poprawnosci wpisanych danych powinno byc poza klasa?? moze to zly przyklad ale chodzi mi o to ze jezeli teraz chcialbym sprawdzic czy pola zostaly wypelnione to czy mam to napisac w klasie czy poza nia??

[acztery]

pare uwag.

1)
czemu tak:

[PHP] pobierz, plaintext 
<?php
function policz_znaki() {				
	  $ile_znakow = strlen($this -> nazwa);		
	  return $ile_znakow;			
}
?>
[PHP] pobierz, plaintext 

nie lepiej

[PHP] pobierz, plaintext 
<?php
function policz_znaki() {				
	  return  $ile_znakow = strlen($this -> nazwa);	  
}
?>
[PHP] pobierz, plaintext 

2) czemu w metodach masz np "bledny adres e-mail" nie lepiej false albo true
3) czemu $this -> policz_znaki(). anie self::policz_znaki().
4) metoda sa public static czy jak ..

acha a metody sprawdz_adres policz_znaki moge byc private chyba

nie jestem specem ale tyle mi sie rzuciło

Ten post edytował acztery 28.12.2006, 17:40:39

[Ace]

@acztery jesli sie czepiasz to czepiaj sie dokladnie...

proponujesz

[PHP] pobierz, plaintext 
<?php
function policz_znaki() {				
	  return  $ile_znakow = strlen($this -> nazwa);	  
}
?>
[PHP] pobierz, plaintext 

to ja zaproponuje

[PHP] pobierz, plaintext 
<?php
function policz_znaki() {				
	  return  strlen($this -> nazwa);	  
}
?>
[PHP] pobierz, plaintext 

po co $ile_znakow?? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) zmienna zbedna

poza tym to jest srednia obiektowosc, ubranie funkcji w obiekt.

Formularz to jedno
Walidacja danych to drugie
Co z tymi danymi robimy to trzecie...

Formularz moze skladac sie z wielu obiektow.. kazde pole moze byc obiektem skladajacym sie z metod walidowania tego pola

[acztery]

przeoczylem te zmienna ale to mialem na mysli..

[skowron-line]

dzeki za uwagi jak zawsze bardzo cenne ale i tak nikt nie odpowiedzial na moje pytanie

poza tym to jest srednia obiektowosc, ubranie funkcji w obiekt.

@Ace mozesz to rozwinac

[Ace]

Jasne,

Najprostrzy przyklad, twoja klasa "obsluga_formularza" jest odpowiedzialna za wyswietlenie DANEGO formularza, oraz metody do walidacji ma w sobie. Czemu?

Pomysl lepiej nad obiektem formularza, ktory bedziesz mogl rozszerzac. Walidacja w formularzu? Zle zagranie. Walidacje powinienes trzymac w zupelnie innym miejscu - inna klasa? ktora bedzie zawierac zbior wszystkich metod walidacji czego zywnie sobie chcesz...

Ludzie chcacy pisac obiektowo, piszac proceduralnie po prostu ubieraja zbior funkcji w obiekt... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pomysl inaczej, Formularz to jakis obiekt ma napewno jakies wlasnosci - lista pol, akcja ktora stanie sie po submit, wczesniej walidacja danych, kazde pole ma przypisane do siebie sposob walidacji, moze nie miec walidacji, moze miec 1 sposob walidacji, albo kilka walidacji...

Twojego przykladu nie da sie rozszerzac. W obiektowosci liczy sie to, ze dobrze napisane obiekty mozesz wykorzystac wiele razy, pozatym latwosc rozbudowy/przebudowy jest wielkim plusem OOP. Wszystko sklada sie z obiektow. Wiec obsluga_formularza powinna tez dzielic sie na wiele mniejszych obiektow. Chociazby lista pol, albo walidacje w klasie do obslugi formularzy.

[skowron-line]

Kod

Najprostrzy przyklad, twoja klasa "obsluga_formularza" jest odpowiedzialna za wyswietlenie DANEGO formularza, oraz metody do walidacji ma w sobie.

[PHP] pobierz, plaintext 
<?php
echo'<form action="" method="post">
		nazwa:<input type="text" name="nazwa">
		adres:<input type="text" name="adres">
	<input type="submit" name="submit">';
 
if(isset($_POST['submit'])){
 
		$parm = new obsluga_formularza($_POST['nazwa'], $_POST['adres']);
 
		echo $parm -> pokaz();
 
}
?>
[PHP] pobierz, plaintext 

formularz jest poza klasa dopiero po nacisnieciu submita jest wywolywana klasa.


czyli "mowisz" ze formularz tez powinien byc w klasie??, a walidacja w innej??

dopiero zaczynam przygode z OOP i samo jej uzycie jest dla mnie malo zrozumiale

Kod

Twojego przykladu nie da sie rozszerzac

w jakim sensie nie da sie go rozszerzac?? to co powinno tam jeszcze sie znajdowac??

Kod

Wiec obsluga_formularza powinna tez dzielic sie na wiele mniejszych obiektow. Chociazby lista pol, albo walidacje w klasie do obslugi formularzy.

nie kapuje dlaczego??

[koala]

czyli "mowisz" ze formularz tez powinien byc w klasie?? - nie
a walidacja w innej?? - tak

[PHP] pobierz, plaintext 
<?php
class textInputComponent implements ioptComponent
	{
		protected $message = NULL;
		protected $tagParameters = array();
		protected $tpl;
 
		public function __construct()
		{
			$this -> message = NULL;		
		} // end __construct();
 
		public function setOptInstance(optClass $tpl)
		{
			$this -> tpl = $tpl;
		} // end setOptInstance();
 
		public function set($name, $value)
		{
			switch($name)
			{
				case 'message':
					$this -> message = $value;
					break;
				default:
					$this -> tagParameters[$name] = $value;		
			}
		} // end set();
 
		public function push($value, $desc, $selected = false)
		{
			$this -> set($value, $desc);
		} // end push();
 
		public function setDatasource(&$source)
		{
			if(is_array($source))
			{
				if(isset($source['name']))
				{
					$this -> tagParameters['name'] = $source['name'];
				}
				if(isset($source['value']))
				{
					$this -> tagParameters['value'] = $source['value'];
				}
				if(isset($source['message']))
				{
					$this -> message = $source['message'];
				}
			}
		} // end setDatasource();
 
		public function begin()
		{
			return '<input type="text"'.generateTagElementList($this->tagParameters).' />';
		} // end begin();
 
		public function onmessage($pass_to)
		{
			if($this -> message == NULL)
			{
				return 0;
			}
			$this -> tpl -> vars[$pass_to] = $this -> message;
			return 1;
		} // end onmessage();
 
		public function end()
		{
			return '';		
		} // end end();
	}
?>
[PHP] pobierz, plaintext 

Ten kod pochodzi z OpenPowerTemplate - plik opt.components.php
input type="text" - jest klasą
input type="select" - jest kolejną klasą
itd. czyli każdy "input" to osobna klasa
a Twoja klasa mogaby się nazywać "Zend_Filter_Input" (zaglądnij do Zend-a). Za pomocą tej klasy możesz sprawdzić czy podany przez użytkownika email to faktycznie email, jeśli pytasz o wiek to czy to co podal użytkownik to napewno jest "int", czy nick ma minimum np. 5 znaków itd.
Czyli wszystko osobno - każdy input = osobna klasa a walidacja w innej.

Gdybyś swój formularz wstawil do do tej swojej klasy to miabyś klasę do obsugi tego konkretnego formularza a do kolejnego musiabyś stworzyć kolejną taką samą klasę która różnilaby się tylko ilością :
input type="?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) " name="?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) "
czyli jeśli w serwisie masz 10 formularzy musisz stworzyć 10 niemalże identycznych klas a tak masz zestaw konkretnych klas (elementów formularzy) i walidację w osobnej i czy w serwisie "XXX" czy "YYY" czy "ZZZ' ciągle korzystasz z tych samych klas nie musząc ciągle tworzyć nowych.

[skowron-line]

Kod

Czyli wszystko osobno - każdy input = osobna klasa

Kod

klasę która różnilaby się tylko ilością :
input type="?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) " name="?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) "

a wczoraj jak zasypialem wpadlem na cos takiego ale jak znam zycie to pewnie to rozwiazanie nie jest wlasciwe i ktos je skrytykuje

[PHP] pobierz, plaintext 
<?php
 
class formularz {
 
	public $describe;
	public $name;
	public $rows;
	public $cols;
 
 
	public function text($name, $describe) {
 
		return $describe. '<input type="text" name="'.$name.'">';
 
	}
 
	public function textarea($name, $describe, $rows, $cols) {
 
		return $describe. '<textarea rows="'.$rows.'" cols="'.$cols.'" name="'.$name.'"></textarea>';
 
	}
 
}
 
$parm = new formularz;
 
echo $parm -> text("dupa", "dupa");
echo $parm -> textarea("dupa","dupa","5","10");
 
?>
[PHP] pobierz, plaintext 

[Ace]

Powiem tak,

dla mnie piszac obiekt do oblugi formularzy chce, latwo tworzyc rozne formularze, dodawac walidacje, zrodla danych...

Formularz to jeden obiekt, mam tez obiekty kazdego z typow pol - bo kazde pole jest inne... Mam Managera do tworzenia walidacji, Kazda walidacja jest przypisana do pola, po $_POST ida dane do formularza, obiekt jest tworzony, dane sa uzupelniane i walidujemy + wiemy ktore pola sa zle wypelnione. Proste...

Nie tworz nowych metod dla roznych typow Pol... To jest ZLE. Chodzi w skrocie o to, ze jak w jednym projekcie bedziezs mial jakies tam typy pol, a w innym inny wyglad tych pol. To czemu masz modyfikowac klase formularzy? Szablony do tego zastosuj. W kolejnych projektach nie mozesz modyfikowac klasy formularza. Musisz dobudowywac do niej pewna funkcjonalnosc, ale samej jej nie mozesz modyfikowac.

Narazie pracuje nad swoimi komponentami wiec wiecej nie moge powiedziec - bo jeszcze nie skonczone i nie przetestowane.

[Dandelion]

zalozmy ze za pomoca describe table wygenerowalem sobie cos takiego w php


Field Type

int(11) id
char(50) tytul
longtext tresc
datetime data_dodania
char(40) autor
int(40) id_kat

chcialbym napisac klase ktora na podstawie tych danych sama wygeneruje formularz do oblugi wybranej tabeli

troche nad tym myslalem i zastanawia mnie obsluga pol

przyklad:

longtext tresc

jak zaprogramowac klase zeby wiedziala ze polu longtext w formularzu odpowiada textarea

Uzywanie if do kazdego typu pola w bazie danych mija sie z celem dlatego szukam jakiejs alternetywy

Ten post edytował Dandelion 2.01.2007, 23:39:57

[NuLL]

szukam jakiejs alternetywy

switch ? (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

[Ace]

Chyba troszke sie ograniczasz...

Pole int jak odwzorujesz? Bo moze to byc select box z wybraniem np: id kategorii, badz po prostu ilosc (textbox)

Lepiej chyba sam stworzyc definicje ze pole takie typ - taki, pole inne typ - inny...