代码之家 › 专栏 › 技术社区 › ip696

如何摆脱对空值的检查,如果此代码中有签名,是否还有其他检查?

clean-architecture code-cleanup design-patterns java

ip696 · 技术社区 · 7 年前

我得到复杂的对象响应。看起来像这样:

    class ComplexObject{
        private PartOne partOne;
        private PartTwo partTwo;
        private PartThree partThree;
    }

我需要处理这个反应。现在我这样做:

    if (partOne!= null) {
        processOne(partOne);
    } else if (partTwo != null) {
        processTwo(partTwo);
    } else if (partThree != null) {
        processThree(partThree);
    }

但看起来很糟糕。如果我可以影响外部服务,我将添加枚举 Status.PART_ONE, Status.PART_TWO, Status.PART_THREE 但我做不到。

如何重写此代码以使其更清晰?

2 回复 | 直到 7 年前

davidxxx 7 年前

如果 PartOne , PartTwo , PartThree 不要成为基类的一部分,您不能利用多态性。
在这种情况下,您的实际方法是可以接受的折衷。

我只是这样写:

if (partOne!= null) processOne(partOne);
else if (partTwo != null) processTwo(partTwo);
else if (partThree != null) processThree(partThree);

SamHoque 7 年前

如何使用 java.util.Optional 由Java 8提供的,如注释中建议的“iTFRADAK”,代码看起来像

Optional.ofNullable(partOne).ifPresent(p -> processOne(p));
Optional.ofNullable(partTwo).ifPresent(p -> processTwo(partTwo));
Optional.ofNullable(partThree).ifPresent(p -> processThree(partThree));

如果您使用的是Java 9或更高版本,您可以这样做

Optional.ofNullable(partOne).ifPresentOrElse(p -> processOne(partOne),
        () -> Optional.ofNullable(partTwo).ifPresentOrElse(p -> processTwo(partTwo), 
                () ->  Optional.ofNullable(partThree).ifPresent(p -> processThree(partThree))));

推荐文章

Tenten Ponce · 在全球范围内,将所需数据(通过API传递)的附加放在哪里?

8 年前

Utsav Shrestha · 如何在gradle建立清洁建筑项目?

8 年前

Tenten Ponce · 干净的体系结构、数据请求协调器、演示者或用例/交互者?

8 年前

Mehmed Andrew Lam · 干净的体系结构:在哪里实现分页逻辑?

8 年前

w00ly · 干净的体系结构、用例和实体

8 年前

Tony Raimo · 域实体是否应该调用存储库?

8 年前

nairavs · 干净的体系结构Maven模块依赖关系

8 年前

Ritesh Shakya · 使用Dagger2和setRetainInstance时获得NullPointException(true);

8 年前

Adnan Bal · Clean architecture rxjava firebase实现

9 年前

Chandra Gopalaiah · 在Bob叔叔的体系结构中,将表示层中的模型映射到域层中的对象

9 年前