Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

FenixPredicateBuilder 是否考虑支持Lamda表达式获取字段名 #42

Open
threefish opened this issue Apr 28, 2021 · 15 comments
Open

Comments

@threefish
Copy link

例如:

List<Blog> blogs = blogRepository.findAll(builder ->
                builder.andIn(Blog::getId, ids, ids != null && ids.length > 0)
                        .andLike(Blog::getTitle, params.get("title"), params.get("title") != null)
                        .andLike(Blog::getAuthor, params.get("author"))
                        .andBetween(Blog::getCreateTime, params.get("startTime"), params.get("endTime"))
                .build());
@blinkfox
Copy link
Owner

blinkfox commented May 7, 2021

@threefish 暂不支持哈,在考虑到底是否有必要添加。

@threefish
Copy link
Author

好的,我先用着,有机会的话我来个pr吧

@blinkfox
Copy link
Owner

blinkfox commented May 7, 2021

要加的话,很多重载方法都得加 Supplier 的函数式接口

@hexian
Copy link
Contributor

hexian commented Jul 3, 2021

好的,我先用着,有机会的话我来个pr吧

这个我看了下mybatis plus的代码,找时间我提个pr吧,会不会抢活了,哈哈哈。

@threefish
Copy link
Author

好的,我先用着,有机会的话我来个pr吧

这个我看了下mybatis plus的代码,找时间我提个pr吧,会不会抢活了,哈哈哈。

期待,哈哈

@hexian
Copy link
Contributor

hexian commented Jul 3, 2021

好的,我先用着,有机会的话我来个pr吧

这个我看了下mybatis plus的代码,找时间我提个pr吧,会不会抢活了,哈哈哈。

期待,哈哈

好的,明天我先在fork分支上写完,等前面一个功能合并了,我再提个pr。

@hexian
Copy link
Contributor

hexian commented Jul 4, 2021

好的,我先用着,有机会的话我来个pr吧

这个我看了下mybatis plus的代码,找时间我提个pr吧,会不会抢活了,哈哈哈。

期待,哈哈

我提交代码了,等待作者合并代码
8544bf6

@bearmr
Copy link

bearmr commented Jan 12, 2022

@threefish 暂不支持哈,在考虑到底是否有必要添加。

强烈建议加上

@203579397
Copy link

可以抄抄MybatisPlus。加上JPA的多对多功能,mybatis就没啥市场了

@bearmr
Copy link

bearmr commented Feb 24, 2022

主要是我是mybatis-plus 用的习惯了,但是这个公司是用jpa,我之前没搞过jpa,直接下重手仿照有点不太敢

@203579397
Copy link

我也是用MP用的多,但这几年用下来,尤其是微服务开发,还是觉着JPA的 @manytomany 很方便,然后就发现了Fenix。所以就希望Fenix 抄一下 MP

@pengten
Copy link
Contributor

pengten commented Feb 25, 2022

这个不是已经合并了吗?咋还在讨论,有新问题建立新的issue吧

@bearmr
Copy link

bearmr commented Feb 28, 2022

这个不是已经合并了吗?咋还在讨论,有新问题建立新的issue吧

没有说如何使用哈

@bearmr
Copy link

bearmr commented Feb 28, 2022

这个不是已经合并了吗?咋还在讨论,有新问题建立新的issue吧
这边2.6.1没有看到有相关代码合并发版

@blinkfox
Copy link
Owner

blinkfox commented Apr 1, 2022

我在新版本中也没看到,不知道是不是有次清理某些与之前版本不兼容的代码时给清掉了。

后续版本我再看看,或者大家有时间可以再提交下。代码、注释和单元测试写好点儿,否则我通常不会合并。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants