我们应该在配置文件中初始化一个 class object 吗？

Question

初始化 object 以进行身份验证 class 是否是一种好习惯，该身份验证在配置文件中执行各种操作，例如注册用户、登录用户等？

配置文件主要用于设置数据库用户名、密码和站点名称等。

如果您认为这不是标准做法，请提供原因。 只是为了让您知道，除了这个特定的 class 之外，大部分代码都是程序性的。

    // MySQL connection settings
    $db_host = "localhost";
    $db_user = "someuser";
    $db_pass = "some pass";
    $db_name = "somedb";



    // Name of the site
    $site_name = "MyDomainName";   



    $auth = new auth();//initialize Authentication class
$auth->set_auth();//Check if user autorized

Answer 1

在我看来，这不是一个好主意。 配置文件用于配置，而不是用于执行。 我会将这段代码放入 bootstrap.php 中，因此，当您将来必须编辑代码时，您确切知道在哪里可以找到配置和实例化。

索引.php：

include('bootstrap.php');

引导程序.php：

include('config.php');
$auth = new auth();
$auth->set_auth();

config.php：

$db_host = "localhost";
$db_user = "someuser";
$db_pass = "some pass";
$db_name = "somedb";

Answer 2

身份验证 class 应该负责身份验证，而不是处理用户特定的功能，如注册/编辑/删除。 配置 object 应该对需要配置数据的 object 可用，而不是相反。

配置文件不应包含除配置数据之外的任何内容。

Answer 3

我不会这样做的。 出于多种原因，最好将“代码”与“配置文件”完全分开。 一是您可能在配置文件中包含敏感信息（例如密码），并且希望将它们保存在与代码本身不同的存储库中。 将初始化（即代码）放入 Config 将意味着它在两个源存储库中进行控制。 其次，“运行”的东西可能不是幂等的。 new auth()可能会设置一些 state 这将使“重新加载”配置有点痛苦。

最好保留一个“配置加载器”，它会在从配置文件读取信息后创建一个包含实例化类等的配置 object，而不是将初始化代码直接放入文件中。 此外，我不建议使用可执行代码作为配置文件（尽管这似乎是 PHP 世界的做法）。 我会使用.ini或yaml之类的格式并加载它，而不是冒险评估我的核心应用程序之外的代码。

Answer 4

将您的代码抽象为具有单一目的的块是一种很好的做法，在您的情况下：配置 object 和身份验证处理程序。 这与KISS 原则有关，该原则要求设计简单。 在这种情况下：代码简单。 当您混合代码时，就像您所描述的那样，您的代码会变得更加复杂，因为它会做更多的事情。

虽然起初它可能看起来并不复杂，但这种设计最终会导致复杂的代码。 例如：如果另一个开发人员加入您的团队怎么办？ 您需要添加另一个配置和条件到 select 正确的配置。 在使用不同的环境（开发/生产）时也是如此。 如果在某些情况下您不想使用身份验证处理程序怎么办？ 也许您正在执行不需要身份验证的操作，或者更好的是：当不需要身份验证时。 更多的代码，更多的复杂性。

通过创建代码块，无论是 class、function 还是包含，您都在使用松散耦合。 这意味着这些块彼此不知道，但它们可以交互。 这种交互通常是通过为身份验证处理程序提供一个它可以使用的配置 object 来完成的，而不是让处理程序决定它使用什么配置。 这称为依赖注入，被认为是一种非常好的解耦代码的方法。

其次，质量代码主要是文档本身。 假设我必须在您的代码中工作，我不知道包含配置文件实际上会进行身份验证。 我可以花几分钟甚至几个小时试图弄清楚为什么会创建 session，而在查看代码时，似乎没有任何迹象表明这一点。

在你的例子中，我会使用这样的东西：

<?php

include 'config.php'; // contains function that returns config object/array
include 'auth.php';   // contains functions for authentication

// determine environment, based on server variables (ip, script path, etc)
$environment = getEnvironment($_SERVER);

// get config based on environment
$config = getConfig($environment);

// see if authentication is needed
if (needsAuthentication()) {

    // perform authentication based on request variables (eg: submitted form fields)
    $auth = new auth();
    $auth->doAuth($config, $_REQUEST);
}