Forum PHP.pl > [MySQL][PHP]Problem z Skryptem

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [MySQL][PHP]Problem z Skryptem

saperdak

7.03.2015, 20:07:31

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

Z góry dziękuję

Kshyhoo

7.03.2015, 20:12:05

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

pasman

7.03.2015, 20:15:38

Cytat(saperdak @ 7.03.2015, 20:07:31 )

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

Z góry dziękuję

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

saperdak

7.03.2015, 20:20:27

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

Kshyhoo

7.03.2015, 20:23:34

Manual PHP: isset

pasman

7.03.2015, 20:48:27

Cytat(saperdak @ 7.03.2015, 20:20:27 )

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

Pyton_000

7.03.2015, 21:01:28

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

7.03.2015, 21:15:46

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

markonix

8.03.2015, 10:39:06

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

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.