遇到“极其坑爹”的程序代码(Matlab)怎么改?

2013-10-22 22:03:49 +08:00
 dragonszy
不得不说,这坨代码就是一坨shit。

各种A,a,I,i,Z,z,B,Trace,trace等等,变量名完全是随便取,完全分不清啥是啥。

各种复制粘贴,该写成函数的不写函数。不该写成函数的写成函数。

load RLC.txt load RLC1.txt load RLC2.txt load RLC3.txt 随便取名至少0123吧?

各种全局变量滥用,各种重复变量k=size(RLC),A=size(RLC)。真是信手拈来。

函数名也是随意取,什么F(Z),里面画了8个图。也是复制粘贴。写个函数传个参数画图能跪?

有些变量名还和Matlab自带的函数名重叠,比如前面的trace。(虽然赋值后不影响)

其间还用了一个“遗传算法”拟合。感觉就是完全不懂就在用,即使用其他的数据总能拟合对。

更别说矩阵的优化了,感觉程序里面写的“最有效率”的就是直接下载的遗传算法工具包。

还有缩进,简直无情。Sublime Text的Reindent也无能为力。

-------------分割线-------------

导师要求我和某同学辅助某研究生学长完成这个任务,属于外面在职读博的人的博士论文的一部分。只不过真心没办法改这程序,学长说这程序也是上届流传下来的,很多地方看不懂。我只能说这坨代码真是朽木不可雕。

让我想起了《人月神话》,不同的是人月还有成功的可能性,这坨shit丝毫没有。

但是导师很急,因为外面的那个博士很急。虽然本身不是我的任务(从其他项目上调过来)。

但没有意义啊,即使重新写一遍把程序搞定了,但其实还是错的(拟合)。

感觉在浪费生命,而且是别人的责任。Fck!还不如去打LOL。

明天还要汇报,怎么说。

有点准备今天晚上熬夜重写一遍了。
2891 次点击
所在节点    问与答
7 条回复
9hills
2013-10-22 22:19:08 +08:00
不要尝试重写,你会后悔的<_<
Golevka
2013-10-22 23:31:50 +08:00
我大三时也遇见了类似的情况, 卧槽数学专业的学长写的代码简直没法看啊.

于是我花了一个寒假的时间把代码全推翻重写, 然后把未完成的部分也实现了.
sinxccc
2013-10-22 23:55:48 +08:00
千万不要重写,除非你保证下面一段时间能有充足的时间和精力投进去。

首先你得让它跑起来,然后再想办法让它跑出正确的结果。这个过程中可以适度的重命名一些变量,手工调整缩进(别用自动工具),分割文件,提取小的函数,但千万不要动到代码逻辑,一点都不要,Matlab 这种糙快猛的语言里有无数的坑。

最后记得改代码之前先建个 git repo,改一点 commit 一点…虽然麻烦但保险。
bcxx
2013-10-23 00:03:00 +08:00
把已有的加一层 wrapper 好好保护吧! 23333
dragonszy
2013-10-23 08:52:15 +08:00
谢谢楼上们的建议,我昨晚直接睡了,睡了一觉也想通了,Matlab这种程序,能用就行,管他内部多混乱,反正别人论文又不需要附带程序。有时间的话再增个函数什么的,变量名已经真心改不了了。
dragonszy
2013-11-13 19:27:45 +08:00
LZ已经重写完毕了,从算法结构命名函数注释文档等各方面提升。发现Matlab其实90%情况可以不用for循环。

可惜就是曲线拟合太慢。
dragonszy
2013-12-07 22:50:41 +08:00
LZ找出了瓶颈,修改了目标函数的一些算法,把拟合速度提升了50倍左右(5m->30s),增加了一系列其他功能。

至此,正式完结。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/86509

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX