PHP函数滥用？

Question

我有一个大的系统，我已经编码，我希望使代码(我敢说)更简单和容易阅读。不幸的是，在此之前，我并没有经常使用函数。

我有许多不同的MySQL查询在我的代码中运行，我觉得如果我将各种显示变成函数并将它们存储在一个单独的文件中，将使代码更易于维护(实际上，我知道它会)。

我唯一想知道的是，这是否是一种常见的做法，以及你是否认为这会对我的长期表现和其他因素造成伤害。下面是我目前正在使用的一个示例：

$result = mysql_query("SELECT * FROM table");

while($row = mysql_fetch_array($result)){

/* Display code will go here */

}

正如你可以想象的那样，这可能会变得很长。我正在考虑创建一个函数，它将接受结果变量，并完成这一任务，然后返回结果，如下所示：

$result = mysql_query("SELECT * FROM table");

dsiplayinfo($result);

你认为这是正确的方法吗？

编辑

函数将非常不同，因为每个函数都需要以不同的方式显示数据。需要在每个场景中显示数据库的不同字段。你觉得即使考虑到这个因素，这种方法仍然是一个好方法吗？AKA模块设计还没有完全完成，但是易于维护。

Answer 1

使用更多的功能是有帮助的，也可能是有害的。从理论上讲，它让你更倾向于模块化设计，这意味着你可以在多个应用程序中反复使用一个函数，而不必重写它。

我真诚地鼓励您更多地使用更大的约定，比如MVC框架。科哈纳是一个很棒的。它使用帮助器进行额外的外部功能，使用模型查询数据库，并使用控制器执行所有的逻辑--最终结果将传递给视图，并使用HTML/CSS进行格式化，并添加Javascript。

Answer 2

你要记住的一件事是干的原则:不要重复自己。

如果您发现有一个代码块是您多次使用的，或者是非常类似的代码，可以使用相同的代码块，那么它是移动到函数中的理想候选代码。

Answer 3

• 不要使用"SELECT *“--枚举您想要的字段，因为性能原因以及维护原因。
• 在您的示例中，显示代码可能与SQL查询非常紧密地耦合在一起，所以您可以将它们封装在一起。
• 您可能会考虑使用ORM (如CakePHP)的某种MVC框架，这将比编写一堆函数更有利于模型重用
• 你在正确的轨道上！写你的代码，然后重构它，使它更好--非常聪明。