5 replies

Simplifying Option.liftPredicate Usage in Pipe

Very simple question. Say you have code like this:

  const max = pipe(
    [1, 2, 3, 4, 5] as Iterable<number>, // the type cast is just for example purposes to demonstarte starting a pipe with an iterable
    Array.fromIterable,
    (a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a),
    Option.map(Array.max(Order.number))
  )

  const max = pipe(
    [1, 2, 3, 4, 5] as Iterable<number>, // the type cast is just for example purposes to demonstarte starting a pipe with an iterable
    Array.fromIterable,
    (a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a),
    Option.map(Array.max(Order.number))
  )

  const max = pipe(
    [1, 2, 3, 4, 5] as Iterable<number>, // the type cast is just for example purposes to demonstarte starting a pipe with an iterable
    Array.fromIterable,
    (a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a),
    Option.map(Array.max(Order.number))
  )

  const max = pipe(
    [1, 2, 3, 4, 5] as Iterable<number>, // the type cast is just for example purposes to demonstarte starting a pipe with an iterable
    Array.fromIterable,
    (a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a),
    Option.map(Array.max(Order.number))
  )

Is there any better thing to do than the

(a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a)

(a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a)

(a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a)

(a) => Option.liftPredicate(Array.isNonEmptyReadonlyArray)(a)

?

By "better" I mean something that would avoid the lambda in the conversion from an

Array<number>

Array<number>

Array<number>

Array<number> to an

Option<NonEmptyReadonlyArray<number>>

Option<NonEmptyReadonlyArray<number>>

Option<NonEmptyReadonlyArray<number>>

Option<NonEmptyReadonlyArray<number>>

Simplifying Option.liftPredicate Usage in Pipe

Similar Threads

Similar Threads

Similar Threads