用于AJAX调用和表单操作的PHP文件夹结构

Question

我正在尝试养成良好的代码组织习惯，并专门在php中使用OOP，但是我似乎无法将自己的想法缠住。

这是我正在使用的内容的简化描述：

• 我所有的课程文件都放在文件夹“ / resources / Classes”中
• 我所有的html和javascript分别位于'/ public_html'和'/ public_html / script'中

我的问题与文件有关，这些文件是表单或AJAX请求的操作。 例如“ formAction.php”和“ ajaxURL.php”。 这些文件不是类，也不包含任何html或其他此类GUI。

我一直将它们放在文件夹“ resources / actions”中，但是我的直觉告诉我有关此的信息还不是完全面向对象的。

如果我尝试完整的OOP，这些文件的使用方式是否不正确？ 如果是这样，我该如何采取不同的方法。

这是我的项目中的实际文件，作为一个具体示例：

//file: getBalance.php

<?php
/**
 * This script gets the balance of an account from the server
 */

if (!isset($Database)) {
    $Database = require_once "../clear_finance_pkg.php";
}

/** @var User $User */
$User = $Database->getUserByID("1");//TODO: user should be set dynamically

$User->setAccounts($Database->getAccountsByUser($User));

if (isset($arg1)) {
    $accountID = $arg1;

    foreach ($User->getAccounts() as $Account) {

        if ($Account->getId() == $accountID) {

            $RbcChequing = RbcAccount::accountToRbcAccount($Account, "Chequing");

            echo '$' . Money::toDollars($RbcChequing->getBalance());
            break;
        }


    }
} else throw new Exception('Account ID is not set. Could not get balance');

Answer 1

很难说您的代码是否是完整的OOP，但我认为不是。 您正在使用类，对象和方法，因此您似乎处于正确的轨道。 这是OOP的基础。 不再有大的if / else语句和许多没有意义的变量，而是没有代码重复的对象和方法。

我认为您的问题更多地与逻辑和视图的分离有关。 首先。 通常，为功能创建文件是可以的，但是在大型应用程序中，您将失去概述。 您正在做的是将与视图相关的内容和与逻辑相关的内容组合在一个文件中，但这实际上不是您想要的。 理想的情况是逻辑和视图完全分离，因此您可以基于同一逻辑创建多个单独的视图。

我的建议是看一下MVC模式。 看一下这个链接。 这将帮助您对MVC模式有一个基本的了解。 请注意，您不再需要为每个功能都拥有一个文件。 而且您可以将逻辑和视图元素分开，因为您可以在多个视图中使用相同的模型（尽管这可能不是最佳示例）。