PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Convert GlobalCache to its own class

Offen
#2,385 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

technical
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

  1. Make the functions more generic (e.g. for mergeDB here, have it be mergeTables and call it twice for conditions and multipliers)
  2. Move the functions to a separate file specifically for managing the GlobalCache (might be worth turning GlobalCache into a proper OO class potentially)
  3. 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

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Ö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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.