@ Fehlerunterdrückung bei $_GET Variablen und set_error_handler

[Forumsgast]

Hi Community !

Bin grad dabei mich ums Error Handling meiner PHP Engine zu kümmern.

Hab zwar gelesen das Klammeraffen im Code nicht professionell sind aber trotzdem kennen ich einen guten Grund um sie anzuwenden.
Und zwar bei nicht immer definierten $_GET Variablen um ein ständiges if(isset zu vermeiden.

$test = @$_GET['test']; 

statt

if(isset($_GET['test']))
    $test = $_GET['test'];
else
    $test = ''; 

Das Problem:
Seit dem ich den set_error_handler anwende werden Fehler mit @ Zeichen vor dem Code im handler angezeigt.
Was natürlich unpraktisch ist weil ich alle Fehler loggen will wenn sie vorkommen.

Was kann man da tun ?

greetz,
silec

[Forumsgast]

Seit php 7 gibt's auch den ??-Operator:

Statt

if(isset($_GET['test']))
    $test = $_GET['test'];
else
    $test = ''; 

kann man dann einfach:

$test = $_GET['test'] ?? ''; 

machen.

Grüße.

[Forumsgast]

Danke für die schnelle Antwort.
Nette Sache !
Leider zerbrezelt dieser Operator in meinem veralteten Espresso das Code Highlighting.
Gibt es keine Methode dem set_error_handler zu sagen er soll das @ nicht ignorieren ?

[Forumsgast]

aber trotzdem kennen ich einen guten Grund um sie anzuwenden.

Das ist alles, aber kein guter Grund.

Leider zerbrezelt dieser Operator in meinem veralteten Espresso das Code Highlighting.

Auch das ist kein Grund für eine unsaubere Programmierung.

[Forumsgast]

Hab zwar gelesen das Klammeraffen im Code nicht professionell sind aber trotzdem kennen ich einen guten Grund um sie anzuwenden.
Und zwar bei nicht immer definierten $_GET Variablen um ein ständiges if(isset zu vermeiden.

Nein, das ist kein Grund. Programmiere einfach sauber und lass die schmutzigen Hacks.

Als weitere Alternative sollte man noch php_input_filter() nennen.

[Forumsgast]

Ich versteh nicht warum die Mehrheit die PHP Entwickler für Idioten halten und manche Funktionen (in jedem Fall) verteufeln.
Als würden die PHP Entwickler Funktionen einbauen die einem bei Benutzung AUTOMATISCH ins Bein schießen.
Das Ergebnis von

$test = @$_GET['test']; 

und

if(isset($_GET['test']))
    $test = $_GET['test'];
else
    $test = ''; 

ist ident. Bis darauf das der error handler die @ nicht wahrnimmt.
Und das da irgendein anderer Fehler entstehen könnte ist für mich nicht denkbar.
Also suche ich noch immer nach einem Befehl dem Error Handler zu sagen die @´s zu beachten.

[Forumsgast]

Als würden die PHP Entwickler Funktionen einbauen die einem bei Benutzung AUTOMATISCH ins Bein schießen.

Ja, das tun sie immer wieder. Siehe z.B. das Magic Quotes Desaster, das inzwischen endlich wieder entfernt wurde.

Das Ergebnis von

$test = @$_GET['test']; 

und

if(isset($_GET['test']))
$test = $_GET['test'];
else
$test = ''; 

ist ident.

Nein, ist es nicht.

Und das da irgendein anderer Fehler entstehen könnte ist für mich nicht denkbar.

Aha, also wenn etwas für dich nicht denkbar ist, dann kann es nicht existieren?

Also suche ich noch immer nach einem Befehl dem Error Handler zu sagen die @´s zu beachten.

Das ist dein eigenes Problem, das du dir künstlich erzeugt hast. Programmiere sauber, dann hast du auch keine Probleme.

[Forumsgast]

hellbringer

http://php.net/manual/de/language.op...rorcontrol.php :

There is no reason to NOT use something just because "it can be misused". You could as well say "unlink is evil, you can delete files with it so don't ever use unlink".

It's a valid point that the @ operator hides all errors - so my rule of thumb is: use it only if you're aware of all possible errors your expression can throw AND you consider all of them irrelevant.

A simple example is
<?php

$x = @$a["name"];

?>
There are only 2 possible problems here: a missing variable or a missing index. If you're sure you're fine with both cases, you're good to go. And again: suppressing errors is not a crime. Not knowing when it's safe to suppress them is definitely worse.


Wenn man unfähig ist Variablen richtig zu schreiben sollte man @ wohl nicht vor Variablen stellen.

[Forumsgast]

https://stackoverflow.com/questions/...-error-handler

für PHP 7:
Im Error Handler überprüfen ob error_reporting() != 0 ist

if(error_reporting()!=0) 

[Forumsgast]

Mach das, was in deinem geposteten Link steht: Vermeide @.

[Forumsgast]

Hellbringer bitte unterlasse es auf meine Topics zu posten.

[Forumsgast]

Hellbringer bitte unterlasse es auf meine Topics zu posten.

hellbringer, du böser, böser User. Du kannst doch nicht einfach schreiben, dass silec unsauber programmiert und das ändern soll. Also ehrlich, das geht ja mal garnicht.

[Forumsgast]

Hellbringer hat mir schon auf php-forum.de die Laune verdorben. Deshalb habe ich mich hier angemeldet.
Und ausserdem brauche ich von niemandem die Hilfe der "die Hölle bringt"

[Forumsgast]

Laune hin oder her. Seine Anmerkungen sind korrekt, gerechtfertigt und auch wichtig. Für die Anfänger die hier mitlesen soll nicht der Eindruck entstehen dass die Verwendung des @-Operators eine gute Idee ist - unabhängig davon ob du ihn trotzdem verwendest. Das ist nämlich nicht "dein" Topic sondern eine frei zugängliche Ressource die u.U. Einfluss auf die Entscheidungen anderer nimmt.

[Forumsgast]

silec

ein Vorschlag zur Güte:

bau dir eine Function, bspw.:

$test = checkGet('test');