Warning: file_get_contents(/data/phpspider/zhask/data//catemap/8/design-patterns/2.json): failed to open stream: No such file or directory in /data/phpspider/zhask/libs/function.php on line 167

Warning: Invalid argument supplied for foreach() in /data/phpspider/zhask/libs/tag.function.php on line 1116

Notice: Undefined index: in /data/phpspider/zhask/libs/function.php on line 180

Warning: array_chunk() expects parameter 1 to be array, null given in /data/phpspider/zhask/libs/function.php on line 181

Warning: file_get_contents(/data/phpspider/zhask/data//catemap/4/sql-server-2008/3.json): failed to open stream: No such file or directory in /data/phpspider/zhask/libs/function.php on line 167

Warning: Invalid argument supplied for foreach() in /data/phpspider/zhask/libs/tag.function.php on line 1116

Notice: Undefined index: in /data/phpspider/zhask/libs/function.php on line 180

Warning: array_chunk() expects parameter 1 to be array, null given in /data/phpspider/zhask/libs/function.php on line 181
Design patterns 如果代码隐藏了程序流,我应该重构代码吗?_Design Patterns_Refactoring_Code Readability - Fatal编程技术网

Design patterns 如果代码隐藏了程序流,我应该重构代码吗?

Design patterns 如果代码隐藏了程序流,我应该重构代码吗?,design-patterns,refactoring,code-readability,Design Patterns,Refactoring,Code Readability,我已经构建了restful web服务。服务端点具有类似的逻辑: 如果输入验证成功,请执行数据库操作并返回结果 如果db操作失败,则返回db错误 如果验证失败,则返回包含失败验证信息的响应 我的代码是明确的,您可以阅读代码并理解程序的流程 一位同事对其进行了重构,以便在一个实用程序类中重构此基本流,该实用程序类的主要功能包括请求主体、db操作和处理程序 现在,代码不表示流程,但基本逻辑已重构 最佳做法是什么?你会为了重构而牺牲可读性吗 是的,单元测试也会减少一点 谢谢。您认为包含重构代码的实用程

我已经构建了restful web服务。服务端点具有类似的逻辑:

如果输入验证成功,请执行数据库操作并返回结果

如果db操作失败,则返回db错误

如果验证失败,则返回包含失败验证信息的响应

我的代码是明确的,您可以阅读代码并理解程序的流程

一位同事对其进行了重构,以便在一个实用程序类中重构此基本流,该实用程序类的主要功能包括请求主体、db操作和处理程序

现在,代码不表示流程,但基本逻辑已重构

最佳做法是什么?你会为了重构而牺牲可读性吗

是的,单元测试也会减少一点


谢谢。

您认为包含重构代码的实用程序类是否可以在许多其他类似的操作场景中重用,例如

验证输入, 正在运行数据库操作, 返回结果 如果它可以在其他类似场景中重用,那么重构就可以了


我想重构会更好,从可维护代码的角度来看,重构服务器是一个更好的目的。如果时间允许,我想,您也可以让新代码可读,以使流程明确。

我建议将前后版本发布在。同时查看的定义。将重构与可读性进行对比是没有意义的。重构不是一项功能。谢谢,我在这里发布了一个问题: