OnMouseMove MouseMove

[DocShoe]

Ich hab ähnliche Probleme mit dem MouseMoveEvent, das feuert bei einer Komponente auch ständig. Ich hab´s dadurch gelöst, dass ich mir die Mauskoordinaten lokal merke und nur bei echten Änderungen auf das Event reagiere:

void __fastcall MyForm::OnMouseMove( TObject* Sender, MouseButton Button, TShiftState Shift, int X, int Y )
{
   static TPoint last = TPoint( -1,-1 );

   TPoint curr = TPoint( X, Y );
   if( curr != last )
   {
      last = curr;
      // tu was
   }
}

[hustbaer]

@DocShoe sagte in OnMouseMove MouseMove:

static TPoint last = TPoint( -1,-1 );

*Schmerzen*

[DocShoe]

@hustbaer sagte in OnMouseMove MouseMove:

@DocShoe sagte in OnMouseMove MouseMove:

static TPoint last = TPoint( -1,-1 );

*Schmerzen*

Alternative?

[hustbaer]

Membervariable?

[DocShoe]

@hustbaer

Dann schlepp ich das ja da mit, wo ich es überhaupt nicht brauche. Mit meiner Lösung ist der "Schaden" auf die einzige Funktion begrenzt, wo er gebraucht wird. Race conditions können nicht auftreten, da der Aufruf immer aus dem GUI Thread kommt, ich sehe also nix, was dagegen spricht. Außer persönlicher Präferenz.

[DocShoe]

@hustbaer

Was hältst du denn von solchen Konstrukten?

void f()
{
   static bool PreventReentrance = false;

   if( !PreventReentrance ) 
   {
      PreventReentrance = true;
      ...
      PreventReentrance = false;
   }
}

[SeppJ]

Ist das nicht eine Race Condition, die nur darauf wartet, dass sie explodiert?

[DocShoe]

Jo, denke auch. Hab sowas mal in ´nem Snippet gesehen, hängt vermutlich vom Einsatzbereich ab, wo das eingesetzt wird.

[SeppJ]

@DocShoe sagte in OnMouseMove MouseMove:

Jo, denke auch. Hab sowas mal in ´nem Snippet gesehen, hängt vermutlich vom Einsatzbereich ab, wo das eingesetzt wird.

Nein, das ist einfach immer falsch, aber viele Leute haben halt einfach kein Bewusstsein für Race Conditions. Fehler dieser Art wirst du tausendfach in Codebeispielen finden, besonders oft sieht man dies bei Dateihandling.

[DocShoe]

@SeppJ

Du gehst von Multithreading aus, das kann man auch zum Verhindern von endloser Rekursion benutzen. Wenn man seinen Programmablauf nicht im Griff hat.

[SeppJ]

@DocShoe sagte in OnMouseMove MouseMove:

@SeppJ

Du gehst von Multithreading aus, das kann man auch zum Verhindern von endloser Rekursion benutzen. Wenn man seinen Programmablauf nicht im Griff hat.

Naja, der gezeigte Code ist ganz eindeutig auf Multithreading bezogen, wo er falsch ist. Ansonsten bist du bei "Leute benutzen if mit globalen Variablen", was natürlich auch oft richtig eingesetzt wird

[DocShoe]

@SeppJ

Ich weiß zufällig, dass er nicht in Multithreading eingesetzt wird

[SeppJ]

@DocShoe sagte in OnMouseMove MouseMove:

@SeppJ

Ich weiß zufällig, dass er nicht in Multithreading eingesetzt wird

Häh? Wieso dann Reentrance (falsch) verhindern?

das kann man auch zum Verhindern von endloser Rekursion benutzen

Beziehst du dich darauf? Nutzt hier jemand statische Variablen in einer Rekursion? Das spricht ja noch mehr dafür, dass viele Leute einfach keine Ahnung haben.

[DocShoe]

@SeppJ

Jetzt muss ich aufpassen, was ich sage. Da könnte jemand sauer auf mich werden. Aber ja, damit wird Rekursion verhindert.

[SeppJ]

Dieser Beitrag wurde gelöscht!

[hustbaer]

@DocShoe sagte in OnMouseMove MouseMove:

@hustbaer

Dann schlepp ich das ja da mit, wo ich es überhaupt nicht brauche. Mit meiner Lösung ist der "Schaden" auf die einzige Funktion begrenzt, wo er gebraucht wird. Race conditions können nicht auftreten, da der Aufruf immer aus dem GUI Thread kommt, ich sehe also nix, was dagegen spricht. Außer persönlicher Präferenz.

Mir ging's weniger um Race-Conditions sondern darum dass das u.U. nimmer so wie es soll funktioniert wenn's mal zwei oder mehr Instanzen von dem Fenster gibt.

[hustbaer]

@DocShoe sagte in OnMouseMove MouseMove:

@hustbaer

Was hältst du denn von solchen Konstrukten?

void f()
{
   static bool PreventReentrance = false;

   if( !PreventReentrance ) 
   {
      PreventReentrance = true;
      ...
      PreventReentrance = false;
   }
}

Auch *Schmerzen*
Mit thread_local und RAII wäre es OK.

[hustbaer]

@SeppJ sagte in OnMouseMove MouseMove:

@DocShoe sagte in OnMouseMove MouseMove:

@SeppJ

Du gehst von Multithreading aus, das kann man auch zum Verhindern von endloser Rekursion benutzen. Wenn man seinen Programmablauf nicht im Griff hat.

Naja, der gezeigte Code ist ganz eindeutig auf Multithreading bezogen, wo er falsch ist. Ansonsten bist du bei "Leute benutzen if mit globalen Variablen", was natürlich auch oft richtig eingesetzt wird

Ich kenn solchen Code eher dafür Rekursion zu vermeiden.
Wobei ich die Variable dann nicht PreventReentrance sondern IsRecursiveCall oder so nennen würde.

[Skator123]

@Th69 Ich habe das Problem anders gelöst. Das verschieben des TButton auf dem Formular löst immer wieder MouseMove aus... ich verschiebe jetzt ein TBitBtn damit geht es.

Danke für eure ganzen Antworten

Gruß Ska