.Net开发人员必须避免的5个常见的编程错误

￡神魔★判官ぃ 2021-11-19 13:54 211阅读 0赞

来源：http://www.amazedsaint.com/2010/02/top-5-common-programming-mistakes-net.html

Some time back, I asked a question in Stackoverflow.com, about common programming mistakes .NET developers must avoid. The response was awesome, to say the least. I’m just listing down the top 5 developer crimes I picked, from the answers I received (regardless the votes).

### 1. Breaking the stack unnecessarily when you re-throw an exception ###

This a pretty ‘old thing’ - but surprisingly, still a very common mistake. As [TheSoftwareJedi][] mentioned, you don’t really need to break the stack while throwing exceptions. I’ve done this myself when I was a beginner - and I’ve seen this more often than anything else when I do code reviews these days.

To clarify the point - What is the difference between

*try \{ ..\} catch (Exception ex) \{ throw ex; \}*

and

*try \{..\} catch(Exception ex) \{ throw; \}* ?

[![image][]][image 1]

And when you lose the stack trace, you can’t debug your app – and even worse, you can’t log your error details properly to your error log.

The MSDN guidelines on exception handling clearly states

> Do not rethrow by throwing the same exception object. This causes the stack trace for the original exception to be lost--use a lone "throw;" statement (without an object following it) to "rethrow" a catch exception.

### 2. Not using *using* to dispose objects ###

When ever you initialize an IDisposable object, it is a good practice to initiate it in a using statement to ensure the object is getting disposed properly, like this.

[![image][image 2]][image_image 2]

Most developers won’t do that. [Greg Dean][] pointed the same.

[![image][image 3]][image_image 3]

Here is a little bit of re-cap from MSDN

> As a rule, when you use an IDisposable object, you should declare and instantiate it in a using statement. The using statement calls the [Dispose][] method on the object in the correct way, and it also causes the object itself to go out of scope as soon as [Dispose][] is called. Within the usingblock, the object is read-only and cannot be modified or reassigned.
> 
> The using statement ensures that [Dispose][] is called even if an exception occurs while you are calling methods on the object. You can achieve the same result by putting the object inside a try block and then calling [Dispose][] in a finally block; in fact, this is how the using statement is translated by the compiler.

Also, do you know that a using statement is expanded by the compiler to a try\{..\} finally\{..\} block at compile time? Read more tips about [using statement][] in MSDN.

### 3. Not unhooking event handlers appropriately after wiring them. ###

I think [Scott Langham][] brought up a great point about not unhooking event handlers. People don’t really ‘remember’ to unhook events.

[![image][image 4]][image_image 4]

In C\#, when you register an event handler, you are creating a strong reference from the event source to the listener. So, the listener won’t get garbage collected even if you don’t have any pointers to the listener - unless you unhook the event by yourself, and this might even result in a memory leak.

So, if you are thinking about how to deal with situations where your Source is having a longer life span than the listener – there are multiple ways to deal with it, and Daniel Grunwald covers them nicely in [this excellent Codeproject Post.][]

### 4. Forgetting that Strings are immutable ###

In .NET, we know that strings are immutable – which means, once a string is created, its value can’t be changed. All string operations you perform, actually returns a new string containing the modification.

[John Sonmez][] have a nice point with an example.

[![image][image 5]][image_image 5]

### 5. Not Overriding GetHashCode when overriding Equals method in C\# ###

Why this is important? If Hashcode of two items does not match, they’ll be never considered equal, and the Equals method will never be called to execute your custom comparison logic – let us say, in scenarios where your objects are used as a key in a dictionary or so. Hence, in first place, you should override GetHashCode and return the same value for two equal objects based on comparison logic. I’ve seen this a couple of times during code review.

So, though [Mark Gravell][] has several interesting points in his answer, my pick is about not overriding GetHashCode when you override Equals method in C\#.

[![image][image 6]][image_image 6]

You may read my other [Back To Basics][] posts here, you may find them interesting.

Note: As mentioned, these are my favorite picks of common problems, with some custom commentary. I suggest you to go through [the entire thread here in Stack Overflow][] if you are interested. Happy coding!!

转载于:https://www.cnblogs.com/net205/archive/2011/09/15/2176834.html

[TheSoftwareJedi]: http://stackoverflow.com/users/18941/thesoftwarejedi
[image]: http://lh5.ggpht.com/__Mw4iY-4nuY/S28cZwaOcMI/AAAAAAAAAmA/wQYZuVrYeo4/image_thumb%5B3%5D.png?imgmax=800
[image 1]: http://lh5.ggpht.com/__Mw4iY-4nuY/S28cYvY_CcI/AAAAAAAAAl8/6eIkBnMIKWI/s1600-h/image%5B5%5D.png
[image 2]: http://lh6.ggpht.com/__Mw4iY-4nuY/S28ccveoQjI/AAAAAAAAAmI/PD7UrT6Ldq0/image_thumb%5B12%5D.png?imgmax=800
[image_image 2]: http://lh5.ggpht.com/__Mw4iY-4nuY/S28cbRLwMKI/AAAAAAAAAmE/-I76qmCq9UM/s1600-h/image%5B20%5D.png
[Greg Dean]: http://stackoverflow.com/users/4430/greg-dean
[image 3]: http://lh6.ggpht.com/__Mw4iY-4nuY/S28cfs4ImLI/AAAAAAAAAmQ/H6aufMBhRzg/image_thumb%5B6%5D.png?imgmax=800
[image_image 3]: http://lh5.ggpht.com/__Mw4iY-4nuY/S28cdgVLCyI/AAAAAAAAAmM/v6QtjRCUrCI/s1600-h/image%5B10%5D.png
[Dispose]: http://msdn.microsoft.com/en-us/library/system.idisposable.dispose.aspx
[using statement]: http://msdn.microsoft.com/en-us/library/yh598w02.aspx
[Scott Langham]: http://stackoverflow.com/users/11898/scott-langham
[image 4]: http://lh3.ggpht.com/__Mw4iY-4nuY/S28ciFoXu6I/AAAAAAAAAmY/rs8RrloxOCQ/image_thumb%5B9%5D.png?imgmax=800
[image_image 4]: http://lh5.ggpht.com/__Mw4iY-4nuY/S28cg5nuO4I/AAAAAAAAAmU/HX3tuThnw2A/s1600-h/image%5B15%5D.png
[this excellent Codeproject Post.]: http://www.codeproject.com/KB/cs/WeakEvents.aspx
[John Sonmez]: http://stackoverflow.com/users/45365/john-sonmez
[image 5]: http://lh6.ggpht.com/__Mw4iY-4nuY/S28clSQRIDI/AAAAAAAAAmg/plWTtciuk4M/image_thumb%5B15%5D.png?imgmax=800
[image_image 5]: http://lh6.ggpht.com/__Mw4iY-4nuY/S28cjb25zbI/AAAAAAAAAmc/c1mdjoCHxt0/s1600-h/image%5B25%5D.png
[Mark Gravell]: http://stackoverflow.com/users/23354/marc-gravell
[image 6]: http://lh6.ggpht.com/__Mw4iY-4nuY/S28coEp6IeI/AAAAAAAAAmo/IrLlYiln5Ak/image_thumb%5B18%5D.png?imgmax=800
[image_image 6]: http://lh4.ggpht.com/__Mw4iY-4nuY/S28cmZFDrXI/AAAAAAAAAmk/tS2Zi5OCaC8/s1600-h/image%5B30%5D.png
[Back To Basics]: http://amazedsaint.blogspot.com/search/label/Back%20To%20Basics
[the entire thread here in Stack Overflow]: http://stackoverflow.com/questions/380819/common-programming-mistakes-for-net-developers-to-avoid