In which case I'd put it in a lib, but how would this be named ?
I'd also make a second implementation that would enforce ID uniqueness and use Map to map IDs with indexes instead of running find : while the first implementation would be fine for static data, the second one would be more suitable for dynamic data.
Seems like a great idea.
I often find myself using objects when I'm building data (like a horrible CSV processor). It's easy to pull in a row of data, and potentially modify existing data (yeh, really horrible CSV).
But when it gets to processing the data (transformers and database), I prefer arrays.
A lot of it comes back to typescript, where Object.keys().forEach means the key isn't necessarily usable to access the item in the object (the whole, a key becomes a string, but an object can be keyed by string/number/symbol). .
So, whatever you come up with HAS to have first class Typescript support.
Fair enough, I can understand that.
I used to enjoy the flexibility that JS provides. And IDEs do a pretty good job of filling the holes!
My last project, I went all in on typescript. And I have caught so many more errors before even compiling.
It's like having tests. It gives a hell of a lot more confidence.
Some days I miss just slinging objects around, and just coercing types tho (without explicitly casting/converting... Object keys being a big one!)
My goal is to be able to both easily get an item by ID (using data[id]) and easily get an item by properties (using data.find()).
Meanwhile, just like Object requires Object.values(object) before calling find, Map requires [...map.values()], and it also have the additional inconvenient of requiring a call to map.get to get an item by ID, so no, it's even worse for my goal, but thanks.
I'm pretty sure I've been in your situation but haven't created a dictionary/array hybrid.
Without any more details about your use case and situation, I can imagine a few pitfalls with your solution:
Serialization might not behave as you would expect (JSON.stringify).
3rd-party functions might not be able to deal with your data structure properly (again, potentially unexpected behavior).
You can't easily access array methods (find, filter, map etc).
How do you distinguish between ID access and index access? ArrayLike([ {id: "1" }, { id: "2" } ])[1] returns { id: "2" }, how do you access { id: "1" } by ID?
It's harder to reason about access time of lookups. However, this might not be a concern of yours.
It may cause confusion if you're working with other developers.
That being said, unless you work in an environment where your code should be easily understandable by others, the best way to find out if this is a good idea or not, is to try :)
Me personally, I usually use an associateBy function, when I need a values-by-ID structure. Of course this is not as convenient to use as your approach, but it's good enough for me.
// this WILL drop elements if key is not unique
function associateBy(array, key) {
return array.reduce((acc, el) => ({
...acc,
[el[key]]: el
}), {});
}
associateBy([
{id: "foo"},
{id: "bar"}
], "id").foo; // -> {id: "foo"}
Serialization might not behave as you would expect (JSON.stringify)
Actually, my implementation idea relying on Proxy don't have this issue.
As for other implementations, they do have this issue, but as they say, hybrid array use cases don't generally involve the need to stringify, and even when they do, using Object.defineProperty with { enumerable: false } (or replacing the toJSON method) would fix it.
3rd-party functions might not be able to deal with your data structure properly (again, potentially unexpected behavior).
You can’t easily access array methods (find, filter, map etc).
Actually, it would still be an Array, so no, there shouldn't be any problems, and yes, those methods definitely work, which is precisely what said I want to achieve.
How do you distinguish between ID access and index access?
If your IDs are integers then there is no need for an hybrid at all, precisely because all you have to do is put each item at the same position as their ID.
It’s harder to reason about access time of lookups. However, this might not be a concern of yours.
I'll definitely run benchmarks so that users would be aware of performance losses, if any. But use cases of hybrid arrays are mostly small datasets so it usually shouldn't be a concern indeed.
It may cause confusion if you’re working with other developers.
If I implement this, it will be as a documented and commented package, so it shouldn't be that much of a problem.
I think I misunderstood your initial post (and definitely didn't read it as carefully as I should have 😅).
Do I understand your correctly that your goal is a companion object for your arrays that simplifies access? Not a new data structure that you'd user instead of arrays? If so, most of my points are moot.
If your IDs are integers then there is no need for an hybrid at all, precisely because all you have to do is put each item at the same position as their ID.
If you don't treat IDs as opaque values, this is true.
I'll definitely run benchmarks so that users would be aware of performance losses, if any. But use cases of hybrid arrays are mostly small datasets so it usually shouldn't be a concern indeed.
I think my point is actually wrong (it was really late when I was writing my initial response). Performance would be O(n), since that's the worst case scenario.
Anyways, I hope you could take something useful from my answer.
If you're not married to functional code, for searching a keyed object by property for the key, you can always write:
for (const key in data) {
if (data[key].firstName === 'Frederic') {
console.log(key);
}
}
No need to transform the whole object with Object.entries, and you could turn that into a function reasonably easy, something like findKey(object, property, value) or some such.