下面 Java 代码怎么用 Lambda 表示

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 2154 天前的主题，其中的信息可能已经有所发展或是发生改变。

if (null != shopView.getProvinceId()) {
            Region region = regionService.getById(shopView.getProvinceId());
            shopView.setProvinceDisp(region.getRegionName());
        }
        if (null != shopView.getCityId()) {
            Region region = regionService.getById(shopView.getCityId());
            shopView.setCityId(region.getRegionName();
        }
        if (null != shopView.getDistrictId()) {
            Region region = regionService.getById(shopView.getDistrictId());
            shopView.setDistrictDisp(region.getRegionName());
        }
        if (null != shopView.getStreetId()) {
            Region region = regionService.getById(shopView.getStreetId());
            shopView.setStreetDisp(region.getRegionName());
        }

32 条回复 • 2018-12-14 11:04:27 +08:00

chanchan

2018-12-13 14:57:21 +08:00

Optional.ofNullable(obj).ifPresent(item -> {

});

wleexi

2018-12-13 17:19:47 +08:00

@chanchan 每个属性都要写一个？

chanchan

2018-12-13 17:27:23 +08:00

这种还是 StringUtils.isNoneEmpty(,,,,)吧

xwbz2018

2018-12-13 17:28:58 +08:00

```java
String[] regionNames = Stream.of(shopView.getProvinceId(), shopView.getCityId(), shopView.getDistrictId(), shopView.getStreetId())
.filter(Objects::nonNull)
.map(regionService::getById)
.map(Region::getRegionName)
.toArray(String[]::new);
```
这样可行不，设置值到 shopView 那部分比较魔法

wleexi

2018-12-13 17:45:03 +08:00

@chanchan 类型是 Long。。。
我就是来问问有什么更好的办法做检查并设置值

wleexi

2018-12-13 17:46:20 +08:00

@xwbz2018 那查询完了之后设置值呢。。按照 index ？

xwbz2018

2018-12-13 17:50:27 +08:00

@wleexi 魔法一下可以的（这块可以写到 shopView 类里），总不能上反射吧

wleexi

2018-12-13 17:51:37 +08:00

@xwbz2018 恩我疑惑的地方就在这边。最后都是调用一样的方法想精简

chanchan

2018-12-13 17:52:01 +08:00

@wleexi 兄嘚，你就不能开动一下你的脑筋吗

wleexi

2018-12-13 17:58:21 +08:00

@chanchan 表达式这块实在不熟正在学习中谢谢提点

xwbz2018

2018-12-13 18:06:25 +08:00

@wleexi 这 getter，setter 是没法省了。。。regionService.getById 如果支持多参数的话，还可以优化一下

wleexi

2018-12-13 18:07:57 +08:00

@xwbz2018 这个倒是可以

youngxhui

2018-12-13 18:14:17 +08:00 via Android

这么多空判断不如换成 kotlin

corningsun

2018-12-13 18:15:57 +08:00

Optional.ofNullable(showView.getProvinceId()).map(RegionService::getById).map(Region::getRegionName).ifPresent(showView::setProvinceDisp);
Optional.ofNullable(showView.getCityId()).map(RegionService::getById).map(Region::getRegionName).ifPresent(showView::setCityDisp);
Optional.ofNullable(showView.getDistrictId()).map(RegionService::getById).map(Region::getRegionName).ifPresent(showView::setDistrictDisp);
Optional.ofNullable(showView.getStreetId()).map(RegionService::getById).map(Region::getRegionName).ifPresent(showView::setStreetDisp);

lhx2008

2018-12-13 18:19:42 +08:00 via Android

楼上的写法应该是楼主想要的了，不过说实话不如原来的

johnniang

2018-12-13 18:19:43 +08:00

可以考虑改造 getById()方法

```java
Optional<Entity> getById(Integer id) {

if(id == null || id <= 0) {

return Optional.empty();

}

...
}
```

结合一楼

```java
regionService.getById(shopView.getProvinceId()).ifPresent(region -> shopView.setProvinceDisp(region.getRegionName()));

...
```

lhx2008

2018-12-13 18:24:52 +08:00 via Android

想了下，还可以定义一个静态方法
static void <T> setProperty(Callable<Long> idGetter, Callable<T> regionGetter, Runnable setter)
里面是 14 楼的代码
然后传的时候用 lambda
具体怎么玩楼主研究一下，手机不想打了

lhx2008

2018-12-13 18:26:13 +08:00 via Android

不一定是 17 楼的代码，原来的代码也行，反正是抽象出来了，可读性也还行吧，泛型不一定要

lhx2008

2018-12-13 18:26:52 +08:00 via Android

@lhx2008 17->14

lhx2008

2018-12-13 18:36:31 +08:00 via Android

static <T,R> void setProperty(Callable<Long> idGetter, Callable<T> regionGetter, Consumer<R> setter)
重新谢了一遍

xwbz2018

2018-12-13 18:43:58 +08:00 via Android

@xwbz2018 抱歉，随手一写写出 bug 了。应该是：
String[] regionNames = Stream.of(shopView.getProvinceId(), shopView.getCityId(), shopView.getDistrictId(), shopView.getStreetId())
.map(id -> id == null ? null : regionService.getById(id))
.map(Region::getRegionName)
.toArray(String[]::new);

Kaiv2

2018-12-13 19:53:37 +08:00 via Android

这不太符合 Lambda 的使用场景。。。

Nickwongfree

2018-12-13 21:24:46 +08:00

可以简洁一点，分支太多，4 条基本可以抽象出来新函数
void setLocation(String locationId, Function<String, Boolean> setFunction) {
if (locationId!=null) {
Region region = regionService.getById(locationId);
setFunction(region.getRegionName());
}
}

4 条调用如下
setLocation(shopView.getProvinceId(), showView::setProvinceDisp)
setLocation(shopView. getCityId(), showView:: setCityDisp)
....

Nickwongfree

2018-12-13 21:34:38 +08:00

@Nickwongfree
line5 应为 setFunction.apply()

Charkey

2018-12-13 21:36:43 +08:00

函数式接口似乎不错，楼上有了

binbinyouliiii

2018-12-13 23:34:47 +08:00

有时候强上 Lambda 不觉得可读性会变差吗

wleexi

2018-12-14 08:52:55 +08:00

@binbinyouliiii 你说的对。这个题目的目的是学习

wleexi

2018-12-14 09:03:59 +08:00

@Nickwongfree 好像不行，setCityDisp 是 void

008px

2018-12-14 10:00:22 +08:00 via Android

首先你要明确 Lambda 表达式的使用条件，不是所有都能用的

CasualYours

2018-12-14 10:05:40 +08:00

@wleexi Function 是接收一个参数，返回一个结果。Consumer 接收一个参数，无返回结果。

改成这样就行了

```
void setLocation(String locationId, Consumer<String> setFunction) {
//...
}
```

CasualYours

2018-12-14 10:08:27 +08:00

@CasualYours 方法内部还要改一句 setFunction.accept(region.getRegionName());

Raymon111111

2018-12-14 11:04:27 +08:00

不是 lambda 的场景别硬套了