作者a88241050 (再回头已是百残身)
看板Soft_Job
标题[请益] MVC新手请教..
时间Fri May 31 16:02:02 2019
最近工作上刚写了一支程式
当整个程式流程跑完
我总共需要insert和update7个table
然後我每次对table做操作都会透过ajax传
json物件呼叫controller的方法
每个controller再调用service的insert或update
所以我一个流程就会发送7次ajax需求呼叫controller
现在程式写完遇到两个问题
一个是程式执行时间有点久..
一个是若程式出错没办法把资料还原
後来有人告诉我我应该把全部流程包在一个controller的方法里
请问这样就能解决了吗？
还是应该写在service里?
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 101.13.209.157
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1559289725.A.A2C.html
1F：推 cz75: 小弟也是菜鸡，提供一下我的想法，第一点互相没有影响的tabl05/31 16:19
2F：→ cz75: e也许可以用非同步提高效率？05/31 16:19
3F：→ cz75: 第二点不晓得用transaction scope包起来有没有用05/31 16:19
4F：推 DCTmaybe: 你流程的每一步都会影响到下一步吗？可以讲详细一点05/31 16:46
现在问题是我那七次操作都是分开的，但是我现在需要的是只要一个出错那前面的操作都
要还原所以似乎不能这样写,只是我不太清楚我是要把全部逻辑写在controller的一个方
法里还是写在service里，而且似乎就是因为送了太多ajax请求导致程式跑很慢
5F：→ lwtech: 呼叫七个是将逻辑切乾净，你现在要呼叫七个就看你要在前端05/31 16:47
6F：→ lwtech: 封装还是後端都可以，另外服务和controller应该是不一样的05/31 16:47
7F：→ lwtech: 东西，很类似但服务是controller的封装层05/31 16:47
8F：→ lwtech: db慢你要自己会用工具看效能的部分，通常设定可以解决05/31 16:52
※ 编辑: a88241050 (101.13.209.157), 05/31/2019 17:18:30
9F：推 vi000246: google sql transaction05/31 17:20
10F：→ lwtech: ajax 是 异步...05/31 17:22
11F：推 asdasdasd80: 前面都先暂存，等最後步骤再一起写进DB行吗05/31 18:03
12F：→ chocopie: DB不要太频繁写入，不然日後还要再做优化，多一次工05/31 18:06
13F：推 pilor: db 写交易05/31 18:07
14F：推 DCTmaybe: 听起来是要包transaction,但七次操作是分开的，可以先在05/31 18:08
15F：→ DCTmaybe: 前端暂存最後才打ajax吗？05/31 18:09
16F：→ DCTmaybe: 或是你要另外开一张暂存的db来储存这些操作的结果，最後05/31 18:10
17F：→ DCTmaybe: 才一次写入到要存的table05/31 18:11
18F：推 puring0815: 有点模糊，每个步骤会依上个步骤结果有不同商业逻辑 05/31 18:48
19F：→ puring0815: 吗？ 05/31 18:48
不会，只有写入的值有差
20F：推 puring0815: 出错就要还原，那出错的定义是什麽？有可能只存四个 05/31 18:49
21F：→ puring0815: 资料表但是是代表成功？还是说要嘛没资料要嘛七个表 05/31 18:49
22F：→ puring0815: 都有资料？ 05/31 18:49
七次操作只要有一次出exception那之前做的都要还原
23F：推 pathy: 如果7次的操作是有相关性的，那用一次ajax比较合理 05/31 18:50
24F：→ pathy: 另外如果第三步执行失败，前面已经执行过的两步改动要复原 05/31 18:51
25F：→ pathy: 的话，整个7次操作要用transaction包起来，自己查下transac 05/31 18:51
26F：→ pathy: tion的用法吧 05/31 18:51
好，我会研究一下，感恩~
※ 编辑: a88241050 (101.13.209.157), 05/31/2019 19:35:36
27F：→ lazarus1121: 应该1次发送，後端1次跑7个模组吧 05/31 19:48
28F：推 eateat33: 整包包起来 05/31 19:56
29F：推 puring0815: 也同意可以先试试用包同一包ajax并用transaction处理 05/31 20:10
30F：推 forewero: 1次Request,7次操作比较合理,然後AP or db做transaction 05/31 20:18
31F：→ lwtech: ajax 有 async,await, 七个你错哪一个你要去db 看... 05/31 20:21
32F：→ lwtech: async 里的 sync,没 es6+ 就用 polyfill when,promise. 05/31 20:24
33F：推 clamperni: M_V_C 05/31 20:33
所以我原本的写法每次操作资料库就送一次ajax这不是普遍的做法?
本来就应该要把所有流程包在一个service里面?
※ 编辑: a88241050 (101.13.209.157), 05/31/2019 20:45:24
34F：推 SHANGOYANYI: 从你的描述看起来 这7张表的操作是属於「一个交易」 05/31 21:05
35F：→ SHANGOYANYI: 所以发一个ajax去後端做整包的处理会比较好做 （失05/31 21:05
36F：→ SHANGOYANYI: 败处理透过orm的transaction机制rollback即可） 05/31 21:05
37F：推 ashlikewing: 这和MVC没什麽关系，你没有搞清楚程式流程的主要目 05/31 21:15
38F：→ ashlikewing: 的 05/31 21:15
39F：→ vi000246: 主要是你的流程要做什麽 不是这做法普不普遍05/31 22:29
40F：→ vi000246: 像有些网站的setting不需要存档 只要更新就自动存档 05/31 22:30
41F：→ vi000246: 就会有改一个栏位就call一次api的做法 05/31 22:31
42F：→ vi000246: 但你又要还原rollback 就像一次更新整个form的七个栏位 05/31 22:31
43F：→ vi000246: call七次 一次失败就不存档 那你可以直接改成一次送出 05/31 22:32
44F：→ vi000246: 整个form 不用分七次request 就逻辑设计的问题05/31 22:32
所以也是因为我送request的次数太多导致程式执行时间过长吗？
※ 编辑: a88241050 (101.13.209.157), 05/31/2019 22:47:41
45F：推 googoo1102: 总之送7次肯定是不好的写法 05/31 22:53
46F：→ lwtech: 不然这样好了，我们九点开盘，九点零一收盘，大家节省时间 05/31 23:13
47F：推 vi000246: 执行时间你可以开浏览器的F12看 如果是等response的时间 06/01 00:37
48F：→ vi000246: 太长 可能是後端有问题 06/01 00:38
49F：推 sxy67230: 理论上，不要去让View处理太复杂的逻辑，正确的作法都是 06/01 01:50
50F：→ sxy67230: 把商业逻辑封装在service里面，controller不做逻辑，只 06/01 01:50
51F：→ sxy67230: 负责流程控制。最後，善用sql语法、必要的时候做index加06/01 01:50
52F：→ sxy67230: 速query、大量资料记得用sql做分页，否则你会写出很肥06/01 01:50
53F：→ sxy67230: 的逻辑，人家一行就搞定了。06/01 01:50
54F：推 iansrc: 应该是一个方法包起来吧，在 API 里呼叫方法，AJAX 只要叫 06/01 10:49
55F：→ iansrc: 一次 API 就好。用 transaction 包那些 SQL （或是ORM) 语 06/01 10:49
56F：→ iansrc: 法，出错就会还原了 06/01 10:49
57F：→ johnny4753: 如果是用entityframework的话，只需呼叫一次SaveChang 06/01 12:44
58F：→ johnny4753: e()就帮你搞定rollback问题了 06/01 12:44
感谢各位，我改出来了..刚Debug才发现，sevice里面的操作做完要跳回controller才会
真正去更新资料库，虽然说我不知道为什麽..反正把全部逻辑写在service就能解决rollb
ack的问题了@@
※ 编辑: a88241050 (101.13.209.157), 06/01/2019 15:24:11
59F：→ lwtech: 学点工具做後端流程监控吧. 06/01 15:44
61F：→ reichs: 写一个stored procedure处理这7张table. 06/01 18:56
62F：→ DendiQ: 我猜你是不是把所有 db sql 操作都写在 client...？ 06/01 21:39
63F：推 ian90911: asp.net MVC 要特别写出来 06/02 13:03
64F：→ beau03: 各阶段用cookie将值加密暂存，最後再将cookie送後端或aja 06/02 16:05
65F：→ beau03: x 06/02 16:05
66F：→ beau03: 使用者如prev上一步，直接由cookie复原画面栏位值 06/02 16:06
67F：推 blackie1019: 看一下Saga pattern 可以救你很多 06/04 00:22
68F：→ eggeggss: 前6次写入暂存档,第7次包transaction写入正式档 08/17 16:01