转载请注明出处:葡萄城官网,葡萄城为开发者提供专业的开发工具、解决方案和服务,赋能开发者。
原文出处: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["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["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原则,并随时抱着不要重复自己代码的想法去完成开发工作。
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。