看板 Soft_Job作者 a88241050 (再回頭已是百殘身)標題 [請益] 這是個很低級的錯誤嗎?時間 Tue May 7 20:42:01 2019
是這樣的
最近工作上需要寫一支程式寫資料進資料庫
並產生流水號當key值寫進欄位中
我的想法很單純
就是直接抓資料庫資料筆數
然後拿筆數+1當新增的流水號
寫完後交給SA測試
結果他一測馬上就出問題了
後來我才知道 原來流水號是要抓最大值+1
而不是一直照順序編下去
因為這個我被SA噹的體無完膚
請問這個是很基本的觀念嗎?
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 49.214.145.27
※ 文章代碼(AID): #1SqNqRbb (Soft_Job)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1557232923.A.965.html
※ 同主題文章:
[請益] 這是個很低級的錯誤嗎?
05-07 20:42 a88241050
推 g2581856: 我覺得是啦
不過以後別再犯就OK了吧,用不著噹1F 05/07 20:47
我沒想到他會手動刪除資料啊
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 20:50:06
推 abccbaandy: SA需求不明確還敢噹人喔...
產生流水號規則有說/寫清楚嗎?4F 05/07 20:50
→ meowyih: ... 呃, 這也太新手了6F 05/07 20:51
→ abccbaandy: 另外第二種作法有考慮multi thread嗎?7F 05/07 20:52
→ meowyih: 還有這不是auto incremental自動會幫你寫嗎,為啥要自己手動寫?8F 05/07 20:53
推 sourbait: 資料庫一般來說要考慮到刪除的可能11F 05/07 20:54
→ weinine32: 有考慮lock問題嗎? 流水號會重複喔!12F 05/07 20:55
推 alihue: 我就算第一次學也不會犯這種錯14F 05/07 20:58
推 yyc1217: 是 不過刪除也有分直接刪除或是註記刪除
如果能用資料庫本身的流水號更好 除非流水號有要求格式一般來說要刪除也要保留資料比較好 畢竟記憶體不貴
查到了 叫soft delete15F 05/07 20:59
→ alan3100: 流水號由DB產生是基本, 除非你有其他特殊需求19F 05/07 21:07
→ supernow: 是的,這很基本,不抓最大的就會有重複的問題20F 05/07 21:11
推 t64141: 滿基本的,不過多了一個機會去研究流水號的問題也是好事21F 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
→ alan3100: 抓最大+1也是錯的 除非你一次只insert一筆22F 05/07 21:14
是一次一筆沒錯,不會有多筆的情況
→ meowyih: google 'sql auto_increment'23F 05/07 21:21
推 benjamin99: 比較好奇 SA 為啥會有 max+1 就正確的概念?24F 05/07 21:23
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 21:26:17
→ Chris926926: 好奇不用auto increment的原因是?有特殊需求?26F 05/07 21:28
→ godddddd: 看你資歷 沒事請用sql內建流水號28F 05/07 21:30
推 q26766: 一樣錯啊哈哈 ,別理有些酸推文,誰沒新手過30F 05/07 21:54
→ mackliu: 你們的流水號會不會是設計成文字格式,而且還沒設主鍵?XD31F 05/07 21:56
→ GoodFriday: 抓最大值+1不就超容易抓到重複 只有第一筆塞得進去32F 05/07 21:59
推 ripple0129: 你新手就算了,SA也新手,拜託找個資深的來帶一下吧33F 05/07 22:00
→ jyunwei: 那你以後就會想到了,如果你資歷一年內的話啦35F 05/07 22:07
推 crossdunk: 他的流水號應該是前面還需要加其他東西38F 05/07 22:15
推 abraxas: 他那做法,直接新增一筆溢位筆數減一的資料不就炸掉了39F 05/07 22:15
推 Label: 他還沒測transation呢40F 05/07 22:16
推 bill0205: 通常db 的 key都是auto increment吧= =43F 05/07 22:23
推 abccbaandy: 好奇到底什麼使用情境可以想出這種神奇作法...44F 05/07 22:24
→ bill0205: 我自己是會做兩種序號 一種是db 一種是顯示的SN45F 05/07 22:24
可是我的序號前面還要加其他字串欸,等於說我在寫入資料庫前就要取得db產生的key值
組成我要的流水號後再寫入db,這有辦法嗎?
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 22:33:10
推 rahit: 這東西一般資料庫都能自動給…46F 05/07 22:31
推 cloudgoogle: 單純抓max一樣也是有問題 多人同時操作的執行順序...47F 05/07 22:35
推 drajan: 蠻嚴重的錯誤 不過如果入行不到半年可以容忍48F 05/07 22:38
→ ChungLi5566: 要加字串還是補左邊零的寫在程式就好 幹嘛放在table佔DB磁碟空間49F 05/07 22:42
→ bill0205: 有序的就照C大方法 或是把字串丟到另外欄位 撈出來時後再組起來 無序比較麻煩51F 05/07 22:43
推 cloudgoogle: 如果可以撈出來另外處理當然是最好 不過有的Table
是要給別人存取的,可能就沒辦法控制在自己這邊
即便如此,還是可以像bill大說的存兩欄可能好一點54F 05/07 22:46
→ you878787: 如果我看到這種code應該會把這個人幹到火星去吧= =58F 05/07 22:51
→ bill0205: 題外話 大家是怎麼做無序的流水序號60F 05/07 22:52
推 you878787: 開始寫前請先訂好test case, 另外有基礎知識應該根本不能這樣抓....61F 05/07 22:55
→ lukelove: 比較大的問題是 開發前沒有先查怎麼設計就土砲做63F 05/07 22:56
推 CloudyWing: 總筆數+1在跳號的情況下有可能序號重複,有經驗的設計應該就要注意這件事...
今天就算是Soft delete,也有可能因為一些意外因素導68F 05/08 00:01
推 jlhc: 還是不懂為什麼不是auto increment71F 05/08 00:06
→ CloudyWing: 致資料跳號,不過我好奇設計上有介面可以輸入序號
還是SA直接加資料庫?72F 05/08 00:07
推 molopo: 流水號一直上去就好75F 05/08 00:12
→ gpctv: 這個也要刁,改就好了啊,在我們公司SA地位像狗一樣76F 05/08 00:27
→ sachung28: serial primary key就會自己產生序號+避免重複了77F 05/08 00:28
→ sachung28: 可以在SQL寫入DB時順便用return回傳+改格式 前提是DB有支援...另外 刪除並重塞資料測序號產生邏輯ok 但我第一次看到QA自己塞數字當流水號
如果是UI呈現美觀 流水號要補字串 這可以API hard code處理 或是另外開個流水號前綴字的定義表儲存 查詢時將前綴和序號串起來
建議你去惡補一下DB觀念 學會用DB能省很多寫程式的時間79F 05/08 00:33
推 mathrew: 4 這種問題就代表 你根本寫的時候 很多狀況都沒想過但是為什麼流水號不給DB自動去編88F 05/08 06:23
推 brianhsu: 流水號不要自己生,問題很多的。交給 DB,例如 auto Inc 之類的功能處理。90F 05/08 07:26
噓 deray: 流水號幹嘛最大值+1 auto increment92F 05/08 08:36
→ qpowjohn: 抓最大值+1想了一下應該有可能,可是要開transaction保證不會有人寫表...有錯請指教93F 05/08 08:44
→ xdraculax: 總數加一很瞎,最大加一也只是50步笑百步95F 05/08 08:51
→ y3k: 這就是菜而已吧96F 05/08 08:57
→ lion0208: 為何不用 auto increment?另外取最大+1,同時間確定只會有一個 process/thread 在寫?98F 05/08 09:10
推 mago: 如果有大量同時寫入問題,兩個都不行100F 05/08 09:14
→ CodingMan: 換個想法你就知道答案了 難不成這是高級問題嗎?101F 05/08 09:19
→ lazarus1121: auto increment如果需要跨table當key不就不能用了102F 05/08 09:20
推 silent5566: 直接用DB的sequence去滾不就好了
你這兩種作法未來table在應用上會很差
多人使用的時候沒咬住key值很容易有重複的問題103F 05/08 09:34
→ hakama99: 我一開始寫也不知道有自動產生的功能 也是跟你一樣XD106F 05/08 10:18
→ testPtt: 設預設值不要用sql寫就好啦107F 05/08 10:45
推 f124: 叫資料庫自己跑流水號就好 還自己編幹嘛...109F 05/08 11:25
推 iamshiao: 是,而且拿 max 也可能會有問題,用資料庫本身提供的自動增加機制比較好110F 05/08 11:34
推 seedli: 除非系統只有一個人用,不然抓max+1可能有race condition112F 05/08 12:13
→ worf: ...113F 05/08 14:08
推 amyt: 是114F 05/08 14:25
推 laker780304: 不嫌麻煩可以建立Sequence,新增時取Sequence用即可115F 05/08 15:20
推 indexcome: 其實就算沒講清楚需求。這種可能發生的情況寫code的人應該都會注意到的.117F 05/08 17:24
推 BignoZe: 照常理不太會這樣寫119F 05/08 17:59
推 xo1100: 抓最大+1的話 你之後碰用到多人使用的系統還會再被噹一次122F 05/08 22:15
推 hooll111: 這個還蠻基本的耶 取筆數超抖...124F 05/09 01:32
→ deanh: 你們兩個都錯了啊,搞笑125F 05/09 04:44
推 mdkn35: 我都用uuid128F 05/09 14:23
推 DerLuna: 這兩個方法都不好吧129F 05/09 15:54
推 cholux: 是有一點,不過看問題可能會有更好的作法130F 05/09 18:04
推 panpan: 笑死 看到兩個菜雞戶啄131F 05/09 20:32
推 Frecci: 妳太菜是真的 但是不代表他能夠直接進入db刪資料
假設是用ui刪除的話 那完全就妳問題了132F 05/09 22:45
推 paulshain04: 這怎麼會是自己新增 DB不是有自動流水號 column不用填啊134F 05/10 09:09
--