Jestem świeży w OOP, dlatego chętnie usłyszałbym kilka porad na temat tego co stworzyłem (licze się z tym, że może to być totalnie błędne).

Ponadto próbuje stworzyć do tej "samopiszącej się formy" validacje, która sama się będzie pisać. Jakieś porady :/ ?

[PHP] pobierz, plaintext 
define( 'PHP_SELF', $_SERVER['REQUEST_URI'] );
 
class FormCreator
{	
	var $result;	
	public function addfield ($type, $vars) {
			$this->$type($vars);
	}	
    public function __construct()
    {
		$this->result = '';       
    } 
    public function __destruct()    {       
    }	
	private function startForm ($formname){
		$this->result .= '<form action="'.PHP_SELF.'" method="post" id="'.$formname.'" class="form" enctype="multipart/form-data">
				<input type="hidden" name="'.$formname.'" value="1">';	
	}
	private function endForm ($formname){
		$this->result .= '</form>';
	}	
	private function input($vars){	
		$this->result .= '<input type="'.$vars['type'].'" class="'.$vars['style'].'" name="'.$vars['name'].'" id="'.($vars['id'] ? $vars['id'] : $vars['name']).'" data-req="'.$vars['required'].'">';
	}	
	private function select($vars) {		
		$loopvar = count ($vars['options']);		
		array_unshift($vars['options'], $loopvar);		
		$this->result .= '<select name="'.$vars['name'].'" id="'.$vars['name'].'" data-req="'.$vars['required'].'">';		
		for ($i=1;$i<=$vars['options'][0];$i++){
			$this->result .= '<option>'.$vars['options'][$i].'</option>';
		}		
		'</select>';		
	}
	private function label_start($vars){		
		$this->result .= '<label for="'.$vars['for'].'">'.$vars['s_text'].'';
	}
	private function label_end($vars){			
		$this->result .= $vars['e_text'].'</label>';
	}	
	private function button($vars){			
		$this->result .= '<input type="submit" value="'.$vars.'">';
	}	
	public function getResult(){	
		return $this->result;		
	}
 
}
 
 
 
 
// Create a new object
$obj = new FormCreator("test");
$input1 = array ( type => 'email', style => 'test', required=> 'yes', name => 'c_mail' );
$input2 = array ( type => 'radio', style => 'test', required=> 'yes', name => 'radion1', id=> 'r1' );
$select1 = array ( options => array("test", "test2"), style => 'test', required=> 'no', name => 'c_select' );
$label1['s_text'] = 'Some text';
$label1['for'] = 'r1';
 
$obj -> addfield("startForm", "autoform");
$obj -> addfield("input", $input1);
$obj -> addfield("label_start", $label1);
$obj -> addfield("input", $input2);
$obj -> addfield("label_end", $label1);
$obj -> addfield("select", $select1);
$obj -> addfield("button", "Send Me Now");
$obj -> addfield("label_start", array(s_text => 'checkbutton'));
$obj -> addfield("input", array(type=>"checkbox",required=>"no", name=> "somecheck"));
$obj -> addfield("label_end", array());
$obj -> addfield("endForm", "");
echo $obj->getResult();
[PHP] pobierz, plaintext 

Brakuje zmiany form method post na get, dodania kolejnych atrybutów, zmiany klasy, url w action
Począwszy od linii 54, indeksy w tablicach też się bierze w ciapki

Nie ma czegoś tak podstawowego, jak odbieranie danych.

var $result - mamy juz XXI wiek i slowka VAR naprawde są przezytkiem minionej epoki

Każde pole powinno być oddzielną klasą.
Button nie ma nazwy

itd itd

Z OOP nie ma to wiekszego związku. Ot zebranie paru funkcji w jednej klasie. Przenosze

Jak mówi dobre internetowe przysłowie - "You're doing it wrong" - szczerze i bez ogródek

Twój kod do OOP ma się bardzo mało - owszem jest w nim 1 obiekt... ale to nie jest OOP.

Nie będę analizował kodu ale podpowiem Ci jak to powinno wyglądać (idea):

- pierwszy obiekt to obiekt formularza, w którym możesz zmieniać jego parametry (action, method itp.), własności i zarządzać jego elementami (dodawać, usuwać, odnosić się)

- drugi obiekt to ogólny obiekt elementu formularza zawierający podstawowe metody i własności (np: wyświetl, waliduj itp)

- kolejne obiekty to obiekty konkretnych typów pól formularza (text, select, radio, radio-group, label, plaintext itp), powinny one dziedziczyć z ogólnego obiektu elementu formularza i uzupełniać go o specyficzne dla siebie opcje

Tak zbudowany system formularza będzie już bliżej OOP. Daje dużo plusów i możliwości. Uzywanie polegało by na utworzeniu nowego formularza, każdego z pól osobno i dodaniu ich do formularza w odpowiedniej kolejności. Co więcej możesz każde pole wyposażyć w opcje przypisywania wartości a następnie formularz w przypisywanie wartości jego polom - wówczas możesz uzyskać fajną edycję danych

Pomyśl o tym