[skrypt]generowanie formularzy Opcje

[AcidBurnt]

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

Ten post edytował AcidBurnt 19.05.2006, 16:48:40

[nospor]

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

[AcidBurnt]

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

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

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

tzn bo nie rozumem?

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

pewnie wyszlo by w praniu masz racje mozna dodac

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

jak Twoim zdaniem bylo by wygodnie?

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

zgodze sie, mozna ta czynnosc troche zautomatyzowac

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

poszukaj, wszelkie uwagi mile widziane

cenne uwagi to negatywne uwagi zaraz bedziemy coś probowac poprawic

[nospor]

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