Hallo zusammen!
Keine Ahnung ob das wer liest, scheint ja gewaltig viel los zu sein in diesem Forum, aber ich hab mich grad über einen Codeabschnitt in Ihrer aktuellen Ausgabe ziemlich geärgert, da muss ich mir jetzt Luft machen ;-)
Also, es geht um den Artikel
"Short tasks vs. Long tasks" in Ausgabe 11/2010 des .Net Magazins
Seite 54, Listing 2 die Methode "Execute"
(Ich erspar mir jetzt das Abschreibendes Codes, falls sich eine Diskussion ergibt darf das jeder der im Besitz der Sourcen ist gerne übernehmen)
Diese Methode hat einen ganz schweren Design Fehler:
JEDE Art von Exception wird in eine ArgumentException umgewandelt, und der Stacktrace abgeschnitten
Ausserdem ist die try/Finally Verwendung für den OverrideCursor mehr als Suboptimal gelöst.
Ich weiss aus eigener Erfahrung (auch als Autor), dass man das Thema Exceptionhandling in vielen Beispielen aus Übersichtsgründen komplett weglässt, das finde ich ja noch OK!
Aber wenn man ein Beispiel mit Exceptionhandling veröffentlicht, in welchem es zum Teil explizit um dieses Handling geht (Cursor/Try/Catch/Finally Handling) dann sollte der Code wenigstens keine OFFENSICHTLICHEN Fehler enthalten.
Speziell wenn man sich im Abspann des Artikels noch damit brüstet, dass dieser Code einen Innovation Award gewonnen hat
Schöne grüße
Werner Mairl
PS.: Ja, das ist mein erstes Post in diesem Forum, aber dafür stehe ich auch mit meinem Realname dafür ein
Keine Ahnung ob das wer liest, scheint ja gewaltig viel los zu sein in diesem Forum, aber ich hab mich grad über einen Codeabschnitt in Ihrer aktuellen Ausgabe ziemlich geärgert, da muss ich mir jetzt Luft machen ;-)
Also, es geht um den Artikel
"Short tasks vs. Long tasks" in Ausgabe 11/2010 des .Net Magazins
Seite 54, Listing 2 die Methode "Execute"
(Ich erspar mir jetzt das Abschreibendes Codes, falls sich eine Diskussion ergibt darf das jeder der im Besitz der Sourcen ist gerne übernehmen)
Diese Methode hat einen ganz schweren Design Fehler:
JEDE Art von Exception wird in eine ArgumentException umgewandelt, und der Stacktrace abgeschnitten
Ausserdem ist die try/Finally Verwendung für den OverrideCursor mehr als Suboptimal gelöst.
Ich weiss aus eigener Erfahrung (auch als Autor), dass man das Thema Exceptionhandling in vielen Beispielen aus Übersichtsgründen komplett weglässt, das finde ich ja noch OK!
Aber wenn man ein Beispiel mit Exceptionhandling veröffentlicht, in welchem es zum Teil explizit um dieses Handling geht (Cursor/Try/Catch/Finally Handling) dann sollte der Code wenigstens keine OFFENSICHTLICHEN Fehler enthalten.
Speziell wenn man sich im Abspann des Artikels noch damit brüstet, dass dieser Code einen Innovation Award gewonnen hat
Schöne grüße
Werner Mairl
PS.: Ja, das ist mein erstes Post in diesem Forum, aber dafür stehe ich auch mit meinem Realname dafür ein
Comment