[PHP] pobierz, plaintext 
$error = array();
global $error;
 
function validateNames($value) {
  if(!ereg("^[A-ZŁŚ]{1}[a-ząęółśżźćń]{1,30} [A-ZŁŚ]{1}[a-ząęółśżźćń]{1,30}$", $value) && strlen($value) < 3)
		return $error[] = 1;
}
 
function validateEmail($value) {
  if(filter_var($value, FILTER_VALIDATE_EMAIL) === FALSE && strlen($value) == 0)
		return $error[] = 1;
}
 
function validateSubject($value) {
  if(empty($value))
		return $error[] = 1;
}
 
function validateContent($value, $min) {
	if(empty($min)) $min = 10;
  if(strlen($value) < $min)
		return $error[] = 1;
}
 
function validateCaptcha($code, $security) {
  if(strtolower($code) != strtolower($security))
		return $error[] = 1;
}
 
echo validateSubject('');
echo validateContent('', '');
echo '>>>'.sizeof($error).'<<<'; // Tutaj chcę mieć ilość errorów, ale ciągle jest zero :(
[PHP] pobierz, plaintext 

Co tutaj więcej pisać ?! ...

Do każdej funkcji dodałem globalną zmienną $error i działa

Ciekawi mnie ja zwracasz napis o błędzie ,dla konkretnego pola.
Nie powinno być:
return $error[0] = 1;
return $error[1] = 1;
return $error[2] = 1; itd.?

heh
Poprzez wywołanie funkcji echem
echo funkcja(tekst);

Aktualna wersja kodu:

[PHP] pobierz, plaintext 
$error = array();
 
function validateNames($value) {
  if(!ereg("^[A-ZŁŚ]{1}[a-ząęółśżźćń]{1,30} [A-ZŁŚ]{1}[a-ząęółśżźćń]{1,30}$", $value) && strlen($value) < 3) {
		global $error;
		return $error[] = 1;
	}
}
 
function validateEmail($value) {
  if(filter_var($value, FILTER_VALIDATE_EMAIL) === FALSE && strlen($value) == 0) {
		global $error;
		return $error[] = 1;
	}
}
 
function validateSubject($value) {
  if(empty($value)) {
		global $error;
		return $error[] = "puste pole #1";
	}
}
 
function validateContent($value, $min) {
	if(empty($min)) $min = 10;
  if(strlen($value) < $min) {
		global $error;
		return $error[] = "puste pole #2";
	}
}
 
function validateCaptcha($code, $security) {
  if(strtolower($code) != strtolower($security)) {
		global $error;
		return $error[] = 1;
	}
}
[PHP] pobierz, plaintext 

Gra i buczy
jeżeli chcę sprawdzić poprawność pola formularza/tekstu to wywołuję funkcję np.

[PHP] pobierz, plaintext 
echo validateSubject($_POST['subject'])
[PHP] pobierz, plaintext 

jeżeli pole jest błędnie wypełnione to dostaję treść błędu

Dramat... Płacz i zgrzytanie zębów... Dur brzuszny i gorączka krwotoczna...


eregi się nie używa, ta funkcja nie będzie za chwile wspierana, w nowszych wersjach php jej użycie produkuje warning..
Ale to jeszcze nic, najgorsze jest to że walisz w skrypcie znakami które nie są ASCII. To nie ma prawa zadziałać, poczytaj sobie trochę o kodowaniu, o unicode, o stronach kodowych, może zrozumiesz w czym problem.


Global też się raczej nie używa, a na pewno nie w ten sposób.

Może źle mi się wydaje, ale ta wiadomość tekstowa o błędzie jest prawdopodobnie jednolita dla każdego pola.

ereg sie juz nie uzywa, od tego jest preg_match

Zacznij używać klas, stosowanie globali nie jest wskazane. Stwórz sobie na początek klase (pisane z palca):

[PHP] pobierz, plaintext 
class Validate
{
 
prrivate $errors = array(); // tablica na błędy
 
// tutaj te Twoje funkcje przerobione, np:
public function validateSubject($value) {
if(empty($value))
{
$this->setError('subject', 'wartość nie może być pusta')
return false;
}
return true;
}
 
public function setError($name, $msg) // ustaw error
{
$this->errors[$name] = $msg; 
}
public function getErrors() 
{
return $this->errors; 
}
public function countErrors() // ile errorów
{
return count($this->errors); 
}
}
[PHP] pobierz, plaintext 

Oczywiście taka klasa jest prymitywna, ale z czasem sobie ją rozbudujesz albo jeszcze lepiej - napiszesz od nowa z zupełnie innym podejściem (bardzo wskazane).

No fajnie
Dzięki za uwagi
A jak byście w takim razie ułożyli warunek preg_match() dla walidacji imienia i nazwiska by przepuszczała tylko taką formę "Imię Nazwisko"
Czyli pierwsza litera imienia i nazwiska musi być duża, reszta mała + spacja...
Tak jak mam to zrobione w swoim kodzie z eregami

http://php.net/manual/pl/intro.pcre.php
www.bitcetera.com/page_attachments/0000/0030/regex_in_a_nutshell.pdf
pluton.pol.lublin.pl/~beatap/PHP/PHP_w3.pdf

Dzięki ale już dawno sobie z tym poradziłem:

[PHP] pobierz, plaintext 
preg_match("/^[A-ZŁŚ]{1}[a-ząęółśżźćń]{1,30}[ ][A-ZŁŚ]{1}[a-ząęółśżźćń]{1,30}$/", $value)
[PHP] pobierz, plaintext 

Może ten sposób nie należy do 'najładniejszych' ale przynajmniej jest skuteczny
Dzięki wszystkim za odpowiedzi,
pozdrawiam.

_____
EDIT:
Zasłużone pomógł poleciały

Właśnie że nie jest skuteczny. Przestanie działać któregoś dnia bez ostrzeżenia.