DRY原则的一个简单实践
转载请注明出处：葡萄城官网，葡萄城为开发者提供专业的开发工具、解决方案和服务，赋能开发者。
原文出处：https://dzone.com/articles/dry-dont-repeat-yourself
我们之前就发过一篇相关的文章：https://www.cnblogs.com/powertoolsteam/p/12758496.html 其中也提到了包括DRY在内的一些软件开发的原则。
DRY 是软件开发的原则之一，其目的主要是为了避免代码重复，指导开发者尽量以抽象的思维去解决重复，基本上是，当您发现自己一遍又一遍地编写相同的代码时，可能会有更好的方法。
实际案例
让我们先看一个例子，看看这个例子是否可以改进，以及如何通过重构来避免代码重复。
这里有一个简单的Report类，该类接收一些数据并通过控制台以格式化的方式直接输出。
我们这里使用php的一个代码片段来举例，相信大家对代码的结构和想要完成的工作都不难理解，所以为了大家更容易理解，我只对一些下面用到的php函数定义做一个解释：
- echo() 函数输出一个或多个字符串
- ucwords()函数把字符串中每个单词的首字符转换为大写。
- strtolower() 函数把字符串转换为小写。
- file_put_contents() 函数把一个字符串写入文件中。
- floor() 函数向下舍入为最接近的整数。
class Report
{
public function show(array $data)
{
echo "Report: " . ucwords(strtolower($data["name"])) . "\n";
echo "Product: " . ucwords(strtolower($data["product"])) . "\n";
echo "Start date: " . date("Y/m/d", $data["startDate"]) . "\n";
echo "End date: " . date("Y/m/d", $data["endDate"]) . "\n";
echo "Total: " . $data["total"] . "\n";
echo "Average x day: " . floor($data["total"] / 365) . "\n";
echo "Average x week: " . floor($data["total"] / 52) . "\n";
}
}
可以看到，上面的代码完成目标是没有任何问题的。
这时我们对Report类提出一个新的需求：把所有字符串也可以保存到文件中。
我们经过一通复制和粘贴上面的代码，新建一个名为saveToFile的函数，就可以很快的完成这个需求，代码如下：
class Report
{
public function show(array $data)
{
echo "Report: " . ucwords(strtolower($data["name"])) . "\n";
echo "Product: " . ucwords(strtolower($data["product"])) . "\n";
echo "Start date: " . date("Y/m/d", $data["startDate"]) . "\n";
echo "End date: " . date("Y/m/d", $data["endDate"]) . "\n";
echo "Total: " . $data["total"] . "\n";
echo "Average x day: " . floor($data["total"] / 365) . "\n";
echo "Average x week: " . floor($data["total"] / 52) . "\n";
echo "Average x month: " . floor($data["total"] / 12) . "\n";
}
public function saveToFile(array $data)
{
$report = '';
$report .= "Report: " . ucwords(strtolower($data["name"])) . "\n";
$report .= "Product: " . ucwords(strtolower($data["product"])) . "\n";
$report .= "Start date: " . date("Y/m/d", $data["startDate"]) . "\n";
$report .= "End date: " . date("Y/m/d", $data["endDate"]) . "\n";
$report .= "Total: " . $data["total"] . "\n";
$report .= "Average x day: " . floor($data["total"] / 365) . "\n";
$report .= "Average x week: " . floor($data["total"] / 52) . "\n";
$report .= "Average x month: " . floor($data["total"] / 12) . "\n";
file_put_contents("./report.txt", $report);
}
}
那么，上面的代码能够满足我们提出的需求吗？答案当然“是的”。但是从技术角度来看，这段代码似乎是有些问题的，它的重复代码到处都是。无论是对代码阅读及后期维护来讲，这都是一场噩梦。
所以我们需要进行一些重构，抽象能抽象的方法，让冗繁的代码变得更简洁。
首先，我们对Report类进行功能上的抽象，生成报告并输出一共可以分为两个功能，一个只负责创建Report，一个只负责如何处理Report，那么让我们开始重构吧。
class Report
{
public function show(array $data)
{
echo $this->createReport($data);
}
public function saveToFile(array $data)
{
file_put_contents("./report.txt", $this->createReport($data));
}
private function createReport(array $data): string
{
$report = '';
$report .= "Report: " . ucwords(strtolower($data["name"])) . "\n";
$report .= "Product: " . ucwords(strtolower($data["product"])) . "\n";
$report .= "Start date: " . date("Y/m/d", $data["startDate"]) . "\n";
$report .= "End date: " . date("Y/m/d", $data["endDate"]) . "\n";
$report .= "Total: " . $data["total"] . "\n";
$report .= "Average x day: " . floor($data["total"] / 365) . "\n";
$report .= "Average x week: " . floor($data["total"] / 52) . "\n";
$report .= "Average x month: " . floor($data["total"] / 12) . "\n";
return $report;
}
}
现在看起来更清楚一些，对吗？
下面我们还有函数使用重复的问题要解决，例如，Report和Products的名称函数使用重复：
$report .= "Report: " . ucwords(strtolower($data["name"])) . "\n"; $report .= "Product: " . ucwords(strtolower($data["product"])) . "\n";
我们可以将这些转换抽象为一个新的函数：
private function normalizeName($name): string
{
return ucwords(strtolower($name));
}
另一个重复：日期格式。
$report .= "Start date: " . date("Y/m/d", $data["startDate"]) . "\n";
$report .= "End date: " . date("Y/m/d", $data["endDate"]) . "\n";
让我们将其抽象为：
private function formatDate($date): string
{
return date("Y/m/d", $date);
}
最后一个：平均值计算。
$report .= "Average x day: " . floor($data["total"] / 365) . "\n"; $report .= "Average x week: " . floor($data["total"] / 52) . "\n"; $report .= "Average x month: " . floor($data["total"] / 12) . "\n";
尽管计算结果并不完全相同，但执行的操作大家是一致的，所以可以抽象为如下：
private function calculateAverage(array $data, $period): string
{
return floor($data["total"] / $period);
}
所以，经过了一番重构，最终的Report类变为了如下：
class Report
{
public function show(array $data)
{
echo $this->createReport($data);
}
public function saveToFile(array $data)
{
file_put_contents("./report.txt", $this->createReport($data));
}
private function createReport(array $data)
{
$report = '';
$report .= "Report: " . $this->normalizeName($data["name"]) . "\n";
$report .= "Product: " . $this->normalizeName($data["product"]) . "\n";
$report .= "Start date: " . $this->formatDate($data["startDate"]) . "\n";
$report .= "End date: " . $this->formatDate($data["endDate"]) . "\n";
$report .= "Total: " . $data["total"] . "\n";
$report .= "Average x day: " . $this->calculateAverage($data, 365) . "\n";
$report .= "Average x week: " . $this->calculateAverage($data, 52) . "\n";
$report .= "Average x month: " . $this->calculateAverage($data, 12) . "\n";
return $report;
}
private function formatDate($date): string
{
return date("Y/m/d", $date);
}
private function calculateAverage(array $data, $period): string
{
return floor($data["total"] / $period);
}
private function normalizeName($name): string
{
return ucwords(strtolower($name));
}
}
这是一个简单的例子，实际情况可能比这要更加复杂的多，但我仅想通过这个实例向大家说明一个问题，那就是避免重复代码的重要性及我们如何通过重构去处理重复代码。
有时候重复一次相同的代码可能没问题，但是当第三次我们写出相同的代码时，那就说明是时候重构你的代码了。
结论：
请记住DRY原则，并随时抱着不要重复自己代码的想法去完成开发工作。
版权声明：本文为powertoolsteam原创文章，遵循 CC 4.0 BY-SA 版权协议，转载请附上原文出处链接和本声明。