入职新公司已五天有余，看了公司项目的部分代码，说下自己的想法。我不是大牛，也不是要说公司代码不好。只是希望自己以后写代码的时候能注意下面这些小问题。
1. AppDelegate过于冗杂
- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions；
程序启动方法内过于杂乱，注册友盟极光环信、是否进入引导页等等有很多操作，一坨一坨的很长。
- 解决方法一
可以将相同功能代码封装在一个方法内,例如将关于友盟的初始化模块都放在- (void)registerUM方法内，然后在需要初始化的时候调用这段代码。这样最起码代码看上去会整洁很多，看上去没有那么散乱。 - 解决方法二
写一个AppDelegate类的扩展，将方法一中的- (void)registerUM放到扩展中，然后在AppDelegate.m中调用，这样会减少AppDelegate.m中代码量。 - 解决方法三
写一个AppDelegate类的扩展，用method swizzling，使得在调用系统方法之前先调用自己定义的方法。在自己的方法中初始化第三方平台。这样一点都不会增加AppDelegate.m代码，但是要注意的是自己定义方法中也不能写的杂乱无章。
2. 硬编码太多
项目中需要根据用户信息中的type来判断用户的类型比如是学生还是老师，后台返回的type是字符串"student","teacher"这样，不知道为什么不返回一个typeID，客户端在判断用户的时候直接[userModel.type isEqualToString:@"student"];来判断用户的类型。因为项目中有很多场景都需要这样的判断，假如后台有天想把返回的"student"改为"stu"，那怎么办。
解决方法一
无论后台的type返回的是字符串还是typeID,最后都在客户端做一个映射，在model中定义一个枚举，然后在type的setType方法中将根据后台type的类型映射成枚举中的一个值。这样判断用户的时候就可以[userModel.etype == userTypeStudent"];判断。这样即使后台修改了type的值，我们也只需要在model的setType方法中将对应的值修改下就行。解决方法二
如果执意用[userModel.type isEqualToString:@"student"];那最好也将“student”定义一个宏吧。这样后台做修改之后，也只需要修改下宏。
3. ViewController中的- (void)viewDidLoad
不说viewController瘦身，最起码要不要做到控件的初始化，布局，和业务逻辑都写随意在- (void)viewDidLoad。会使方法内代码行数过多，看了容易晕车。有点类似
- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions；
中。
- 解决方法
参考问题一，可以将初始化控件、布局操作、业务逻辑都封装成各自的方法然后调用相应方法即可，虽然没有减少VC中的代码量，但是最起码没有出现不同功能代码相互掺杂，我中有你你中有我的情况。
4. END
这些问题基本都和技术牛不牛没有多大关系，但是都是值得注意的一些小问题。最起码要让自己的代码看起来整齐，易读，易维护。最高境界就是要达到让我们的代码截图到可以做手机壁纸的程度
😝。
