一个2w行的action
各位oc的朋友们大家好,现请教大家一个问题,想来想去,不知该如何处理,希望经验丰富的大神指点一二。
先谢过大家了!
做公司的项目,一个Action中居然出现了2w行的代码,我分析了一下缺点
1.svn提交时候会产生冲突。
2.代码不好找
暂时就想到这些了。
我已经没什么话说了,好无语的感觉,大家怎么看?
上次我看到一个代码规范,说是一个方法不要超过30行,一个屏幕看方便些,我一看我们的这个,300行都超过了。
怎么办?当初设计就没想到这个类会变得臃肿
如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。
绑定邮箱获取回复消息
由于您还没有绑定你的真实邮箱,如果其他用户或者作者回复了您的评论,将不能在第一时间通知您!
发布评论
评论(9)
其实就是没有规划好。
我个人从不写超过30行的函数,只要超过20行就说明里面的逻辑混乱,可以继续拆分。
所谓的编程效率其实体现在编程思路和可维护性上面,代码运行速度或者多少行其实不是判断标准。
现在产品迭代这么快,需求不停变更,可维护性比运行提升那么百分之零点几或者少占用几M内存更关键。
曾经接手处理过某个项目的一个bug,那哥们就是将他负责开发的N个模块的代码写在一个类中,行数也是1万2千多行。修正bug后研究优化精减了一下代码,减少到8000多行吧。这里的所谓优化,其实就是理解现有功能、抽取公用方法,然后在需要的地方调用相应方法。
编译都浪费时间
看清楚哦,万一,一改到处问题就完了。
看得懂,运行正常就不要动。强迫症除外。
如果没有必要,我劝你不要动他
2W行~?~?~~!?~!那也不全是设计的问题了!!!!
也没有什么30行的规范,有些程序难免要有一堆if/else或switch什么的,30行也太少了
个人觉得尽量控制在2位数就好,遇到复杂的、实在拆分不了的,最多最多200左右
重构吧,或者把里面的功能拆解成小的方法。业务逻辑一般不写在Action上面,Action只负责分发
svn冲突,用ide有插件的啊,点查看下一个冲突,马上就定位到了
代码不好找,有搜索
优化之前,请先想下:究竟要优化什么?
如果仅仅是简短代码行数,就是抽象出公共方法,公用方法,还有业务逻辑分离出来