Refactoring Tips - Tip 3
Tip 3
In a class,if the expression is repeating across the methods. Then encapsulate/wrap that with either a Property or Method as shown below.
This increases the code reuse,easy to extend and maintain
Bad practice
public class A
{
private int Var1;
private int Var2;
private void Method1()
{
string s = this.Var1 * this.Var2
}
private void Method2()
{
string s = this.Var1 * this.Var2
}
}
Good practice
public class A
{
private int Var1;
private int Var2;
private int Expression
{
get
{
return this.Var1 * this.Var2;
}
}
private void Method1()
{
string s = this.Expression
}
private void Method2()
{
string s = this.Expression;
}
}
I hope this helps!.
Regards,
-Vinayak
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.