如何组织我的代码（我为网站上的各种用途创建的方法）

Question

我编写了一些代码来将应用程序连接到其数据库，然后创建了一些代码来使用连接代码并检索、更新或添加一些值到数据库中，此外，我可能还有一些代码来处理其他内容，而不是处理数据库

代码有点复杂，也许很简单，但并不短，例如编写一段好的代码来检索单个值，这样我就可以用这个值设置控制器我使用了这个：

SqlConnection sqlconnection= new SqlConnection(ConfigurationManager.ConnectionStrings["DefConnectionString"].ConnectionString);
        SqlCommand sqlcommand = new SqlCommand("SELECT name FROM message  WHERE id = 3", sqlconnection);

        try
        {
            sqlconnection.Open();
            lbl_name.Text = (string)sqlcommand.ExecuteScalar();
            Status.Text = "Done";
        }

        catch (Exception ex)
        {
            Status.Text = ex.Message;
        }

        finally
        {
            sqlconnection.Close();
        }

我什至可以添加一些代码为了在数据库中存储有关抛出任何异常的一些信息，我认为代码非常基本，但它并不小，因为它是我需要的最小的部分，编写了一些巨大的代码来添加或编辑新项目，并且还要考虑到这不是我在页面中编写的唯一代码，该页面还有更多需求：D！

我希望我提供了一个详细的案例！...那么您认为使我的页面快速、易于阅读并具有有组织的代码的最佳方法是什么 我已经开始将每个代码放在适当的方法中，但有些方法是常见的，因此我在每个需要它们的页面中再次创建它们，例如 ConnectionToDataBase 方法，同时我认为我只是组织了页面，因此我可以主要查看 Page_Load看看被调用的方法是什么，然后向下滚动以阅读定义，但它仍然是一个大页面并记下最佳实践，我认为

对不起所有大问题，我只是提供详细信息，以便我可以为我的问题获得合理的答案，我希望每个人都能从中受益，因为大多数问题都非常基础，像我这样的新手需要一些详细的案例和答案才能获得更好的开始......感谢您的宝贵时间！

编辑：我知道我是新来的，所以非常欢迎对我的代码提出任何评论！

Answer 1

一些快速观察：

1. 不要使用内联 SQL，因为它很难维护（使用存储过程，或者如果您正在执行基本的 CRUD 操作，请考虑 ORM）

另外（根据下面的评论，这是非常相关的）如果您必须使用内联 SQL，如果您最终将值连接到命令文本中，您可能会遭受 SQL 注入攻击（一般来说，在任何情况下都应该避免内联 SQL）。

1. 不要全局捕获和吞下异常 - 仅捕获实际处理异常的地方 - 否则，将它们冒泡备份。

2. 您需要将数据访问代码抽象为独立于业务对象的数据访问层（我是存储库模式的粉丝），或者（最好）考虑 ORM（我强烈推荐 NHibernate，可能是 Entity） Framework 4 具有更好的 POCO 支持）。

一般来说，我构造此类数据的方式是让业务对象引用存储库对象 - 这被定义为接口，因此我可以通过依赖项注入交换我的测试和实际实现。

然后，数据存储库会执行相关的繁重工作（通常通过 ORM，有时在 ADO.Net 中处理我们的一些遗留代码）——因此，如果我需要客户列表，我只需在存储库上调用“GetCustomers”方法即可。业务层不了解数据访问，它只知道当它调用一个方法时，它会返回一个很好的强类型列表。

[回答请求 RE 组织]

当我在 MVC 网站中使用此模式时，我通常让我的控制器调用业务逻辑类。它的目的是封装从我的数据访问层检索的非数据特定计算等。

业务逻辑类将依次引用一个或多个存储库，这些存储库封装了我的数据和业务类之间的数据传输（例如保存/检索数据等）。

存储库负责提供数据（同样，ADO.Net、NHibernate、EF，任何适合您的东西）。它应该没有业务逻辑，只需足以从我的数据库中获取数据，对其进行适当的调整，然后将基本的非数据特定对象返回到业务层。

我的博客上有一些关于这个组织的示例（可在我的个人资料中找到），虽然它们是 NHibernate 特定的，但展示了存储库模式的基本用法。

希望能有所帮助:)