jak to naprawic? Opcje

[-Gość_kuba-]

czesc... mam taki oto problem.
mam skrypt, ktory w zaleznosci od wartosci zmiennej $link (za pomoca switch'a) obsluguje mi dany formularz.
np. jezeli mam $link = 'prace' powinno mi obsluzyc formularz i w razie bledow, wyswietlic je nad formularzem...
problem twki w tym, ze po nieprawidlowym wypelnieniu formularza przenosci mnie na admin.php a nie wyswietla jeszcze raz formularza i notek, co zle zrobilem - $download_message.
gdzie blad?

[PHP] pobierz, plaintext 
<?php
//obsluz formularz dla 'download'
if (isset($_POST['download_wyslij'])) {
$download_message = NULL;
if (strlen($_POST['download_przedmiot']) > 0) {
if ($_POST['download_przedmiot'] != 'angielski' OR $_POST['download_przedmiot'] != 'chemia' OR $_POST['download_przedmiot'] != 'fizyka' OR $_POST['download_przedmiot'] != 'geografia' OR $_POST['download_przedmiot'] != 'historia' OR $_POST['download_przedmiot'] != 'matematyka' OR $_POST['download_przedmiot'] != 'niemiecki' OR $_POST['download_przedmiot'] != 'po' OR $_POST['download_przedmiot'] != 'polski' OR $_POST['download_przedmiot'] != 'religia' OR $_POST['download_przedmiot'] != 'wos') {
$download_przedmiot1 = escape_data($_POST['download_przedmiot']);
} else {
$download_przedmiot1 = FALSE;
$download_message .= &#092;"pole przedmiot musi mieć jedn&plusmn; z podanych warto&para;ci: <span color=\"#000;\">angielski, chemia, fizyka, geografia, historia, matematyka, niemiecki, po, p
lski, religia, wos</span>.<br />n\";
}
} else {
$download_przedmiot1 = FALSE;
$download_message .= \"podaj przedmiot, okre&para;laj&plusmn;cy dziedzinę pracy.<br />n\";
}
if (strlen($_POST['download_nazwa']) > 0) {
$download_nazwa1 = escape_data($_POST['download_nazwa']);
} else {
$download_nazwa1 = FALSE;
$download_message .= \"podaj nazwę pliku.<br />n\";
}
if (strlen($_POST['download_link']) > 0) {
$download_link1 = escape_data($_POST['download_link']);
} else {
$download_link1 = FALSE;
$download_message .= \"podaj nazwę pliku.<br />n\";
}
if (strlen($_POST['download_wielkosc']) > 0) {
if (is_numeric($_POST['wielkosc'])) {
$download_wielkosc1 = $_POST['download_wielkosc'];
} else {
$download_wielkosc1 = FALSE;
$download_message .= \"wielko&para;ć musi być warto&para;ci&plusmn; liczbow&plusmn;.<br />n\";
}
} else {
$download_wielkosc1 = FALSE;
$download_message .= \"podaj wielkosc pliku.<br />n\";
}
//jezeli dane sie zgadzaja
if ($download_przedmiot1 && $download_nazwa1 && $download_link1 && $dawnload_wielkosc1) {
$query_download = \"INSERT INTO download_spis (data, przedmiot, link, nazwa, wielkosc) VALUES (NOW(), '$download_przedmiot1', '$download_link1', '$download_nazwa1', '$download_wielkosc1')\";
$result_download = @mysql_query($query_download);
if ($result_download) {
header (\"Location: http://\" . $_SERVER['HTTP_HOST'] . dirname($_SERVER['PHP_SELF']) . \"/admin.php\");
exit();
} else {
$message_sprawdziany = 'awaria systemu.' . mysql_error() . '<br />' . &#092;"n\";
}
}
}
?>
[PHP] pobierz, plaintext 

i moze ktos ma jakis pomysl, jak moge skrocic:

[PHP] pobierz, plaintext 
<?php
if ($_POST['download_przedmiot'] != 'angielski' OR $_POST['download_przedmiot'] != 'chemia' OR $_POST['download_przedmiot'] != 'fizyka' OR $_POST['download_przedmiot'] != 'geografia' OR $_POST['download_przedmiot'] != 'historia' OR $_POST['download_przedmiot'] != 'matematyka' OR $_POST['download_przedmiot'] != 'niemiecki' OR $_POST['download_przedmiot'] != 'po' OR $_POST['download_przedmiot'] != 'polski' OR $_POST['download_przedmiot'] != 'religia' OR $_POST['download_przedmiot'] != 'wos') {
?>
[PHP] pobierz, plaintext 

z gory dziekuje za pomoc.

[-Gość_kuba-]

zapomnialem.... tak w razie czego - to jest kod formularza:

[PHP] pobierz, plaintext 
<?php
echo &#092;"<form action=\"\" . $_SERVER['PHP_SELF'] . \"\" method=\"post\">n\";
echo &#092;"<table cellspacing=\"0\" cellpadding=\"0\">n\";
echo &#092;"<tr>n\";
echo &#092;"<td class=\"download_lewa\">przedmiot:</td>n\";
echo &#092;"<td class=\"download_prawa\"><input type=\"text\" name=\"download_przedmiot\" size=\"40\" />\"; if (isset($_POST['download_przedmiot'])) {echo stripslashes($_POST['download_przedmiot']);} echo \"</td>n\";
echo &#092;"</tr>n\";
echo &#092;"<tr>n\";
echo &#092;"<td class=\"download_lewa\">nazwa:</td>n\";
echo &#092;"<td class=\"download_prawa\"><input type=\"text\" name=\"download_nazwa\" size=\"40\" />\"; if (isset($_POST['download_nazwa'])) {echo stripslashes($_POST['download_nazwa']);} echo \"</td>n\";
echo &#092;"</tr>n\";
echo &#092;"<tr>n\";
echo &#092;"<td class=\"download_lewa\">link:</td>n\";
echo &#092;"<td class=\"download_prawa\"><input type=\"text\" name=\"download_link\" size=\"40\" />\"; if (isset($_POST['download_link'])) {echo stripslashes($_POST['download_link']);} echo \"</td>n\";
echo &#092;"</tr>n\";
echo &#092;"<tr>n\";
echo &#092;"<td class=\"download_lewa\">wielko&para;ć:</td>n\";
echo &#092;"<td class=\"download_prawa\"><input type=\"text\" name=\"download_wielkosc\" size=\"5\" /><\"; if (isset($_POST['download_wielkosc'])) {echo stripslashes($_POST['download_wielkosc']);} echo \"/td>n\";
echo &#092;"</tr>n\";
echo &#092;"<tr>n\";
echo &#092;"<td id=\"download_razem\" colspan=\"2\"><input type=\"submit\" name=\"download_wyslij\" value=\"dodaj prace do spisu\" /></td>n\";
echo &#092;"</tr>n\";
echo &#092;"</table>n\";
echo &#092;"</form>n\";
?>
[PHP] pobierz, plaintext 

moze problem twki w wartosci action w <form> ? strzelam, bo nie wiem... wiec prosze nie smiac sie

[cim]

[PHP] pobierz, plaintext 
<?php
function sprawdz($wybrane){
$jest = false;
$przedmioty = array('angielski', 'chemia', 'fizyka', 'geografia', 'historia', 'matematyka', 'niemiecki', 'po', 'polski', 'religia', 'wos');
 
foreach($przedmioty as $value){
if($wybrane == $value) break; $jest = true;
}
 
return $jest;
}
 
?>
[PHP] pobierz, plaintext 

[php]if(sprawdz($_POST['download_przedmiot'] == true){ .... }else{ .... }

wiele krótsze to chyba nie jest, ale bardzej poręczne i milsze dla oka

[popbart]

Albo tak

[PHP] pobierz, plaintext 
<?php
$przedmioty = array('angielski', 'chemia', 'fizyka', 'geografia', 'historia', 'matematyka', 'niemiecki', 'po', 'polski', 'religia', 'wos');
if(!in_array($_POST['download_przedmiot']),$przedmioty)
{
	   $download_przedmiot1 = escape_data($_POST['download_przedmiot']);
}
else ....
 
?>
[PHP] pobierz, plaintext 

Ten post edytował popbart 17.09.2004, 19:49:24

[-Gość_kuba-]

dzieki za kody...
ale jeszcze jedna kwestia - dlaczego nie dziala?