读同事移交的程序有感

1万 · 2019-9-10 19:35

aolin 发表于 2019-9-10 15:23
非常认同楼主的看法，这种代码做个演示可以，但绝对不是产品上使用的。
不单只楼主同事，我见到过的很多毕 ...

别污蔑教科书，教科书最多在按键那里狂加delay(), 中断那里都是有强调不能乱来的，计算机专业的教科书都讲了阻塞的概念，主要是制度问题，小公司没有个优秀的技术主管，都是先搞定销售然后搭草台班子。。。

1万 · 2019-9-10 19:35

现实就是这么的混乱，还是重写吧，将来也好维护

只看该作者 · 2019-9-10 20:23

你确定不是他故意加的诈弹

只看该作者 · 2019-9-10 20:28

本帖最后由 fly1974 于 2019-9-10 20:54 编辑

yongwong9901 发表于 2019-9-10 20:23
你确定不是他故意加的诈弹

不存在这个可能性，说句他听到了可能不高兴的话，他写代码的水平就是这样，感觉要不是他代码写得少，要不就是他写代码功能实现就行，有点bug从不深究的，不考虑完善的！  这个同事两年多一共就写了两个设备的代码，另一个设备的代码和这个风格很一致（1毫秒的定时中断里面有大量的函数执行时间在几毫秒到几十毫秒的，我开始时还奇怪为啥没有中断重入的问题，后来仔细一看，原来是进1mS中断后不清标志位，出中断时才清，所以就不会有中断重入的问题，但是所谓的1MS中断就只能是个大概值了！）：
  void TIM3_IRQHandler(void)                //1ms中断
{
//u16 flag=0;
      if(TIM3->SR&0X0001)//Òç³öÖÐ¶Ï
      {

            sect_strim();                //2017.04.06       执行时间可能远大于1mS
            if(dq_flag==1)                               //2019.07.16  add
            {
               qpshu_dis();             // 执行时间可能远大于1mS
            }       ....
   }
      TIM3->SR&=~(1<<0);//Çå³ýÖÐ¶Ï±êÖ¾Î»
}

只看该作者 · 2019-9-10 20:54

这个工资值不值，老板说了算……
代码质量好不好，重点看产品是否稳定……

只看该作者 · 2019-9-10 21:01

本帖最后由 R2D2 于 2019-9-10 22:16 编辑

关键是能不能通过测试，能通过你还怕什么呢？没有测试你refact出来的代码搞不好还不如人家呢。

我自以为写的最糟糕的代码是一个设备的配件，甲方项目需求写的还行，但是到现场发现很多需求里面没有写到的东西，甲方非常严格，整机所有可能发生的工况都要自动跑一遍，一次就是20个小时。然后我就根据不符合要求的地方改代码，在甲方那里住了半个月，终于能通过所有工况的测试了，但是代码被我改的一塌糊涂我自己都不认识了。后来我自己想改，但是没有测试条件，整套设备好几千万买不起，但是这个软件在甲方那里运行的非常好，连续采购我这个配件超过10年了。这么烂的代码这么多年甲方居然没说过一个不字也是神奇了。

只看该作者 · 2019-9-10 21:12

NickyGrout 发表于 2019-9-10 20:54
这个工资值不值，老板说了算……
代码质量好不好，重点看产品是否稳定…… ...

这个产品稳定不稳定怎么讲呢，产品刚出了3-4台，我这边看到的是程序里面加了看门狗，当然不是说不能加看门狗，只是我们公司的产品以前就没加过看门狗，以前没有因为死机出过问题（产品都做过EMC和EMS的），后来发现我这个同事在按键扫描单元省掉了一颗芯片，直接从单片机引脚引线接到面膜上，长度大概70公分左右，相当于把单片机引脚当天线用，如果手上带静电比较厉害，单片机就可能在按键被按下时被电死机，所以必须要加入看门狗，但是又不能经常性的复位，所以我猜测他是故意把温度传感器扫描时间设得比较长，1分钟左右，以减少复位几率，这个解决问题的思路，依我来看简直是头疼医头，脚痛医脚了！

只看该作者 · 2019-9-10 21:17

R2D2 发表于 2019-9-10 21:01
关键是能不能通过测试，能通过你还怕什么呢？没有测试你refact出来的代码搞不好还不如人家呢。

我自以为写 ...

连我们公司研发主管那都一堆意见，你觉得我这个同事的设备算是通过测试了吗？我们研发主管说我这个同事太难沟通，问题记了一堆，但是只有出货急需要改的让我同事改了，其它好几条都是等他离职了交给我改的！

只看该作者 · 2019-9-10 21:43

额，，，这代码确认不妥当，，，但是再机器上已经验证过能正常工作了。。。。。你就要考虑重构的成本值不值的了

只看该作者 · 2019-9-10 22:10

fly1974 发表于 2019-9-10 21:12
这个产品稳定不稳定怎么讲呢，产品刚出了3-4台，我这边看到的是程序里面加了看门狗，当然不是说不能加 ...

产品不稳定就另说了。综合产品的问题来分析程序中存在的隐患是正确的打开方式

有时候有些很奇怪、不合理的处理，往往是为了解决一些奇怪的问题……这方面要了解，虽然办法不太对，但是出发点是好的

只看该作者 · 2019-9-10 23:07

本帖最后由 fly1974 于 2019-9-10 23:09 编辑

l435799304 发表于 2019-9-10 21:43
额，，，这代码确认不妥当，，，但是再机器上已经验证过能正常工作了。。。。。你就要考虑重构的成本值不值 ...

两个设备，一个已经出了五台，目前收集反馈改进中，一个只是样机阶段，这次好像是机构件开模，样机有变动，还在改！出了五台的这个设备是老产品的改进型，就是人机界面从面膜按键加液晶屏改为面膜加串口触摸屏，刚开始的第一版PCB是我做的，程序只是从MSP430移植到STM32F103，不是从头搭的，msp430的代码是我给的！

只看该作者 · 2019-9-11 08:13

fly1974 发表于 2019-9-10 17:57
资深工程师9-10K一个月，我们公司在二三线小城市，薪水差不多就这样了，郁闷的是好长时间不涨了，也没啥 ...

要是二、三线城市，这个工资水平已经很不错了。

只看该作者 · 2019-9-11 08:53

感觉有点乱，应该是没有经过系统的训练，水平有点LOW,你们有代码规范，应该要求按照代码规范

只看该作者 · 2019-9-11 08:55

中断里面加个延迟其实也没啥大惊小怪把，虽然是影响实时性，不过很多外部信号检测要求实时性特别高的，不加系统，很多都是中断加延迟防抖在判断的，当然很多也是用定时器扫描方式。虽然我不这么写，不过从事电子行业很多年，见过不少这样写的，见怪不怪，影响除了实时性也没你说的辣么夸张。不过如果不是很复杂的案子，重写不就得了，没必要抱怨，如果涉及很深的专业知识的，碰到烂代码是很头疼。