Click here to Skip to main content
11,642,381 members (67,168 online)
Click here to Skip to main content

Tagged as

Refactoring Tips - Tip 11

, 10 Mar 2010 CPOL 4.7K 1
Rate this:
Please Sign up or sign in to vote.
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)

Share

About the Author

Vinayaka Krishna Shenoy
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.

You may also be interested in...

Comments and Discussions

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

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

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

| Advertise | Privacy | Terms of Use | Mobile
Web03 | 2.8.150731.1 | Last Updated 11 Mar 2010
Article Copyright 2010 by Vinayaka Krishna Shenoy
Everything else Copyright © CodeProject, 1999-2015
Layout: fixed | fluid