请教大家一个 ORM 代码抽离的问题

问答 / 0 / 7 / 创建于 4年前 / 更新于 4年前

原来在service层的这段代码,想把Test表的sql抽离到model层,怎么做比较好? (type后面的’.’是打错了)

我是这么写的,不知道对不对
service层

model层

课程读者 59 声望

看起来 @ 不太聪明的亚子

暂无个人描述~

《L05 电商实战》

从零开发一个电商项目，功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等

《G01 Go 实战入门》

从零开始带你一步步开发一个 Go 博客项目，让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。

推荐文章：

更多推荐...

用 Laravel12 Startkit 做了一个 composer 私有包托管平台 😂 点赞超过 20 个开源，看看需要的人多不多 28 / 19 |

如何打造令后端面试官印象深刻的简历？ 14 / 10 |

花了四个月打磨的 Laravel Plus 开源 30 / 98 |

冯老师的困惑 —— 一个跑了两年的 BUG 19 / 16 |

在laravel下实现全双工的websocket开发 21 / 8 |

无需修改任何代码和扩展将你的Laravel项目性能提高20倍 101 / 98 |

讨论数量: 7

WmKong

课程读者 20 声望 / RD @ Baidu

github.com/Tucker-Eric/EloquentFil... 可以试试这个包

4年前评论

fangmuke

课程读者 109 声望

先改造一下代码，把if换成when()吧还有这个是什么玩意？

file

4年前评论

houmuxu （楼主）

额,随便写的例子,逗号打错了,其他框架没when方法

ma1232006

课程读者 4 声望

model层不能写任何查询代码；即使要写也只能新加一层里面写。 request不能在service里面出现，request是请求层的代码；不能侵入。

4年前评论

houmuxu （楼主）

那个只是变量名叫$request,传过来的时候已经是数组了...

小彭友

课程读者 35 声望

file User $user可以依赖注入，参数最好加个isset判断

4年前评论

王成涛

课程读者 165 声望 / coder @ 蚂蚁嗨客

很明显属于过度封装，违法面向对象特性 -- 封装，model 掺杂业务逻辑，方法复用性不高，随着业务复杂性增加，类似这样的方法会增加

4年前评论

王成涛（作者）

@houmuxu 看到很多人都在用 service 层，但是我觉得没必要，多余的

s51983

@王成涛有些Model光是关联就很多了，贫血模型+Service层还是有必要的。

xiaopi

@王成涛前俩天看了Laravel 框架作者 Taylor Otwell写得《From Apprentice To Artisan - Advanced Application Architecture With Laravel 4》，里面有提到目录结构，可以了解一下。结合实际，觉得还是有必要service层的，职责单一。如果想要代码可复用性高，还是得想办法抽离 xueyuanjun.com/post/9711

乘风破浪

代码复用维护，使用 Service 层封装逻辑很有必要，再加一个 Repository 层封装数据库操作，真的很有必要，不然后期只能重构了

aen233

课程读者 399 声望 / php @ abc

小白弱弱的说，感觉完全没必要那么封装。。。。。model里别放具体业务查询了吧（除非特别公用的.....
感觉不仅没抽离，还越来越混乱了呢......

4年前评论

houmuxu （楼主）

嗯,确实,一些简单高可用的可以放在model里,这种复杂判断的还是不放了

lyxxxh

1.3k 声望

file
(个人讨厌的写法同事也跟我抱怨过为什么写这么多余代码(封装))

抽离的话,可以考虑下apiato (单人开发最好别碰)。

route -> controller -> action -> task -> repository -> model | criterias -> transformers
……

还是 mvc 香，简单。

(单人开发,随便怎么封装都可以)
封装可以参考下:基于教程实现的一个简单网站实例

4年前评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容，与人为善，比聪明更重要！

帮助

59 声望

看起来 @ 不太聪明的亚子

纠错改进

成为赞助商