Rozwiązanie z użyciem Map

[DanStak]

Cześć! Chciałem Wam zaprezentować swoje rozwiązanie. Wykorzystałem obiekt Map z uwagi na jego wydajność w dużych zbiorach danych (w końcu Święty Mikołaj zapewne ma łogromną bazę danych). Ujdzie czy da radę jeszcze to ulepszyć ? :)

const HASH_KEY = 'KID_'

interface Kid {
    id: number,
    gifts: Map<string, string>
}

const constructHashKey = (id: number) => `${HASH_KEY}${id}`

export class GiftRegistry {
    private list = new Map<string, Kid>()

    private getKidFromList(kidID: number): Kid {
        const kidHash = constructHashKey(kidID);
        return {...this.list.get(kidHash) || {id: kidID, gifts: new Map<string, string>()}};
    }

    private saveKid(kid: Kid) {
        const kidHash = constructHashKey(kid.id)
        this.list.set(kidHash, kid);
    }

    addGift(kidID: number, giftName: string) {
        const searchedKid = this.getKidFromList(kidID);

        searchedKid.gifts.set(giftName, giftName);

        this.saveKid(searchedKid)
    }

    getGiftsForChild(kidID: number) {
        const searchedKid = this.getKidFromList(kidID);
        
        return Array.from(searchedKid.gifts.values());
    }

    removeGift(kidID: number, giftName: string) {
        const searchedKid = this.getKidFromList(kidID);
        
        if(!searchedKid.gifts.has(giftName)) throw Error('Gift not found');

        searchedKid.gifts.delete(giftName)
        
    }
}

https://github.com/DanStak/advent-of-frontend/blob/main/tasks/2023-12-01/index.ts

[maciek30-byte]

ogolnie calkiem spoko zrobilem podobnie, brakuje tylko obslugi niektorych wyjatkow, ale to chyba nie jest celem, wiec jak najbardziej fajnie :)

[DanStak]

Tak, olałem wyjątki właśnie z uwagi że nie były przedmiotem zadania, choć są nie mniej ważne :) No i super, dzięki za opinię !