javascript - Function to pick N random items from array in Typescript - Stack Overflow

admin管理员组

文章数量:1352150

This is the code. Got the idea from this answer.

Typescript playground

I'm using the generic type parameter: T extends unknown[], since this function should work for any kind of array.

export const pickRandomItems = <T extends unknown[]> (arr: T, n: number): T => {
  const shuffled = Array.from(arr).sort(() => 0.5 - Math.random());
  return shuffled.slice(0, n);
};

But I'm getting the following error on the return statement:

It goes away if I do a type assertion. But why is that necessary? Am I doing something wrong?

NOTE:

The following line is evaluated as unknown[], so why is the type assertion necessary anyway?

const result = shuffled.slice(0, n);

This is the code. Got the idea from this answer.

Typescript playground

I'm using the generic type parameter: T extends unknown[], since this function should work for any kind of array.

export const pickRandomItems = <T extends unknown[]> (arr: T, n: number): T => {
  const shuffled = Array.from(arr).sort(() => 0.5 - Math.random());
  return shuffled.slice(0, n);
};

But I'm getting the following error on the return statement:

It goes away if I do a type assertion. But why is that necessary? Am I doing something wrong?

NOTE:

The following line is evaluated as unknown[], so why is the type assertion necessary anyway?

const result = shuffled.slice(0, n);

• javascript
• arrays
• typescript

Share Improve this question Follow edited Nov 3, 2020 at 11:44 cbdeveloper asked Nov 3, 2020 at 11:39 cbdevelopercbdeveloper 31.5k4545 gold badges202202 silver badges396396 bronze badges 1

• How do I ask a good question?: "DO NOT post images of code, data, error messages, etc. - copy or type the text into the question. " – Andreas Commented Nov 3, 2020 at 11:47

Add a ment  | 

2 Answers 2

Sorted by: Reset to default 9

Change the type parameter to be the array item, not the whole array:

export const pickRandomItems = <T extends unknown> (arr: T[], n: number): T[] => {
  const shuffled = Array.from(arr).sort(() => 0.5 - Math.random());
  return shuffled.slice(0, n);
};

Playground Link

While T extends unknown[] does mean T can be any array, it could also be a subtype of Array, which means slice will not return the appropriate type (it will return Array instead of whatever T is). Generally you can't assign a concrete type to a generic type parameter, because the generic type parameter could be any sub type that is decided by the caller, while you are using a specific concrete type in your implementation.

The issue you're having is that T can be any subtype of unknown[], however your function returns an array. Thus, typescript warns you that an array cannot be cast to T, which could be some subclass of an array. Instead, you can change T to be the item, and take an iterable as an argument and return an array:

// trailing ma in generic so ts doesn't confuse with jsx
export const pickRandomItems = <T,> (arr: Iterable<T>, n: number): T[] => {
  const shuffled = Array.from(arr).sort(() => 0.5 - Math.random());
  return shuffled.slice(0, n);
};

本文标签： javascriptFunction to pick N random items from array in TypescriptStack Overflow