In C++ wie Variablen nach X Sekunden in einer Klasse zurückgesetzt werden?

Question

Ich muss einige Werte in einer Klasse zurücksetzen, nachdem X Sekunden vergangen sind. Ich bin mit so etwas, aber ich weiß nicht, welche Probleme es haben könnte:

class Points 
{ 
    public: 
     Points(); 
     ~Points(); 
     void reset(); 
     ... 
    private: 
     int number; 
     static void resetAfter5seconds(Points* points); 
}; 

Im Innern des contructor ich einen freistehenden Thread initialisieren:

Points::Points(){ 
    thread start(resetAfter5seconds, this); 
    start.detach(); 
} 

void Points::resetAfter5seconds(Points* points){ 

    while(true){ 
    std::this_thread::sleep_for((std::chrono::seconds(5))); 
    points->reset(); 
    } 

} 

Dies funktioniert, aber ich weiß nicht, ob es ist der richtige Ansatz. Hat es Speicherprobleme? Was sind die Alternativen?

Answer 1

Ihr Code ist nicht threadsicher, und es ist wahrscheinlich zu viel, um es threadsicher zu machen, wenn der Rest des Codes single threaded ist. Vielleicht könntest du ein einfacheres Muster haben, das keine Threading-Funktion benötigt, wie zum Beispiel das Überprüfen, ob die Zeit von 5 Sekunden seit der Erstellung verstrichen ist, etwa so;

class Points 
{ 
public: 
    Points() {creationtime = time(0); /* do the rest of init */ } 
    ~Points(); 
    void method1() { test_and_reset(); /* do the rest of logic...*/ } 
    void method2() { test_and_reset(); /* do the rest of logic...*/ } 

     ... 
private: 
    time_t creationtime; 
    void test_and_reset() { 
      time_t now = time(0); 
      if (now > creationtime + 5) { /*reset*/ } 
    } 
}; 

Die obige einfache Implementierung auch bedeuten, dass Sie das Problem des Fadens ungültigen Speicherzugriff nicht haben, wenn das Objekt gelöscht wird, und Sie werden nicht haben, um herauszufinden, wie die Lauffläche zu töten.

Answer 2

Tun Sie es träge. Keine Notwendigkeit für Threads hier. Füge Getter hinzu (privat oder öffentlich) und greife nur über den Getter (und den entsprechenden Setter, wahrscheinlich) auf die Variable zu.

Im Getter überprüfen, ob die Rücksetzzeit aktiv ist (einfaches boolesches Flag, um den Overhead zu minimieren) und dann prüfen, ob genug Zeit vergangen ist, und dann den Wert zurücksetzen. Schließlich geben Sie den Wert wie in einem einfachen Getter zurück.

---

Oder, wenn Ihre Anwendung Ereignisschleife und ereignisgesteuerte Architektur (wahrscheinlich nicht, nur das Hinzufügen von hier der Vollständigkeit halber), Timer-Ereignis fügen Sie den Wert zurücksetzen. Immer noch keine Notwendigkeit für Fäden, um die Schläger zu komplizieren.