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());
}
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
This topic created in 2884 days ago, the information mentioned may be changed or developed.
 |
chanchanDec 13, 2018
1
Optional.ofNullable(obj).ifPresent(item -> {
}); |
|
chanchanDec 13, 2018
3
这种还是 StringUtils.isNoneEmpty(,,,,)吧
|
 |
xwbz2018Dec 13, 2018
4
```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 那部分比较魔法 |
 |
youngxhuiDec 13, 2018 via Android
13
这么多空判断不如换成 kotlin
|
 |
corningsunDec 13, 2018
14
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); |
 |
lhx2008Dec 13, 2018 via Android
15
楼上的写法应该是楼主想要的了，不过说实话不如原来的
|
 |
johnniangDec 13, 2018
16
可以考虑改造 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())); ... ``` |
|
lhx2008Dec 13, 2018 via Android
17
想了下，还可以定义一个静态方法
static void <T> setProperty(Callable<Long> idGetter, Callable<T> regionGetter, Runnable setter) 里面是 14 楼的代码 然后传的时候用 lambda 具体怎么玩楼主研究一下，手机不想打了 |
|
lhx2008Dec 13, 2018 via Android
18
不一定是 17 楼的代码，原来的代码也行，反正是抽象出来了，可读性也还行吧，泛型不一定要
|
|
lhx2008Dec 13, 2018 via Android
20
static <T,R> void setProperty(Callable<Long> idGetter, Callable<T> regionGetter, Consumer<R> setter)
重新谢了一遍 |
|
xwbz2018Dec 13, 2018 via Android
21
@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); |
 |
Kaiv2Dec 13, 2018 via Android
22
这不太符合 Lambda 的使用场景。。。
|
 |
23
可以简洁一点，分支太多，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) .... |
1 |
NickwongfreeDec 13, 2018
24
@Nickwongfree
line5 应为 setFunction.apply() |
 |
CharkeyDec 13, 2018
25
函数式接口似乎不错，楼上有了
|
 |
binbinyouliiiiDec 13, 2018
26
有时候强上 Lambda 不觉得可读性会变差吗
|
 |
27
@binbinyouliiii 你说的对。这个题目的目的是学习
|
|
28
@Nickwongfree 好像不行，setCityDisp 是 void
|
 |
008pxDec 14, 2018 via Android
29
首先你要明确 Lambda 表达式的使用条件，不是所有都能用的
|
 |
CasualYoursDec 14, 2018
30
@wleexi Function 是接收一个参数，返回一个结果。Consumer 接收一个参数，无返回结果。
改成这样就行了 ``` void setLocation(String locationId, Consumer<String> setFunction) { //... } ``` |
|
CasualYoursDec 14, 2018
31
@CasualYours 方法内部还要改一句 setFunction.accept(region.getRegionName());
|
 |
Raymon111111Dec 14, 2018
32
不是 lambda 的场景 别硬套了
|
Select Language