[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 (IMG:style_emoticons/default/smile.gif)

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

[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.