Soft_Job 板


LINE

如題 目前做的project架構長這樣 Loader1 Loader2 Loader3 ........... Loader30 Area 1 Area 2 Area 3 .... .... Area 10 就是同一個Loader 會算出結果給不同Area的人 有分成前端跟後端,前後端都是我維護 (前端 .Net Framework java script MVC,會 用到一點 jquery,其他大多套套件用現成的物件再加工,後端就純C# query DB用的工具是公版的不用自己寫,就負責input 要打的SQL跟DB,然 後output datatable或是對DB做insert/delete/update....) Loader每支老闆認知都不算很大所以也是一人包辦XD Loader每支的邏輯也類似: Start--->會記錄log表示loader開始 Initial setting----> query一些接下來SQL會用到的config QueryRawData---->把上一個initial function的config帶下來在這裡組成SQL string 並且query進data table ProcessData--->對上一步的data 做運算然後轉換成可能只有一個kpi值的結果 或是pre-sum過後的結果 (舉例來說query hourly的資料轉換成daily的資料) End---->只是記錄log表示上面的function都執行完了 看似很穩定 但loader卻需要不斷的更版 原因是因為SQL全部都是寫死的,接水其他系統的資料 https://imgur.com/a/xjTfPfG 以這個圖為例,USER只看的到最終的TABLE,沒有太多空間可以彈性修改query的條件 也沒有像左上角有彈性調整query資料的方法 (有config但是很少) 所以需求常常長得像這樣: "tinasfishs 啊你幫我改一下,我發現今天的KPI有錯,因為 今天開始要加採計'AA' 這種類別,麻煩再幫我改" 原始SQL通常長這樣 SELECT COUNT(1) CNT FROM STORE_SALES WHERE 1=1 SELECT * FROM STOR_SALES WHERE 1=1 ( AND TYPE IN ('A','AB','BC',....'DZ','AA') AND SALE_DATE BETWEEN SYSDATE-3/24 AND SYSDATE-1/24 AND MT_CODE IN ('2','5') AND...... CASE WHEN .... )A, ( SELECT EMPLOYEE_DEG,EMPLOYEE_NAME FROM EMPLOYE_L WHERE AREA='AREA1' AND RES_PRODUCT NOT LIKE '%AAZc' .... .... ... )B, ( SELECT EMPLOYE_ID,EMPLOYEE_SHIFT FROM SHIFT_ARRANGE_LIST WHERE RECTIME>SYSDATE-5 AND AREA='AREA1' if (AREA=='AREA1' or AREA=='AREA2') { RECTIME<SYSDATE-1 } )C WHERE A.SALE_EMPLOYEE=B.EMPLOYEE_NAME AND B.EMPLOYEE_ID=C. EMPLOYEE_ID SQL通常都很長而且用sub-query,會把商業邏輯埋在hard-code的SQL裡面 不能用SQL處理的邏輯則是在ProcessData那步處理 假設綠色的地方是config,黃色的地方則是這次修改的部分 重點是改後端的SQL都需要重新上code,雖然有提供公版的工具把SQL另外儲存 只要呼叫那個外部工具就可以把SQL取得 但前人為了開發方便 當然全部都把SQL直接擺在程式裡 然後複雜的是,同樣算這個KPI,各個AREA 的條件是不盡相同的 用藍色字的地方就是舉例,沒有使用config而是用C# 的code 直接hard code 這兩個 area 需要加此條件 所以也會衍生像是 "我們是AREA3,AREA1 跟 AREA2 看的那個條件我們從下禮拜開始也想跟著採計 ,你們可以重新上code嗎" 有爬文像是Stored procedure 之類的做法,歹勢對於這個架構完全不適用 理論上像這樣各個area的條件有不少差異的情況下,應該是個別維護各area的loader 但這整套是寫在一起的...,每次上code為了維護版控也會在各area同步上同一套 也就會發生改A壞B,假設area1跟area2 共用同一段SQL條件,答應area1刪掉 其中一句之後,area2 就跑來說他們的KPI異常 這個專案負責接水30幾支上游的TABLE,就有30幾支LOADER..而且還在逐漸增加中 所以談需求跟開發的時候,根本也不是從 "為什麼要新增AA這個條件,除了改SQL有什麼更好的方式" 去切入 因為那要去詢問上游以及user為什麼要新增這個條件太花時間 而是 "幫我改SQL吧比較快"---> 所以從來都只在做改SQL這種很末端的工作 一個禮拜大概有4-5個這樣的"假需求"---->我定義為只是改SQL,請user驗驗看是否 符合他們期望 user有時候也不確定改了條件是否就會對,改了也不能上code 總是改了再測測了再改 這樣的需求都做完了,才有時間去處理可能真的是loader本身的問題 ----->效能的問題啊、要新增一支新loader 與user談規格之類的....orz 談規格也是我一人,頂多再加0.5個PM PM也跟我想像會把規格談到不會讓日後充滿這些維護 剛好相反 最後規格談一談還是常常訴諸短解 能夠hard code SQL就hard code,理想性的應該要跟上游負責系統的人看資料 怎麼接軌的溝通過程完全沒有= = 我有堅持過想先看清資料流再做新需求也因為求時效性被打槍 有被訓練得比較有警覺性,但一個SQL 至少20行,下面又有條件讓user高興改就改 實在很難用很大的框架去處理這樣的事,SQL有記得拔出來放在另外存的地方 不用重新上code就很不錯了 BTW SQL邏輯跟要接水什麼報表通常都是由USER提供 ----------- 有跟老闆溝通過像這樣接水別人資料的匯總型AP 應該要重構,或者是一支一支 拿出來看有什麼可以enhance的 但user常常會認知KPI錯誤就來說是系統問題,久而久之老闆也好像覺得是我的問題 而沒有深究之所以KPI會錯誤,就是因為SQL在上code流程的不彈性,還有讓user在 不能彈性重上CODE的情況下還是一再想修改SQL (不能彈性的原因是要重上code老闆都會看,跟他解釋這個是要改SQL就會被 question,但老闆也不看CODE也不會負責建議有建設性的方法,只會打高空問 說可不可以不要改SQL--->問我的格局只能做到這樣...整個黑人問號) 私以為想修改SQL或是想改變KPI的算法,都應該是大家坐下來看data flow是什麼 是不是只是加了一句SQL就可以改變 但文化上大家也不太有耐心這樣梳理問題 不太清楚老闆是否了解目前系統的設計是長這樣,總之在溝通上也覺得不太有救 想問問有沒有維護或是開發過這種DATA LOADER的前輩 可以分享一下心得的QQ --



※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 114.25.93.94 (臺灣)
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1669050189.A.15C.html
1F:推 neo5277: 我看到1=1了耶... 11/22 01:29
2F:推 neo5277: 也不要用* 11/22 01:30
3F:→ peter98: 把條件弄成dynamic 不要hardcoded 設定好權限 讓個別 11/22 01:32
4F:→ peter98: user可以選要加甚麼條件在filter裡 我做過這個 很難寫 11/22 01:33
5F:→ peter98: 難在一堆無聊人士搞audit 設定權限他們也不懂 11/22 01:33
6F:推 neo5277: 辛苦啦,我是建議你讓sql參數化單純一點,把條件的東西搬去 11/22 01:34
7F:→ neo5277: c#用策略模式做 11/22 01:34
8F:→ neo5277: 然後多使用虛擬類別,多的kpi用介面去加 11/22 01:35
9F:→ neo5277: 這樣你sql就相對單純 11/22 01:35
10F:→ neo5277: 現在這樣筆數多撈起來會很慢吧... 11/22 01:36
11F:推 neo5277: 然後你sql要板控的話可以加一個sql,專案在方案裡 11/22 01:49
12F:→ neo5277: 最終還是sp化接參數再透過策略選擇呼叫那一種就好 11/22 01:49
13F:→ neo5277: 好維護,改起來也快,而且會好閱讀很多吧 11/22 01:50
14F:→ airtsubasa: 1=1是為了 後面串條件方便吧 應該是後面可能不會出現 11/22 06:52
15F:→ airtsubasa: 條件 11/22 06:52
16F:→ play714: 不太懂分那麼多支loader 的原因為何? 11/22 08:39
17F:推 CRPKT: 先把上游資料轉成同樣格式,query 的部分再跟著 area 版控 11/22 09:22
18F:→ CRPKT: 現在這樣維護成本就是 loader 與 area 數量乘開 11/22 09:23
19F:→ CRPKT: 把東西拆到讓維護成本變成加法是唯一的活路 11/22 09:24
20F:推 enthos: https://github.com/pardeike/Harmony/wiki 11/22 09:32
21F:→ enthos: Game Live patch:做多種MOD讓使用者自行訂閱 11/22 09:32
22F:推 giacch: Loader1.GetSQLText(Area1), Loader2.GetSQLText(Area1) 11/22 11:41
23F:推 wulouise: query可以獨立分開放在檔案裡讀吧? 11/22 12:08
24F:噓 sniper2824: JavaScript 為什麼你要分開寫 11/22 12:35
25F:→ brucetu: 你做好一版 馬上把所有可能被影響的loader/area重跑 比 11/22 17:12
26F:→ brucetu: 對實際被影響的資料是否符合你預期 先建立這樣的系統避免 11/22 17:12
27F:→ brucetu: 改A壞B沒發現 再來談重構 11/22 17:12
28F:→ brucetu: 會發生改a壞b表示你有共用的code卻允許共用的其中一個需 11/22 17:14
29F:→ brucetu: 求端任意更改 這當然不正常, 如果一種東西會被多個需求 11/22 17:14
30F:→ brucetu: 端各自提需求修改 那這種東西不論有多相似 都應該建立單 11/22 17:14
31F:→ brucetu: 獨的class 11/22 17:14
32F:→ brucetu: 你的整個重構會很複雜不是你一人一兩個月就能完成 一定要 11/22 17:15
33F:→ brucetu: 先建立確保不會改A壞B的機制 11/22 17:15
34F:→ brucetu: 你這正常的架構就是要先定義原始資料類別有哪些, 聚合資 11/22 17:16
35F:→ brucetu: 料的filter有哪些 11/22 17:16
36F:→ brucetu: 你的loader程式應該要給每個最終報表有適當的命名 同時 11/22 17:19
37F:→ brucetu: 有文件紀錄每個報表會有哪些單位查看 11/22 17:19
38F:→ brucetu: 每一支報表由哪些資料源以及哪些filter算出中間的結果以 11/22 17:25
39F:→ brucetu: 及最終報表 要有文件 11/22 17:25
40F:→ brucetu: 當有人跟你說 他要增加採計AA類別 你就去查程式 採計AA 11/22 17:30
41F:→ brucetu: 類別這個行為是哪一支filter在處理, 有其他單位用到嗎? 11/22 17:30
42F:→ brucetu: 如果沒有 那大概就是直接修改這支filter 最終報表就會是 11/22 17:30
43F:→ brucetu: 他要的結果 也不影響別的單位. 如果有其他單位用到 你可 11/22 17:30
44F:→ brucetu: 能需要拆分成兩支filter給兩個單位使用 或是你的這個filt 11/22 17:30
45F:→ brucetu: er可以吃參數 那就是兩個area吃不同config即可. 最重要是 11/22 17:30
46F:→ brucetu: 不管你的架構如何 改完就是要測試確保沒有動到不該動的 11/22 17:30
47F:→ brucetu: 東西 這就是測試的覆蓋率的重要性 11/22 17:30
48F:→ brucetu: 最後建議 直接換工作 11/22 17:31
49F:推 jellyfishuan: 不太懂 老闆聽你這樣解釋之後還是認為是你的問題嗎 11/22 17:58
50F:→ jellyfishuan: 通常這樣看你敘述下來就知道是架構問題了吧? 11/22 17:58
51F:推 f821027: SQL WHERE 用 python 的 f-string 把 list 傳進去,這樣 11/22 18:43
52F:→ f821027: 只要更新這個list就好 11/22 18:43
53F:推 bills1987: 像是水流一樣,資料一站一站往下流,就很好追蹤!可能會 11/22 21:02
54F:→ bills1987: 變動的地方可以另開TABLE存放參考物件AA ,BB...後續就 11/22 21:02
55F:→ bills1987: 算新增參考,也只需更動參考TABLE內容,MAIN SQL部分只 11/22 21:02
56F:→ bills1987: 要寫好關聯,後續就不用一直改 11/22 21:02
57F:→ tinasfishs: 已經有參數化,loader唯一有設計的就是把同一支loade 11/22 23:58
58F:→ tinasfishs: r的可能10幾、20幾支SQL擺在同一個class,然後用retu 11/22 23:58
59F:→ tinasfishs: rn string的方式去return SQL,只是除了用class的殼 11/22 23:58
60F:→ tinasfishs: 包住&部分用參數之外就沒做其他的了。參數有的帶了6- 11/22 23:58
61F:→ tinasfishs: 7個結果還是沒辦法把SQL拆到自由度完全釋放 11/22 23:58
62F:→ tinasfishs: 我的確很想by Area去把SQL list拆出來,但是可能和其 11/23 00:06
63F:→ tinasfishs: 他公司不同,SQL除了擺在公司共用的SQL池以外就是只 11/23 00:06
64F:→ tinasfishs: 有hard code一法,SQL池只是儲存string跟參數位置, 11/23 00:06
65F:→ tinasfishs: 假設要把本來是hard code的條件改成帶參數,就是原本 11/23 00:06
66F:→ tinasfishs: C#裡面call SQL池的地方要改。例如string SQL=sql po 11/23 00:06
67F:→ tinasfishs: ol(type,gender)要改成SQL=sqlpool(type,gender,age) 11/23 00:06
68F:→ tinasfishs: 就要重新上code,不能自行維護一份txt檔就好 11/23 00:06
69F:→ tinasfishs: 感覺前人在這樣的infra環境下,選擇hard code或是擺S 11/23 00:10
70F:→ tinasfishs: QL pool需要的effort是差不多的,為了快速完成,也不 11/23 00:10
71F:→ tinasfishs: 會去選擇把TABLE裡每個欄位都拆出來寫成class…table 11/23 00:10
72F:→ tinasfishs: 跟table我覺得關聯性很差,常常出現10幾個column的ta 11/23 00:10
73F:→ tinasfishs: ble,我資料庫不強不知道這算不算是架構不好的象徵QQ 11/23 00:10
74F:→ tinasfishs: 但大家的建議都很棒,我會多尋找機會點的!謝謝 11/23 00:12
75F:推 giacch: sql = "" + 11/23 00:55
76F:→ giacch: (age != null ? "" + 11/23 00:55
77F:→ giacch: "\n and age = @age" + 11/23 00:55
78F:→ giacch: "" : "") + 11/23 00:55
79F:→ giacch: ""; 11/23 00:56
80F:推 giacch: 藍色的if可以寫成上面哪樣, 字串一直串 11/23 00:59
81F:→ giacch: 不想整個Code重上, 可以拆成不同project (dll) 11/23 01:04
82F:→ giacch: 若你將所有C# Code都寫在一個cs檔,就可以實現維護一份text 11/23 01:14
83F:→ giacch: 參數太多可以考慮用一個物件來放 11/23 01:21
84F:→ giacch: (type, class or Dictionary<string, object> args) 11/23 01:21
85F:推 hanshsu: c# call sqlpool interface 改成傳 object 就可以解決改 11/23 11:16
86F:→ hanshsu: 來改去的問題? 11/23 11:16
87F:推 giacch: 樓上是講我嗎? "實現維護一份text"那一行只是玩笑話 11/23 11:28
88F:推 giacch: 不太懂樓上的意思 11/23 11:40
89F:→ lazarus1121: 另外花點時間寫一支自己理解後的模組? 11/23 12:10
90F:→ lazarus1121: 然後兩邊比較結果,久了沒問題就能把舊的取代 11/23 12:10
91F:→ qss05: 如果你會很常異動邏輯,通常會把核心做成view有需要再篩選 11/24 22:53
92F:→ qss05: 吧,這樣就從資料庫改就好,子查詢也可以做成function放在 11/24 22:53
93F:→ qss05: 資料庫 11/24 22:53
94F:推 ck237: 這個架構也太神奇了吧? 11/25 01:09
95F:推 jack0204: 為啥不從程式去處理要用SQL去處理? 11/26 09:11







like.gif 您可能會有興趣的文章
icon.png[問題/行為] 貓晚上進房間會不會有憋尿問題
icon.pngRe: [閒聊] 選了錯誤的女孩成為魔法少女 XDDDDDDDDDD
icon.png[正妹] 瑞典 一張
icon.png[心得] EMS高領長版毛衣.墨小樓MC1002
icon.png[分享] 丹龍隔熱紙GE55+33+22
icon.png[問題] 清洗洗衣機
icon.png[尋物] 窗台下的空間
icon.png[閒聊] 双極の女神1 木魔爵
icon.png[售車] 新竹 1997 march 1297cc 白色 四門
icon.png[討論] 能從照片感受到攝影者心情嗎
icon.png[狂賀] 賀賀賀賀 賀!島村卯月!總選舉NO.1
icon.png[難過] 羨慕白皮膚的女生
icon.png閱讀文章
icon.png[黑特]
icon.png[問題] SBK S1安裝於安全帽位置
icon.png[分享] 舊woo100絕版開箱!!
icon.pngRe: [無言] 關於小包衛生紙
icon.png[開箱] E5-2683V3 RX480Strix 快睿C1 簡單測試
icon.png[心得] 蒼の海賊龍 地獄 執行者16PT
icon.png[售車] 1999年Virage iO 1.8EXi
icon.png[心得] 挑戰33 LV10 獅子座pt solo
icon.png[閒聊] 手把手教你不被桶之新手主購教學
icon.png[分享] Civic Type R 量產版官方照無預警流出
icon.png[售車] Golf 4 2.0 銀色 自排
icon.png[出售] Graco提籃汽座(有底座)2000元誠可議
icon.png[問題] 請問補牙材質掉了還能再補嗎?(台中半年內
icon.png[問題] 44th 單曲 生寫竟然都給重複的啊啊!
icon.png[心得] 華南紅卡/icash 核卡
icon.png[問題] 拔牙矯正這樣正常嗎
icon.png[贈送] 老莫高業 初業 102年版
icon.png[情報] 三大行動支付 本季掀戰火
icon.png[寶寶] 博客來Amos水蠟筆5/1特價五折
icon.pngRe: [心得] 新鮮人一些面試分享
icon.png[心得] 蒼の海賊龍 地獄 麒麟25PT
icon.pngRe: [閒聊] (君の名は。雷慎入) 君名二創漫畫翻譯
icon.pngRe: [閒聊] OGN中場影片:失蹤人口局 (英文字幕)
icon.png[問題] 台灣大哥大4G訊號差
icon.png[出售] [全國]全新千尋侘草LED燈, 水草

請輸入看板名稱,例如:Tech_Job站內搜尋

TOP