Forum PHP.pl > Porównanie pobranych z formularza danych do NULLa

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Porównanie pobranych z formularza danych do NULLa

beel

3.07.2011, 21:38:11

Pobieram trzy wartości w JavaScript, dwie daty i jeden numeric (kwota). Jeśli nic nie zostanie wprowadzone to gdy porównam numeric do NULL'a, wie, że nic nie podano i odpowiednio reaguje (czyli do zapytania SQL wstawia NULLa), z datami niestety już to nie działa wstawia mi w zapytaniu coś takiego:

[PHP] pobierz, plaintext 
data = ''
[PHP] pobierz, plaintext

a chciałbym aby było:

[PHP] pobierz, plaintext 
data = NULL
[PHP] pobierz, plaintext

Kod wygląda tak:

[PHP] pobierz, plaintext 
      $dataZaw = $_POST['sUGO_DATA'];
      $kwota = $_POST['sUGO_KWOTA'];
      $terminSpl = $_POST['sUGO_TERMIN'];
 
 
        if (!is_int($_SESSION['S_tabSelectUgoda']))
        {   
            if ($dataZaw == NULL)
                    $sKomendaSQL = "UPDATE ugody SET data_umowy=NULL, termin_sp='$terminSpl', kwota=$kwota WHERE ug_id_proc = $cid";
            if ($terminSpl == NULL)
                    $sKomendaSQL = "UPDATE ugody SET data_umowy='$dataZaw', termin_sp=NULL, kwota=$kwota WHERE ug_id_proc = $cid";
            if ($kwota == NULL)
                    $sKomendaSQL = "UPDATE ugody SET data_umowy='$dataZaw', termin_sp='$terminSpl', kwota=NULL WHERE ug_id_proc = $cid";
            else
            $sKomendaSQL = "UPDATE ugody SET data_umowy='$dataZaw', termin_sp='$terminSpl', kwota=$kwota WHERE ug_id_proc = $cid";
...
...
[PHP] pobierz, plaintext

Fifi209

4.07.2011, 00:37:29

A w bazie masz ustawione, że data może być null?

beel

4.07.2011, 05:40:25

Tak, mam odhaczone opcje NOT NULL.
Zresztą jak ręcznie wklepię zapytanie z NULLami (dla daty, terminu i kwoty) to przechodzi.

nospor

4.07.2011, 06:32:39

Twoje IFy są bez sensu.
Robiąc IFa pod IFem bez żadnych else, wykona się każdy IF pokolei (jeśli tylko jest prawdziwy) nadpisując poprzedniego IFa
A już najlepszy jest ostatni IF, gdzie dałeś ELSE. W takim przypadku zawsze ci się wykona ten ostatni IF lub ELSE i tu już na 100% masz nadpisywane zapytanie z poprzednich IF

by_ikar

4.07.2011, 10:00:32

Kod

$dataZaw = !is_null($_POST['sUGO_DATA']) ? $_POST['sUGO_DATA'] : NULL;
$kwota = !is_null($_POST['sUGO_KWOTA']) ? $_POST['sUGO_KWOTA'] : NULL;;
$terminSpl = !is_null($_POST['sUGO_TERMIN']) ? $_POST['sUGO_TERMIN'] : NULL;;

Generalnie, lepiej napisać dla czegoś takiego funkcje/metodę, ponieważ wiele razy będziesz się powtarzał, a tak w swojej funkcji/metodzie będziesz mógł dorzucić od razu jakąś domyślną wartość, escapowanie itp itd. przykładowo:

Kod

function getPost($keyName, $default = null)
{
return !isset($_POST[$keyName]) || is_null($_POST[$keyName]) ? $default : $_POST[$keyName];
}

I wtedy już tylko bezpośrednio w zapytaniu:

Kod

$sKomendaSQL = "UPDATE ugody SET data_umowy='".getPost('sUGO_DATA')."', termin_sp='".getPost('sUGO_TERMIN')."', kwota='".getPost('sUGO_KWOTA')."' WHERE ug_id_proc = $cid";

Czy coś. Od tego masz funkcje, klasy i tym podobne, żeby nie pisać x razy tego samego.

beel

4.07.2011, 12:12:49

Pomyślę nad ostatnią propozycją, dzięki.
A co do przedostatniej podpowiedzi, no tak, ostro dałem ciała z IFami.

Pozdrawiam

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.