网站前端切页面时间,wordpress英文版如何变成中文版,网站备案帐号是什么意思,免费logo图片生成器 自动生成在学习编程的路上#xff0c;相信大家这几个词一定不少听#xff0c;什么 面相对象、封装继承多态、内功心法21种设计模式 等等 。但是却很少用到#xff0c;或者说用到的都是被动使用。大牛们在写代码前早就构思好了#xff0c;接口#xff0c;基类等等。自己写代码的时候…在学习编程的路上相信大家这几个词一定不少听什么 面相对象、封装继承多态、内功心法21种设计模式 等等 。但是却很少用到或者说用到的都是被动使用。大牛们在写代码前早就构思好了接口基类等等。自己写代码的时候很少有把面向对象想的面很全很容易在遇上不够优秀的代码这时候就需要重构了。 但是我们却很少去重构可能原因有很多比如很重要的一点不想改出Bug不想增加工作量我是要5点半下班的男人女朋友还在等我做饭时间很紧先实现功能先代码是82年的不敢动 其实重构可以写出更健壮的代码、减少后面的工作量、让开发者更好阅读。看了很多重构的文章发现很多是一些基本的命名规范或者拆函数什么的。这篇文章写下我重构的一些思路和重构之前代码对比。好了废话不多说上项目。威武、威武、威武........ 简单介绍一下项目项目就是一个客户端小工具用来审核编写递交的说明是否规范。这里说下面向对象是一种思想和语言无关。只要是面向对象语言无论你是C#、Java、TypeScript、Python都是可以用这种思想重构的。 上图就是小工具了一个部分截图有若干个栏每个栏都是填写一些对应的修改内容在审核校验时会检查写的内容是否符合标准。 前辈已经完成一些栏的校验我的任务是完成剩下栏写正则表达式然后在不标准的时候提示就好了是不是觉得根本没有必要重构依葫芦画瓢就好了在我拿到代码的时候是这样的代码 // 下面变量是和界面绑定的变量RaisePropertyChanged作用在变量改变的时候通知前端重新渲染
// 不熟悉C#代码的只用知道这些变量就是和前台绑定的就是了private string _editDescription;/// summary/// 修改说明内容/// /summarypublic string EditDescription{get { return _editDescription; }set{_editDescription value;RaisePropertyChanged(() EditDescription);}}private string _editDescriptionRules;/// summary/// 修改说明校验规则/// /summarypublic string EditDescriptionRules{get { return _editDescriptionRules; }set{_editDescriptionRules value;RaisePropertyChanged(() EditDescriptionRules);}}private bool _editDescriptionHasValidationError;/// summary/// 修改说明校验标志/// /summarypublic bool EditDescriptionHasValidationError{get { return _editDescriptionHasValidationError; }set{ _editDescriptionHasValidationError value;RaisePropertyChanged(() EditDescriptionHasValidationError);}}private string _integratedNote;/// summary/// 集成注意内容/// /summarypublic string IntegratedNote{get { return _integratedNote; }set{_integratedNote value;RaisePropertyChanged(() IntegratedNote);}}private string _integratedNoteRules;/// summary/// 集成注意规则/// /summarypublic string IntegratedNoteRules{get { return _integratedNoteRules; }set{_integratedNoteRules value;RaisePropertyChanged(() IntegratedNoteRules);}}private bool _integratedNoteHasValidationError;/// summary/// 集成注意校验标志/// /summarypublic bool IntegratedNoteHasValidationError{get { return _integratedNoteHasValidationError; }set{_integratedNoteHasValidationError value;RaisePropertyChanged(() IntegratedNoteHasValidationError);}}// 这里随便举了两栏的变量后面还有若干栏。
复制代码依葫芦画瓢以后呢我发现原来是这样的。每一栏用了单独三个变量直接去绑定编写的内容、是否标准的标志、不标准提示语。我是一个懒人啊在我画了两个瓢以后就很烦一直在那复制变量在那改来改去这些个变量都是差不多一个意思。为啥让我重复在复制修改呢 明显每栏有相同项啊对不对一个是内容一个状态一个是错误提示语啊咆哮 这时候我想起了书本上的一句话 早重构常重构 我已经安奈不住我那颗懒惰的心里因为下面还有“狠多狠多”栏每一行有三个类似的变量我这依葫芦画瓢这个星期加班就在复制粘贴去了。我是要上进要每天进步的人不能这样 我为啥不能将这相同的共性抽象出来呢是不咯这个时候我想起了《葵花宝典》的第一页的第一句“万物皆对象”于是本能的告诉我每一栏看做一个对象只是每栏的值不一样。然后我就写了一个”鸡肋“基类代码如下 /// summary/// 栏 基类/// /summarypublic class Base: ObservableObject{private string _content;/// summary/// 内容/// /summarypublic virtual string Content{get { return _content; }set{_content value;RaisePropertyChanged(() Content);}}private string _errorTip;/// summary/// 错误提示/// /summarypublic virtual string ErrorTip{get { return _errorTip; }set{_errorTip value;RaisePropertyChanged(() ErrorTip);}}private bool _isError;/// summary/// 是否错误/// /summarypublic virtual bool IsError{get { return _isError; }set{_isError value;RaisePropertyChanged(() ErrorTip);}}}
复制代码virtual是为了让子类能够重写get和set如果有需求的话为后面扩展做准备然后字段从3个变到了1个了。 /// summary
/// 修改说明栏
/// /summary
public class EditDescription : Base { }
private EditDescription _editDescriptions;
/// summary
/// 修改说栏绑定变量
/// /summary
public EditDescription EditDescriptions
{get { return _editDescriptions; }set{_editDescriptions value;RaisePropertyChanged(() EditDescriptions);}
}// 其他的一样我就不多写了
复制代码 那我们来算一下账原先的变量每一栏有3个变量一个变量有6行代码的话假如我这个有100栏就是 重构前 100栏x3x6 1800 行代码 阿西吧。 重构后 100栏x1x6 600 行代码 。 小学算数 1800 - 600 1200 1200行你说干点啥不好啊 那秀儿们算下你花这么多时间在那复制粘贴不敢去动前辈们的代码还是勇敢一点呢 然后是不是感觉到一个继承就简单了很多呢这只是一个面向对象的简单运用就让我们少写了这么多代码。 前辈和我说写代码就像练武功就像你会了“九阳神功”或者吸星大法学其他武功看一眼就会。也就是说当你理解了面向对象以后呢你自然而然的就会写出很精简的代码了。阿西吧又扯远了。 变量好了抬头一看函数我的脸便有点抽搐啊这函数有毒函数是这样的 // 此处函数用来设置每一栏报错时边框变红private void SetValidateFlag(){// 绑定的实体类判断状态if (tsEntity.EditDescriptionHasValidationError){// 控件边框改颜色EditDescriptionHsExpander.BorderThickness new Thickness(1);EditDescriptionHsExpander.BorderBrush _readBrush;}if (tsEntity.TestSuggestionHasValidationError){TestSuggestionHsExpander.BorderThickness new Thickness(1);TestSuggestionHsExpander.BorderBrush _readBrush;}if (tsEntity.IntegratedNoteHasValidationError){IntegratedNoteHsExpander.BorderThickness new Thickness(1);IntegratedNoteHsExpander.BorderBrush _readBrush;}// 此处省略一万个if每个栏都有一个if}
复制代码然后大家懂的嘛在我改了两栏以后我又耐不住性子了。再一看感觉似曾相识燕归来的感觉啊有没有每个if中都有三个类似的东西。我那个心啊又忍不住悸动了起来像是等初恋的感觉想她来来了又不知道要干哈。然后我发现其实if中判断的就是每栏的状态括号里面是对控件栏的设置。然后想嘛能不能搞个类似循环的东西只要写一个for就好了呢。 思考以后是这样的我把控件也变成了栏的一个属性了这样if判断里面就都是判断一个类了。代码就变成了这样 public class Base: ObservableObject{/// summary/// 模块控件新增/// /summarypublic object Control { get; set; }//其他的和上面的Base一样}
复制代码然后 SetValidateFlag() 函数就变成了这样 /// summary/// 设置校验界面效果/// /summaryprivate void SetValidateFlag(){// listBase 所有栏实体集合foreach (var item in listBase){if (item.IsError){var control item.Control as HsExpander;if (control null)continue;// 将控件的边框变成红色control.BorderThickness new Thickness(1);control.BorderBrush _readBrush;}}}
复制代码好了嘞好好的一个if的葫芦瓢给咱给整没了。这时候我们来算算我们这个重构省了多少事。老样子 原先的一个if算5行代码我这个有100栏就是 重构前 100栏x 5 500 行代码 全是if啊。 重构后 我数了一下没有错的话应该是14行代码 。 重构一下以后变量得到了减少且对外统一。感觉一颗心得到了小小满足感觉灵魂得到了升华不知道是自己太容易满足还是代码世界里给我的成就感。感觉“乾坤大挪移”瞬间上了两层。 这就是我在写代码的时候一个小小的重构思路希望能够帮助到大家一点。然后这个是我的Github如果有帮助到大家的项目可以给我点个star, 小生在这边谢谢啦
