witam

z recji, że pisze pewien projekcik w ktory bede musial stworzyc kilka duuuzych formularzy napisalem pewna klase do ich generowania:

http://nie.bbgroup.pl/test/

na dole stronki

index.php - czyli wywolanie klasy i przygotowanie formularza:

[PHP] pobierz, plaintext 
<?php
require_once('functions/functions.php');
 
function puste($t){
	return empty($t);
}
 
 
$smarty = new Smarty;
$smarty -> template_dir = './tpl';
$smarty -> compile_dir = './tpl_c';
$smarty -> cache_dir = './tpl_cache';
$smarty -> config_dir = './tpl_configs';
 
$form = new form('formularz_testowy', 'post', true, $_POST);
/* rozpoczynanie formularza */
$formularz_testowy[begin] = $form -> formBegin();
 
/* generowanie pola typu input */
$formularz_testowy[login] = $form -> genInput('puste','text', 'login', $_POST[formularz_testowy][login], 'test', 'Pole Login nie mo�e by� puste', 'a tutaj na przyklad jakies javascriptowegowienko');
/* za pomoca genInput generujemy tez pole typu checbox ;) */
$formularz_testowy[checked] = $form -> genInput('puste','checkbox', 'sex', $_POST[formularz_testowy][sex], '', 'teeeeeeeeeeeeeeerreeeeeeee', '');
$formularz_testowy[wyslij] = $form -> genInput('', 'submit', 'go', 'wy�lij', '');
 
/* generowanie pola typu texarea 
w warto�� $line[4] podajemy np <p>%</p> wtedy tam gdzie % zostanie wstawione pole <input type="radio" .....
taka o dekoracyjna funkcyjka jakby trzeba bylo dodac jakies <br> albo <p> przy radio  dobry jestem co nie ;p
*/
 
$formularz_testowy[tresc] = $form -> genTextarea('puste', 'tresc', $_POST[formularz_testowy][tresc], 'as', 'Tre�� nie mo�e by� pusta', '');
/* generowanie pola typu radio */
/* 0 - value, 1 - class, 2 - add(jakas java), 3 - checked, 4 - decorate */
 
$items = array();
$items[] = array('pierwsza', 'class', '', 'jakis java', '%<br />');
$items[] = array('drua', 'aasd', '', 'checked', '%<hr>');
$items[] = array('pierwsza', 'asd', '');
$formularz_testowy[radio] = $form -> genRadio('', 'radio', $items, 'wiadomosc bledu');
 
/* generowanie pola typu select */
/* 0 - value, 1 - tresc, 2 - class , 3 - add(jakas java), 4 - selected */
 
$items = array();
$items[] = array('', '-= wybierz =-', '', '');
$items[] = array('1', 'tresc', 'asd','', 'selected');
$items[] = array('2', 'tresc 1', '');
$items[] = array('3', 'tresc 2', '');
$formularz_testowy[select] = $form -> genSelect('puste', 'select', 'class', $items, 'wiadomosc bledu');
 
/* przes�anie ewentualnych b��d�w */
$formularz_testowy[errors] = $form -> errorMsg;
 
/* musi byc wywolane jako ostatnie */
$formularz_testowy[end] = $form -> formEnd();
 
if($form -> parse){
	echo "formulaz przeszed� test prawid�owo";
}
else{
	$smarty -> assign('formularz_testowy', $formularz_testowy);	
}
 
$smarty -> display('index.tpl');
?>
[PHP] pobierz, plaintext 

form.class.php - klasa formularza

[PHP] pobierz, plaintext 
<?php 
class form{
	var $form_id;
	var $method;
	var $debug;
	var $data;
	var $errorMsg = array();
 
	var $parse = false;
 
	function __construct($form_id, $method, $debug, $data = array()){
		$this -> form_id = $form_id;
		$this -> method = $method;
		$this -> debug = $debug;
 
		$this -> data = $data;
	}
 
	function formBegin($class =''){
		$form_begin = '<form id="'.$this -> form_id.'" method="'.$this -> method.'" '.($class ? 'class="'.$class.'"' : '').'>';
		return $form_begin;
	}
 
	function genInput($checkFunction = '',$type, $name, $value = '', $class = '', $error_msg= '', $add =''){
		if($this -> debug && !empty($this -> data) && !empty($checkFunction)){
			if($checkFunction($this -> data[$this -> form_id][$name])){
				$this -> errorMsg[] = $error_msg;
				$error = 'style="border: 1px solid red"';
			}
		}
		if($type == 'checkbox'){
			$input = '<input type="'.$type.'" name="'.$this -> form_id.'['.$name.']" '.($value == 'on' ? 'checked' : '').' '.($class ? 'class="'.$class.'"' : '').' '.$add.' '.$error.'/>';
		}
		else{
			$input = '<input type="'.$type.'" name="'.$this -> form_id.'['.$name.']" '.($value ? 'value="'.$value.'"' : '').''.($class ? 'class="'.$class.'"' : '').' '.$add.' '.$error.'/>';
		}
		return $input;
	}
 
	function genTextarea($checkFunction = '', $name, $value ='', $class ='', $error_msg = '', $add = ''){
 
		if($this -> debug && !empty($this -> data) && !empty($checkFunction)){
			if($checkFunction($this -> data[$this -> form_id][$name])){
				$this -> errorMsg[] = $error_msg;
				$error = 'style="border: 1px solid red"';
			}
		}
		$textarea = '<textarea name="'.$this -> form_id.'['.$name.']" '.($class ? 'class="'.$class.'"' : '').' '.$add.' '.$error.'>'.($value ? ''.$value.'' : '').'</textarea>';
		return $textarea;
	}
 
	function genRadio($checkFunction = '', $name,$items , $error_msg = ''){
		if($this -> debug && !empty($this -> data) && !empty($checkFunction)){
			if($checkFunction($this -> data[$this -> form_id][$name])){
				$this -> errorMsg[] = $error_msg;
				$error = 'style="border: 1px solid red"';
			}
		}
		$radio = '';
		foreach ($items as $a => $lines){
			if(empty($lines[4])){
				$radio .= '<input type="radio" name="'.$this -> form_id.'['.$name.']" value="'.$lines[0].'" '.(!empty($lines[1]) ? 'class="'.$lines[1].'"' : 'as').' '.$lines[3].' '.$error.'/>';
			}
			else {
				$radio .= str_replace('%','<input type="radio" name="'.$this -> form_id.'['.$name.']" value="'.$lines[0].'" '.($lines[1] ? 'class="'.$lines[1].'"' : '').' '.$lines[3].' '.$error.'/>', $lines[4]);
			}
		}
		return $radio;
	}
 
	function genSelect($checkFunction,$name, $class,$items, $error_msg = ''){
		if($this -> debug && !empty($this -> data) && !empty($checkFunction)){
			if($checkFunction($this -> data[$this -> form_id][$name])){
				$this -> errorMsg[] = $error_msg;
				$error = 'style="border: 1px solid red"';
			}
		}
		$select = '<select name="'.$this -> form_id.'['.$name.']"  '.$error.'>';
		foreach ($items as $a => $lines){
			$select .= '<option value="'.$lines[0].'" '.($lines[0] ? 'class="'.$lines[0].'"' : '').' '.$lines[3].' '.$lines[4].'>'.$lines[1].'</option>';
		}
		$select .= '</select>';
		return $select;
	}
 
	function formEnd(){
		$form_end = '</form>';
		if(empty($this -> errorMsg) && !empty($this -> data)){
			$this -> parse = true;
		}
		return $form_end;
	}
}
?>
[PHP] pobierz, plaintext 

index.tpl - czyli umieszczenie formularza na www

[HTML] pobierz, plaintext 
{if $formularz_testowy}
{$formularz_testowy.begin}
<table>
      {if $formularz_testowy.errors}
      <tr>
            <td colspan="2">
                  {section name=a loop=$formularz_testowy.errors}
                        <p style="color:red">{$formularz_testowy.errors[a]}</p>
                  {/section}
            </td>
      </tr>
      {/if}
            <tr>
                  <td>Login:</td>
                  <td>{$formularz_testowy.login}</td>
 
            <tr>
                  <td>tresc</td>
                  <td>{$formularz_testowy.tresc}</td>
            </tr>
            <tr>
                  <td></td>
                  <td>{$formularz_testowy.wyslij}</td>
            </tr>
            <tr>
                  <td>wybierz</td>
                  <td>{$formularz_testowy.radio}</td>
            </tr>
            <tr>
                  <td>wybierz</td>
                  <td>{$formularz_testowy.select}</td>
            </tr>
            <tr>
                  <td></td>
                  <td>{$formularz_testowy.checked}</td>
            </tr>
      </table>
      {$formularz_testowy.end}
{/if}
[HTML] pobierz, plaintext 

wszelkie opinie mile widziane

same minusy:

1)Ten formularz generuje mase noticów. CZemu? ano temu:
$_POST[formularz_testowy][login]
powinno byc:
$_POST['formularz_testowy']['login']
i reszta podobnie

2) Uzytkownik musi sam zadbac o zapamietanie wartosci po wyslaniu forma. To powinien robic juz formularz, a usera nie powinno to obchodzic

3) Przyjales, ze pole checkbox nie bedzie przyjmowalo zadnych wartosci. A co jesli jednak okaze sie, ze będą jakies wartosci?

4) deklarowanie wszystkich pol jako tablica, ktorej glownym indexem jest nazwa fomularza. to zle/zbedne rozwiazanie. pola powinno sie rozroniac glownie po id. zbieranie ich do kupy w jednej tablicy jest zbedne i nie wygodne

5) uzytkownik musi sam zapamietywac generowane pola i potem je przypisywac do szablonu. to tez powiino byc robione przez formularz.

6) i jeszcze bym cos pewnie znalazl, ale nie chce mi sie juz szukac


ocena koncowa:
niska

jak i wiele bardzo duzych projektow ;p no ale masz racje mzna sie tego bez problemu pozbyc



tzn bo nie rozumem?



pewnie wyszlo by w praniu masz racje mozna dodac



jak Twoim zdaniem bylo by wygodnie?



zgodze sie, mozna ta czynnosc troche zautomatyzowac



poszukaj, wszelkie uwagi mile widziane

cenne uwagi to negatywne uwagi zaraz bedziemy coś probowac poprawic

ad2)
NO chodzi oto, ze user musi sam wpisywac wartosc z posta, przy generowaniu pola. A to powinien za niego robic juz sam formularz


ad4)
Jak wygodnie? No olac robienie z pol tablicy, ktorej indexem jest nazwa fomularza. Identyfikowac pola juz tylko po ID i nadawac jej normalne nazwy anie:
name="nazwaformularza[nazwapola]"
powinno byc:
...name="nazwapola" id="idpola" ....

7)
Pole select nie pamieta co zaznaczono
Pozatym brak jest mozliwosci stworzenia multiselecta, co w zasadzie wynika z braku pamietania co zaznaczono

8)zaznaczenie radia tez nie jest pamietane