Làm thế nào để thực thi logic trên Tùy chọn nếu không có?

Tôi muốn thay thế mã sau bằng java8 Optional:

public Obj getObjectFromDB() {
    Obj obj = dao.find();
    if (obj != null) {
        obj.setAvailable(true);
    } else {
        logger.fatal("Object not available");
    }

    return obj;
}

Mã giả sau đây không hoạt động vì không có orElseRunphương pháp nào , nhưng dù sao nó cũng minh họa mục đích của tôi:

public Optional<Obj> getObjectFromDB() {
    Optional<Obj> obj = dao.find();
    return obj.ifPresent(obj.setAvailable(true)).orElseRun(logger.fatal("Object not available"));
}

Với Java 9 trở lên, ifPresentOrElserất có thể là những gì bạn muốn:

Optional<> opt = dao.find();

opt.ifPresentOrElse(obj -> obj.setAvailable(true),
                    () -> logger.error("…"));

Currying bằng vavr hoặc tương tự có thể nhận được mã gọn gàng hơn, nhưng tôi chưa thử.

Tôi không nghĩ rằng bạn có thể làm điều đó trong một tuyên bố. Làm tốt hơn:

if (!obj.isPresent()) {
    logger.fatal(...);   
} else {
    obj.get().setAvailable(true);
}
return obj;

Đối với Java 8 Spring cung cấp ifPresentOrElsetừ "Phương pháp tiện ích để làm việc với Tùy chọn" để đạt được những gì bạn muốn. Ví dụ sẽ là:

import static org.springframework.data.util.Optionals.ifPresentOrElse;    

ifPresentOrElse(dao.find(), obj -> obj.setAvailable(true), () -> logger.fatal("Object not available"));

Bạn sẽ phải chia điều này thành nhiều câu lệnh. Đây là một cách để làm điều đó:

if (!obj.isPresent()) {
  logger.fatal("Object not available");
}

obj.ifPresent(o -> o.setAvailable(true));
return obj;

Một cách khác (có thể được thiết kế quá mức) là sử dụng map:

if (!obj.isPresent()) {
  logger.fatal("Object not available");
}

return obj.map(o -> {o.setAvailable(true); return o;});

Nếu obj.setAvailablethuận tiện trả về obj, thì bạn có thể chỉ cần lấy ví dụ thứ hai để:

if (!obj.isPresent()) {
  logger.fatal("Object not available");
}

return obj.map(o -> o.setAvailable(true));

Trước hết, bạn dao.find()phải trả về một Optional<Obj>hoặc bạn sẽ phải tạo một.

ví dụ

Optional<Obj> = dao.find();

hoặc bạn có thể tự làm như:

Optional<Obj> = Optional.ofNullable(dao.find());

cái này sẽ trở lại Optional<Obj>nếu có hoặc Optional.empty()nếu không có.

Vì vậy, bây giờ chúng ta hãy đi đến giải pháp,

public Obj getObjectFromDB() {
   return Optional.ofNullable(dao.find()).flatMap(ob -> {
            ob.setAvailable(true);
            return Optional.of(ob);    
        }).orElseGet(() -> {
            logger.fatal("Object not available");
            return null;
        });
    }

Đây là một lớp lót mà bạn đang tìm kiếm :)

Có là một .orElseRunphương pháp, nhưng nó được gọi .orElseGet.

Vấn đề chính với mã giả của bạn là .isPresentnó không trả về Optional<>. Nhưng .maptrả về một Optional<>có orElseRunphương thức.

Nếu bạn thực sự muốn làm điều này trong một tuyên bố, bạn có thể thực hiện điều này:

public Optional<Obj> getObjectFromDB() {
    return dao.find()
        .map( obj -> { 
            obj.setAvailable(true);
            return Optional.of(obj); 
         })
        .orElseGet( () -> {
            logger.fatal("Object not available"); 
            return Optional.empty();
    });
}

Nhưng điều này thậm chí còn phức tạp hơn những gì bạn có trước đây.

Tôi đã có thể đưa ra một vài giải pháp "một dòng", ví dụ:

    obj.map(o -> (Runnable) () -> o.setAvailable(true))
       .orElse(() -> logger.fatal("Object not available"))
       .run();

hoặc là

    obj.map(o -> (Consumer<Object>) c -> o.setAvailable(true))
       .orElse(o -> logger.fatal("Object not available"))
       .accept(null);

hoặc là

    obj.map(o -> (Supplier<Object>) () -> {
            o.setAvailable(true);
            return null;
    }).orElse(() () -> {
            logger.fatal("Object not available")
            return null;
    }).get();

Nó trông không đẹp lắm, một cái gì đó giống như orElseRunsẽ tốt hơn nhiều, nhưng tôi nghĩ rằng tùy chọn đó với Runnable có thể chấp nhận được nếu bạn thực sự muốn giải pháp một dòng.

Đối với những người bạn muốn thực hiện một hiệu ứng phụ chỉ khi không có tùy chọn

tức là tương đương ifAbsent()hoặc ifNotPresent()ở đây là một sửa đổi nhỏ đối với các câu trả lời tuyệt vời đã được cung cấp.

myOptional.ifPresentOrElse(x -> {}, () -> {
  // logic goes here
})

Bạn cần Optional.isPresent () và orElse () . Đoạn mã của bạn đã thắng; không hoạt động vì nó không trả về bất kỳ thứ gì nếu không có.

Điểm của Tùy chọn là trả về nó từ phương thức.

Với Java 8, Optionalnó có thể được thực hiện với:

    Optional<Obj> obj = dao.find();

    obj.map(obj.setAvailable(true)).orElseGet(() -> {
        logger.fatal("Object not available");
        return null;
    });

ifPresentOrElse cũng có thể xử lý các trường hợp nullpointers. Cách tiếp cận dễ dàng.

   Optional.ofNullable(null)
            .ifPresentOrElse(name -> System.out.println("my name is "+ name),
                    ()->System.out.println("no name or was a null pointer"));

Tôi cho rằng bạn không thể thay đổi dao.find()phương thức để trả về một phiên bản Optional<Obj>, vì vậy bạn phải tự tạo một phiên bản thích hợp.

Đoạn mã sau đây sẽ giúp bạn. Tôi đã tạo lớp OptionalAction, lớp này cung cấp cơ chế if-else cho bạn.

public class OptionalTest
{
  public static Optional<DbObject> getObjectFromDb()
  {
    // doa.find()
    DbObject v = find();

    // create appropriate Optional
    Optional<DbObject> object = Optional.ofNullable(v);

    // @formatter:off
    OptionalAction.
    ifPresent(object)
    .then(o -> o.setAvailable(true))
    .elseDo(o -> System.out.println("Fatal! Object not available!"));
    // @formatter:on
    return object;
  }

  public static void main(String[] args)
  {
    Optional<DbObject> object = getObjectFromDb();
    if (object.isPresent())
      System.out.println(object.get());
    else
      System.out.println("There is no object!");
  }

  // find may return null
  public static DbObject find()
  {
    return (Math.random() > 0.5) ? null : new DbObject();
  }

  static class DbObject
  {
    private boolean available = false;

    public boolean isAvailable()
    {
      return available;
    }

    public void setAvailable(boolean available)
    {
      this.available = available;
    }

    @Override
    public String toString()
    {
      return "DbObject [available=" + available + "]";
    }
  }

  static class OptionalAction
  {
    public static <T> IfAction<T> ifPresent(Optional<T> optional)
    {
      return new IfAction<>(optional);
    }

    private static class IfAction<T>
    {
      private final Optional<T> optional;

      public IfAction(Optional<T> optional)
      {
        this.optional = optional;
      }

      public ElseAction<T> then(Consumer<? super T> consumer)
      {
        if (optional.isPresent())
          consumer.accept(optional.get());
        return new ElseAction<>(optional);
      }
    }

    private static class ElseAction<T>
    {
      private final Optional<T> optional;

      public ElseAction(Optional<T> optional)
      {
        this.optional = optional;
      }

      public void elseDo(Consumer<? super T> consumer)
      {
        if (!optional.isPresent())
          consumer.accept(null);
      }
    }
  }
}