作者a88241050 (再回頭已是百殘身)
看板Soft_Job
標題[請益] 這是個很低級的錯誤嗎？
時間Tue May 7 20:42:01 2019
是這樣的
最近工作上需要寫一支程式寫資料進資料庫
並產生流水號當key值寫進欄位中
我的想法很單純
就是直接抓資料庫資料筆數
然後拿筆數+1當新增的流水號
寫完後交給SA測試
結果他一測馬上就出問題了
後來我才知道 原來流水號是要抓最大值+1
而不是一直照順序編下去
因為這個我被SA噹的體無完膚
請問這個是很基本的觀念嗎？
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 49.214.145.27
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1557232923.A.965.html
1F：推 g2581856: 我覺得是啦05/07 20:47
2F：→ g2581856: 不過以後別再犯就OK了吧，用不著噹05/07 20:47
3F：→ diabloevagto: 你自己測試測不出來嗎？？？05/07 20:48
我沒想到他會手動刪除資料啊
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 20:50:06
4F：推 abccbaandy: SA需求不明確還敢噹人喔...05/07 20:50
5F：→ abccbaandy: 產生流水號規則有說/寫清楚嗎？05/07 20:51
6F：→ meowyih: ... 呃, 這也太新手了05/07 20:51
7F：→ abccbaandy: 另外第二種作法有考慮multi thread嗎？05/07 20:52
8F：→ meowyih: 還有這不是auto incremental自動會幫你寫嗎,為啥要自己手05/07 20:53
9F：→ meowyih: 動寫？05/07 20:53
10F：推 wayne12345: 是05/07 20:53
11F：推 sourbait: 資料庫一般來說要考慮到刪除的可能05/07 20:54
12F：→ weinine32: 有考慮lock問題嗎？ 流水號會重複喔！05/07 20:55
13F：→ MOONY135: 要看需求吧05/07 20:56
14F：推 alihue: 我就算第一次學也不會犯這種錯05/07 20:58
15F：推 yyc1217: 是 不過刪除也有分直接刪除或是註記刪除05/07 20:59
16F：→ yyc1217: 如果能用資料庫本身的流水號更好 除非流水號有要求格式05/07 21:00
17F：→ yyc1217: 一般來說要刪除也要保留資料比較好 畢竟記憶體不貴05/07 21:02
18F：→ yyc1217: 查到了 叫soft delete05/07 21:05
19F：→ alan3100: 流水號由DB產生是基本, 除非你有其他特殊需求05/07 21:07
20F：→ supernow: 是的，這很基本，不抓最大的就會有重複的問題 05/07 21:11
21F：推 t64141: 滿基本的,不過多了一個機會去研究流水號的問題也是好事 05/07 21:12
請問DB產生的意思是? SA是跟我說抓最大的號碼加1,比如說現在有0000,0001,0002,0003,
0004五筆，這五筆都是由程式產生出來的，所以照理來說程式產生的下一筆就是0005,但
現在問題是SA可能在資料庫直接手動新增資料，比如說現在他手動新增了一筆1234,這時
再跑程式的話就應該要產生1235,而不是0006,他的意思是這樣
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 21:19:45
22F：→ alan3100: 抓最大+1也是錯的 除非你一次只insert一筆 05/07 21:14
是一次一筆沒錯，不會有多筆的情況
23F：→ meowyih: google 'sql auto_increment' 05/07 21:21
24F：推 benjamin99: 比較好奇 SA 為啥會有 max+1 就正確的概念？ 05/07 21:23
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 21:26:17
26F：→ Chris926926: 好奇不用auto increment的原因是?有特殊需求? 05/07 21:28
27F：推 googoo1102: 保險起見把流水號欄位設成unique 05/07 21:29
28F：→ godddddd: 看你資歷 沒事請用sql內建流水號 05/07 21:30
29F：推 x000032001: 同時插兩筆抓max也是爆炸 05/07 21:50
30F：推 q26766: 一樣錯啊哈哈 ，別理有些酸推文，誰沒新手過 05/07 21:54
31F：→ mackliu: 你們的流水號會不會是設計成文字格式,而且還沒設主鍵?XD 05/07 21:56
32F：→ GoodFriday: 抓最大值+1不就超容易抓到重複 只有第一筆塞得進去 05/07 21:59
33F：推 ripple0129: 你新手就算了，SA也新手，拜託找個資深的來帶一下吧 05/07 22:00
34F：→ LinuxKernel: 慘 05/07 22:06
35F：→ jyunwei: 那你以後就會想到了，如果你資歷一年內的話啦 05/07 22:07
36F：推 crossdunk: 很基本阿，可是為什麼是SA測試呀 05/07 22:13
37F：→ ChungLi5566: 非本科? 實務上流水號都給db自己編啊 05/07 22:14
38F：推 crossdunk: 他的流水號應該是前面還需要加其他東西 05/07 22:15
39F：推 abraxas: 他那做法，直接新增一筆溢位筆數減一的資料不就炸掉了 05/07 22:15
40F：推 Label: 他還沒測transation呢05/07 22:16
41F：推 kurtsgm: 405/07 22:19
43F：推 bill0205: 通常db 的 key都是auto increment吧= =05/07 22:23
44F：推 abccbaandy: 好奇到底什麼使用情境可以想出這種神奇作法... 05/07 22:24
45F：→ bill0205: 我自己是會做兩種序號 一種是db 一種是顯示的SN 05/07 22:24
可是我的序號前面還要加其他字串欸，等於說我在寫入資料庫前就要取得db產生的key值
組成我要的流水號後再寫入db,這有辦法嗎？
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 22:33:10
46F：推 rahit: 這東西一般資料庫都能自動給… 05/07 22:31
47F：推 cloudgoogle: 單純抓max一樣也是有問題 多人同時操作的執行順序... 05/07 22:35
48F：推 drajan: 蠻嚴重的錯誤 不過如果入行不到半年可以容忍 05/07 22:38
49F：→ ChungLi5566: 要加字串還是補左邊零的寫在程式就好 幹嘛放在table 05/07 22:42
50F：→ ChungLi5566: 佔DB磁碟空間 05/07 22:42
51F：→ bill0205: 有序的就照C大方法 或是把字串丟到另外欄位 撈出來時後 05/07 22:43
52F：→ bill0205: 再組起來 無序比較麻煩 05/07 22:43
53F：推 ashlikewing: 流水號自己做太扯了啦 05/07 22:44
54F：推 cloudgoogle: 如果可以撈出來另外處理當然是最好 不過有的Table 05/07 22:46
55F：→ cloudgoogle: 是要給別人存取的，可能就沒辦法控制在自己這邊 05/07 22:46
56F：→ cloudgoogle: 即便如此，還是可以像bill大說的存兩欄可能好一點 05/07 22:47
57F：推 onlyeric23: 菜到不行 05/07 22:49
58F：→ you878787: 如果我看到這種code應該會把這個人幹到火星去吧= = 05/07 22:51
59F：→ ChungLi5566: 通常別系統來存取 是走API而不會直連DB 05/07 22:51
60F：→ bill0205: 題外話 大家是怎麼做無序的流水序號 05/07 22:52
61F：推 you878787: 開始寫前請先訂好test case, 另外有基礎知識應該根本 05/07 22:55
62F：→ you878787: 不能這樣抓.... 05/07 22:55
63F：→ lukelove: 比較大的問題是 開發前沒有先查怎麼設計就土砲做 05/07 22:56
64F：推 jinmin88: 菜到不行的問題 05/07 23:10
65F：推 blackie1019: 兩個都會笑到翻過去 05/07 23:39
66F：推 crossdunk: 無序怎麼叫做流水XD 05/07 23:57
67F：→ Darkword1987: 資料筆數減少不就GG了 05/07 23:58
68F：推 CloudyWing: 總筆數+1在跳號的情況下有可能序號重複，有經驗的設計 05/08 00:01
69F：→ CloudyWing: 應該就要注意這件事... 05/08 00:02
70F：→ CloudyWing: 今天就算是Soft delete，也有可能因為一些意外因素導 05/08 00:05
71F：推 jlhc: 還是不懂為什麼不是auto increment 05/08 00:06
72F：→ CloudyWing: 致資料跳號，不過我好奇設計上有介面可以輸入序號 05/08 00:07
73F：→ CloudyWing: 還是SA直接加資料庫？ 05/08 00:07
74F：推 bibo9901: uuid.. 05/08 00:09
75F：推 molopo: 流水號一直上去就好 05/08 00:12
76F：→ gpctv: 這個也要刁，改就好了啊，在我們公司SA地位像狗一樣 05/08 00:27
77F：→ sachung28: serial primary key就會自己產生序號+避免重複了 05/08 00:28
78F：→ viper9709: 流水號不用也不能自己做吧... 05/08 00:28
79F：→ sachung28: 可以在SQL寫入DB時順便用return回傳+改格式 前提是DB 05/08 00:33
80F：→ sachung28: 有支援...另外 刪除並重塞資料測序號產生邏輯ok 但我 05/08 00:33
81F：→ sachung28: 第一次看到QA自己塞數字當流水號 05/08 00:33
82F：→ sachung28: 如果是UI呈現美觀 流水號要補字串 這可以API hard code 05/08 00:36
83F：→ sachung28: 處理 或是另外開個流水號前綴字的定義表儲存 查詢時將 05/08 00:36
84F：→ sachung28: 前綴和序號串起來 05/08 00:36
85F：→ sachung28: 建議你去惡補一下DB觀念 學會用DB能省很多寫程式的時 05/08 00:40
86F：→ sachung28: 間 05/08 00:40
88F：推 mathrew: 4 這種問題就代表 你根本寫的時候 很多狀況都沒想過 05/08 06:23
89F：推 mathrew: 但是為什麼流水號不給DB自動去編 05/08 06:26
90F：推 brianhsu: 流水號不要自己生，問題很多的。交給 DB，例如 auto Inc 05/08 07:26
91F：→ brianhsu: 之類的功能處理。 05/08 07:26
92F：噓 deray: 流水號幹嘛最大值+1 auto increment 05/08 08:36
93F：→ qpowjohn: 抓最大值+1想了一下應該有可能，可是要開transaction保 05/08 08:44
94F：→ qpowjohn: 證不會有人寫表...有錯請指教 05/08 08:44
95F：→ xdraculax: 總數加一很瞎，最大加一也只是50步笑百步 05/08 08:51
96F：→ y3k: 這就是菜而已吧 05/08 08:57
97F：→ luluking: 你不覺得這樣寫反而麻煩 05/08 09:03
98F：→ lion0208: 為何不用 auto increment？另外取最大+1，同時間確定只 05/08 09:10
99F：→ lion0208: 會有一個 process/thread 在寫？ 05/08 09:10
100F：推 mago: 如果有大量同時寫入問題,兩個都不行 05/08 09:14
101F：→ CodingMan: 換個想法你就知道答案了 難不成這是高級問題嗎? 05/08 09:19
102F：→ lazarus1121: auto increment如果需要跨table當key不就不能用了 05/08 09:20
103F：推 silent5566: 直接用DB的sequence去滾不就好了 05/08 09:34
104F：→ silent5566: 你這兩種作法未來table在應用上會很差 05/08 09:38
105F：→ silent5566: 多人使用的時候沒咬住key值很容易有重複的問題 05/08 09:40
106F：→ hakama99: 我一開始寫也不知道有自動產生的功能 也是跟你一樣XD 05/08 10:18
107F：→ testPtt: 設預設值不要用sql寫就好啦 05/08 10:45
108F：→ johnny9144: 4 05/08 11:23
109F：推 f124: 叫資料庫自己跑流水號就好 還自己編幹嘛... 05/08 11:25
110F：推 iamshiao: 是，而且拿 max 也可能會有問題，用資料庫本身提供的自 05/08 11:34
111F：→ iamshiao: 動增加機制比較好 05/08 11:34
112F：推 seedli: 除非系統只有一個人用，不然抓max+1可能有race condition 05/08 12:13
113F：→ worf: ... 05/08 14:08
114F：推 amyt: 是 05/08 14:25
115F：推 laker780304: 不嫌麻煩可以建立Sequence，新增時取Sequence用即可 05/08 15:20
116F：推 zg0608x: 好有創意的方法 05/08 17:17
117F：推 indexcome: 其實就算沒講清楚需求。這種可能發生的情況寫code的人 05/08 17:24
118F：→ indexcome: 應該都會注意到的. 05/08 17:24
119F：推 BignoZe: 照常理不太會這樣寫 05/08 17:59
120F：→ rocwild: 基本 05/08 18:30
121F：→ GinginDenSha: 兩個都4 05/08 19:00
122F：推 xo1100: 抓最大+1的話 你之後碰用到多人使用的系統還會再被噹一次 05/08 22:15
123F：推 stupid0319: 是SA太淺了 05/09 00:09
124F：推 hooll111: 這個還蠻基本的耶 取筆數超抖... 05/09 01:32
125F：→ deanh: 你們兩個都錯了啊，搞笑 05/09 04:44
126F：→ gettheworld: 覺得是想展現優越感 05/09 12:53
127F：推 zerrofighter: 這種事情就是炸過一次就知道了 05/09 14:13
128F：推 mdkn35: 我都用uuid 05/09 14:23
129F：推 DerLuna: 這兩個方法都不好吧 05/09 15:54
130F：推 cholux: 是有一點，不過看問題可能會有更好的作法 05/09 18:04
131F：推 panpan: 笑死 看到兩個菜雞戶啄 05/09 20:32
132F：推 Frecci: 妳太菜是真的 但是不代表他能夠直接進入db刪資料 05/09 22:45
133F：→ Frecci: 假設是用ui刪除的話 那完全就妳問題了 05/09 22:45
134F：推 paulshain04: 這怎麼會是自己新增 DB不是有自動流水號 column不用 05/10 09:09
135F：→ paulshain04: 填啊 05/10 09:09
136F：噓 owen5566: == 這樣寫怎麼可能沒問題 05/12 07:50
137F：噓 rickboo0809: 超低級 05/12 10:41
138F：推 Tony427: 不管基本不基本，犯錯要搞懂為何不能這樣做，以及找出最 05/13 14:25
139F：→ Tony427: 佳做法，至少下次不要再犯 05/13 14:26
140F：→ newhandfun: 想必閣下跟我一樣是沒人帶的菜鳥吧，能夠發現問題提早 05/15 08:51
141F：→ newhandfun: 解決真是件好事。 05/15 08:51
142F：推 silent5566: 大家也不用嘲諷吧 每個人也是爆炸中成長QQ 05/16 10:35
143F：→ silent5566: 推Tony的心態 05/16 10:35