Dotaz: PHP Upřesnění scriptu

[Libor Kron]

Zdravím,

mám kod

$page .= "<td><input type=\"text\" name=\"name\" value=\"$_POST['name']\" /></td>";

a PHP na mě křičí, že Notice: Undefined index: name

Co to přesně znamená? že prom $_POST['name'] neexistuje, nebo ze existuje ale je prazdna nebo co přesně? Jak to můžu opravit?

Když tu prom nadefinuji

$_POST['name'] = 'nevim co sem napsat';

tak uz ji pak opravdová prom. nepřepíše, což je logické, ale nevim jak to pořádně ošetřit. Mužu jeste napsat podminku pokud neexistuje tak ji nějak nadefinuji (nevim sice jak) a pokud existovat bude tak se podminka nevykona. Ale nevím no jestli je to OK.

Děkuji za pomoc ;)

Odpovědi

[poky74]

Asi nejjednodušší bude něco jako

if(!isset($_POST['name']))$_POST['name']="";

[voda]

Když už tak raději takto:

$page .= '<td><input type="text" name="name" value="'(.isset($_POST['name']) ? $_POST['name'] : '').'" /></td>';

[polymorf]

Daj pred dolar zavinac: @$_POST['name']

[l4m4]

Vynikající nápad, jak do toho udělat ještě větší díru.

[camel1cz]

No pojal bych to jinak - když píšete aplikaci, musíte mít za všech okolností jasno, co je v proměnné... o těch, poslaných přes REQUEST to platí dvojnásob!

Řešil bych to tedy zpracováním vstupu. Vlastně "překopírováním" hodnot do proměnných aplikace, přičemž současně bych udělal validaci dat, escapování, atd.

Pokud uděláte výše zmíněné, tak ten problém co zmiňujete "sám" zmizí...

[pht]

To je dost zející XSS díra. Minimálně kolem toho dejte htmlentities().

[Ash]

Je to upozornění, že v poli $_POST[] není index 'name' definován, tedy to neznamená že by byl prázdný, ale prostě vůbec neexistuje, neboť nevznikl. Takže ve formuláři nejspíš nebyl prvek se jménem 'name' formou POST odeslán či přijat. Obecně není dobré se na existenci takovýchto proměnných spoléhat, i kdyby byl formulář udělaný správně, vždycky může přijít někdo s nekalými úmysly a podvrhnout upravený požadavek.

Zmíněné isset() je možný způsob jak to ošetřit.

Vámi navržené řešení "nějak ji nadefinovat" může a nemusí být správné, z uvedených informací to nelze určit. Měl byste být sám schopen posoudit, jestli to 'name' je něco co potřebujete znát, nebo to lze nějak vygenerovat. Příklad: pokud vypisuji informace o produktech třeba z databáze a produkty jsou identifikovány pomocí 'name', nejspíš bez 'name' nebudu vědět co vypsat. Mohu vypsat informace o náhodném produktu, ale většinou spíš vypíšu chybovou hlášku, že uvedený produkt nebyl nalezen. Naopak pokud 'name' označuje třeba jméno souboru, která chci uložit třeba na server, tak mohu vygenerovat náhodné jméno a oznámit, že soubor byl uložen pod tímto jménem.

[Libor Kron]

Tak ze je lepsi na zacátku kodu promenou -vzdy definovat $prom=""; -nebo dat podminku jestli je plna if(!empty($prom) $add = $prom;)); -nebo dat podminku jestli existuje if(isset($prom) $add = $prom;));

Díky

[Libor Kron]

Tak ze je lepsi na zacátku kodu promenou

-vzdy definovat $prom="";
-nebo dat podminku jestli je plna if(!empty($prom) $add = $prom;));
-nebo dat podminku jestli existuje if(isset($prom) $add = $prom;));

Díky

[Ash]

Bavíme se o proměnné, která nemusí existovat.

Buďto lze před každým jejím použitím udělat if(isset($prom)) {... } else {...}.

Nebo (zejména pokud by výše uvedené kód dost komplikovalo) pokud lze kód napsat tak, že se bude chovat "dobře" třeba pro prázdnou proměnnou, a současně to nebude kolidovat se situací, kdy uživatel zadal prázdnou proměnnou, lze ji (v případě že neexistuje) na začátku nastavit na prázdnou, zbytek kódu je pak jednodušší.

Místo na prázdnou ji lze nastavit na nějakou defaultní či nějak vygenerovanou hodnotu, to záleží na situaci.

Vždy záleží na tom, co reálně děláte. Je rozdíl nepodat daňové přiznání (proměnná nedefinovaná) a podat daňové přiznání s nulovým doplatkem/přeplatkem (proměnná prázdná). Někdy je "nedat někomu žádnou facku" (prázdná proměnná) totéž, jako dát mu nezjištěný počet facek (nedefinovaná proměnná), ale někdy to jedno není.

Můžete si stanovit nějakou politiku (třeba vždy proměnnou nějak nastavit), ale rovnou počítejte s tím, že nastane situace, kdy by se stav "undefined" docela hodil ;)

[Ash]

Když čtu váš dotaz tak v podstatě jste všechny ty možnosti uvedl, takže odpověď je "ano, je to OK" :) Můžete psát podmínky, můžete proměnnou nastavit... Pokud víte co píšete, tak budete i vědět, kterou z uvedených možností zvolit. Trochu z toho dotazu mám pocit že úplně nevíte, co píšete ("nevím jak ji mám nastavit"), takže doporučuji se lépe seznámit s tím kódem, co že to vlastně dělá a jaká mohou být vstupní data.