Witam.
Mam formularz wysyłany za pomocą skryptu.php na mejla. Wszystko działało ok.
W ostatnich tygodniach coraz częściej zamiast wypełnionego formularza przychodzi formularz całkowicie pusty tj. wszystkie pola są puste a wiem od ludzi wysyłających formularz, że został wypełniony. Są tam checkobx-y i input-y.
Żeby móc wysłać formularz trzeba wypełnić wymagane pola (required) więc teoretycznie bez tego nie powinien zostać wysłany a dochodzi.
Na 8 wypełnionych prawidłowo przychodzą 2 całkiem puste.
Gdzie może leżeć tego przyczyna?
Pozdrawiam

Wszystkie przychodza puste zawsze ?

Problem dotyczy około 15-20% formularzy, które docierają do mnie na mejla. Przychodzą z pustymi polami jakby ktoś nic nie wypełnił a wysłał. Nie chodzi o to, że przychodzi pusty mejl bez niczego tylko dochodzi niewypełniony formularz typu:

Imię:
Nazwisko:
Adres:

a te poprawne są z wypełnionymi danymi adresowymi.
Nie wiem czy to możliwe, że wpisane dane giną gdzieś po drodze... Reszta formularzy dochodzi jak należy: wybrane checkbox-y zaznaczone a wymagane pola do wypełnienia (required) wypełnione prawidłowo.

Sprawdź z jakich przeglądarek są wysyłane te puste emaile.

Ale sądzę że powinieneś zweryfikować skrypt wysyłania emaila aby wykonywał się gdy formularz zawiera treści.
Pusty formularz mimo że go oskryptujesz, nadal można wysłać.

Zasadnicze pytanie - kiedy robisz walidację wprowadzonych danych? W JavaScripcie czy w PHP?

Nie mam walidacji pól Sorki, mogłem o tym wspomnieć... Jestem świadom tego, że można w te pola walnąć spacjami i wysłać: przyjdzie pusty formularz. Nawet przypuszczałem, że ktoś sobie robi żarty. Ale dziś po raz pierwszy zdałem sobie sprawę, że to nie jaja. Formularz został wypełniony, wysłany, dotarł ale niewypełniony. Nie mam kontaktu z osobą od której przyszedł ani z innymi nadawcami tych "pustych formularzy". Nie jestem w stanie sprawdzić z jakiej przeglądarki przychodzą formularze. Próbuję to wyjaśnić z Waszą pomocą.

Czy to wg. Waszej wiedzy jest teoretycznie możliwe, że problemem może być konkretna przeglądarka, wysyłanie z mobila itp. sprawy?
Wysyłam z firefoxa, opery, chrome'a: wszystko dochodzi.

Nie sądzę by to była wina przeglądarki jakkolwiek. Tylko i wyłącznie walidacja po stronie PHP - dopiero jak ją wdrożysz to możesz mieć pewność, że zrobiłeś wszystko by formularza działał poprawnie i dopiero w tedy, gdy nadal będą dochodziły puste wiadomości, zastanawiać się co jest nie tak.

Najprosciej dodac walidacje kazdego pola w php; sprawdzic czy nie jest puste i jesli nie puscic dalej.

Przeciez sam mozesz to sprawdzac nie musisz kontaktowac sie z nikim.

Będzie off top ale czy pomoże ktoś z modyfikacją skryptu? Mam rozumieć, że grzebać muszę tylko w pliku skryptu czy w pliku html też coś trzeba pozmieniać? Nie chcę szaleć, chciałbym aby sprawdzało poprawność wypełnienia tylko dwóch pól: imię i kod. Jak to zadziała to będę wiedział, że formularz przychodzi pusty z innego powodu i nie jest nim "spacjowanie" pól dla żartu.

W html mam pola input:
name="imię"
name="kod"

Mój skrypt.php wygląda tak:
<?php

$do = 'mojmejl@pocztaxxxxxxx.pl';
$imię = $_POST['imię'];
$ulica = $_POST['ulica'];
$kod = $_POST['kod'];
$miejscowosc = $_POST['miejscowosc'];
$telefon = $_POST['telefon'];
$k01 = $_POST['01'];
$k02 = $_POST['02'];
$k03 = $_POST['03'];
$k04 = $_POST['04'];
$k05 = $_POST['05'];

$tresc .= "Imię:".$imię."\n\n";
$tresc .= "Ulica: ".$ulica."\n\n";
$tresc .= "Kod: ".$kod."\n\n";
$tresc .= "Miejscowość: ".$miejscowosc."\n\n";
$tresc .= "Telefon: ".$telefon."\n\n";
$tresc .= "biały: ".$k01."\n\n";
$tresc .= "czarny: ".$k02."\n\n";
$tresc .= "niebieski: ".$k03."\n\n";
$tresc .= "zielony: ".$k04."\n\n";
$tresc .= "czerwony: ".$k05."\n\n";
@mail($do, 'ankieta', $tresc, "MIME-Version: 1.0\r\n"."Content-type: text/plain; charset=ISO-8859-2\r\n"."From: $mail\r\n");
header('Location: http://www.mojastrona.pl/dziekuję.html');
echo $tresc;
?>

Tutaj coś znalazłem ale nie mam pojęcia gdzie to powklejać i czy to w ogóle jest poprawne. Chodzi tylko o imię i kod i aby sprawdzało czy zamiast imienia i kodu ktoś nie klepnął kilka razy spacji:

function poprawneImie($imię)
{
if (!preg_match('/^[a-z]$/', $imię))
{
return false; //zwraca "false" jeśli imię jest błędne
}
return true; //zwraca "true" jeśli imię jest poprawne
}

function poprawnyKod($kod)
{
if (!preg_match('/^[0-9]{2}+-[0-9]{3}$/', $kod))
{
return false;
}
return true;
}



//czy puste:
if(empty($_POST['imię']))
{
echo "Wypełnij pole imię";
}
if(empty($_POST['kod']))
{
echo "Wypełnij pole kod";
}



if(!poprawneImie($_POST['imię']))
{
echo "Błędne imię";
}
if(!poprawnyKod($_POST['kod']))
{
echo "Błędny kod";
}


Z góry dzięki.

Mozesz sprawdzac kazde pole warunkami poprzez:

[PHP] pobierz, plaintext 
if( ($zmienna1 =! "") ||  ($zmienna2 =! "") ||  ($zmienna3 =! "") )
 
{
}
 
[PHP] pobierz, plaintext 

ale to toporne rozwiazanie. Przy kliku zmiennych ok ale jak sie ma ich kilkanascie to słabo to wygląda. Jeszcze gorzej jak dla kazdej zmiennej trzebabuy wyswietlic z osobna informacje.

. To jest szybsze i krotsze. Oczywiscie do wyswietlenia kazdego z bledów nalezaloby nieco zmienic (przyklad 2). Ten przyklad wyswietli tylko informacje ze chocby jedno pole jest puste

[PHP] pobierz, plaintext 
<?php
 
 
 
			if(!isset($_POST['submit']))
			{
 
				echo "<form action " . $_SERVER['SCRIPT_NAME'] . " method=post>";
				echo "<input type=\"text\" name=\"pole1\"><br>";
				echo "<input type=\"text\" name=\"pole2\"><br>";
				echo "<input type=\"text\" name=\"pole3\"><br>";
				echo "<input type=\"text\" name=\"pole4\"><br>";
				echo "<input type=\"submit\" name=\"submit\" value=\"dalej\"><br>";
 
 
			} else {
 
			unset($submit);
			$myPostArgs = filter_input_array(INPUT_POST);
 
			foreach($myPostArgs as $key => $value)
			{
 
				if(trim($value) == "")
				{
 
					exit('wypelnij wszystkie pola');
 
				}
 
			}
 
			}
 
 
 
?>
[PHP] pobierz, plaintext 

Tutaj przykla gdzie wyswietlaja sie kazde błędy.W tym momencie przyklad jest oparty na glownym warunku i osadzony w jednym pliku. Warunek sprawdza czy polsano zmienna $submit i jesli nie to wyswietla sie formularz a jak tak czyli jak wcisniesz przycisk wykonuje sie reszta kodu. Jelsi bedziesz mial dwa pliki z czego w jednym formularz a w drugim kod do sprawdzenia i cala resztew to trzeba zmienic kilka rzeczy. Na pewno usunac formularz ) Przyklad na dwa pliki na samym koncu

[PHP] pobierz, plaintext 
<?php
 
 
			if(!isset($_POST['submit']))
			{
 
 
 
				echo "<form action " . $_SERVER['SCRIPT_NAME'] . " method=post>";
				echo "<input type=\"text\" name=\"login\"><br>";
				echo "<input type=\"text\" name=\"haslo\"><br>";
				echo "<input type=\"text\" name=\"plec\"><br>";
				echo "<input type=\"text\" name=\"wiek\"><br>";
				echo "<input type=\"submit\" name=\"submit\"  value=\"dalej\"><br>";
 
             exit;
			} 
 
			$ArrInfo = array('login' => 'Wypelnij pole login',
				'haslo' => 'Wypelnij pole haslo',
				'plec' => 'Wypelnij pole plec',
				'wiek' => 'Wypelnij pole wiek'
				);
 
			unset($submit);
			$myPostArgs = filter_input_array(INPUT_POST);
			$return = array();
			foreach($myPostArgs as $key => $value)
			{
 
				if(trim($value) == "")
				{
 
					$return[] = $ArrInfo[$key];
 
					$match = true;
 
 
				}
 
			}
 
 
			if(isset($match))
			{
				print 'Znaleziono błędy: <br>';
 
			for($i=0; $i<count($return);  $i++)
				{
 
				print ($return[$i] . '<br>');
 
				}
			exit;
 
			} 
			print 'Tutaj reszta kodu jak pola zostaly wypelnione';
?>
[PHP] pobierz, plaintext 

Osadz jeden z tych kodów (jesli drugi to uzupełnij tablice pomocniczą kluczami i wartosciami) przed tym co masz przepuscic dalej. Dodaj name=submit do swojego submita w formularzu.

Przykład 3 na dwa pliki ((osobno z formularzem i osobno plik z resztą kodu) plik do ktorego bedzie kierowal formularz:

[PHP] pobierz, plaintext 
 
<?php
 
 
			if(!isset($_POST['submit']))
			{
 
				exit('wystąpił błąd');
 
 
			} 
 
			$ArrInfo = array('login' => 'Wypelnij pole: login',
				'haslo' => 'Wypelnij pole: haslo',
				'plec' => 'Wypelnij pole: plec',
				'wiek' => 'Wypelnij pole: wiek'
				);
 
			unset($submit);
			$myPostArgs = filter_input_array(INPUT_POST);
			$return = array();
			foreach($myPostArgs as $key => $value)
			{
 
				if(trim($value) == "")
				{
 
					$return = $ArrInfo[$key];
 
					$match = true;
 
 
				}
 
			}
 
 
			if(isset($match))
			{
				print 'Znaleziono błędy: <br>';
 
			for($i=0; $i<count($return);  $i++)
				{
 
				print ($return[$i] . '<br>');
 
				}
			exit;
 
			} 
 
			print 'Tutaj reszta kodu jak pola zostaly wypelnione';
 
 
?>
[PHP] pobierz, plaintext 

Za takie coś dostałbys z liścia gdybym z Tobą pracował.

Co Ty nie powiesz. Anulowanie wyswietlenia w tym przykladzie bledu jest niczym nadzwyczajnym. Nie pracujemy tutaj nad aplikacją dla NASA. Zluzuj poślady.

[PHP] pobierz, plaintext 
if(!isset($_POST['submit']))
[PHP] pobierz, plaintext 

zadowolony ?

Dzięki za propozycję. Ta cała walidacja trochę mnie przerasta i wolałem dotąd jej nie tykać skoro wsio działało (i w sumie działa dalej) jak należy. Ale jak trzeba... A czy to co zaproponowałeś Dublinka zabezpieczy przed wysłaniem formularza z polami "wypełnionymi" spacjami lub innymi głupotami? Stąd zainteresowałem się tymi wzorcami wyrażeń regularnych. Nie chcę się porywać z motyką na słońce, krok po kroku.

Na początek chciałem aby przy próbie wysłania za cholerę nie przepuściło formularza pustego. No to mam required przy inputach. Ale dooopa bo można pola wyspacjować i formularz pójdzie i tu przydałoby się zastosować jakieś wyrażenie regularne, choćby tylko przy kodzie pocztowym typu: pattern="^[0-9]{2}-[0-9]{3}$" stąd.

Mam nadzieję, że tylko to wystarczy aby nie dało się wysłać formularza z wpisanymi głupotami. Przy imionach, nazwiskach, miejscowościach jest tyle opcji, że głowa mała - nie chcę atakować wypełniającego formularz dziesiątkami alertów. Dlatego wystarczyłoby samo zabezpieczenie na kod. To nie ma być jakiś kosmos z zabezpieczeniami tylko próba dowiedzenia się z czego wynika poruszony na początku wątku problem.

Najlepsze jest to, ze mimo ze dales malpe, to linijke nizej i tak dajesz sprawdzanie ISSET na wyniku tej malpy... Moze naucz sie wiec pisac porzadnie i robic isset gdzie nalezy

Zacznijmy od tego ze to mozna w ogole wylaczyc/ominac wiec na czyms takim nigdy nie powinienies polegac.

ps: przenosze

Poprawione jakbys nie zauwazyl.


Skopiuj przyklad i odpal. Przyklad nie przepuszcza tzw. białych znakow" spacji i tabulacji z początku i końca lini

[PHP] pobierz, plaintext 
$submit = @$_POST['submit'];
 
			if(!isset($submit))
[PHP] pobierz, plaintext 

No wlasnie oswiec mnie co poprawiles?

Jeśli chcesz pomagać to pomagaj dobrze. Jak to mawiają: "jeśli masz zamiar zrobić coś byle jak to nie rób tego wcale."

@nospor poprawił kody wyżej.

Oswiecam

Dajcie spokój. Po cholerę te przepychanki i dogryzanie sobie? Szkoda nerwów na głupoty.

Mały update. Coś się ruszyło bo wpisałem w pliku html to: pattern="^[0-9]{2}-[0-9]{3}$" i mam teraz:
<input required="" name="kod" pattern="^[0-9]{2}-[0-9]{3}$"
Przy wypełnianiu nie pozwala przejść dalej ani wysłać formularza

@maniana juz ci pisalem ze takie zabepzieczenia sa o kant 4 liter. Kazdy moze je ominac. Zabezpieczenie masz miec przedewszystkim po stronie php

Jak ktos wylaczy js w przegladarce to wszystko przejdzie. Tylko i wylacznie sprawdzanie po stronie serwera.

@dublinka akurat te parametry nie sa obslugiwane przez js wiec wylaczenie JS nie wplynie na ich zachowanie. Nie mniej jednak jak juz mowilem mozna je bez problemu ominac/usunac. Mozna tez wyslac formularz bez przegladarki

To co masz czyli:

[PHP] pobierz, plaintext 
$do = 'mojmejl@pocztaxxxxxxx.pl'; 
$imię = $_POST['imię']; 
$ulica = $_POST['ulica']; 
$kod = $_POST['kod']; 
$miejscowosc = $_POST['miejscowosc']; 
$telefon = $_POST['telefon']; 
$k01 = $_POST['01']; 
$k02 = $_POST['02'];
$k03 = $_POST['03'];
$k04 = $_POST['04']; 
$k05 = $_POST['05']; 
 
$tresc .= "Imię:".$imię."\n\n"; 
$tresc .= "Ulica: ".$ulica."\n\n"; 
$tresc .= "Kod: ".$kod."\n\n"; 
$tresc .= "Miejscowość: ".$miejscowosc."\n\n"; 
$tresc .= "Telefon: ".$telefon."\n\n"; 
$tresc .= "biały: ".$k01."\n\n"; 
$tresc .= "czarny: ".$k02."\n\n"; 
$tresc .= "niebieski: ".$k03."\n\n"; 
$tresc .= "zielony: ".$k04."\n\n"; 
$tresc .= "czerwony: ".$k05."\n\n";
@mail($do, 'ankieta', $tresc, "MIME-Version: 1.0\r\n"."Content-type: text/plain; charset=ISO-8859-2\r\n"."From: $mail\r\n");
header('Location: [url="http://www.mojastrona.pl/dziekuję.html&#39%3b%29;"]http://www.mojastrona.pl/dziekuję.html');[/url]
echo $tresc;
[PHP] pobierz, plaintext 

Daj tam gdzie wstawilem informacje na samym dole a na samym poczatku to co ja dalem.

[PHP] pobierz, plaintext 
 
			if(!isset($_POST['submit']))
			{
 
				exit('wystąpił błąd');
 
 
			} 
 
			$ArrInfo = array('login' => 'Wypelnij pole: login',
				'haslo' => 'Wypelnij pole: haslo',
				'plec' => 'Wypelnij pole: plec',
				'wiek' => 'Wypelnij pole: wiek'
				);
 
			unset($submit);
			$myPostArgs = filter_input_array(INPUT_POST);
			$return = array();
			foreach($myPostArgs as $ParamsKey => $ParamsValue)
			{
 
				if(trim($ParamsValue) == "")
				{
 
					$return[] = $ArrInfo[$ParamsKey];
 
					$match = true;
 
 
				}
 
			}
 
 
			if(isset($match))
			{
				print 'Znaleziono błędy: <br>';
 
			for($i=0; $i<count($return);  $i++)
				{
 
				print ($return[$i] . '<br>');
 
				}
			exit;
 
			} 
 
			// teraz tu co Ty masz tylko pamietaj ze musisz pododawac do tablicy wartosci jakie masz czyli nazwa pola z formularza i wartosc jaka sie wyswietli jesli błąd. Rowniez nazwij swoj input name=submit
 
 
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$ArrInfo = array('login' => 'Wypelnij pole: login',
				'haslo' => 'Wypelnij pole: haslo',
				'plec' => 'Wypelnij pole: plec',
				'wiek' => 'Wypelnij pole: wiek'
				);
[PHP] pobierz, plaintext 

login, haslo, plec, wiek -> to sa nazwy z pol tekstowych formularza. Pozmieniaj/.dodaj swoje


Mnie ogolnie chodzilo o blokowanie przez js

Z twojego posta wynikalo, ze chodzilo rowniez o parametry, ktore podal maniana. Grzecznie wiec sprostowalem twoje slowa.

W poście #14 cytując mnie odniosłeś się tylko do "required". Wiem, że to lipa. O pattern="^[0-9]{2}-[0-9]{3}$" nie wspomniałeś więc go użyłem.

A jak dodam walidację za pomocą PHP stąd:

$sprawdzKodPocztowy = $_POST['kodpocztowy'];
if ( !preg_match('/^[0-9]{2}-?[0-9]{3}$/Du', $sprawdzKodPocztowy) ) {
print 'Wprowadzono błędny kod pocztowy';
}
?>

do istniejącego pliku php to będzie gitara? Jeszcze nie próbowałem.

Dublinka będę w tym grzebał na 100%, Twoje rady nie pójdą do kosza. A co z tego wyjdzie zobaczymy.

Mialem wymienic wszystkie parametry po kolei? Chodzilo generalnie o walidacje po stronie przegladarki - to jest lipa. Czy to zrobisz w JS czy w html to jest lipa. To conajwyzej jest mily dodatek do walidacji po stronie serwera i nic wiecej.

Jak dodasz w tej dokladnie postaci to nie. Nie blokujesz tutaj forma w zaden sposob a jedynie wyswietlasz jakis komunikat

Dobra to zabieram się za rady Dublinki. Chłopaki dzięki za wyrozumiałość i cierpliwość.

Dublinka, zrobiłem jak napisałeś. Przy próbie wysłania wywala błąd w 15 linii:
Parse error: syntax error, unexpected ''telefon'' (T_CONSTANT_ENCAPSED_STRING), expecting ')' in /mój adres/skrypt.php on line 15

Początek pliku php wygląda jak należy:

<?php

if(!isset($_POST['submit']))
{

exit('wystąpił błąd');


}

$ArrInfo = array('imie' => 'Wypelnij pole: imie',
'ulica' => 'Wypelnij pole: ulica',
'kod' => 'Wypelnij pole: kod',
'wiek' => 'Wypelnij pole: miejscowosc'
'telefon' => 'Wypelnij pole: telefon'
);

unset($submit);

Moze rzeczywiscie malo precyzyjne to nbylo


Zwroc uwage na konstrukcje tablicy. Po kazdym kluczu i jego wartosci jest przeinek. Ostatni klucz ma byc bez przecinka

W 15 linii błąd zapomniałeś dać przecinka po miejscowość

O Jezusiczku... racja, przecinek

A i jeszcze muszę zmienić w html przy każdym inpucie ("Rowniez nazwij swoj input name=submit") to:
Czyli z:

<small>imię
</small></td>
<td style="vertical-align: top;"><input
required="" name="imie"

na taką wersję:

<small>imię
</small></td>
<td style="vertical-align: top;"><input
required="" name=submit "imie"

Czy tak? Zaraz to odpalę.

Submit. Ten ktory klikasz.

[PHP] pobierz, plaintext 
<input type="submit" name="submit" value="dalej">
[PHP] pobierz, plaintext 

Nie kazde pole tylko submit

Odnośnie buttona WYŚLIJ to mam w htmlu coś takiego:

<button name="wysylam"><b>WYSLIJ</b>
</button> </div>

Jak to pozmieniać?

No to zmien na input. Jaki problem.

Wywal button i wstaw to

[PHP] pobierz, plaintext 
<input type="submit" name="submit" value="dalej">
[PHP] pobierz, plaintext 

Albo zostaw button tylko dodaj do buttona wartosc.

[PHP] pobierz, plaintext 
<button type="submit" name="submit" value="byleco">Dalej</button>
[PHP] pobierz, plaintext 

Noi i co. Działa ?

Działa piknie! Trzeba było z pracy w końcu wyjść W domu naniosłem poprawki, kilka wrzutów na ftp-a, poprawek i w końcu śmiga. Dzięki jeszcze raz za pomoc.
Trochę chamska ta strona z komunikatami o niewypełnionych polach co się pokazuje po kliku w submit-a. Cała biała jak śmierć i tylko:

Znaleziono błędy:
Wypełnij pole: imię
Wypełnij pole: miejscowość

No ale ważne, że działa.
Aha, zapomniałem: niby działa ale nadal można walnąć w pola głupoty typu: ... .. lub . . "Spacjowanie" faktycznie wyrugowane.
No ale przynajmniej będę wiedzieć, że TAJEMNICZE COŚ nie czyści mi wypełnionych pól po drodze bo coś w tych polach musi przyjść.
W kodzie pocztowym zostawiłem: pattern="^[0-9]{2}-[0-9]{3}$" to tam bzdety nie przejdą, przynajmniej tak mi się wydaje.

To sobie te część sformatuj w tagach html.

Np:

[PHP] pobierz, plaintext 
if(isset($match))
			{
				print "<body style=\"background-color: lightblue\"><div style=\"background-color: darkblue; color:white; padding: 10px\">";
				print 'Znaleziono błędy: <br>';
 
			for($i=0; $i<count($returnNames);  $i++)
				{
 
					print ("<b>" . $returnNames[$i] . '</b><br>');
 
				}
 
				print "<a style=\"color: gold\" href=java script:history.go(-1)>Wstecz</a></div></body>"; // forum dzieli string javascript na dwa czlony. Polacz to w jedno
 
			exit;
 
			} 
[PHP] pobierz, plaintext 

To ze mozna wpisac nie znaczy ze to przejdzie.




Wszystkie pola ktore przesylasz tzn wartosci tych pól mozesz swobodnie (po tym co dałem) filtrowac i sprawdzac względem wymagań jakie tam chcesz w tym adres @
Przykładowo:

[PHP] pobierz, plaintext 
$email = "jakis_email@costam..pl"; // bledny adres z dwoma kropkami
 
if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
  exit('Błędny adres email');
} 
print 'dalsza czesc strony do ew. filtrowania';
// tua jdajesz kod ktory ma sie dalej wykonywac
[PHP] pobierz, plaintext 

A jest jakiś sposób na umieszczenie na tej "stronie z błędami w formularzu" jakiegoś linku/buttona typu Powróć lub Popraw. Tak sobie testuję i nawet backspace nie wraca w operze i chrome. Tylko klik w < w menu przeglądarki działa. Pewnie znowu od cholery zabawy o ile to w ogóle możliwe. Ale wolę zapytać.

No przeciez napisalem Ci przyklad wyzej

Tzn? Wpisuję co chcę i przechodzi. Alert o znalezionych błędach jest tylko gdy pole jest puste/wyspacjowane. Wystarczy walnąć w dowolne pole (oprócz rzecz jasna kodu pocztowego bo to zabezpieczyłem przed głupotami) kropkę i idzie formularz na mejla w stylu:
Imię: . . ..
Nazwisko: 8-)
Kod: 00-100
Miejscowość: tralala ..

Kropka to nie spacja czy tabulator. To są tzw białe znaki. Reszta wpisywana z klawiatury jest normalnym znakiem. Miales problem z pustymi wartosciami ktore byly przesylane na mejla a to czy ktos Ci wpisze w pole MIASTO = blablabla to juz od tego sie nie uchronisz bo musialbys miec jakas tablice wszystkich miast, miejscowosci, wsi etc i pozwlaac tylko z tych.
Jesli chcesz filtrowac względem dozwolonych znaków to trzeba filtrowac po tym przykladzie ktory podalem. On sprawdza CZY COS ZOSTAŁO WYSŁANE lub jak kto woli czy POLA FORMULARZA NIE ZOSTAŁY WYSŁANE PUSTE.
Do pustych nie nalezą kropki czy inne znaki. Puste sa to białe znaki. Jesli przyklad nie wychwyci nic znaczy sie ze pola formularza zostaly wypelnione.


To przykład ktory sprawdzi czy pole zawiera TYLKO male litery i liczby - reszta jest niedozwolona

[PHP] pobierz, plaintext 
<?php
$co = "login---";
if(preg_match('/[^0-9a-z]/i',$co))
{
	exit('Wystepuja nieprawidlowe znaki');
} 
 
print 'Dalsza czesc strony jesli ok';
?>
[PHP] pobierz, plaintext 

W podobny sposob mozesz dodawac znaki ktore moga byc akceptowane jak rowniez dlugosc minimalna i maksymalna.
W tym przypadku akceptowane sa:
Male litery
Liczby
Znak podkreslenia
Kropka
Minimalna dlugosc 6 a max 15 znakow

[PHP] pobierz, plaintext 
<?php
$co = "logi99._-";
if(!preg_match("/^[a-z0-9_.]{6,15}$/", $co))
	{
	exit('Wystepuja nieprawidlowe znaki. Minimalna liczba znakow to 6 a max to 15');
} 
 
print 'Dalsza czesc strony jesli ok';
?>
[PHP] pobierz, plaintext 

Dzięki. Wezmę się za to ale nie dziś.

Wracając do "przycisku/odnośnika WSTECZ (w moim przypadku to będzie odnośnik POPRAW POLA) na stronie z błędami zrobiłem na szybko wg. Twoich rad. Dodałem:

print "<a style=\"color: green\" href=java script:history.go(-1)><mark>POPRAW POLA</mark></a></div></body>";

na końcu. Jest ok poza tym, że po kliku w ten link wywala błąd:

Not Found
The requested URL /ankieta/java was not found on this server.
www.mojastronaxxxxxxxx.pl

Robi się off top ale chyba nie ma co tworzyć nowego wątku. Co z ta javą na moim serwerze?

Ty chyba nie czytasz tego co sie pisze.
Dalem komentarz do tego linku. Skrypt forum dzieli wyraz javascript na dwa. Wez to polacz w jeden i bedzie git.

Dlaczego wynikiem jest tylko 'java' ? Ano dlatego ze jesli sie nie obejmie apostrofami url (w tym przypadku tag 'a') a występuje w ciagu spacja wtedy bedzie odczytany pierwszy ciąg przed spacją. Ale to juz tak gwoli scislosci. Akurat to nie ma znaczenia w tym przykladzie. Skrypt forum dzieli niektore wyrazy poprostu. Trzeba je polączyc.

Uściślając Twój 1-szy regexp akceptuje duże i małe litery + cyfry (operator `i`)

Czytałem, czytałem Ze zrozumieniem to już inna sprawa. Łączyłem ten cały ciąg (myślałem, że to jest string) z exit, na kilka sposobów i nic. Po Twoim ostatnim poście dotarło do mnie żeby połączyć słowo java script w javascript.

W międzyczasie pojawiła się kolejna zagwozdka:
nie chcąc zmuszać wypełniającego do podawania telefonu usunąłem z tego fragmentu:

$ArrInfo = array('imie' => 'Wypełnij pole: imię',
'ulica' => 'Wypełnij pole: ulica',
'kod' => 'Wypełnij pole: kod',
'miejscowosc' => 'Wypełnij pole: miejscowość',
'telefon' => 'Wypełnij pole: telefon',
);

ciąg dot. telefonu, żeby nie pokazywał błędu Popraw/Wypełnij. Teraz mam:

$ArrInfo = array('imie' => 'Wypełnij pole: imię',
'ulica' => 'Wypełnij pole: ulica',
'kod' => 'Wypełnij pole: kod',
'miejscowosc' => 'Wypełnij pole: miejscowość',
);

Sęk w tym, że to pole musi być opcjonalne, zero przymusu. Problem w tym, że nie wypełniając pola telefon i próbie wysłania wyskakuje okno:

Znaleziono błędy:

POPRAW POLA


Nie ma nic o błędzie, pusta linijka. Dopóki nie wklepie się w to pole czegoś to formularz nie przejdzie.

W dodatku w formularzu html mam pole textarea, też do OPCJONALNEGO wypełnienia. Miejsce na uwagi od wypełniającego:
<textarea name="uwagi" cols="50" rows="3"></textarea>

Występuje taka sama sytuacja jak z polem telefon. Dopóki nie wklepiesz tam czegoś dopóty formularz nie pójdzie dalej i błąd:

Znaleziono błędy:

POPRAW POLA

Pomożecie koledzy?

Pokaz aktualny kod.
Byloby rowniez BARDZO milo jakbys raczyl uzywac BBCODE dla wstawianego kodu

[PHP] pobierz, plaintext 
<?php 
 
			if(!isset($_POST['submit']))
			{
 
				exit('wystąpił błąd');
 
 
			} 
 
			$ArrInfo = array('imie' => 'Wypełnij pole: imię',
				'ulica' => 'Wypełnij pole: ulica',
				'kod' => 'Wypełnij pole: kod',				
				'miejscowosc' => 'Wypełnij pole: miejscowość',
				);
 
			unset($submit);
			$myPostArgs = filter_input_array(INPUT_POST);
			$returnNames = array();
			foreach($myPostArgs as $ParamsKey => $ParamsValue)
			{
 
				if(trim($ParamsValue) == "")
				{
 
					$return[] = $ArrInfo[$ParamsKey];
 
					$match = true;
 
 
				}
 
			}
 
 
			if(isset($match))
			{
				print 'Znaleziono błędy: <br>';
 
			for($i=0; $i<count($return);  $i++)
				{
 
				print ($return[$i] . '<br>');
 
				}
 
print "<a style=\"color: green\" href=java script:history.go(-1)><mark>POPRAW</mark></a></div></body>"; 
			exit;
 
			} 
 
      $do = 'mojmejl@pocztaxxx.pl'; 
      $imie = $_POST['imie']; 
      $ulica = $_POST['ulica']; 
      $kod = $_POST['kod']; 
      $miejscowosc = $_POST['miejscowosc']; 
      $telefon = $_POST['telefon']; 
      $k01 = $_POST['k01']; 
      $k02 = $_POST['k02'];
      $k03 = $_POST['k03'];
      $k04 = $_POST['k04']; 
      $k05 = $_POST['k05']; 
 
      $tresc .= "Imię:".$imie."\n\n"; 
      $tresc .= "Ulica: ".$ulica."\n\n"; 
      $tresc .= "Kod: ".$kod."\n\n"; 
      $tresc .= "Miejscowość: ".$miejscowosc."\n\n"; 
      $tresc .= "Telefon: ".$telefon."\n\n"; 
      $tresc .= "biały: ".$k01."\n\n"; 
      $tresc .= "czarny: ".$k02."\n\n"; 
      $tresc .= "niebieski: ".$k03."\n\n"; 
      $tresc .= "zielony: ".$k04."\n\n"; 
      $tresc .= "czerwony: ".$k05."\n\n";
      @mail($do, 'ankieta', $tresc, "MIME-Version: 1.0\r\n"."Content-type: text/plain; charset=ISO-8859-2\r\n"."From: $mail\r\n");
      header('Location: [url="http://mojastronawww/Dziekuję.html&#39%3b%29;"]http://mojastronawww/Dziekuję.html');[/url]
      echo $tresc;
      ?>
[PHP] pobierz, plaintext 

1) w kodzie nadal masz telefon w tablicy wiec nie oczekuje ze zniknie
2) Twoj kod

[PHP] pobierz, plaintext 
			foreach($myPostArgs as $ParamsKey => $ParamsValue)
			{
 
				if(trim($ParamsValue) == "")
				{
 
					$return[] = $ArrInfo[$ParamsKey];
 
					$match = true;
 
 
				}
 
			}
[PHP] pobierz, plaintext 

Leci po wszystkicj danych z POST zamiast leciec tylko po danych z tablicy
Zmiana na szybko, choc i tak mi sie nie podoba:

[PHP] pobierz, plaintext 
			foreach($myPostArgs as $ParamsKey => $ParamsValue)
			{
 
				if(isset($ArrInfo[$ParamsKey]) && trim($ParamsValue) == "")
				{
 
					$return[] = $ArrInfo[$ParamsKey];
 
					$match = true;
 
 
				}
 
			}
[PHP] pobierz, plaintext 

Generalnie caly ten kod do poprawki by byl ale od biedy narazie starczy

Zapomniałem usunąć telefon, sorki. Jasne, że go tam teraz nie ma w kodzie skryptu.
Zaraz pozmieniam to co podałeś.

Kurka wodna! Działa. Dzięki!
Co do estetyki kodu pliku php to wiem, jest jaki jest... Ale nie wnikam póki działa bo zaraz coś się wykrzaczy i będzie płacz...

To teraz nie pozostaje mi nic tylko podziękować wszystkim i zobaczymy czy w przyszłości przyjdą na skrzynkę jeszcze jakieś niewypełnione formularze. Oby nie!

Puste pola juz nie przejdą. A to czy bedziesz dalej filtrowal dane wg. swoich wymagan to juz inne buty.

Tak, zostaje kwestia filtrowania pól, o której kilka wpisów tu jest ale to nie jest teraz najpilniejsza sprawa. Wiesz co było priorytetem a filtrowanie inputów wyszło przy okazji. To delikatna sprawa: tyle wyjątków i różnych kombinacji przy każdym polu, nie wspominając, że ludzie często walą w form-ie wszystko jak leci z małej litery (co dla mnie nie jest żadnym problemem). Niech mają luz. Jak 1 na 100 będzie chciał wpisać bzdety nie widzę problemu - niech sobie ulży, jego sprawa.

[PHP] pobierz, plaintext 
$co = "logini77";
if(!preg_match('/^[a-z0-9]+$/', $co))
{
	print 'blad';
} else {
	print 'ok';
}
[PHP] pobierz, plaintext 

Do tego sluza wyrazenia regularne. Cala sila w nich jest taka ze nie trzeba wlasnie miec naście kombinacji - jedna regula i tyle.