将非结构化重复代码重构为可重用组件的技巧

Question

我查了又查，没有得到任何相关的答案。希望有人可以提供帮助。

我开始编写类作为一种保存数据的单元，并编写另一个 util 类来从数据库填充该类。好吧，对于小型代码库来说，这似乎不是什么大问题，但展望未来，对于我来说，成长为一名有能力的开发人员，牢牢掌握面向对象的设计和开发，我需要将其发展为适当的设计

让我们开始最初的课程

public class Foo
{
   public int FooID {get;set;}
   public string FooName {get;set;}
   public string FooDescription {get;set;}
}

public class NotRelatedToFoo
{
   public int SomeID {get;set;}
   public string SomeName {get;set;}
   public string SomeDescription {get;set;}
}


public class Util
{
   public Util(){} 

   public List<NotRelatedToFoo> GetNotRelatedToFoos(string SomeName)
   {
     .... DB Code to return Data from DB 

     List<NotRelatedToFoo> notfoos = new List<NotRelatedToFoo>()
     (select * from NotRelatedToFoo where SomeName like @SomeName)
     foreach var item in db
     {
       NotRelatedToFoo notfoo = new NotRelatedToFoo ();
       notfoo.SomeID = item.SomeID;
       notfoo.SomeName = item.SomeName
       notfoo.SomeDescription = item.SomeDescription 
       notfoos.Add(notfoo);
      }
      return notfoos ;
   }

}

我的问题是从这个设计中前进的最佳方式是什么。

编辑 目前，Util 类一团糟，我需要建议如何按照结构化设计提取功能并将其分组在一起，

这行不通，或者可以吗？

public class Foo
{
   public int FooID {get;set;}
   public string FooName {get;set;}
   public string FooDescription {get;set;}

   public List<Foo> GetFoos(string FooName)
   {
     .... DB Code to return Data from DB 

     List<Foo> foos = new List<Foo>()
     (select * from foo where FooName like @FooName)
     foreach var item in db
     {
       Foo foo = new Foo();
       foo.FooID = item.FoodID;
       foo.FooName = item.FooName
       foo.FooDescription = item.FooDescription 
       foos.Add(foo);
      }
      return foos;
   }
}

谢谢

Answer 1

我的问题是，推进此设计的最佳方式是什么。

名为 Util 或 Manager 的类或您拥有的其他类通常具有设计味道。您应该能够描述该类正在做什么（它应该具有单一职责），并且这应该是该类的名称。如果你必须使用通用的、模糊的、包罗万象的术语来描述类正在做什么，那么你就做错了，需要重新设计。

您的 Util 类做得太多了。它从数据库中获取两种不同类型的数据。通过以这种方式设计类，您将违反开放/封闭原则（每次向模型添加新类型时都必须修改此类）。

至少，您需要单独的类来从数据库获取 Foo 和 NotRelatedToFoo。 等等怎么样

abstract class Repository<T> {
     IEnumerable<T> GetAll();
}

abstract class SqlRepository<T> : Repository<T> {
    protected readonly SqlConnection connection;
    public SqlRepository(SqlConnection connection) {
        this.connection = connection;
    }
}

class FooRepository : SqlRepository<T> {

    public FooRepository(SqlConnection connection) : base(connection) { }

    public IEnumerable<T> GetAll() {
    }
}

？真正令人作呕的是，您甚至可以将许多重复的代码从 IDataReader 中提取到一个公共类中（提示：第一个版本将使用反射）。

除此之外，请阅读SOLID。如果您真的想深入了解，请尝试企业应用程序架构模式（Fowler）和设计模式（Gamma 等人）

Answer 2

访问数据库的常见方式是通过数据访问对象。 http://java.sun.com/blueprints/corej2eepatterns/Patterns/DataAccessObject.html

不知道，是否有其他类用于保存、更新或删除数据，但我通常通过 DAO 进行 CRUD 操作。

Answer 3

您的 Util 类负责很多事情（正如 Jason 之前指出的那样）。

看来，问题的关键在于，Util 试图成为 Foo 和 UnrelatedToFoo 对象的存储库。

我个人会考虑将其抽象为一些继承结构并引入泛型。

如果您要创建大量以重复且基于约定的方式映射到数据库表的 POCO 对象，请考虑使用 ORM，例如具有 FluentNHibernate 映射的 NHibernate。