Forum PHP.pl

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?