-
-
Save ssippe/1f92625532eef28be6974f898efb23ef to your computer and use it in GitHub Desktop.
const f = (a: any[], b: any[]): any[] => | |
[].concat(...a.map(a2 => b.map(b2 => [].concat(a2, b2)))); | |
export const cartesianProduct = (a: any[], b: any[], ...c: any[]) => { | |
if (!b || b.length === 0) { | |
return a; | |
} | |
const [b2, ...c2] = c; | |
const fab = f(a, b); | |
return cartesianProduct(fab, b2, c2); | |
}; |
Indeed, the inference does not work with your example.
But my simple recommendation here:
Try to avoid mixing types when you can. Here for a cartesian product, I'm not sure it makes a lot of sense to mix strings and numbers.
I want a type system to help me, not make my work harder.
A type system is also here to help you to design things the right way. If the implementation becomes too complex, it generally means that something is wrong. You do not implement things the same way when using typings. If you're not happy with that, you should go back to JS.
Tell me that you cannot solve the problem of the cartesian product in TypeScript without telling me that you cannot solve the problem of the cartesian product in TypeScript.
Which is fine, by the way. I don't know the solution either. I came here looking for the solution and make the note that the proposed solution is wrong, in the sense explained in the comment above. Nothing else.
If you find the right solution, I'd be glad if you posted it here.
Mixed types are tricky.
Give this a try.
The key is [...T]
export const cartesianProduct = <T extends any[][]>(a: [...T]): Array<{[K in keyof T]: T[K][number]}> => a.reduce((a, b) => a.flatMap(d => b.map(e => [d, e].flat())))
@cr0cK
First of all, the inference is not working properly. But even if we aid the compiler, the types are still awful:
We know with certainty that the second element of the cartesian product is a
string
. However, your implementation loses that information. That's not what I expect from a type system. I want a type system to help me, not make my work harder.That is how you "expect" it to work?