PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Convert GlobalCache to its own class
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
In general, Common.lua should be for functions that can be used on any table or number or other type. Many of these functions there are very specific to the GlobalCache. A few options I'd rather see
- Make the functions more generic (e.g. for
mergeDBhere, have it bemergeTablesand call it twice for conditions and multipliers) - Move the functions to a separate file specifically for managing the GlobalCache (might be worth turning GlobalCache into a proper OO class potentially)
- Some of the functions are only used in one place. It might be better off having those functions in the file that uses them either as a local function or a function defined in that file instead of polluting the global namespace
Originally posted by @Wires77 in https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/2294#discussion_r602679854
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der Prüfung der GlobalCache-spezifischen Funktionen in Common.lua und verfolge ihre Aufrufstellen. Vergleiche die drei vorgeschlagenen Richtungen – generische Hilfsfunktionen, eine eigene GlobalCache-Datei oder -Klasse und das Lokalisieren von nur einmal verwendeten Funktionen –, bevor du dich für ein Design entscheidest. Als erledigt gilt die Aufgabe, wenn die Verantwortlichkeiten getrennt sind, ohne das bestehende Verhalten zu ändern; dieses Issue definiert jedoch keinen spezifischen Abnahmetest.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100