Click here to Skip to main content
15,886,362 members
Articles / Programming Languages / C#
Tip/Trick

Refactoring Tips - Tip 11

Rate me:
Please Sign up or sign in to vote.
3.33/5 (8 votes)
10 Mar 2010CPOL 11.6K   1   2
Refactoring Tips - Tip 11Tip 11Refactoring 'OR' type of conditions - In a method,if we have multiple conditional statements,with the same result. apply this rule.Bad practiceprivate int Method(){ int result = 1; if( condition1) result = 0; if( condition2) ...
Refactoring Tips - Tip 11

Tip 11

Refactoring 'OR' type of conditions - In a method,if we have multiple conditional statements,with the same result. apply this rule.

Bad practice

private int Method()
{
  int result = 1;
  if( condition1)
    result = 0;
  if( condition2)
    result = 0;
  if( condtion3)
    result = 0;

 return result;
}



Good practice


private int OrCondition()
{
 if( condition1 || condition2 || condition3)
  return 0;
 else
  return 1;
}
private int Method()
{
 return OrCOndition();  
}


I hope this helps!.

Regards,
-Vinayak

License

This article, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)


Written By
Architect MindTree Ltd
India India
Motivated achiever who guides organizations in applying technology to business settings, provides added value, and creates project deliverables in a timely manner. An experienced Technical Consultant, have successfully led large project teams of more than 20 people from requirements gathering to implementation and support using C#, .NET ,ADO.NET, ADO.NET Entity Framework,ASP.NET,ASP.NET MVC, WCF and SQL Server.

Comments and Discussions

 
GeneralReason for my vote of 1 very very simple Pin
Member 27222256-Sep-10 0:53
Member 27222256-Sep-10 0:53 
Generalmultiple returns ... Pin
Stefan_Lang11-Mar-10 6:21
Stefan_Lang11-Mar-10 6:21 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.