1

我正在使用 CodeIgniter，但这个问题也适用于一般意义上。

我有一个带有列的交易表

item_name | type | date | price | document

我想在两个完全独立的情况下执行以下操作。

1）获取特定日期范围内的交易列表。2）获取特定日期范围内每笔交易的总价格。类型。

前者可以通过简单地使用带有 > datetimestamp 的 select 语句来实现

后者可以通过选择 SUM 并按类型分组来实现，同时就像实现任何所需的 where 条件，例如使用 > datetimestamp

虽然是一个简单的案例，但要实现这一点，我需要有两种方法，但是这两种方法的大部分（即 WHERE 子句）在两种方法中都是重复的。

就速度等而言，这并不重要，但这似乎是毫无意义的代码复制。

第二个例子如下。

我以前有一个方法get_data($ID)可以根据传入的 ID 从表中获取一行。

因此，在一个单独的方法中，我会得到我的 100 个项目。返回一个数组，遍历它们并为每个调用 get_data。

这种设置意味着许多不同的方法可以从不同的来源获取不同的列表，然后仍然使用相同的 get_data 函数和循环来获取所需的数据。

这最大限度地减少了代码重复，但效率极低，因为它意味着循环加载大量数据项和数百个数据库查询。

在我当前的设置中，我只是在我的每个方法中加入数据表 - 代码重复但明显提高了效率。

最后一个例子如下

在 codeigniter 中，我可以有如下功能：

get_thing($ID)
{
$this->load->database();
$this->db->where('ID',$ID);
$this->db->get('table');
}

但是在其他情况下，我可能只想获取特定文件夹中的项目..因此使功能更通用的效果更好..例如

get_thing($array)
{
$this->load->database();
$this->db->where($array);
$this->db->get('table');
}

但是我可能想在两个不同的上下文中使用这个功能，例如用户页面和管理页面，管理员可以看到所有项目，甚至是未经验证的项目。我的代码现在变成：

get_thing($array,$show_unverified = false)
{
$this->load->database();
$this->db->where($array);
if($show_unverified == false)
{
$this->db->where('verified','YES');
}
$this->db->get('table');
}

正如您可能看到的那样，这很快就会失控，并且方法可能变得过于复杂、混乱和充满条件。

我的问题如下 - 减少代码重复的最佳实践是什么，以及如何将它们应用于上述情况？我花了好几个小时试图让我的代码更有效率，但我一无所获，因为我无法锻炼我真正应该努力实现的目标。

干杯

4

3 回答 3

0

我对数据库访问函数中的代码重复的想法是，最好将其分开。

我在这里的规则是特别是函数不应该根据参数返回不同类型的数据，例如它不应该有时返回单个用户，有时不返回用户数组。但它可能会返回错误代码（假）。

如果函数实现了不同的访问级别，这可以在多个页面之间共享。

于 2013-05-10T13:06:00.867 回答
0

这基本上总是回到常识。您应该尽量减少重复代码并尝试降低单个函数的复杂性。保持它们小而简单。

所以基本上每次你尝试泛化这样的函数时，你都必须问重复代码的问题是否比函数过于复杂的问题更大。

在这种情况下，我会在你的第二点停下来，接下来你可以为最常见的任务创建一些包装器（但要小心不要让包装器迷宫）

//you generic function
function get_thing($array)
{
$this->load->database();
$this->db->where($array);
$this->db->get('table');
}

// a nice and friendly wrapper
function get_thing_by_id($id)
{
  get_thing(array('id' => $id));
}

// this is just getting silly. don't go crazy with wrappers, only for very often used things.
// and yes the function name is purposely crazy ;)
function get_thing_verified_by_name_and_city_and_some_more($name, $city, $somethingElse)
{
  get_thing(array('name' => $name, 'city' => $city, 'somethingelse' => $somethingElse));
}
于 2013-05-10T13:16:19.060 回答
0

这回答了你问题的第一部分。假设您使用的是 mysql_fetch_assoc 或类似的。当您迭代结果集时，您可以将计数值存储在循环中的变量中，以获取每种交易类型的总价格。

第二部分，只要您没有无限重复代码，这会导致您在维护代码库时出现问题，就可以了。对于您的函数，您始终可以测试传递给函数的变量类型并相应地设置条件行为。

查看与软件设计模式相关的工厂模式或策略模式以进一步了解。

于 2013-05-10T13:19:48.807 回答