[klasa] Walidacja pola select w PHP Opcje

[wlodziu]

Witam, chciałbym przedstawić moją klasę na walidacje pola select.

Chodzi o to, by w naszym polu select nie została wpisana wartość, której nie chcemy, oczywiście możecie to sprawdzić za pomocą kilki warunków (czym dłuższy select tym dłuższy warunek)
Na potrzeby swojego skryptu napisałem klasę, która waliduje nasz select.

Jak z niej korzystać?
Dołączamy plik selectvalidate.class.php

[PHP] pobierz, plaintext 
require_once('selectvalidate.class.php');
[PHP] pobierz, plaintext 

Następnie tworzymy obiekt z nazwą selecta, ja nazwałem wojewodztwo:

[PHP] pobierz, plaintext 
$pole = new Select('wojewodztwo'); 
[PHP] pobierz, plaintext 

Dodajemy do niego optiony metodą:

[PHP] pobierz, plaintext 
$pole->AddOption('mazowieckie', 'maz'); // dodajemy optiony: nazwa | wartosc
$pole->AddOption('lodzkie', 'lodz'); // jw
$pole->AddOption('slaskie', 'sl'); // jw
$pole->AddOption('pomorskie', 'pom'); // jw
[PHP] pobierz, plaintext 

Następnie wyświetlamy nasz select w formularzu:

[PHP] pobierz, plaintext 
$pole->showSelect()
[PHP] pobierz, plaintext 

Jeśli chcemy go sprawdzić, na poczatku pobieramy wartośc z tego selecta o nazwie wojewodztwo:

[PHP] pobierz, plaintext 
$select = $_POST['wojewodztwo'];
[PHP] pobierz, plaintext 

I sprawdzamy, czy wartośc jest poprawna:

[PHP] pobierz, plaintext 
if ($pole->validateSelect($select)) { // sprawdzamy nasz select
		echo 'wykonujemy dalsza czesc skryptu.';
	} else {
		echo 'Ojj, mamy blad, ktos probuje wpisac inna wartosc';
	}
[PHP] pobierz, plaintext 

selectvalidate.class.php:

[PHP] pobierz, plaintext 
<?php
/*
Author: wlodziu
Date: 17/02/2013
Contact: wlodziuss@wp.pl
*/
	class Select {
		public function __construct ($name) {
			$this->name = $name;
		}
		public function AddOption ($text, $value) {
			$this->option[] = $text;
			$this->value[] = $value;
		}
		public function showSelect () {
			$count = count($this->option);
			$string = '<select name="'.$this->name.'">';
			for ($i = 0; $i <= $count - 1; $i++) {
				$string .= '<option value="'.$this->value[$i].'">'.$this->option[$i].'</option>';
			}
			$string .= '</select>';
			return $string;
		}
		public function validateSelect ($val) {
			$count = count($this->option);
			$x = false;
			for ($i = 0; $i <= $count - 1; $i++) {
				if ($this->value[$i] == $val) {
					$x = true;
				}
			}
			return $x;
		}
	}
?>
[PHP] pobierz, plaintext 

Przykład zastosowania:

[PHP] pobierz, plaintext 
<?php
require_once('selectvalidate.class.php'); // dolaczamy plik z klasą
$pole = new Select('wojewodztwo'); // tworzymy obiekt, z parametrem nazwy selecta
$pole->AddOption('mazowieckie', 'maz'); // dodajemy optiony: nazwa | wartosc
$pole->AddOption('lodzkie', 'lodz'); // jw
$pole->AddOption('slaskie', 'sl'); // jw
$pole->AddOption('pomorskie', 'pom'); // jw
if ($_POST['submit']) {
	$select = $_POST['wojewodztwo'];
	if ($pole->validateSelect($select)) { // sprawdzamy nasz select
		echo 'wykonujemy dalsza czesc skryptu.';
	} else {
		echo 'Ojj, mamy blad, ktos probuje wpisac inna wartosc';
	}
} else {
	// wyswietlamy nasz utworzony select: $pole->showSelect()
	echo '
	<form method="post" action="">
		'.$pole->showSelect().'
		<input type="submit" name="submit" value="OK">
	</form>
	';
}
 
?>
[PHP] pobierz, plaintext 

Jest to moja pierwsza zabawka w php obiektowym, prosiłbym o oceny

Ten post edytował wlodziu 17.02.2013, 09:54:59

[CuteOne]

[PHP] pobierz, plaintext 
 
$array = array(
   'maz' => 'mazowieckie',
   'pom' => 'pomorskie',
   'war' => 'warmińsko-mazurskie'
);
 
echo '<select name="select">';
         foreach($array as $key => $value)  echo '<option value="'.$key.'">'.$value.'</option>';
echo '</select>';
 
if(!empty($_POST)) {
 
     if(!array_key_exists($_POST['select'], $array)) {
 
            echo 'błąd';
     }
}
[PHP] pobierz, plaintext 

Ta klasa to żadna walidacja.
- brak obsługi grup
- brak obsługi multiple
- brak ustawienia domyślnego "selected"
- jak już sam wspomniałeś im więcej wartości tym więcej problemów. Wytłumacz mi w jaki sposób mam dodać 50, 150 lub 300 opcji za pomocą tego:

[PHP] pobierz, plaintext 
$pole->AddOption('mazowieckie', 'maz'); // dodajemy optiony: nazwa | wartosc
$pole->AddOption('lodzkie', 'lodz'); // jw
$pole->AddOption('slaskie', 'sl'); // jw
$pole->AddOption('pomorskie', 'pom'); // jw
[PHP] pobierz, plaintext 

?
- sama walidacja to nieporozumienie

[wlodziu]

A jak dodasz w swoim kodzie 50 wpisów? Tak samo jak w moim? Czy użyjesz magii?

- brak ustawienia domyślnego "selected"

zobacz swój kod.

Nie chodzi mi tutaj o dodawanie wpisów w inny sposób niż z palca, a do tego przeczytaj końcówkę mojego posta.

[phpion]

Generalnie zgadzam się z CuteOne, ale skoro koniecznie chcesz opinii to:

1. Wszystkie składowe są publiczne. Nie deklarujesz ich jako protected/private więc domyślnie są publiczne.
2. Po co Ci 2 składowe option i value? Możesz to przecież zrobić na 1 tablicy.
3. Masz metodę dodawania opcji, a gdzie metoda usuwania?
4. Metoda showSelect nie pozwala na wstawienie atrybutów HTML select'a. Nie można nadać mu np. klasy albo id.
5. $i <= $count - 1 jest w sumie bez sensu. To samo osiągniesz poprzez $i < $count.
6. W validateSelect robisz zbędne iteracje pętli. Jeśli dany element istnieje to od razu zwróć TRUE, a nie przeszukuj kolejnych elementów.
7. W validateSelect wystarczyłoby skorzystać z in_array(). Jeśli option i value przerobiłbyś na 1 tablicę to wystarczyłoby isset / array_key_exists.

[CuteOne]

A jak dodasz w swoim kodzie 50 wpisów? Tak samo jak w moim? Czy użyjesz magii?

[PHP] pobierz, plaintext 
public function __construct ($name, $options=array()) {
	$this->name = $name;
 
        if(!empty($options)) {
 
              $this->setOptions($options);
        }
}
 
protected function setOptions($options) {
 
      if(!empty($options['multioptions'])) {
 
             $this->addOptions($options['multioptions']);
      }
 
      if(!empty($options['default_value'])) {
 
             $this->setDefaultValue($options['default_value']);
      }
}
 
public function addOptions($options) {
 
      if(!is_array($options)) throw new MyException('Not array blablabla'); // jak nie array to won
 
      if(!empty($options)) {
 
             foreach($options as $k => $v) {
 
                   if($this->hasValue($k) && $this->allowOverwrite() === false) { // zabezpieczenia przeciw nadpisywaniu danych
                         throw new MyException('Not array blablabla');
                   }
 
                   $this->addOption($k, $v); // tu twoja funkcja
             }
      }
 
      return $this;
}
 
 
$options = array(
     'multioptions' => array(
           'maz' => 'mazowieckie',
           'pom' => 'pomorskie',
           'war' => 'warmińsko-mazurskie'
     ),
     'default_value' => 'pom'
);
 
 
$select = new Select('moj_select', $options);
 
//lub
 
$select -> addOptions($options['multioptions']);
[PHP] pobierz, plaintext 

[wlodziu]

I tak czy siak mussiz przypisać je do tablicy, więc możesz także powielać mój kod i wpisywać, sytuacja moze się zmienić, jeśli chcesz to użyć jeszcze dalej, ale to mozna dopisać odpowiednią metodę i nie będzie problemu

[CuteOne]

[PHP] pobierz, plaintext 
$result = $pdo -> fetchAll();
 
$select = new Select('select', array('multioptions'=>$result));
[PHP] pobierz, plaintext 

ps. zobacz sobie do Zenda 1.xx jak tam rozwiązano walidację elementów formularzy i bierz z nich przykład (chociażby po to aby poszerzyć swoją wiedzę)