公司40岁的程序員到底在写什么代码

去年在前公司玩了一年(基本兩三個月一個需求),除了日常維護就一些特別簡單的功能開發,到年底也沒見到公司黃(國企背景),沒辦法只好裸辭,現在這個公司各方面还不错,但是令我不得不吐槽是10年開發人員,技術真的是令人吃驚,先看看他的項目接口
项目结构
在这里插入图片描述
运行效果
在这里插入图片描述

1、這個接口項目的架構 是三層架構(10年前的技术啊),數據庫鏈接方式還是傳統的ADO.NET操作数据库 ,不必我多说,大学都不教这些
2、代码逻辑错误,十年开发,能力也不怎么样啊,实习生都不会犯这样的错误:
在这里插入图片描述这里首先他给自己埋了一个雷,他的循环里并没有对每一条数据进行判断,比如10条数据,7条成功,3条没有成功(第4,6,9条数据失败)程序依然会返回成功。foreach 循环没有处理失败的数据。
我们继续看看他的SaveJobHead究竟是什么鬼玩意。
在这里插入图片描述看到这里的逻辑,我如果要更新或新增一条数据,他的写法是先去查这条数据有没有,然后再更新或者添加;
与数据库交互是:往数据操作一条数据,要和数据库做两次交互。
就算不用orm框架,自己全手写数据库操作,也说得过去(自己愿意嘛),可以完全整合在一个SQL里面,使用、Merge into

我们继续往下看GetProcessFlow 接口

两个一模一样的逻辑方法,只是执行的数据库不一样,所以加一个Type不就行了,完全可以写在一个里面,提高代码质量
在这里插入图片描述前面取json 参数我不想多说啥了,垃圾啊,8行代码序列化两遍取4个参数,这代码质量真的令人震惊
GetProcessFlowByProductOFN
GetProcessFlowByProductAOC
从这两个方法名有点类似,应该是同一种逻辑,有点像类型不一样,看看下面的分析
在这里插入图片描述
从上面我100%确定这个同事绝对没有10年软件开发经验,懂得都懂。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 1
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值