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?

[matix]

Z nudów, napisałem sobie klasę, opierając się na tym, co napisał @Seth:

[PHP] pobierz, plaintext 
<?
  interface Inputs {
	  public function __toString();
  }
 
  class Input implements Inputs {
	  public function __construct($type, $name, $value = '')
	  {
		  $this->sInput = sprintf('<input type="%s" name="%s" value="%s"/>', $type, $name, $value);
	  }
 
	  public function __toString()
	  {
		  return $this->sInput;
	  }
  }
 
  class form {
 
	  private $sResult = '';
 
	  function add(Inputs $input = null)
	  {
		  $this->sResult .= $input.$this->sLimit;
		  return $this;
	  }
 
	  function __toString()
	  {
		  return $this->sResult;
	  }
 
	  function setLimit($string)
	  {
		  $this->sLimit = $string;
	  }
 
  }
 
  $f = new form;
 
  $f->setLimit("<br/>\n");
 
  $f
	  ->add (new Input('text', 'login'))
	  ->add (new Input('text', 'password'))
	  ->add (new Input('text', 'mail'))
	  ->add (new Input('submit', 'send', 'sendit'));
 
  echo $f;
 
  ?>
[PHP] pobierz, plaintext