Proste Tworzenie formularza Opcje

[zietas]

Raczkuje w OOP.
Napisałem nasępujący powiedzmy sobie szczerze 'bardzo prosty generator formularza'.

[PHP] pobierz, plaintext 
<?php
class FormGenerator
{
 
public function formOpen($action,$file)
	{
		if($file)
			{
				return $formStart = '<form method="post" action="'.$action.'" enctype="multipart/form-data">';
			}
		else
			{
				return $formStart = '<form method="post" action="'.$action.'">';
			}
	}
 
public function formInput($label,$type,$name)
	{
		return $input = ' <label for="'.$name.'">'.$label.'</label>: <input type="'.$type.'" name="'.$name.'" id="'.$name.'" />';
	}
 
public function formTextarea($label,$name,$char_width,$num_lines)
	{
		return $textarea = '<label for="'.$name.'">'.$label.'</label>: <textarea name="'.$name.'" cols="'.$char_width.'" rows="'.$num_lines.'" id="'.$name.'"></textarea>';
	}
 
public function formSelect($label,$name,$value,$option)
	{
		return $select = '<label for="'.$name.'">'.$label.'</label>: <select name="'.$name.'" id="'.$name.'"><option value="'.$value.'">'.$option.'</option></select>';
	}
 
public function formBtn($type,$value,$close)
	{
		if($close)
			{
				return $input = '<input type="'.$type.'" value="'.$value.'" /></form>';			
			}
		else
			{
				return $input = '<input type="'.$type.'" value="'.$value.'" />';
			}
	}
}
?>
[PHP] pobierz, plaintext 

No i moj formularz tworze nastepujaco:

[PHP] pobierz, plaintext 
<?php
require_once('class.FormGenerator.php');
 
$form = new FormGenerator();
 
			echo $form->formOpen('contact.php',false);
			echo $form->formInput('Pierwszy','text','pierwszy_text');
			echo $form->formInput('Drugi','text','drugi_text');
			echo $form->formBtn('submit','Send',true);
?>
[PHP] pobierz, plaintext 

Czy jest to w miare poprawne?
Czy musze inaczej do tego podejsc? Jezeli tak, to jak?

[zietas]

Można by dodać do tej klasy jeszcze labele i np. paragrafy <p> zamiast tego setLimit() (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .

to tam jest najmniejszy problem. Z dodaniem klas itd.

Problem tkwi w napisaniu tego poprawnie w OOP (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ja sobie powoli cos 'powaznego' z tego tworze (wykorzystujac np. labele, css).


Dzieki Seth i matrix za pomoc.