下面 Java 代码怎么用 Lambda 表示

[wleexi]

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());
        }

[chanchan]

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

});

[wleexi]

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

[chanchan]

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

[xwbz2018]

```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]

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

[wleexi]

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

[xwbz2018]

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

[wleexi]

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

[chanchan]

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

[wleexi]

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

[xwbz2018]

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

[wleexi]

@xwbz2018 这个倒是可以

[youngxhui]

这么多空判断不如换成 kotlin

[corningsun]

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]

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

[johnniang]

可以考虑改造 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]

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

[lhx2008]

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

[lhx2008]

@lhx2008 17->14

[lhx2008]

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

[xwbz2018]

@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]

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

[Nickwongfree]

可以简洁一点，分支太多，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]

@Nickwongfree
line5 应为 setFunction.apply()

[Charkey]

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

[binbinyouliiii]

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

[wleexi]

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

[wleexi]

@Nickwongfree 好像不行，setCityDisp 是 void

[008px]

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

[CasualYours]

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

改成这样就行了

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

[CasualYours]

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

[Raymon111111]

不是 lambda 的场景 别硬套了