Chyba ERRNO 8 – PHP – Fórum – Programujte.com

Ahoj, php mi hází chybu, a nevím jak ji napravit, chyba se zobrazí pouze když nastavím error handler na E_ALL.

ERRNO: 8

Text chyby: Undefined index: page

Soubor: C:\inetpub\wwwroot\sdjizni\template\sd_jizni\index.phtml

Řádek: 36

<?php

  $page2 = $_GET['page'];

  if(empty($page2)){

    $page->zobraz("uvod","obsah");

  }else{

    if($page->zobraz($page2,"modul") == 1){

      @include 'foto.php';

    }elseif($page->zobraz($page2,"modul") == 2){

      @include "video.php";

    }else{

      $page->zobraz($page2,"obsah");

    }

  }

?>

V poli page neexistuje hodnota s indexem (klicem) 'page'. Mel by to vyresit jeden zavinac. taky bys mel pred to empty pridat jeste is_null kontrolu.

  $page2 = @$_GET['page']; 

  if(is_null($page2) || !$page2){ 

    $page->zobraz("uvod","obsah");

CommanderZ
$page2 = @$_GET['page'];
Používat tlumící operátor? Ale fůj :o) Pryč s ním:

$page2 = isset($_GET['page']) ? $_GET['page'] : null;

To plasmo: Zdá sa mi, že error to nebude. Má to nejaké fatálne následky? Proste ukončenie skriptu? Pokiaľ nie, je to notice. Tie môžeš kľudne ignorovať. Väčšinou sa zobrazujú, keď interpreter narazí na neinicializovanú premennú.
EDIT: ...a keď máš samozrejme toto zobrazovanie Notices v php.ini aktivované :) V tvojom prípade direktíva E_ALL

bukaj napsal:
CommanderZ
$page2 = @$_GET['page'];
Používat tlumící operátor? Ale fůj :o) Pryč s ním:

$page2 = isset($_GET['page']) ? $_GET['page'] : null;

plasmo
Taky, není tlumící operátor před tím include zbytečný?

Díky za rady :-) funguje mi to.

CommanderZ napsal:

bukaj napsal:
CommanderZ
$page2 = @$_GET['page'];
Používat tlumící operátor? Ale fůj :o) Pryč s ním:

$page2 = isset($_GET['page']) ? $_GET['page'] : null;

plasmo
Taky, není tlumící operátor před tím include zbytečný?

No jo, ale vsimni si jedne veci - ten tvuj radek je dvakrat delsi a na prvni pohled je o dost mene jednoznacne, co vlastne dela. A ze dela to same co ten muj ;)

To toka : v tomto pripade se neda o spravnosti mluvit

Kod je spravny do te doby, dokud funguje. E_NOTICE a E_WARNING jsou pouze informacni pro vyvoj aplikace.
@ potlacuje jejich vypis, podminkou zase obejdu onu situaci.... spravne je oboje. :) ani jedno neni spravnejsi.
oboje ma svoje vyhody.

Vzhledem k tomu, že se z empty tenhle notice nevyhodí, proč tu proměnnou nevytvořit až v podmínce (myšleno v bloku za podmínkou), kde je jisté, že index existuje?

operátor @ by som nepoužil, pokial by som si nebol istý že ho použiť chcem. V prípade že si neskôr urobíš vlastný error handler tak aj keď v ňom budeš kontrolovať error reporting level každopádne dôjde ku spomaleniu. V tomto prípade to je jedno, ale ak by sa @ ocitol v nejakom cykle a vyhodil len 1000 notices, už by to bolo cítiť...

Pre mňa je jednoznačne isset jediné správne riešenie, ten operátor ako aj možnosť zníženia E_ALL na nižší level by mali zrušiť...

Hmm myslim ze zitra udelam benchmark. Chtel bych videt, kde by se tam to zpomaleni vzalo ;)