前几个月完成对MVVM/RAC的学习之后，最近一直在默默地对项目代码进行重构，写码比较多，过了一段时间回头发现自己的代码风格还有代码质量都有大大的改善。过去几年在一家小公司负责iOS客户端后来负责客户端的研发工作，被杂乱的事情分神比较多，所以到去年的时候，写码已经不太多了。在新公司待了大半年，目前只是写码的小角色，所以精力基本上在写业务代码和业余学习乱七八糟的技术上面。

最近一个月除了专门抽时间和精力重构之外，还有就是遇到需要添加功能的模块的时候，由于项目中的代码历史因素比较多，第一件干的事情往往是重构整理代码，发现很多之前的代码写的时候没有注意的事情特别多，比如全局变量乱用；方法没有层次感，胡乱添加；对业务不了解的情况下，通过打补丁的方式实现功能等等。所以我决定写一篇文章，把自己的觉得实践中需要注意的一些事项，具体总结一下分享给大家。

减少对象属性

这个是最容易改善代码质量的一个点，很多代码一眼看上去就会让人感觉很凌乱，一上来就是几十个不同的对象变量定义在里面，这让不同逻辑之间莫名其妙没法分开。一个是定义的方式不对，很多莫名其妙的内部变量暴露在头文件中，让外部调用者根本不知道哪些才是public可以操作的方法。另外实际上，经过我自己这段时间的重构经验来看，大多数是可以通过局部变量或者__block变量来代替的。

1. 头文件中尽可能少暴露变量或方法，而要使用extension或者category放在.m文件，或者专门的private头文件中

头文件中暴露的信息越少越好，一切不必要的信息都不要暴露出来

m文件的extension中，定义conforms protocol和对象属性，对于对象属性的定义，使用getter/setter 来定义。

2. 使用局部变量或者__block变量代替

局部变量不需要多说，需要写码的时候思路清晰一些，写完之后在commit之前即使review一定要check一遍，对自己的代码质量负责，code review往往检查不出来冗余或者废弃的代码。不添加一个多余的对象属性，不留注释掉的代码，不留没有用途的代码，这些都是基本功，但是很多开发者就是做不到，或者说对写码没有爱，所以很多废弃的代码，我重构代码的时候，虽然对业务不熟悉，但是大多数模块都能删除掉十分之一的代码和大量的对象属性，这个是单纯的不够用心。

关于使用__block变量，这个是Android开发中我感觉到最不满意的地方，这个特性简直太他妈爽了。

比如这里，使用block的时候回传一些变量

再比如这里，我需要记录一个pan手势开始时，headerView的顶部坐标，结合RAC之后，本来需要全局变量来记录的值，使用__block变量即可搞定

3. 可以尽可能避免循环引用

有个地方很多开发者会疏漏，在block中使用_XXX对象变量的时候，block会retain self指针，一不小心就会造成循环引用的出现。所以使用局部变量的话，就能扼杀这种问题在摇篮之中。

减少和模块化对象消息

1. 减少对象消息

减少UI的action类消息，感谢block和RAC，或者blockskit，让我们得以通过hook来把之前target-action模型换为block来实现，UI和action的代码终于可以一起了，使整个逻辑变得紧凑，在查看代码的时候终于不用跳来跳去了。还有就是日常开发中，把自己写的各种protocol或者传递target/selector的地方，尽量使用block来代替，相信我，这个会使代码好读很多。

2. 模块化

使用”#pragma mark - XXX”进行分割不同逻辑之间的界限，让整个文件阅读起来更加结构化。还有一个我现在最常用的就是是设置Xcode的快捷键，把Ctrl + 6 显示文档结构的快捷键改为：Command + J ，搜索来快速跳转到对应的消息和模块，要尽量避免文档结构显示超过两屏幕，超过两屏幕说明有点多了，你肯定考虑一下重构了。

我个人习惯一般划分的模块有: life cycle，ui helper，datasource/delegate，依据功能进行划分的模块等等，如下是我最近重构的一个ViewController的文档结构

虽然不是很懂,但很不错啊

进度在我前面啊，我得抓紧时间了

膜拜大神