Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

这个项目为什么能火? #173

Open
leilj opened this issue Sep 24, 2019 · 5 comments
Open

这个项目为什么能火? #173

leilj opened this issue Sep 24, 2019 · 5 comments

Comments

@leilj
Copy link

leilj commented Sep 24, 2019

1、所有service继承BaseServiceImpl,违反“迪米特法则”;
2、所有service实现BaseService,违反“接口隔离原则”;
3、所有service毫无没做任何业务实现,只是调用了一次dao,业务端不小心就把你整表删除了
。。。。

不明白这个项目为什么能火?????????????????

@ltscu
Copy link

ltscu commented Sep 24, 2019 via email

@moon137
Copy link

moon137 commented Sep 25, 2019

尽信书则不如无书。
BaseService和BaseServiceImpl是所有业务service的公共方法抽离。所有的crud都用到增删改查。
业务开发实用为王。
你提的法则原则,可以用到上层结构上,比如你去搞mybatis源码,那是对业务的高级抽象。

过两年你还会发现,啥数据库三范式啊,冗余字段多得很哦 @leilj

@leilj
Copy link
Author

leilj commented Sep 25, 2019

尽信书则不如无书。
BaseService和BaseServiceImpl是所有业务service的公共方法抽离。所有的crud都用到增删改查。
业务开发实用为王。
你提的法则原则,可以用到上层结构上,比如你去搞mybatis源码,那是对业务的高级抽象。

过两年你还会发现,啥数据库三范式啊,冗余字段多得很哦 @leilj

冗余字段完全没问题。
你也说了业务service,既然是业务,那就应该跟业务相关的,而不是简单的crud,毫无业务逻辑。什么情况下能创建?什么情况下能读取?什么情况写能更新?什么情况下能删除? 。。。。

有时候违反一些原则,并没问题多大,问题它是整个架构都严重违反基本开发原则,而且在这样的情况下还能火起来,是真不能理解。

哪天老大说,我们不用mybatis了,换成hibernate,看看会发生什么?!(虽然这种情况基本不会发生,但我们不能因为不会发生就否认存在的问题)。

@leilj
Copy link
Author

leilj commented Sep 25, 2019

靠的是宣传 best regard! 联系电话:17629196610 发自我的iPhone

😄我并没有什么需要宣传的。

@moon137
Copy link

moon137 commented Sep 25, 2019

停了这个issue吧,你无法就是想让别人同意你的观点。
我说的很客户观了,实用为王
如果你的老板让你换成hibernate,他得考虑时间成本。
做一个普通项目不仅仅考虑代码规范,相反代码规范在大多数场景下不值得考虑。
你的老板会告诉你我不管你用啥实现,用啥技术,我要个结果能交付。
做一个普通的web项目,这个框架不知道开发者,省了多少时间。

  • 甲花了3天写了一段牛逼代码并实现了业务功能很规范

  • 乙花了一天用一堆不合规范不合原则的代码实现了业务功能。

  • 在boss眼里,乙可能更有价值

  • 你要说维护问题了,即使乙辞职了还会招聘丙啊,丙不会可以学啊

  • 丙不高兴可以重构啊,但是要时间啊时间啊时间啊时间啊

  • 如果你要说实现淘宝双十一的秒杀架构,好吧,你雇一个牛逼牛逼的工程师来开发吧,乙和丙干不了

他可能有瑕疵,但并不代表他一无是处。

最后我顺着你的意思说一句,这个项目应该有什么灰色关系,不知道应该是给盖茨送礼了,然后盖茨主推这项目。

Repository owner locked as too heated and limited conversation to collaborators Sep 25, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants