Caught By .Net! - OODesign http://blog.stoneskins.com/ Stoneskin's Notes en-us SunStone Solutions & C LLC Thu, 04 Oct 2012 05:15:33 GMT newtelligence dasBlog 2.3.12105.0 stoneskin@gmail.com stoneskin@gmail.com http://blog.stoneskins.com/Trackback.aspx?guid=60af0a76-d026-4cbe-abf7-3af5b714ef52 http://blog.stoneskins.com/pingback.aspx http://blog.stoneskins.com/PermaLink,guid,60af0a76-d026-4cbe-abf7-3af5b714ef52.aspx stoneskin http://blog.stoneskins.com/CommentView,guid,60af0a76-d026-4cbe-abf7-3af5b714ef52.aspx http://blog.stoneskins.com/SyndicationService.asmx/GetEntryCommentsRss?guid=60af0a76-d026-4cbe-abf7-3af5b714ef52

Notes of read book "clean code"  by Robert C. Martin

SRP: Single responsibility
DRY: Don't repeat yourself
OCP: open close principle

ex:
1. Switch break the SRP and OCP, should limit it in bottom and factory.
2. Try catch should keep it small ,and no code after try catch block
3. Use exception in stand of ErrorCode
4. Each function should has only one return...try avoid  break, continue, never use goto
5. one parameter is better than two parameters and better than three or more
6. Make sense Name and comments
7. First write the code as you think, then refactor to follow the principle
8. Don't return or pass null, empty list is much better
9. Put Error handle code out side the main logic

Something in the book I don't agree is for Name convention,
he don't like name interface with prefix 'I', and don't like put the type in classname such as AbstractProduct or Iproduct.

But I think that make our life easier when we code...if you need a interface,  With the help of Intelli come with VS or ReSharper ,you type 'I', it will jump out, if you need a Enum...type Enum..all Enum named start with Enum will list for you to choice.

Clean Code rules http://blog.stoneskins.com/PermaLink,guid,60af0a76-d026-4cbe-abf7-3af5b714ef52.aspx http://blog.stoneskins.com/2012/10/04/CleanCodeRules.aspx Thu, 04 Oct 2012 05:15:33 GMT <p> Notes of read book "clean code"&nbsp; by Robert C. Martin<br> <br> SRP: Single responsibility<br> DRY: Don't repeat yourself<br> OCP: open close principle<br> <br> ex: <br> 1. Switch break the SRP and OCP, should limit it in bottom and factory.<br> 2. Try catch should keep it small ,and no code after try catch block<br> 3. Use exception in stand of ErrorCode<br> 4. Each function should has only one return...try avoid&nbsp; break, continue, never use goto<br> 5. one parameter is better than two parameters and better than three or more<br> 6. Make sense Name and comments<br> 7. First write the code as you think, then refactor to follow the principle<br> 8. Don't return or pass null, empty list is much better<br> 9. Put Error handle code out side the main logic<br> <br> Something in the book&nbsp;I don't agree is for Name convention, <br> he don't like name interface with prefix 'I', and don't like put the type in classname such as AbstractProduct or Iproduct. </p> <p> But I think that make our life easier when we code...if you need a interface,&nbsp; With the help of&nbsp;Intelli&nbsp;come with VS or ReSharper&nbsp;,you type 'I', it will jump out, if you need a Enum...type Enum..all Enum named start with Enum will list for you to choice.<br> </p> <p> </p> <img width="0" height="0" src="http://blog.stoneskins.com/aggbug.ashx?id=60af0a76-d026-4cbe-abf7-3af5b714ef52" /> http://blog.stoneskins.com/CommentView,guid,60af0a76-d026-4cbe-abf7-3af5b714ef52.aspx Agile C# OODesign http://blog.stoneskins.com/Trackback.aspx?guid=c0fc9862-3e2c-4c35-bba5-b956e03b9bf1 http://blog.stoneskins.com/pingback.aspx http://blog.stoneskins.com/PermaLink,guid,c0fc9862-3e2c-4c35-bba5-b956e03b9bf1.aspx stoneskin http://blog.stoneskins.com/CommentView,guid,c0fc9862-3e2c-4c35-bba5-b956e03b9bf1.aspx http://blog.stoneskins.com/SyndicationService.asmx/GetEntryCommentsRss?guid=c0fc9862-3e2c-4c35-bba5-b956e03b9bf1 I have some thoughts for making better code. I call it 3 rulers plus 2 sub rulers.

1 Readable : small function, single responsibility, straightforward name convention.
2. Re-useable: clear and simple responsibility, DRY
3. Testable : can be tested by UnitTest tools. Also I like write code that I can easily put a break point there.

+

4. flexible :  Prepare for future change,  configurable, lower coupling. using IoC.
5. performance: improve code for better performance or user experience, Threading, Cache, Ajax, Parallel LinQ,


4 and 5 have lower priority than 123.   You should know why ;)

My 3+2 rules in coding http://blog.stoneskins.com/PermaLink,guid,c0fc9862-3e2c-4c35-bba5-b956e03b9bf1.aspx http://blog.stoneskins.com/2012/09/16/My32RulesInCoding.aspx Sun, 16 Sep 2012 06:47:36 GMT I have some thoughts for making better code. I call it 3 rulers plus 2 sub rulers.<br> <br> 1 Readable : small function, single responsibility, straightforward name convention.<br> 2. Re-useable: clear and simple responsibility, DRY<br> 3. Testable : can be tested by UnitTest tools. Also I like write code that I can easily put a break point there.<br> <br> +<br> <br> 4. flexible :&nbsp; Prepare for future change,&nbsp; configurable, lower coupling. using IoC.<br> 5. performance: improve code for better performance or user experience, Threading, Cache, Ajax, Parallel LinQ,<br> <br> <br> 4 and 5 have lower priority than 123.&nbsp;&nbsp; You should know why ;)<br> <p> </p> <img width="0" height="0" src="http://blog.stoneskins.com/aggbug.ashx?id=c0fc9862-3e2c-4c35-bba5-b956e03b9bf1" /> http://blog.stoneskins.com/CommentView,guid,c0fc9862-3e2c-4c35-bba5-b956e03b9bf1.aspx C# OODesign http://blog.stoneskins.com/Trackback.aspx?guid=78011257-92c9-4015-8948-f860d40111a5 http://blog.stoneskins.com/pingback.aspx http://blog.stoneskins.com/PermaLink,guid,78011257-92c9-4015-8948-f860d40111a5.aspx stoneskin http://blog.stoneskins.com/CommentView,guid,78011257-92c9-4015-8948-f860d40111a5.aspx http://blog.stoneskins.com/SyndicationService.asmx/GetEntryCommentsRss?guid=78011257-92c9-4015-8948-f860d40111a5

http://userpages.umbc.edu/~tarr/dp/lectures/OOPrinciples.pdf

 

http://c2.com/cgi/wiki?PrinciplesOfObjectOrientedDesign

O O Design Principles http://blog.stoneskins.com/PermaLink,guid,78011257-92c9-4015-8948-f860d40111a5.aspx http://blog.stoneskins.com/2008/04/08/OODesignPrinciples.aspx Tue, 08 Apr 2008 15:53:09 GMT <p> <a href="http://userpages.umbc.edu/~tarr/dp/lectures/OOPrinciples.pdf">http://userpages.umbc.edu/~tarr/dp/lectures/OOPrinciples.pdf</a> </p> <p> &nbsp; </p> <p> <a href="http://c2.com/cgi/wiki?PrinciplesOfObjectOrientedDesign">http://c2.com/cgi/wiki?PrinciplesOfObjectOrientedDesign</a> </p> <img width="0" height="0" src="http://blog.stoneskins.com/aggbug.ashx?id=78011257-92c9-4015-8948-f860d40111a5" /> http://blog.stoneskins.com/CommentView,guid,78011257-92c9-4015-8948-f860d40111a5.aspx OODesign