首页新闻招聘找找看知识库

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

0
悬赏园豆:10 [待解决问题]

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

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

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

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

 

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

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

问题补充:

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

张林-布莱恩特的主页 张林-布莱恩特 | 初学一级 | 园豆:56
提问于:2018-07-12 17:00

挺无语的!可能有些人就是这样的吧

张林-布莱恩特 5天前
< >
分享
所有回答(11)
0

我是来赚分的

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

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

让我发会呆 | 园豆:1877 (小虾三级) | 2018-07-12 17:26
0

好好教,不能教就换吧。

不动 | 园豆:202 (菜鸟二级) | 2018-07-12 18:18
2

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

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

花飘水流兮 | 园豆:5851 (大侠五级) | 2018-07-12 18:22

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

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

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

支持(0) 反对(0) 花飘水流兮 | 园豆:5851 (大侠五级) | 2018-07-12 18:57

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

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

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

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

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

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

先看需求  在说代码

竜咚咚 | 园豆:343 (菜鸟二级) | 2018-07-12 18:33
1

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

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

这个有啥.

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

吴瑞祥 | 园豆:26431 (高人七级) | 2018-07-12 19:28
0

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

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

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

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

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

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

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

blurs | 园豆:365 (菜鸟二级) | 2018-07-13 10:30

+1

支持(0) 反对(0) 三当家 | 园豆:183 (初学一级) | 2018-07-13 17:32
清除回答草稿
   您需要登录以后才能回答,未注册用户请先注册