Hallo,

ich würde gerne wissen wo man Fehler am besten abfängt, in der Klasse oder in der Anwendung, die diese Klasse benutzt, oder in jeder Methode.

Damit meine ich z.B. wenn in einer Klasse bestimmte Objekte einer anderen Klasse benutzt werden, sollte ich da beim Übergeben der Objekte innerhalb der Klasse prüfen ob es wirklich Objekte sind (also z.B. mit is_object) oder sollte ich das in der Anwendung tun ehe ich das Objekt übergebe oder in jeder Methode der Klasse in der das Objekt benutzt wird?

Oder wenn man z.B. einen Pfad an eine Klasse übergibt und auf diesen Pfad in mehreren Methoden zugegriffen wird, sollte man den Pfad dann bei jeder Methode neu prüfen, da er sich ja u.U. ändern könnte (weil z.B. gerade jemand die Ordnerstruktur verändert etc.) oder nur im Konstruktor?

Also ich denke, wenn man Klassen schreiben will, die wirklich sehr 'unabhängig' von der Anwendung sein sollen und sehr sicher, dann sollte man wahrscheinlich wirklich immer prüfen, aber wie macht man das so im allgemeinen, wenn man z.B. eine Anwendung schreibt und Klassen die etwas 'mehr' auf die Anwendung zugeschnitten ist, prüft man dann trotzdem alles und entsteht da ein Performanceverlust, wenn man viel prüft?

Das ist wahrscheinlich eine ziehmlich blöde Frage, aber vielleicht könnt ihr mir sagen, wie ihr das so macht.

Mister Ad

robo47

argl 2-3 mal ein enter hätte deinem text gut getan

naja fehler fang ich mit ner eigenen klasse für den error_handler ab, so hab ich schonmal die möglichkeit php-seitige fehler, notices etc in ne Datei umzuleiten und den ablauf des scriptes nicht zu stören.

Zum Thema prüfen, naja wenn man sich sicher sein kann, dass der Wert nur übergeben und nicht verändert wird, dann sollte es kein problem sein ihn nur einmal zu überprüfn, jedes mal überprüfen kostet performance, das ist klar, jenachdem wie komplex die prüfung ist, macht es sich bemerkbar.

mfg
robo47

__________________
robo47.net - Blog, Codeschnipsel und mehr
| Caching-Klassen und Opcode Caches in php | Robo47 Components - PHP Library extending Zend Framework

Danke für Deine Antwort

Das hast Du gesagt. <ggg>

Prüfen mußt Du grundsätzlich das, was zweifelhaft ist. Und zweifelhaft sind ausnahmslos alle Daten, die vom Client stammen.

Es ist egal, wo Du prüfst. Hauptsache, Du tust es.

imported_Ben

Das kommt darauf an, was Du machen willst.

Wenn ich z.B. $db->query($sql) aufrufe, dann verwende ich intern (also in der Methode) einen Aufruf einer Error-Klasse bzw. einer entsprechenden Methode, um eventuelle Fehler abzufangen und aufzubereiten.


Also, wenn Du ein Objekt brauchst, dann solltest Du auch überprüfen, ob Du ein Objekt hast. Das ist klar.
Ich mache das generell so, dass ich die Übergabeparameter von Methoden in der Klasse überprüfe. Das scheint mir sinniger zu sein, als die Anwendung, die ja auch für andere Personen als mich gedacht sein können, mit solchen Überprüfungen aufzublasen.

Ein Ansatz ist z.B., dass Du in set-Methoden überprüfst, ob der Datentyp des Übergabeparameters stimmt. Ist dies nicht der Fall wird z.B. eine Exception geschmissen, ausgewertet und es passiert irgendwas

Wenn Du dann später z.B. eine Membervariable verwendest, dann überprüfe ich eigentlich nicht.
Ich reagiere nur entsprechend auf die Fehler, die eventuell auftreten. Indem man sich Exception-Klassen (extends Exception) schreibt kann man da nette Sachen bauen.

Siehe meine obigen Bemerkungen.

Solange Du die Überprüfung nach der Devise "So viel wie möglich, aber nicht mehr als nötig!" ablaufen lässt, sollte Dir der Performance-Einschnitt egal sein.

Nö. Meiner Ansicht ist das eine der wenigen Fragen, die mich in den letzten Wochen in diesem Forum interessiert hat

Grüße Ben.


EDIT:
@robo:
Sorry, aber das ist ein Witz. Ehrlich gesagt denke ich das bei Deinen Texten auf des Öfteren

Frei nach der Signatur von Idefix83:

__________________
Webentwickler-Blog - Buchbesprechung PHP Design Patterns - SQL-Antipatterns

HStev

also ich benutze ganz gerne Klassen in Verbindung mit fast gleichnamigen Exceptionklassen dann kann man gleich mit try catch reagieren im prinzip sind Exception ja fast nichts anderes als eine weitere Kontrolstruktur

zu deinen anderen Fragen: Objekte und Pfade würde ich grundsätzlich bei jeder Übergabe von außerhalb der Klasse überpürfen also im setter

ps: ich find die Frage auch nicht blöd weil ich hab für mich selbst immer noch keine zufriedenstellende Fehlerbehandlung gefunden

__________________
Gewisse Dinge behält man besser für sich, z.B. das man gewisse Dinge für sich behält.

imported_Ben

Hi,
also ich finde eigentlich dieses Beispiel da ganz nett:
http://www.php.net/manual/de/language.exceptions.php

Da kann man sehr gut drauf aufbauen.

Grüße Ben.

__________________
Webentwickler-Blog - Buchbesprechung PHP Design Patterns - SQL-Antipatterns

HStev

ja so in etwa benutze ich das auch nur das meine Exception halt ne StandartException sind wo jediglich der Name anders ist

aber richitg zufrieden stellt mich das immer noch nicht aber man kann mit leben

imported_Ben

Ich würde eben gerne das Script ohne Abbruch etc. weiterlaufen lassen, so dass der User zwar die gewünschte Funktionalität nicht ausführen kann, allerdings trotzdem noch die Möglichkeit hat die Anwendung zu nutzen.

Soweit bin ich aber leider noch nicht.
Wenn da jemand Ideen oder Ansätze hat .. ich bin ganz Ohr

Grüße Ben.

__________________
Webentwickler-Blog - Buchbesprechung PHP Design Patterns - SQL-Antipatterns

Mal im Ernst: welche Fehler erwartest Du eigentlich in einer DB Klasse, die den SQL Server konnektieren und die DB selektieren konnte? Wenn im SQL String alle Werte mit mysql_escape_string behandelt als Strings enthalten sind, sind Fehler nahezu (ich bin vorsichtig) ausgeschlossen. Der einzige Fehler, der dann noch auftreten kann, wäre der, das es kein Ergebnis gibt, weil absolut nix im Datensieb hängengeblieben ist.