[php]programowanie obiektowe (czy jest dobrze), na początek system formularzy Opcje

[Lejto]

Postanowiłem się w końcu zacząć uczyć programowania obiektowego. (IMG:http://forum.php.pl/style_emoticons/default/guitar.gif) Na początek napisałem skrypt który graweruje formularze. Czy kod jest optymalny? Co w nim jest niepotrzebne a co zmienić?

[PHP] pobierz, plaintext 
<?
class mech_form
{
 
	function utworz($action, $nazwa, $metoda)
	{
		echo '<form action="'.$action.'" name="'.$nazwa.'" method="'.$metoda.'">';
	}
 
	function utworzPoleInput($type, $size, $value)
	{
		$a = new tworzHTML();
		$a->PoleInput($type, $size, $value);		
	 }	  
 
}
 
class tworzHTML extends mech_form 
{
 
	function PoleInput($type, $size, $value)
	{
		echo $nazwa.'<input type="'.$type.'" size="'.$size.'" value="'.$value.'">'; 
	}
 
}
 
$mech_form = new mech_form();
   
 
?><br />
<?
	$formularz = $mech_form->utworz('formularz', 'Nazwa', 'post');
	?>
	<table>
		<tr>
			<td>Nick: </td>
			<td><? $mech_form->utworzPoleInput('text', '3', 'ELO'); ?>
			</td>
		</tr>
	</table>
	</form>
[PHP] pobierz, plaintext 

[phpion]

Moim zdaniem tego typu metody lepiej deklarować jako statyczne. Jeśli chciałbyś sobie podejrzeć inne rozwiązania to możesz zobaczyć te, które są wbudowane we framework Kohana:
HTML
Form
Proste, a przy tym cholernie użyteczne i przydatne (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Sedziwoj]

Moim zdaniem tego typu metody lepiej deklarować jako statyczne. Jeśli chciałbyś sobie podejrzeć inne rozwiązania to możesz zobaczyć te, które są wbudowane we framework Kohana:
HTML
Form
Proste, a przy tym cholernie użyteczne i przydatne (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Może i proste, ale nie przydatne, nawet warstwa widoku nie jest wydzielona. Do tego to kod proceduralny zamknięty w obiekcie.
Tak aby nie być gołosłownym, to mamy obiekt formularza, który ma pola, każde pole ma swój typ, nazwę, walidację...
To było by obiektowe, a nie takie coś.

EDIT literówka

Ten post edytował Sedziwoj 27.04.2008, 22:47:40