首页 新闻 会员 周边 捐助

【提问】如果你是老板,写出这种代码会不会开除他,代码非常简单,就是提交一个表单的事情

0
悬赏园豆:10 [已解决问题] 解决于 2018-09-06 15:26

【低级bug、居然能这样写】今天无意间看到这写代码,BS应用的,大概的一个流程就是用户在浏览器端提交一个表单而已,选择多个类型和表述这样的一个表单;服务器端添加一条数据。对,就是这么简单,是在微信上操作的。进入主题:

1.浏览器上的变量保存在服务端,选择一个类型就提交到服务器端保存,如图:

1.先看看前端选择类型的代码,选择一个类型后post到后端,选择一个变量就要保存到服务端,我操你妈的,

2.看看后端是如何保存这个id和name的,居然用的是static,你没有看错是用static保存的

 

 对就是这样保存的,用static,这完全是乱伦,还有王法吗,还有法律吗,张三提交a,李四提交b,结果王五查看自己选择的类型的时候发现有a,b。这三个提交表单的时候就直接把这都提交上去了,哎,我真是无语

当然这是来自一个工作一年的专科狗写的,不知道是怎么发布上线上的,传统软件公司,有些功能写出来,客户也懒的用,没办法。

问题补充:

我的观点是:不是用不用什么static,而已这个bug是在是太低级了

张林-布莱恩特的主页 张林-布莱恩特 | 初学一级 | 园豆:108
提问于:2018-07-12 17:00
< >
分享
最佳答案
0

这个有啥.

之前有一个同事.硬要和我说吧所有的代码都写成static的.就能解决并发问题..

收获园豆:10
吴瑞祥 | 高人七级 |园豆:29449 | 2018-07-12 19:28

666

张林-布莱恩特 | 园豆:108 (初学一级) | 2020-02-26 17:15
其他回答(10)
1

我是来赚分的

chengeng | 园豆:294 (菜鸟二级) | 2018-07-12 17:24
-1

什么语言啊?没太看懂  哈哈……

让我发会呆 | 园豆:2929 (老鸟四级) | 2018-07-12 17:26
0

好好教,不能教就换吧。

Jnetart | 园豆:404 (菜鸟二级) | 2018-07-12 18:18
0

只要保存的是映射到合理关系的集合,不管是内存还是硬盘有何不可~

谁特么说的不能存static,asp.net 本身这种内存对象多了去了,只要存在合理性就是ok的。

花飘水流兮 | 园豆:13617 (专家六级) | 2018-07-12 18:22

这个需求的是提交表单,简单点说就是他把前端选择的变量存到一个static的集合里面去,这就乱伦了,为什么我会说“乱伦”呢,我看到这代码+测试的时候我脑海里就只有这个词。

支持(0) 反对(0) 张林-布莱恩特 | 园豆:108 (初学一级) | 2018-07-12 18:41

@张林-布莱恩特: 默认session 实现和这是相当的——你认为乱伦吗?

支持(0) 反对(0) 花飘水流兮 | 园豆:13617 (专家六级) | 2018-07-12 18:57

@花飘水流兮: 实际的bug会是这样的,a选择的苹果,b选择的是梨子,结果c进来的页面就会实现梨子和苹果,他们前端选择的东西都保存到一个statis的集合上。这个是实际运行中我看到的bug

支持(0) 反对(0) 张林-布莱恩特 | 园豆:108 (初学一级) | 2018-07-12 19:40

@张林-布莱恩特: 看到这才明白你的需求, TypeList只是针对当前用户的, 不是全局的是吗

支持(0) 反对(0) 一叶兰舟飘 | 园豆:351 (菜鸟二级) | 2018-07-13 09:39

@一叶兰舟飘: 所有用户选择的类型(表单中的一个字段)都会保存到服务端公用一个static的TypeList。这代码的我只能知道static的作用的初学者都不会这么做。想不到其他合适的词,只能说乱伦吗

支持(0) 反对(0) 张林-布莱恩特 | 园豆:108 (初学一级) | 2018-07-13 12:03
1

先看需求  在说代码

雷。 | 园豆:775 (小虾三级) | 2018-07-12 18:33
2

我觉得可以捏造出需求,使得这段代码合理化

TCG2008 | 园豆:1150 (小虾三级) | 2018-07-12 18:57
-1

能运行就不错了,我觉得,哈哈。

爱编程的大叔 | 园豆:30844 (高人七级) | 2018-07-12 22:06
0

这个完全满足需求啊。别怪代码套路多!这是老板在考验你呢!

幻天芒 | 园豆:37207 (高人七级) | 2018-07-12 22:37
0

老板在乎的不是如何写代码,老板只在乎如何赚钱。不管你怎么写,能达到一样的效果就可以。除非是一个做技术的老板。

yzy | 园豆:317 (菜鸟二级) | 2018-07-13 09:19
0

专科狗被地图跑击中了。。 吐血而亡

blurs | 园豆:660 (小虾三级) | 2018-07-13 10:30

+1

支持(0) 反对(0) 三当家 | 园豆:383 (菜鸟二级) | 2018-07-13 17:32

@三当家: 陈年往事了

支持(0) 反对(0) 张林-布莱恩特 | 园豆:108 (初学一级) | 2020-02-21 22:37
清除回答草稿
   您需要登录以后才能回答,未注册用户请先注册