如何修复旧的编码风格 php 脚本

Question

对于如何开始修复老式 php 脚本有什么建议吗？

前几天我收到了一个开发旧PHP项目的offer，我所说的老式是指该结构没有使用OOP编码方法，它没有明确的框架。

我不知道从哪里开始，想知道开发旧脚本有哪些方法。

注意：他们不想花很多钱来启动一个新项目。

那么你建议用什么方法来更新旧的 php 脚本？

Answer 1

这取决于您所说的“旧”是什么意思。旧的为 PHP 4 编写的？还是像非 OOP 一样旧？ （或两者兼而有之？）

PHP4 中的旧版本：
只要您筛选它并抑制警告或实际修复不推荐使用的函数调用，一切都应该没问题。这简直就是无聊的工作。简单又便宜。

与非 OOP 一样旧：
理论上，一个人可以在没有 OOP 或明确的 MVC（或其他）框架的情况下开发一个非常稳定且可扩展的应用程序。事实上，如果应用程序规模较小，则没有理由添加 OOP 或框架的意大利面条和肉丸复杂性。使用某些框架在 OOP 中重写所有内容既困难又昂贵。而且很可能是矫枉过正。

Answer 2

你能否给我们更多的细节，也许是一个例子。

即使是程序代码也包含 OOP 元素。您可以识别与同一实体相关的变量和过程。你可以重写它，但他们很难从中找到价值，特别是如果他们像你建议的那样节俭。

Answer 3

也许您的代码现在看起来像这样

你希望它看起来像这样

如果它只是一个脚本而不是整个项目，我会将其转换为 OOP 编码标准。

Answer 4

Joel Spolsky writes:

“[Netscape 犯了] 任何软件公司都会犯的最严重的战略错误：他们决定从头开始重写代码。”

因此，无论您采取何种行动，优先考虑的是使用现有代码。重构将是您可以使用的最佳方法之一。

如果代码库没有更新，那么你绝对不能做什么？要使该操作成为可能，您需要升级多少以及特别需要升级哪些内容？考虑这两个问题。

Answer 5

当我这样做时，这是一个多步骤的过程。通常，有一个现有的产品可以继续运行。从头开始重写很少是一种选择，即使您最终会这样做。

• 开始放弃手动包含语句并在可能的情况下实现自动加载器（需要多次通过）
• 创建一个帮助脚本来模拟魔术引号和注册全局变量。这样您就可以在 PHP 中将其关闭，同时保持现有代码运行
• 如果适用，逐步删除过多的 strip_slashes 或 add_slashes 调用。帮助程序脚本允许您按文件执行此操作。
• 确保您的变量具有适当的范围
• 分离出您的演示文稿代码。考虑 Smarty 或替代模板系统
• 将 DB 调用移至 PDO 并对所有内容使用参数替换
• 查看代码并考虑将前端控制器存根

然后我查看项目并确定我将如何更改逻辑本身。通常，如果根本没有函数，我的第一步是将常见行为封装到静态方法中。无需太多努力即可获得尽可能多的重用，因此我还不关心组织。

减少冗余后，我开始组织。正是在这个阶段，我开始规划我的类模型并将函数重构为干净的方法。这也是自动化测试（phpunit）的时代。一旦我有足够的信心，我就会添加一些控制器并集成模板，然后我就完成了……除非再经过一两次。

对我来说，最重要的是确定我在哪里，我想去哪里，并制定一个可以通过几个小步骤执行的计划。每个人都有自己的目标，所以除了你自己，没有什么神奇的计划可以遵循。

Answer 6

阅读他们的代码。与他们交谈。

根据现有代码查看请求的更改。与他们交谈。

决定你改变多少来做想做的事。与他们交谈。

这样做。与他们交谈。

当他们要求通过重写比修改更容易完成的功能时，这样做。

使用可以帮助重构的 IDE。