Fail Code – WYSI… WTF!

Je mehr Prüfungen man macht, desto sicherer wird der Code:

            if (windowHandelsLocked && !windowHandelsLocked)
                return;

Fail.

About the author

Written by Robert Mühsig

Robert Mühsig (@robert0muehsig) ist Webentwickler und beschäftigt sich mit Web-Frameworks (vor allem dem ASP.NET MVC Framework) und scheut sich auch nicht vor Javascript. Ansonsten bloggt er über all jene Probleme, die ihm über den Weg laufen. Seit 2008 ist er Microsoft MVP für ASP.NET und er arbeitet bei der T-Systems Multimedia Solutions GmbH in Dresden. Treffen kann man ihn online via Twitter (@robert0muehsig) oder dieser Seite oder bei der .NET User Group Dresden.

5 Responses

1. 

   Mariusz

   3. September 2008

   Beide

   Reply
2. 

   Mariusz

   3. September 2008

   …Fälle abgedeckt, besser gehts nicht

   *Irgendwie hat das System mein ersten Kommentar zum Teil aufgefressen :-/

   Reply
3. 

   Stefan Lieser

   3. September 2008

   ReSharper markiert den Ausdruck und schreibt dazu "Expression is always false"

   Reply
4. 

   Peter Bucher

   5. September 2008

   Hehe, der ist mal geil

   Reply
5. 

   Rainer Schuster

   5. September 2008

   Kleine Empfehlung von mir. wenn du Deklarativ programmieren willst, was ich auch tue, leg dir eine Hilfsbibliothek an.  wenn du also ein und Ausgabeparameter auf gültigkeit abprüfst wirst du feststellen, dass du immer wieder über die gleichen Prüfunge stolperst.

   Ich bin der Meinung, ich hätte sowas in den Rhino Tools von Oren Eini gesehen. Mal nachschauen …

   sowas in der Richtung wie:

   Check.StringNotNullOrEmpty(value)
   Check.Not.Null(value)

   Reply