[請益] 這是個很低級的錯誤嗎？

[a88241050 (再回頭已是百殘身)]

是這樣的

最近工作上需要寫一支程式寫資料進資料庫

並產生流水號當key值寫進欄位中

我的想法很單純

就是直接抓資料庫資料筆數

然後拿筆數+1當新增的流水號

寫完後交給SA測試

結果他一測馬上就出問題了

後來我才知道 原來流水號是要抓最大值+1

而不是一直照順序編下去

因為這個我被SA噹的體無完膚

請問這個是很基本的觀念嗎？



--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 49.214.145.27
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1557232923.A.965.html
推 g2581856: 我覺得是啦
→ g2581856: 不過以後別再犯就OK了吧，用不著噹
→ diabloevagto: 你自己測試測不出來嗎？？？
我沒想到他會手動刪除資料啊
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 20:50:06
推 abccbaandy: SA需求不明確還敢噹人喔...
→ abccbaandy: 產生流水號規則有說/寫清楚嗎？
→ meowyih: ... 呃, 這也太新手了
→ abccbaandy: 另外第二種作法有考慮multi thread嗎？
→ meowyih: 還有這不是auto incremental自動會幫你寫嗎,為啥要自己手
→ meowyih: 動寫？
推 wayne12345: 是
推 sourbait: 資料庫一般來說要考慮到刪除的可能
→ weinine32: 有考慮lock問題嗎？ 流水號會重複喔！
→ MOONY135: 要看需求吧
推 alihue: 我就算第一次學也不會犯這種錯
推 yyc1217: 是 不過刪除也有分直接刪除或是註記刪除
→ yyc1217: 如果能用資料庫本身的流水號更好 除非流水號有要求格式
→ yyc1217: 一般來說要刪除也要保留資料比較好 畢竟記憶體不貴
→ yyc1217: 查到了 叫soft delete
→ alan3100: 流水號由DB產生是基本, 除非你有其他特殊需求
→ supernow: 是的，這很基本，不抓最大的就會有重複的問題
推 t64141: 滿基本的,不過多了一個機會去研究流水號的問題也是好事
請問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一筆
是一次一筆沒錯，不會有多筆的情況
→ meowyih: google 'sql auto_increment'
推 benjamin99: 比較好奇 SA 為啥會有 max+1 就正確的概念？
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 21:26:17
→ alan3100: https://www.w3schools.com/sql/sql_autoincrement.asp
→ Chris926926: 好奇不用auto increment的原因是?有特殊需求?
推 googoo1102: 保險起見把流水號欄位設成unique
→ godddddd: 看你資歷 沒事請用sql內建流水號
推 x000032001: 同時插兩筆抓max也是爆炸
推 q26766: 一樣錯啊哈哈 ，別理有些酸推文，誰沒新手過
→ mackliu: 你們的流水號會不會是設計成文字格式,而且還沒設主鍵?XD
→ GoodFriday: 抓最大值+1不就超容易抓到重複 只有第一筆塞得進去
推 ripple0129: 你新手就算了，SA也新手，拜託找個資深的來帶一下吧
→ LinuxKernel: 慘
→ jyunwei: 那你以後就會想到了，如果你資歷一年內的話啦
推 crossdunk: 很基本阿，可是為什麼是SA測試呀
→ ChungLi5566: 非本科? 實務上流水號都給db自己編啊
推 crossdunk: 他的流水號應該是前面還需要加其他東西
推 abraxas: 他那做法，直接新增一筆溢位筆數減一的資料不就炸掉了
推 Label: 他還沒測transation呢
推 kurtsgm: 4
→ ChungLi5566: DB2 https://i.imgur.com/fJG1QA6.jpg
推 bill0205: 通常db 的 key都是auto increment吧=  =
推 abccbaandy: 好奇到底什麼使用情境可以想出這種神奇作法...
→ bill0205: 我自己是會做兩種序號 一種是db 一種是顯示的SN
可是我的序號前面還要加其他字串欸，等於說我在寫入資料庫前就要取得db產生的key值
組成我要的流水號後再寫入db,這有辦法嗎？
※ 編輯: a88241050 (49.214.145.27), 05/07/2019 22:33:10
推 rahit: 這東西一般資料庫都能自動給…
推 cloudgoogle: 單純抓max一樣也是有問題 多人同時操作的執行順序...
推 drajan: 蠻嚴重的錯誤 不過如果入行不到半年可以容忍
→ ChungLi5566: 要加字串還是補左邊零的寫在程式就好 幹嘛放在table
→ ChungLi5566: 佔DB磁碟空間
→ bill0205: 有序的就照C大方法 或是把字串丟到另外欄位 撈出來時後
→ bill0205: 再組起來 無序比較麻煩
推 ashlikewing: 流水號自己做太扯了啦
推 cloudgoogle: 如果可以撈出來另外處理當然是最好 不過有的Table
→ cloudgoogle: 是要給別人存取的，可能就沒辦法控制在自己這邊
→ cloudgoogle: 即便如此，還是可以像bill大說的存兩欄可能好一點
推 onlyeric23: 菜到不行
→ you878787: 如果我看到這種code應該會把這個人幹到火星去吧= =
→ ChungLi5566: 通常別系統來存取 是走API而不會直連DB
→ bill0205: 題外話 大家是怎麼做無序的流水序號
推 you878787: 開始寫前請先訂好test case, 另外有基礎知識應該根本
→ you878787: 不能這樣抓....
→ lukelove: 比較大的問題是 開發前沒有先查怎麼設計就土砲做
推 jinmin88: 菜到不行的問題
推 blackie1019: 兩個都會笑到翻過去
推 crossdunk: 無序怎麼叫做流水XD
→ Darkword1987: 資料筆數減少不就GG了
推 CloudyWing: 總筆數+1在跳號的情況下有可能序號重複，有經驗的設計
→ CloudyWing: 應該就要注意這件事...
→ CloudyWing: 今天就算是Soft delete，也有可能因為一些意外因素導
推 jlhc: 還是不懂為什麼不是auto increment
→ CloudyWing: 致資料跳號，不過我好奇設計上有介面可以輸入序號
→ CloudyWing: 還是SA直接加資料庫？
推 bibo9901: uuid..
推 molopo: 流水號一直上去就好
→ gpctv: 這個也要刁，改就好了啊，在我們公司SA地位像狗一樣
→ sachung28: serial primary key就會自己產生序號+避免重複了
→ viper9709: 流水號不用也不能自己做吧...
→ sachung28: 可以在SQL寫入DB時順便用return回傳+改格式 前提是DB
→ sachung28: 有支援...另外 刪除並重塞資料測序號產生邏輯ok 但我
→ sachung28: 第一次看到QA自己塞數字當流水號
→ sachung28: 如果是UI呈現美觀 流水號要補字串 這可以API hard code
→ sachung28: 處理 或是另外開個流水號前綴字的定義表儲存 查詢時將
→ sachung28: 前綴和序號串起來
→ sachung28: 建議你去惡補一下DB觀念 學會用DB能省很多寫程式的時
→ sachung28: 間
推 bitcch: https://stackoverflow.com/q/5940931
推 mathrew: 4  這種問題就代表  你根本寫的時候  很多狀況都沒想過
推 mathrew: 但是為什麼流水號不給DB自動去編
推 brianhsu: 流水號不要自己生，問題很多的。交給 DB，例如 auto Inc
→ brianhsu:  之類的功能處理。
噓 deray: 流水號幹嘛最大值+1 auto increment
→ qpowjohn: 抓最大值+1想了一下應該有可能，可是要開transaction保
→ qpowjohn: 證不會有人寫表...有錯請指教
→ xdraculax: 總數加一很瞎，最大加一也只是50步笑百步
→ y3k: 這就是菜而已吧
→ luluking: 你不覺得這樣寫反而麻煩
→ lion0208: 為何不用 auto increment？另外取最大+1，同時間確定只
→ lion0208: 會有一個 process/thread 在寫？
推 mago: 如果有大量同時寫入問題,兩個都不行
→ CodingMan: 換個想法你就知道答案了 難不成這是高級問題嗎?
→ lazarus1121: auto increment如果需要跨table當key不就不能用了
推 silent5566: 直接用DB的sequence去滾不就好了
→ silent5566: 你這兩種作法未來table在應用上會很差
→ silent5566: 多人使用的時候沒咬住key值很容易有重複的問題
→ hakama99: 我一開始寫也不知道有自動產生的功能  也是跟你一樣XD
→ testPtt: 設預設值不要用sql寫就好啦
→ johnny9144: 4
推 f124: 叫資料庫自己跑流水號就好 還自己編幹嘛...
推 iamshiao: 是，而且拿 max 也可能會有問題，用資料庫本身提供的自
→ iamshiao: 動增加機制比較好
推 seedli: 除非系統只有一個人用，不然抓max+1可能有race condition
→ worf: ...
推 amyt: 是
推 laker780304: 不嫌麻煩可以建立Sequence，新增時取Sequence用即可
推 zg0608x: 好有創意的方法
推 indexcome: 其實就算沒講清楚需求。這種可能發生的情況寫code的人
→ indexcome: 應該都會注意到的.
推 BignoZe: 照常理不太會這樣寫
→ rocwild: 基本
→ GinginDenSha: 兩個都4
推 xo1100: 抓最大+1的話 你之後碰用到多人使用的系統還會再被噹一次
推 stupid0319: 是SA太淺了
推 hooll111: 這個還蠻基本的耶 取筆數超抖...
→ deanh: 你們兩個都錯了啊，搞笑
→ gettheworld: 覺得是想展現優越感
推 zerrofighter: 這種事情就是炸過一次就知道了
推 mdkn35: 我都用uuid
推 DerLuna: 這兩個方法都不好吧
推 cholux: 是有一點，不過看問題可能會有更好的作法
推 panpan: 笑死 看到兩個菜雞戶啄
推 Frecci: 妳太菜是真的 但是不代表他能夠直接進入db刪資料
→ Frecci: 假設是用ui刪除的話 那完全就妳問題了
推 paulshain04: 這怎麼會是自己新增 DB不是有自動流水號 column不用
→ paulshain04: 填啊
噓 owen5566: == 這樣寫怎麼可能沒問題
噓 rickboo0809: 超低級
推 Tony427: 不管基本不基本，犯錯要搞懂為何不能這樣做，以及找出最
→ Tony427: 佳做法，至少下次不要再犯
→ newhandfun: 想必閣下跟我一樣是沒人帶的菜鳥吧，能夠發現問題提早
→ newhandfun: 解決真是件好事。
推 silent5566: 大家也不用嘲諷吧 每個人也是爆炸中成長QQ
→ silent5566: 推Tony的心態