Sinnloser Code

[Forumsgast]

Gerade allen ernstes folgenden Code gefunden, den ein Dienstleister in einem alten System von nem Kunden eingepflegt hat:

    <script type=”text/javascript”>

    document.write(“<noscript>…</noscript>”);

    </script> 

...was soll man dazu sagen

[Forumsgast]

Muahahaha da hab ich auch einen, wenn man das hier um stinkenden code erweitern darf...

public function load($extra = false, $moreextra = false, $moremoreextra = false, $moremoremoreextra = false, $moremoremoremoreextra = false, $moremoremoremoremoreextra = false, $moremoremoremoremoremoreextra = false, $moremoremoremoremoremoremoreextra = false) {
    $loadmethod = "load" . $this->objecttype;
    $this->data = $this->db->$loadmethod($this->id, $extra, $moreextra, $moremoreextra, $moremoremoreextra, $moremoremoremoreextra, $moremoremoremoremoreextra, $moremoremoremoremoremoreextra, $moremoremoremoremoremoremoreextra);
  } 

Das ist das Mutterobjekt/oder auch Mutterschiff für ein Modul, das ich vor Urzeiten mal geerbt hab..

[Forumsgast]

Grins...ich liebe solche Varibalennamen $moremoremoremoremoremoremoreextra - geiler Code

Edit:

Das ist das Mutterobjekt/oder auch Mutterschiff für ein Modul, das ich vor Urzeiten mal geerbt hab..

Na happy Birthady

[Forumsgast]

Naja, derjenige hat das eiskalt durchgezogen - auch ne Art von "Interface"

[Forumsgast]

...bei den ganzen Extras war das bestimmt ein Hammer Tool

[Forumsgast]

Nur wegen solchem Code gibt es doch diese Seite: DailyWTF

[Forumsgast]

Nur wegen solchem Code gibt es doch diese Seite: DailyWTF

...kannte ich nicht, aber nice...lol...

[Forumsgast]

Es geht auch einfacher, hat mein Kollege mal produziert:

if ( $var != 0 ) {
  $var = 0;
} 

[Forumsgast]

Neulich in einem übernommenen Projekt gefunden:

while (true) {
    break;
} 

try {
    $this->parse();
} catch(\Exception $e) {
    throw $e;
} 

/edit: Da wurde keine einzige Zeile ausgelassen!

[Forumsgast]

try {
    $this->parse();
} catch(\Exception $e) {
    throw $e;
} 

/edit: Da wurde keine einzige Zeile ausgelassen!

Macht das nicht sogar in soweit Sinn, um Datei und Zeile mit reinzubringen?
Schreibt man mehrere derartige Blöcke hintereinander kann man wenigstens unterscheiden, welcher fehlgeschlagen ist (wenn z.B. alle die gleiche Methode aufrufen und somit die gleiche Exception werfen würden).

[Forumsgast]

Ähm... nö. Wozu gibt es Stack Trace?

[Forumsgast]

Naja, kommt darauf an, welche Zeilenangabe du haben willst. Ich würde eher vorziehen, die Zeile der ursprünglichen Exception zu haben. In einem catch-Block würde ich eh nicht noch einmal dieselbe werfen. Das mag vielleicht unter gewissen Umständen ein Design-Pattern sein, aber ich finde es eher verwirrend. Der catch-Block ist zum Abfangen von Exceptions da, d.h. dass hier Maßnahmen ergriffen werden wie z.B. Log-Einträge, Ausgabe vernünftiger Fehlermeldungen oder eben Weiterführung des normalen Programmflusses. Sollte das Werfen einer Exception hier nötig sein, würde ich eher eine andere Exception vorziehen, um das Debuggen zu erleichtern.

[Forumsgast]

Geht man von der "Was du wirfst solltest du auch Fangen"-Devise aus, ist es absurd in einem catch() eine Exception zu werfen, es sei denn du willst überliegende try's als Injektionsebene nutzen und dort Fehlerhandlingtechnisch weiter machen. Aber das das noch irgendwo nach 50-100 "Redirects" wartbar bleibt, vermute ich waage: ist zu bezweifeln.

[Forumsgast]

Naja, so ganz absurd ist es nicht. Man kann ja durchaus versuchen, eine Fehlerbehandlung zu machen. Nicht jede Exception muss ja gleich die gesamte Anwendung abbrechen.
Kann der Fehler nicht behandelt werden, so wirft man erneut eine Exception. Aber sinnvollerweise nicht dieselbe, sondern eher eine CouldntHandle-Exception oder so.

[Forumsgast]

Seit PHP 5.3 wird das werfen von Exceptions in catch-Blöcken durch den $previous Parameter sogar explizit unterstützt. Typische Anwendung z.B. in Datenbank-Mappern: Das Wrappen von PDO-Exceptions in eigener Exception mit zusätzlichen Informationen.

"Was du wirfst solltest du auch Fangen" - sehe ich nicht so, es sollte nur bekannt gemacht werden, was für Exceptions geworfen werden können und da macht eine Vereinheitlichung natürlich Sinn. Es sei denn, du meintest mit "du" gar nicht eine Komponente in sich. Ein guter Grundsatz wäre "Was geworfen wird, solltest du auch fangen" - und zwar möglichst schon eine Ebene höher und nicht erst am unteren Ende des Call Stacks.