Einzelnen Beitrag anzeigen

Benutzerbild von fkerber
fkerber
(CodeLib-Manager)

Registriert seit: 9. Jul 2003
Ort: Ensdorf
6.723 Beiträge
 
Delphi XE Professional
 
#9

AW: MediaPlayer mit PosA and PosB setzen

  Alt 2. Aug 2010, 16:24
Hi!

Aha, gut, dann habe ich so eine ungefähre Vorstellung.

Eine Anmerkung gleich vorweg:

 if form1.LoopAB.checked = false Solche Vergleiche auf true/false sollte man nie machen. Man hat ja bereits eine boolesche Variable und sollte diese direkt nutzen.


Um ehrlich zu sein:
Zumindest in der jetzigen Form passt der Code meiner Meinung nach nicht gut in die CodeLibrary. Dazu gibt es mehrere Gründe - hauptsächlich wären dies die Tatsache, dass der Code nicht kommentiert ist, aber was schwerer wiegt ist die Tatsache, dass hier Code und Darstellung recht stark miteinander verknüpft sind.
Soll heißen: Der Code liegt quasi komplett in Onclick-Prozeduren und ist daher schlecht wiederverwendbar.



Bitte unterlasse es in Zukunft innerhalb kurzer Zeit (= weniger als 24h) zu Pushen - wir haben ja noch ein Leben außerhalb der DP.

Liebe Grüße,
Frederic
Frederic Kerber
  Mit Zitat antworten Zitat