if-Abfrage vereinfachen

[Gruber_Hans_12345]

Zitat von MathiasSimmack:

Ich habe leider kein Delphi zum Prüfen, aber der numerische Wert von "pmBlack" ist dann wohl Null?

ja ist 0 ... ansonsten
FImageEnVect.Bitmap.Canvas.Pen.Mode := TPenMode(integer((not Show)) * (integer(pmNotXor) - integer(pmBlack)) + integer(pmBlack));

[Gruber_Hans_12345]

verwendet pixels überhaupt den PenMode? ... mal googeln

[edit]ok, verwendets


Aber mal eine andere Frage, was sollte den die Funktion genau machen?
du zeichnest einmal direkt schwarz drauf und einmal mit XOR ... ist irgendwie unlogisch für mich, entweder immer per schwarz, wenn das raster auf eine leere fläche gezeichnet wird, oder immer per XOR, wenn das raster über bestehende Grafik gezeichnet wird, und mit einem 2ten xor dann wieder wegmalen das Raster.

[Robert Marquardt]

Ein bischen OT, aber die Funktion ist falsch unterteilt.
DrawDots sollte nicht in eine lokale Funktion ausgelagert werden, da es ja das eigentliche Herzstueck der Methode ist.
Der Pen.Mode hat uebrigens nicht den geringsten Einfluss auf das Setzen der Pixel. Der Pen ist fuer das Zeichnen von Linien verantwortlich.
Was soll uebrigens das Update bewirken? Ist hier das TDWFotoBook.Update gemeint oder FImageEnVect.Update?
Schmeiss das with raus, da es nur Probleme bereitet, aber nicht zur Lesbarkeit beitraegt.

Nachgeprueft an der TCanvas-Implementierung: Es wird Windows.SetPixel aufgerufen das nach Windows-Hilfe keinen Pen.Mode beruecksichtigt.

[Christian Seehase]

Moin Michael,

also ich würde die "Abfrage" so machen:

Delphi-Quellcode:

const
  _aPenMode : array [boolean] of TPenMode = (pmNotXor,pmBlack);

FImageEnVect.Bitmap.Canvas.Pen.Mode := _aPenMode[FShow];

[MathiasSimmack]

Erinnert mich daran, dass ich solche bool'schen Arrays immer gern eingesetzt habe, weil ich auf die C-Variante

Code:

Pen.Mode = (Show)?pmBlack:pmNotXor; // fiktiver Code

neidisch war. Ich glaube, der Gedanke mit dem Array kam von Hagen. Die Variante hätte den Vorteil, dass man anstelle von "pmBlack" auch was anderes nehmen könnte. Diese Sache hier ist nämlich auch klasse, funktioniert aber eben bloß mit pmBlack (= Null) und einem anderen Wert.

[Gruber_Hans_12345]

Zitat von Robert Marquardt:

Der Pen.Mode hat uebrigens nicht den geringsten Einfluss auf das Setzen der Pixel. Der Pen ist fuer das Zeichnen von Linien verantwortlich.

sicher? - ich habs getestet und der pen.mode ist sehr wohl auch für das SetPixel zuständig

[Robert Marquardt]

Eigentlich ja. Ich kann mir nicht vorstellen das Microsoft das zu dokumentieren vergessen haette wenn der Mode Einfluss auf SetPixel haette.
Es macht ja auch keinen Sinn, da der Pen nur zum Zeichnen von Linien zustaendig ist.

[Luckie]

Wenn TPen.Mode keinen Einfluss auf SetPixel hat, warum werden sie dann einmal gemalt und einmal nicht?

Sinn ist der: Ich habe ein Grid zum Ausrichten von Objekten. Dieses soll man anzeigen und nicht nazeigen können. Wenn es angezeigt wird, muss ich es eben entsprechend übermalen, um es wieder verschwiden zu lassen. Dabei kann der Hintergrund jede beliebige Frabe an jeder beliebigen Stelle haben.

[Gruber_Hans_12345]

Zitat von Luckie:

Wenn TPen.Mode keinen Einfluss auf SetPixel hat, warum werden sie dann einmal gemalt und einmal nicht?

Sinn ist der: Ich habe ein Grid zum Ausrichten von Objekten. Dieses soll man anzeigen und nicht nazeigen können. Wenn es angezeigt wird, muss ich es eben entsprechend übermalen, um es wieder verschwiden zu lassen. Dabei kann der Hintergrund jede beliebige Frabe an jeder beliebigen Stelle haben.

um es verschwinden zu lassen, braucht man normalerweise ja nur ein zweites mal mit der selben farbe mit xor drübermalen
also erstes mal mit clBlack und Xor malen -> Grid ist da, zweites mal mit clBlack und Xor malen -> Grid ist weg, das wäre ja der sinn von den xor sachen beim malen.

@Robert: ich war auch der meinung, das es keinen einfluß hat, und auch wenns ms nicht dokumentiert hat, es hat einen einfluß, habs extra vorher noch getestet.

[Robert Marquardt]

pmBlack ist Unsinn. Es bedeutet das immer mit Schwarz gemalt wird, egal was fuer eine Farbe man nimmt. Bitte ausprobieren ob mit einer anderen Farbe als clBlack die andere Farbe benutzt wird oder Schwarz. Andere Farbe heisst das der Pen.Mode keinen Einfluss hat.