[PHP]Notice: Undefined index - jak sobie radzić nie zaciemniając kodu? Opcje

[qrzysztof]

Powiedzmy, że mam kod:

[PHP] pobierz, plaintext 
 
if ( $_POST['wyslij']=='zaloguj' )
{
 //zrob cos
}
if ( $_POST['wyslij']=='wyloguj' )
{
 //zrob cos innego
}
[PHP] pobierz, plaintext 

Powiedzmy też, że postanowiłem ustawić raportowanie błędów na poziomie E_ALL. Powyższy kod spowoduje w tym układzie wyświetlenie Notice: Undefined index.

Jak sobie radzicie z tym problemem? Przez problem rozumiem samo zgłaszanie błędu, nie jego wyświetlanie na wyjściu. Może tak:

[PHP] pobierz, plaintext 
 
if ( isset($_POST['wyslij']) && $_POST['wyslij']=='zaloguj' )
{
 //zrob cos
}
if ( isset($_POST['wyslij']) && $_POST['wyslij']=='wyloguj' )
{
 //zrob cos innego
}
[PHP] pobierz, plaintext 

"Trochę" to zaciemnia kod. Chyba ciut lepiej zadeklarować wszystkie spodziewane zmienne:

[PHP] pobierz, plaintext 
$POST['wyslij'] = (isset($_POST['wyslij']) ? $_POST['wyslij'] : null ;	
if ($_POST['wyslij']=='zaloguj' )
{
 //zrob cos
}
if ($_POST['wyslij']=='wyloguj' )
{
 //zrob cos innego
}
[PHP] pobierz, plaintext 

Ciekawi mnie jak wygląda taki kod u purystów. Bo z faktu, że 95% ma wyłączone przynajmniej E_NOTICE zdaję sobie doskonale sprawę (IMG:style_emoticons/default/smile.gif)

[nospor]

isset($_POST['wyslij']) &&
To jest dobre rozwiązanie.
Zaciemnia kod? Bez przesady

[PHP] pobierz, plaintext 
$wyslij = (isset($_POST['wyslij']) ? $_POST['wyslij'] : null ;	
if ($wyslij=='zaloguj' )
[PHP] pobierz, plaintext 

To też jest dobre rozwiązanie.

[by_ikar]

[PHP] pobierz, plaintext 
<?php
 
function getPostParametr($key, $default = null)
{
    return isset($_POST[$key]) ? $_POST[$key] : $default;
}
[PHP] pobierz, plaintext 

użycie:

[PHP] pobierz, plaintext 
<?php
 
if(getPostParametr('login'))
{
    //do something
} elseif(getPostParametr('logout'))
{
    //do something else
}
[PHP] pobierz, plaintext 

przykład dla tablicy $_GET:

[PHP] pobierz, plaintext 
<?php
 
function getParametr($key, $default = null)
{
    return isset($_GET[$key]) ? $_GET[$key] : $default;
}
[PHP] pobierz, plaintext 

użycie praktycznie takie same, zmienia się jedynie nazwa funkcji:

[PHP] pobierz, plaintext 
<?php
 
if(getParametr('login'))
{
    //do something
} elseif(getParametr('logout'))
{
    //do something else
}
[PHP] pobierz, plaintext 

Ten post edytował by_ikar 9.12.2011, 15:35:02

[qrzysztof]

No proszę, proste ale sprytne (IMG:style_emoticons/default/smile.gif) I nie zaciemnia kodu.

[grzeee]

mi się też podoba, poza "zaciemniania" kodu to zawsze też ograniczy błędy/literówki jeśli tak można to nazwać.

Ten post edytował grzeee 9.12.2011, 15:54:46

[by_ikar]

mi się też podoba, poza "zaciemniania" kodu to zawsze też ograniczy błędy/literówki jeśli tak można to nazwać.

Mało tego, prócz przez was wymienionych "zalet", jest jeszcze jedna - wartość domyślna. Przydatna, kiedy jakiegoś parametru nie ma, a oczekujemy konkretnej wartości. Osobiście w podobny sposób w swoim "systemie" bawię się z widokiem. Powiedzmy że twój system rozbija linki na pewne sekcje. Gdzie z przykładowego routingu:

Kod

/:module/:action.:format

dla linku:

Kod

example.com/myModule/myAction.html

powstaje taka tablica:

Kod

Array
(
    [module] => myModule
    [action] => myAction
    [format] => html
)

A co w przypadku kiedy nie używam w swoim systemie w adresach rozszerzenia html?

Kod

/:module/:action

Kod

example.com/myModule/myAction

Kod

Array
(
    [module] => myModule
    [action] => myAction
)

Odpowiedź prosta, podaje w domyślnym parametrze html, a wygląda to mniej więcej tak:

[PHP] pobierz, plaintext 
$this->options['template'] = $this->moduleName.'.'.$this->actionName.'.'.$this->request->get('format', 'html');
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/wink.gif)

Ten post edytował by_ikar 9.12.2011, 17:10:46