【低级bug、居然能这样写】今天无意间看到这写代码,BS应用的,大概的一个流程就是用户在浏览器端提交一个表单而已,选择多个类型和表述这样的一个表单;服务器端添加一条数据。对,就是这么简单,是在微信上操作的。进入主题:
1.浏览器上的变量保存在服务端,选择一个类型就提交到服务器端保存,如图:
1.先看看前端选择类型的代码,选择一个类型后post到后端,选择一个变量就要保存到服务端,我操你妈的,
2.看看后端是如何保存这个id和name的,居然用的是static,你没有看错是用static保存的
对就是这样保存的,用static,这完全是乱伦,还有王法吗,还有法律吗,张三提交a,李四提交b,结果王五查看自己选择的类型的时候发现有a,b。这三个提交表单的时候就直接把这都提交上去了,哎,我真是无语
当然这是来自一个工作一年的专科狗写的,不知道是怎么发布上线上的,传统软件公司,有些功能写出来,客户也懒的用,没办法。
我的观点是:不是用不用什么static,而已这个bug是在是太低级了
这个有啥.
之前有一个同事.硬要和我说吧所有的代码都写成static的.就能解决并发问题..
666
我是来赚分的
什么语言啊?没太看懂 哈哈……
好好教,不能教就换吧。
只要保存的是映射到合理关系的集合,不管是内存还是硬盘有何不可~
谁特么说的不能存static,asp.net 本身这种内存对象多了去了,只要存在合理性就是ok的。
这个需求的是提交表单,简单点说就是他把前端选择的变量存到一个static的集合里面去,这就乱伦了,为什么我会说“乱伦”呢,我看到这代码+测试的时候我脑海里就只有这个词。
@张林-布莱恩特: 默认session 实现和这是相当的——你认为乱伦吗?
@花飘水流兮: 实际的bug会是这样的,a选择的苹果,b选择的是梨子,结果c进来的页面就会实现梨子和苹果,他们前端选择的东西都保存到一个statis的集合上。这个是实际运行中我看到的bug
@张林-布莱恩特: 看到这才明白你的需求, TypeList只是针对当前用户的, 不是全局的是吗
@一叶兰舟飘: 所有用户选择的类型(表单中的一个字段)都会保存到服务端公用一个static的TypeList。这代码的我只能知道static的作用的初学者都不会这么做。想不到其他合适的词,只能说乱伦吗
先看需求 在说代码
我觉得可以捏造出需求,使得这段代码合理化
能运行就不错了,我觉得,哈哈。
这个完全满足需求啊。别怪代码套路多!这是老板在考验你呢!
老板在乎的不是如何写代码,老板只在乎如何赚钱。不管你怎么写,能达到一样的效果就可以。除非是一个做技术的老板。
专科狗被地图跑击中了。。 吐血而亡
+1
@三当家: 陈年往事了