每个 company 可对应多个 shop 。 因为每个 company 可存在一个特殊店铺,早期历史代码设计的 shop_id=99 (应该是觉得不会有人会开通超过 99 家店吧),并且不记录在表中。 现在有一个客户的店铺超过了 99 家,导致代码中大量判断 shop_id=99 的逻辑执行异常。
不止后端代码,而且客户端代码中也存在这样的逻辑判断。
每个 company 可对应多个 shop 。 因为每个 company 可存在一个特殊店铺,早期历史代码设计的 shop_id=99 (应该是觉得不会有人会开通超过 99 家店吧),并且不记录在表中。 现在有一个客户的店铺超过了 99 家,导致代码中大量判断 shop_id=99 的逻辑执行异常。
不止后端代码,而且客户端代码中也存在这样的逻辑判断。
|
javalaw2010Jan 30
1
先把店铺拆分到两个 company 中去,再想办法解决历史遗留问题
|
|
2
|
|
bootvueJan 30
3
shop_id 如何生成的 baseline 从 100 开启
|
|
liyanggyangJan 30
4
@zhengfan2016 我觉得这是最好的办法
|
|
5
@javalaw2010 变成两个 company 也会影响到业务,现在最低成本的方式确实是二楼的做法
|
|
6
@zhengfan2016 现在确实考虑这么干
|
|
7
@zhengfan2016 这就是天才吗?哈哈哈 还是说以前遇到过
|
|
xiaowoliJan 30
8
(应该是觉得不会有人会开通超过 99 家店吧)
你们公司怪励志的啊 |
|
Gilfoyle26Jan 30
10
为啥一定要从代码的方向来解决呢?超过 99 给他开 2 个账号不就可以了
|
|
ghm2mailJan 30
11
shop = -1 ,-2,-3.....
|
|
justseemoreJan 30
12
|
|
justseemoreJan 30
13
@justseemore #12 就怕不光= 还有>=
|
|
javalaw2010Jan 30
14
@shitcode #5 考虑到如果有代码逻辑是 if(shop_id >= 99) 就危险了。
|
|
montaro2017Jan 30
15
|
|
dcdloveJan 30
16
shop 表的 种子升级到四位数 1000 ,然后历史所有 shop_id 加两个 00
|
|
17
@javalaw2010 这个确实是没有,都是判断 99 的,但是也说不好真存在在哪个角落
|
|
meteora0tkvoJan 30
18
本来应该加多一个字段去标识判断是否是特殊店铺,而后端偷懒搞一个特殊 id 去判断
|
|
leecqanJan 30
19
其实我没看懂,为啥会有异常,有大佬解释一下么
|
|
noahjsnJan 30
20
@meteora0tkvo 《论屎山代码的形成》《论后期偿还技术债的痛苦》
|
|
konakonaJan 30
21
|
|
unusedJan 30
22
特殊店铺 ID 为什么不用 0 或者 1 呢
|
|
23
所有的 99 改成 999 呢
把这个问题留个下一个人 |
|
24
我靠,我也曾经遇到过这种问题,也是类商城系统,也是 99 是特殊值。是不是有哪个培训班的老师喜欢用 99 当特殊值啊,明明 Int32.MaxValue 或者 -1 之类的更合理
|
|
irockytanJan 31 via iPhone
25
@xiaowoli 我 15 年干那家公司,也是电商,接手的代码里面订单号的逻辑是按月加上一个 6 位数字,也是递增,后面业务量起来,一个月超过 100 万订单,于是大量订单号重复
|
|
gongymJan 31 via Android
26
@leecqan 店铺 id 是自增的,特殊店铺 id 是 99 ,特殊店铺有特殊逻辑,但是有一个公司创建了超过 99 家店,导致 id 为 99 的是正常店占了特殊店铺的 id ,特殊逻辑就有问题了(执行特殊逻辑的判断就是 id=99 )
|
|
skallzJan 31
28
这种特殊逻辑蛮多的,包括 2099 年或者 3000 年被视为永久一样,只是一般我们写这种逻辑不敢把数字写的特别小,哈哈
|
|
syboyFeb 1 via Android
29
@zhengfan2016 很显然他这个已经把 99 当成普通店铺了
|