Až nepoměrně často se v PHP kódech setkávám s jedním zlozvykem. Použitím empty konstrukce téměř všude kde je to jen možné. Přitom je to podle mě jedna nejvíce zneužívaných věcí v PHP, která ne jen že snižuje výkon ale je i nebezpečná.

Celkem běžně je možné se setkat s PHP projekty, které v podstatě vůbec nepoužívají běžné porovnání if ($var) místo toho je všude if (!empty($var)) a byl jsem nemile překvapen jak je tohle zakořeněné i u lidí, kteří dělají v PHP několik let a nelze je tedy považovat za začátečníky. Podle mě to vzniká tak, že to prostě někde odkoukali z nějakého podobného projektu a tak narvou empty naprosto všude, vždyť ono to funguje, tak co si s tím lámat hlavu. Dost často ani nevědí, že např. deklarované třídní proměnné mají implicitně hodnotu NULL, takže použití empty je pak naprosto zbytečné.

Další důvod bude ten, že když narazí na nějaký kód, ze kterého se sypou NOTICE chyby o nedeklarovaných proměnných nebo properties, tak tam prostě zase dají vyzkoušené empty a tak se tenhle zlozvyk dále rozvíjí a upevňuje. Tohle by ale určitě zkušenější programátor neměl dělat.

Empty je takový mladší brácha isset a společně jsou nejpoužívanějšími konstrukcemi v PHP vůbec. Což bohužel potvrzuje vžitou představu o PHP vývojářích jako bastličích. Nemá cenu popisovat k čemu vlastně tyhle konstrukce jsou jen pár vlastností, na které se těchto konstrukcí často zapomíná

• isset vrací FALSE i v případě, že proměnná existuje a má hodnotu NULL
• isset může mít více než jeden parametr
• mohlo by se zdát, že se uvnitř isset a empty se nemohou zavolat žádné funkce, ani před PHP 5.5 to nebyla tak úplně pravda

K poslednímu bodu, vždy bylo možné volat funkce a testovat až výsledná pole a objekty, např. tedy empty($this->getItems()->first["title"]); isset(filterNames("name")[0]->surname). Pak se také funkce mohou zavolat "skrytě", empty a isset na objektech mohou volat magické funkce __isset apod., v nich se už může dít cokoli. Možná by jste to ani neřekli, tohle

empty($sessionNamespace->items[0]);

volání nad session zavolá spousty funkcí a metod a nakonec klidně i nastartuje session.

A teď k tomu nejdůležitějšímu, proč empty nepoužívat a jaké jsou jeho vedlejší efekty

Empty vám zamaskuje překlepy

if (!empty($this->configuration["payment"][$type]["isFree"])

Dvojité skryté podmínky

Empty s kladným výsledkem je vlastně zápor

function getItems() {
  foreach () {
   if (.....) {
    foreach (....) {
      $items[] = $item;
    }
   }
 }
 
 return isset($items) ? $items : array();
}