[MySQL][PHP]Problem z Skryptem, przeglądarka pokazuje błąd przy isset Opcje

[saperdak]

Witam. Mam problem, przejdę odrazu do setna sprawy:

[PHP] pobierz, plaintext 
<?php
include_once './functions.php';
if(isset($_POST["klasa"] || $_POST["data"] || $_POST["nrlekcji"] || $_POST["nauczyciel"] || $_POST["zastepstwo"]) && !empty($_POST["klasa"] || $_POST["data"] || $_POST["nrlekcji"] || $_POST["nauczyciel"] || $_POST["zastepstwo"])){
$db = new DB_Functions();
$uname = $_POST["klasa"],$_POST["data"],$_POST["nrlekcji"],$_POST["nauczyciel"],$_POST["zastepstwo"];
$res = $db->storeUser($uname);
 
    if($res){ ?>
         <div id="msg">sukces</div>
    <?php }else{ ?>
         <div id="msg">blad</div>
    <?php }
} else{ ?>
 <div id="msg">Prosze wpisać dane</div>
<?php }
?>
[PHP] pobierz, plaintext 

Wydaje mi się że wszystko jest ok. Ale przeglądarka generuje błąd.

[PHP] pobierz, plaintext 
Fatal error: Cannot use isset() on the result of an expression (you can use "null !== expression" instead) in D:\BazaDanych\htdocs\android\insertuser.php on line 57
[PHP] pobierz, plaintext 

Proszę o pomoc (IMG:style_emoticons/default/wink.gif) Z góry dziękuję

Ten post edytował saperdak 7.03.2015, 20:18:34

[Kshyhoo]

Już pisałem, że masz używać odpowiedniego bb-code dla kodu. Zmień też tytuł wątku...

[pasman]

Wydaje mi się że wszystko jest ok. Ale przeglądarka generuje błąd.

CODE

Fatal error: Cannot use isset() on the result of an expression (you can use "null !== expression" instead) in D:\BazaDanych\htdocs\android\insertuser.php on line 57

Proszę o pomoc (IMG:style_emoticons/default/wink.gif) Z góry dziękuję

a co tu jest dla ciebie niejasne ?
masz podane co jest złe i jak masz to poprawić ...

[saperdak]

Pasman. Wiem gdzie jest błąd. Ale nie wiem jak go naprawić. Jak te isset opisać poprawnie

[PHP] pobierz, plaintext 
if(isset($_POST["klasa"] || $_POST["data"] || $_POST["nrlekcji"] || $_POST["nauczyciel"] || $_POST["zastepstwo"]) && !empty($_POST["klasa"] || $_POST["data"] || $_POST["nrlekcji"] || $_POST["nauczyciel"] || $_POST["zastepstwo"]))
[PHP] pobierz, plaintext 

Dopiero zaczynam z php

Ten post edytował saperdak 7.03.2015, 20:21:01

[Kshyhoo]

Manual PHP: isset

[pasman]

Pasman. Wiem gdzie jest błąd. Ale nie wiem jak go naprawić. Jak te isset opisać poprawnie

[PHP] pobierz, plaintext 
if(isset($_POST["klasa"] || $_POST["data"] || $_POST["nrlekcji"] || $_POST["nauczyciel"] || $_POST["zastepstwo"]) && !empty($_POST["klasa"] || $_POST["data"] || $_POST["nrlekcji"] || $_POST["nauczyciel"] || $_POST["zastepstwo"]))
[PHP] pobierz, plaintext 

Dopiero zaczynam z php

jeśli chcesz sprawdzić czy $_POST coś zawiera to spróbuj:

[PHP] pobierz, plaintext 
if (count($_POST) >0) {...}
[PHP] pobierz, plaintext 

Ten post edytował pasman 7.03.2015, 20:49:50

[Pyton_000]

isset($var1 || $var2)
isset($var1, $var2)

Widzisz różnice?

W empty to nie działa, każda wartość musi być oddzielnie.
@pasman jedna z głupszych porad. Równie dobrze można zastosować if(!empty($_POST)) co ihmo i tak jest bez sensu przynajmniej w tym wypadku.

[Boshi]

Nie lepiej to przelecieć pętlą i sprawdzić czy któraś wartość nie jest pusta i czy istnieje?

[markonix]

Po pierwsze tego typu formularz trzeba porządnie zwalidować, a nie sprawdzić tylko czy pola są utworzone.

Numer lekcji - domyślam się, że to np. liczba naturalna (bez zera w zależności od szkoły).
Data - każdy będzie dodawał swój format i się będzie wysypywać, walidacja powinna być nawet jeżeli korzysta się z datapickera.
Nauczyciel i zastępstwo - to powinno być wg mnie w ogóle w postaci list wyboru.

Każde pole walidować osobno, nie wszystko na raz aby móc wyświetlić konkretne błędy typu "Brak pola X", a nie "Masz gdzieś brak, ale sam szukaj gdzie".