方法/函数的重复利用是这样的吗?

快乐的一只小青蛙 发布于 08/08 16:45
阅读 1K+
收藏 0

接手了一个视频通话的代码,里面各种情况挂断只用了一个只有三个参数方法处理, 然后里面各种判断,引入各种看不懂意图的变量处理。正常情况下当然没问题,但是线上时不时出现各种极端情况, 现在我在这方法上改也不是,不改也不是。简单来说,我想重构,对不同情况分方法处理,类同的地方才用同个方法处理,虽然它们都干了同样的事——挂断。

想问下大佬们:你们遇到这样的情况,是像原代码那样全部塞一起,还是像我这样分开处理?

加载中
0
kiara
kiara

原本的程序引入了业务逻辑使得程序不够单纯,这种一般都是分开处理好,简单点的方式就是入口作为调度层->对应的业务处理程序->执行挂断,这样至少可以确保底层接口复用性。混在一起没什么复用性可言

0
maradona
maradona

使用合适的设计模式来代码复用

maradona
maradona
回复 @快乐的一只小青蛙 : 设计模式本就是解决你说的问题的,难道有问题?如果不遵循一般的编程规范,以后维护你代码的人或许也会有同样的困惑
快乐的一只小青蛙
快乐的一只小青蛙
连设计模式都扯出来了,真牛逼
0
自由PHP
自由PHP

按照设计思路,就一个简单的挂断,当时肯定没想到由那么多可能,后面再改,就遇到和你一样的抉择,在这种情况下前辈选择了稳定,既然你遇到了不改也得改的情况,就看你个人选择了,前提当然是不要增加新的BUG

gitOpen_1
gitOpen_1
回复 @快乐的一只小青蛙 : 当然重构了,否则在屎山添屎早晚埋了自己
快乐的一只小青蛙
快乐的一只小青蛙
No, No, No, 所有代码里,最不简单的就是这个挂断了,至少目前是这样。其他的部分我已经捋清了,按照原来的思路,这个挂断部分我是捋不清了,所有我才想重构。我现在分了3个方法出来,3个方法加一起还没原来一个方法长。原来的方法又臭又长,乱七八糟的判断让人看不清意图
0
q
qlaall

如果可以还是show一下代码片段 

0
快乐的一只小青蛙
快乐的一只小青蛙

经过各种测试,分开写没有问题,原来存在的问题也已解决。我想说,去特么的设计模式和代码复用,能解决问题才是王道

返回顶部
顶部