Jak nastavit error_reporting() tak, při $this->neexistujici_promenna vypsal chybu – PHP – Fórum – Programujte.com

Ahoj,

nejak se mi nedaří nastavit error_reporting() (nebo ini_set('error_reporting',...)
tak, aby vypsal Warning o pristupu k neexistujici promenne.

Priklad:

error_reporting(E_ALL | E_STRICT | E_NOTICE);  

class A

{

	public $promenna;



	function __construct()

	{

		$this->neexistujici_promenna = "nic";

	}

}

$obj = new A();

Jarda
A co direktiva display_errors[1]?

[1] http://cz2.php.net/manual/en/errorfunc.configuration.php#ini.display-errors

To bukaj :
Bohuzel zadna zmena, stale žádná hláška. Dost mě to štve kvůli snadnému zanesení chyby díky překlepu -
chci nastavit proměnnou například width pomocí $this->widht a php mě na překlep neupozorní.

Pokud se pokusim pouzit neinicializovanou promennou,
např.

echo $neinicializovanaPromenna;

class test{

    function __construct(){

        $this->varialbe = 'null';

    }   

    

    function __set( $var ){

        print 'Neexistujici promenna:  $' . $var;   

    }

}

new test();

Jarda
Ano, tak jsem to otestoval a opravdu PHP nikdy nic nezahlásí. Dokonce jde taková nedeklarovaná, ale „vytvořená“ vlasnost využívat pak i v jiných metodách. PHP mě opět zklamalo (už jsem si ale nějak zvykl :o)).

zaachi
Popripade __Get je pro neexistujici metody.
Jen bych poopravil, ne __get(), ale __call()[1].

[1] http://php.net/__call#language.oop5.overloading.methods

Jej pardon. Uz si to moc nepamatuju jak vidim :)
__get je pro vlastnost, ktera neni zpristupnena pro cteni.

To zaachi :
To bukaj :

(zaregistroval jsem se pod nickem Jasir, tedy Jarda ~ Jasir;)

Panové, děkuji, vidím, že budu dělat spoustu překlepů i nadále. :(
V tomto je PHP opravdu s prominutím na prd.

Jak to vidím já, nejlepší je zřejmě vůbec nepřistupovat k veřejným vlastnostem (proměnným :)
pomocí $obj->vlastnost ale pro každou vlastnost vytvořit get/set metody. Je to práce navíc, mírné zpomalení,
na druhou stranu z hlediska návrhu to není vůbec špatné.

Co se týče privátních vlastností, tam to jde v podstatě také, ale už je to docela opruz vyrábět get/set
metody, takže buďto maximálně zvýšená opatrnost, a nebo opravdu pro každý objekt vytvořit __get(), __set() metody s hláškou chyby/překlepu.

Vytvareni getteru/setteru neni vubec spatna praxe. Treba ja ve svych tridach public promenne vubec nevedu.

No a nebo pouzivat lepsi editor, ktery metody a vlastnosti nabizi a tak se jednoduse vyhnout preklepum.

doporucuji si precist tyto dva clanky. jsou protichudne, jeden reaguje na druhy, ale myslenky jsou velmi pekne, a mozna tam najdete nekde i vlastni nazor. :)

programming-psychology-ii-private-methods -http://debuggable.com/posts/programming-psychology-ii-private-methods:481ed862-b0d8-4a0e-9247-165c4834cda3
not-everything-has-to-be-public - http://cakebaker.42dh.com/2008/07/08/not-everything-has-to-be-public/

osobne si mi libi spis prvni nazor. :)

Ja jsem zastance druheho hlavne z praktickeho hlediska. Viz:

$object=new object; 

$object->varA="valA";

$object->varB="valB";

$object->varC="valC";

$object->doSomething();

object()->setA("valA")->setB("valB")->setC("valC")->doSomething();

hrach
Zajímavé články. Hlavně ten první. Ale ještě zajímavější jsou komentáře pod ním. Největší problém validace až při ukládání bych viděl v tom, co napsal Lars Strojny v prvním bodě svého komentáře[1] – není vidět, kde chyba nastala, takže se může táhnou v instanci bůhví odkud a ve větších systémech to bude opravdu těžké zjistit. Teď si nemohu vzpomenout, jak se tomu principu přesně nadávalo v angličitně, ale bylo to něco ve smyslu, že pokud nastane chyba, je potřeba ji oznámit co nejdříve a co nejhlasitěji – je pak jednodušší takovou chybu odstanit. Když nevím, kde chyba nastala nastala, její najití je mnohem těžší, je-li možné.

A další věc, ke které mě přivedl komentář od jazzslidera, je to, že mnoho lidí (autor článku nejspíš taky) nahlíží na instanci jako na strukturu. Když budem na bakovní účet nahlížet jako na strukturu, která v sobě akorát skrývá výši zůstatku, opravdu asi nemá cenu psát gettery a settery – „je to práce na víc“. Pokud ale na instanci opravdu pohlížíme jako na entitu, která nám poskytuje nějaké možnosti s ní pracovat (viz ten interface, který je v tom komentáři), zpřítupňování vlastností by bylo akorát na obtíž.

Z toho článku cítím jediné – díky příkladům ušitým „na míru“ řešení se dá ostatními jednoduše manipulovat :o)

[1] http://debuggable.com/posts/programming-psychology-ii-private-methods:481ed862-b0d8-4a0e-9247-165c4834cda3#comment-48726288-7480-48c2-9976-4f8d4834cda3
[2] http://debuggable.com/posts/programming-psychology-ii-private-methods:481ed862-b0d8-4a0e-9247-165c4834cda3#comment-48737fa1-ee00-42d6-bfd0-7c634834cda3

Zajímavé články, s prvním nesouhlasím, resp. souhlasím s Larsem Strojnym a bukajem :)
Btw, já to řeším nějak takhle, pokud chci třídu která bude mít možnost setovat property tak ji udělám potomkem
této třídy. Nic však není dokonalé ;)

minusy:
- setters/getters fungují i na privátní properties (funkce property_exists() nerozlišuje)
- zpomalení

Co si o takovém řešení myslíte?

class Core_ObjectWithProperties

{

   /**

   * Toto je __stub funkce pro settery a gettery tridy.

   * Problem - public/private, da se nastavit i private

   */

   function __call($fcename,$args)

   {

      $fce=substr($fcename,0,3);

      $var=strtolower(substr($fcename,3,1)).substr($fcename,4);

      $cnt=count($args);

      

      switch($fce)

      {

         case 'set':

            if($cnt != 1)

               throw(new Exception("setter $fcename() needs exactly ONE parameter!"));



            if(!property_exists($this,$var))

               throw(new Exception("$fcename() - class variable \$$var does not exists!"));   

            

            $this->$var = $args[0];



            break;

         case 'get':

            if($cnt != 0)

               throw(new Exception("getter $fcename() does not take any parameter!"));

            if(!property_exists($this,$var))

               throw(new Exception("$fcename() - class variable \$$var does not exists!"));   

         

            return $this->$var;

            

            break;

         default:

            throw(new Exception("unknown method called: $fcename()"));

         

      }

   }



}

To bukaj : Myslí si, že nejlepším řešením je samozřejmě kombinace obojího. Viz. třeba metoda isValid(). Jeji vytvoreni size onen pristp s getery setery nevylucuje, ale primo ji nevyzaduje :) Tak to aspon vnímám já.