一个全局变量很多的程序，如何理清逻辑关系

qingyin2009 · 发表于 2018-1-2 14:31:13

程序是我从原来产品移植过来的，中间经过很多人的手，现在又回到我手上，前些天发现一个严重问题，更改方法也想好了，只是觉得程序太乱了，由于已经量产，也不好重构，只想把它整理清楚，防止后期再有问题。

由于全局变量比较多，画流程图后，还是没办法看出来是否还有问题。请问各位都如何处理此类问题的？

leafstamen · 发表于 2018-1-2 14:41:04

接过别人的球，会有一大群人追着你往前跑的，别管你懂不懂这是什么球。

wind2100 · 发表于 2018-1-2 15:07:30

加你要的功能就好了,一般不会要你去理或是优化程序 , 就为了可读性或是可移植性

ztrx · 发表于 2018-1-2 15:27:18

上操作系统

liansh2002 · 发表于 2018-1-2 17:35:32

血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至少5年以上的程序，因为不好看就顺手加了下{}，结果短路锁死变打嗝，查了半天发现少了一个}，真的是就少了一个，程序还编译通过了。。。

MAD_FISH · 发表于 2018-1-2 17:38:03

liansh2002 发表于 2018-1-2 17:35
血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至 ...

求少了一个}能编译通过的方法

liu672992381 · 发表于 2018-1-2 18:32:23

liansh2002 发表于 2018-1-2 17:35
血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至 ...

同求这样的方法。

sbusr · 发表于 2018-1-3 00:49:36

liansh2002 发表于 2018-1-2 17:35
血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至 ...

这编译器还能这样工作？

q457344370 · 发表于 2018-1-3 07:46:36

liansh2002 发表于 2018-1-2 17:35
血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至 ...

同求少括号还能编译过的方法

zchong · 发表于 2018-1-3 07:51:05

应该不是少了，是括错内容了吧

qq854149876 · 发表于 2018-1-3 08:49:03

我的建议是不要动以前的， N多人改过的代码，肯定有很多特殊的应用场景，或者为了修复某种深层次的bug，这种东西可能很难复现。要加功能就找到对应点修改就可以了。要换平台的话就把接口抽出来，整理一下即可，千万别去重写。

qq335702318 · 发表于 2018-1-3 08:54:00

2楼不仅及格，而且相当优秀！

Ray______ · 发表于 2018-1-3 08:55:04

我想象的那种接手会很难理清，除非上几家都有挺好的风格和清晰的代码。大部分时候只能依靠说明书和使用时所知道的逻辑，针对性打补丁。

liansh2002 · 发表于 2018-1-3 09:45:40

zchong 发表于 2018-1-3 07:51
应该不是少了，是括错内容了吧

哈哈哈，你这么一说还真有可能，编译器检查括号是最基本的功能了，这么低级的问题基本都是人为原因了

霸气侧漏 · 发表于 2018-1-3 12:00:59

liansh2002 发表于 2018-1-2 17:35
血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至 ...

求少了一个}能编译通过的方法

ljc · 发表于 2018-1-3 14:14:32

二楼完美解决问题，哈哈

qingyin2009 · 发表于 2018-1-3 14:54:33

ljc 发表于 2018-1-3 14:14
二楼完美解决问题，哈哈

没有按2楼的做，毕竟原来是我的项目，昨天改了变量命名规则，挪动了部分代码，今天又画了逻辑框图，跟FPGA的RTL图一样，不过对VISIO使用熟悉有限，所以画的比较简单。现在越来越清晰了点

huangqi412 · 发表于 2018-1-3 19:51:11

qingyin2009 发表于 2018-1-3 14:54
没有按2楼的做，毕竟原来是我的项目，昨天改了变量命名规则，挪动了部分代码，今天又画了逻辑框图，跟FPG ...

用si之类改动全局变量名字好理解结构就没精力去改动了只能用到什么加什么

PICTURE · 发表于 2018-1-3 20:36:12

最好能不改就不改了，否则新的软件出了深层次的问题，旧的就没有，这就麻烦了

dwj0 · 发表于 2018-1-5 11:09:41

MAD_FISH 发表于 2018-1-2 17:38
求少了一个}能编译通过的方法

char *str = "{";

styleno1 · 发表于 2018-1-5 11:52:18

有个工具叫Understand，兴许有用

qingyin2009 · 发表于 2018-1-5 12:36:30

styleno1 发表于 2018-1-5 11:52
有个工具叫Understand，兴许有用

o(∩_∩)o 哈哈，用了很久了，它生成的图有的时候比较好，帮助有限

MAD_FISH · 发表于 2018-1-6 18:42:16

dwj0 发表于 2018-1-5 11:09
char *str = "{";

你这条语句是完整的，不要骗我

gsq19920418 · 发表于 2018-1-6 19:01:33

liansh2002 发表于 2018-1-2 17:35
血的教训，N多人改过的程序，要不就重构，要不就别动，然后想一个不动其它任何程序的方法。
曾经拿到一个至 ...

我感觉这个编译器很好，求链接

磊磊映画 · 发表于 2018-1-7 21:38:18

楼主相关一类的变量可以用结构体封装到一起来处理

sjh18256 · 发表于 2018-1-8 15:35:55

最好不要大动，添加你要的功能就好。大动还不如重写呢。

xuzhiping9889 · 发表于 2018-1-8 17:12:24

经过很多人的手的程序，乱就让他乱吧，感觉现在重构很可能会出一堆莫名其妙的问题，而且重新测试也需要很长时间

wang1216 · 发表于 2018-1-16 14:43:10

我一般都重写，这种程序基本没救

fnems · 发表于 2018-1-16 15:46:44

本帖最后由 fnems 于 2018-1-16 15:50 编辑

1 把所有全局变量封装到一个全局结构体里面，然后删除所有全局变量。
2 建立全局结构体变量的赋值和读取函数，区分原程序的全局变量读写。
3 编译时缺少了删掉的全局变量会报错。把报错的语句改成第2步里面建立的读写函数。
4 改进读写函数，增加来源源码文件和来源源码行号参数。调用时传入__FILE__和__LINE__。函数里打印源码调用信息。

5 运行上述修改的程序。这时候会按照预期依次打印出哪个C源文件的哪一行修改/读取了哪个全局变量。

qingyin2009 · 发表于 2018-4-10 22:49:25

fnems 发表于 2018-1-16 15:46
1 把所有全局变量封装到一个全局结构体里面，然后删除所有全局变量。
2 建立全局结构体变量的赋值和读取函 ...

受教了，试试去

bobo89 · 发表于 2018-4-11 20:37:13

这种代码一般不敢乱动！

wangpengcheng · 发表于 2018-4-11 23:39:18

fnems 发表于 2018-1-16 15:46
1 把所有全局变量封装到一个全局结构体里面，然后删除所有全局变量。
2 建立全局结构体变量的赋值和读取函 ...

还不如重构呢

simplorer · 发表于 2018-4-12 00:05:55

fnems 发表于 2018-1-16 15:46
1 把所有全局变量封装到一个全局结构体里面，然后删除所有全局变量。
2 建立全局结构体变量的赋值和读取函 ...

这是一个方法，我也按照这种方式处理一个三年前的老程序。

风铃夜思雨 · 发表于 2018-4-12 00:31:24

改前人的程序是很痛苦的.........但是有强迫症的....还是会忍不住动手改的

一个全局变量很多的程序，如何理清逻辑关系

阿莫论坛20周年了！感谢大家的支持与爱护！！

本帖子中包含更多资源