JanWielgus / JanWielgus/StateDesignPatternLib
Context don't have to be passed in setState method
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 0
- Forks
- 0
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Currently `setState` method (that is run from context object) have a first argument of context. This context is the same object for which we run this method, so in this method it could be accessed by `this` pointer. Using this method with different context than for which it is executed will be invalid, so it is also a bug.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
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
Lokalisieren Sie die Deklaration und Implementierung von setState und überprüfen Sie anschließend die Aufrufer, um zu bestätigen, wie das context-Argument derzeit übergeben wird. Aktualisieren Sie die API konsistent und stellen Sie sicher, dass die bestehende Verwendung weiterhin kompiliert und den besitzenden context verwendet; die Aufgabe ist abgeschlossen, wenn das redundante Argument nicht mehr erforderlich ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100