Typescript - защита общего типа для функции isEmpty - PullRequest
0 голосов
/ 05 сентября 2018

Я не могу правильно реализовать универсальный isEmpty(value) с точки зрения сужения типа, ограничивающего предоставленное значение его пустым аналогом.

Вариант использования:

function getCountryNameById(countries: LookupItem[] = [], countryId?: number): string | undefined {

  if (isEmpty(countries) || !isNumber(countryId)) {

    // within this branch I would like to get countries argument to be narrowed to empty array type. 
    // Same would apply for other function which can have argument type of object or string. Why ? -> to prevent someone to do some mad code hacks like accessing non existent value from empty array ( which would happen on runtime ofc ) on compile time
    // $ExpectType []
    console.log(countries)

    return
  }

  // continue with code logic ...
  // implementation ...
}

аналогичный случай на ограничивающем объекте:

function doSomethingWithObject( data: { foo: string; bar: number } | object ){ 
   if(isEmpty(data)){
     // $ExpectType {}
     data

     // following should throw compile error, as data is empty object
     data.foo.toUpercase()

     return
   }

   // here we are sure that data is not empty on both runtime and compile time
}

Реализация защиты типа isEmpty:

export const isEmpty = <T extends AllowedEmptyCheckTypes>(
  value: T | AllowedEmptyCheckTypes
): value is Empty<T> => {
  if (isBlank(value)) {
    return true
  }

  if (isString(value) || isArray(value)) {
    return value.length === 0
  }

  if (isObject(value)) {
    return Object.keys(value).length === 0
  }

  throw new Error(
    `checked value must be type of string | array | object. You provided ${typeof value}`
  )
}

С определенными типами:

type EmptyArray = Array<never>
type Blank = null | undefined | void

/**
 * // object collects {} and Array<any> so adding both {} and Array<any> is not needed
 * @private
 */
export type AllowedEmptyCheckTypes = Blank | string | object

/**
 * Empty mapped type that will cast any AllowedEmptyCheckTypes to empty equivalent
 * @private
 */
export type Empty<T extends AllowedEmptyCheckTypes> = T extends string
  ? ''
  : T extends any[]
    ? EmptyArray
    : T extends object ? {} : T extends Blank ? T : never

Это немного странно, поскольку оно корректно сужается с точки зрения типа, но не внутри ветви if / else:

isEmpty для строковых значений

isEmpty для значений массива

isEmpty для значений объекта

код можно посмотреть здесь: https://github.com/Hotell/rex-tils/pull/13/files#diff-a3cdcb321a05315fcfc3309031eab1d8R177

Похожие вопросы: Тип Guard для пустого объекта

Ответы [ 3 ]

0 голосов
/ 06 сентября 2018

Итак, после нескольких обсуждений в Твиттере и долгого поиска SO / Github, я нашел следующее решение:

  • прежде всего проверка на наличие значения null / undefined в isEmpty не имеет особого смысла (хотя lodash.isEmpty обрабатывает это, ИМХО, он делает слишком много и не очень явным образом)
  • потому что в принципе нет никакой разницы между {} | object | any[] сужение типа защиты никогда не будет работать, как ожидалось
  • окончательное решение принимает только допустимые значения для проверки в качестве аргументов -> объекты js, а строка и защита возвращают never, поэтому совпадающие значения будут иметь тип never, так как в любом случае не имеет смысла выполнять дальнейший вход в систему в операторе if(isEmpty(value)){ ... } вместо завершения программы или выдачи ошибки

вот окончательная реализация:

const isEmpty = <T extends string | object | any[]>(
  value: T
): value is never => {
  if (isString(value) || isArray(value)) {
    return value.length === 0
  }
   if (isObject(value)) {
    return Object.keys(value).length === 0
  }
   throw new Error(
    `checked value must be type of string | array | object. You provided ${
      // tslint:disable-next-line:strict-type-predicates
      value === null ? 'null' : typeof value
    }`
  )
}

https://github.com/Hotell/rex-tils/pull/13/files#diff-68ff3b6b6a1354b7277dfc4b23d99901R50

0 голосов
/ 06 сентября 2018

Одним из способов решения этой проблемы является отделение пустых проверок (undefined, null) от пустых проверок значений ('', [] {}). Для этого я обычно использую два типа охранников - isDefined и isEmpty.

Первый может выглядеть так. Обратите внимание на проверку typeof - она ​​также работает с необъявленными переменными.

function isDefined<T>(value: T | undefined | null): value is T {
  return (typeof value !== 'undefined') && (value !== null);
}

Для пустых значений можно использовать следующую модель.

namespace Empty {
  export type String = '';
  export type Object = Record<string, never>;
  export type Array = never[];
}

type Empty =
  | Empty.Array
  | Empty.Object
  | Empty.String;

function isEmpty<T extends string | any[] | object>(subject: T | Empty): subject is Bottom<T> {
  switch (typeof subject) {
    case 'object':
      return (Object.keys(subject).length === 0);
    case 'string':
      return (subject === '');
    default:
      return false;
  }
}

type Bottom<T> =
  T extends string
    ? Empty.String
    : T extends any[]
        ? Empty.Array
        : T extends object
            ? Empty.Object
            : never;

Нижние значения выведены правильно.

declare const foo: 'hello' | Empty.String;
declare const bar: [number, number] | Empty.Array;
declare const baz: Window | Empty.Object;

if (isEmpty(foo) && isEmpty(bar) && isEmpty(baz)) {
  console.log(foo, bar, baz);
}

Редактировать: добавлены ограничения на T, как предложено.

0 голосов
/ 05 сентября 2018

Здесь

if (isEmpty(countries) || !isNumber(countryId)) {

у вас есть два условия, только одно из которых является защитой типа для countries, поэтому тип countries не меняется внутри if.

Что касается объектов, {} не представляет пустой объект. Все, кроме null и undefined, может быть присвоено переменной типа {}. Вы можете вместо этого использовать { [prop: string]: never } или { [prop: string]: undefined }.

...