Bild in Graustufen umwandeln

[Rollo62]

50 Shades of gray

Und welches ist jetzt das "offiziell" Richtige

[Neutral General]

Zitat von Celi:

Danke für den Hinweis, doch leider ändert auch das nichts an meinem Problem.

Poste mal den Code wie er jetzt nach der Änderung ist.
Du kannst den Code auch hier als Text einfügen mit [delphi]-Tags drum rum. Dann hast du hier auch Syntax Highlighting

[Redeemer]

Mir ist das ganze sowieso suspekt. Mal ein paar Dinge:

• Als erster Post im Forum ein Post zum Thema Zeigerarithmetik ist seltsam.
• Totengräberei.
• Warum ^TRGBTriple und nicht PBytes wie jeder normale Mensch es nutzen würde?
• Was hat die JPEG-BMP-Methode im Screenshot suchen? (JPEG als Graustufen speichern ist extrem ineffizient. Korrekt wäre PNG Typ 0.)
• Assign ohne var halte ich für kritisch.
• Assign auf sich selbst ergibt keinen Sinn.
• Graustufen produziert ein riesiges Speicherleck.
• Was soll das mit dem Pixelformat überhaupt bringen?
• Inc müsste wenn überhaupt als zweiten Parameter 3 haben oder nicht?

[Darlo]

Zitat von Redeemer:

Mir ist das ganze sowieso suspekt. Mal ein paar Dinge:

• Als erster Post im Forum ein Post zum Thema Zeigerarithmetik ist seltsam.
• Totengräberei.
• Warum ^TRGBTriple und nicht PBytes wie jeder normale Mensch es nutzen würde?
• Was hat die JPEG-BMP-Methode im Screenshot suchen? (JPEG als Graustufen speichern ist extrem ineffizient. Korrekt wäre PNG Typ 0.)
• Assign ohne var halte ich für kritisch.
• Assign auf sich selbst ergibt keinen Sinn.
• Graustufen produziert ein riesiges Speicherleck.
• Was soll das mit dem Pixelformat überhaupt bringen?
• Inc müsste wenn überhaupt als zweiten Parameter 3 haben oder nicht?

Du hast unter Deinen Bulletpoints Dein Fazit vergessen, oder war das schon im ersten Satz?!

[Neutral General]

Zitat von Redeemer:

Mir ist das ganze sowieso suspekt. Mal ein paar Dinge:

• Warum ^TRGBTriple und nicht PBytes wie jeder normale Mensch es nutzen würde?

Ich gebe dir da bei allen Punkten Recht, aber TRGBTriple ist ein Record extra für diesen Zweck.
Von daher würde ich dir da widersprechen.

[Redeemer]

Zitat von Neutral General:

Zitat von Redeemer:

Mir ist das ganze sowieso suspekt. Mal ein paar Dinge:

• Warum ^TRGBTriple und nicht PBytes wie jeder normale Mensch es nutzen würde?

Ich gebe dir da bei allen Punkten Recht, aber TRGBTriple ist ein Record extra für diesen Zweck.
Von daher würde ich dir da widersprechen.

Aber TRGBTriple ist ein einzelnes Element. Ich würde ja eher einen Array-Typ nehmen. Zur Not eben Vcl.Imaging.GIFImg.PBGRArray oder so.

[Blup]

• In "Graustufen()" gehört das "end" der äußeren "for"-Schleife hinter das "end" der inneren "for"-Schleife.
  (Alles ordentlich einrücken, dann passieren solche Fehler nicht.)
• In "Graustufen()" "bitmap.Assign(bitmap);" ist Unsinn und eine mögliche Fehlerquelle.
• In "B_GraustufenClick() zum Schluss "image1.Invalidate" aufrufen, um die Änderung sichtbar zu machen.
• Das Setzen des Pixelformats gehört eher in "Graustufen()".

[bernau]

Zitat von Celi:

Mein Quellcode scheint soweit richtig zu sein, das Programm lässt sich ohne weitere Fehlermeldungen öffnen. Allerdings passiert leider gar nichts, wenn man meinen Button benutzt.

Schon mal mit dem Debugger schritt für Schritt durch den Code gegangen? Damit wirst du schnell sehen, welches End an der falschen stelle steht.

[bernau]

Zitat von Redeemer:

Mir ist das ganze sowieso suspekt. Mal ein paar Dinge:

• Assign ohne var halte ich für kritisch.

Warum? Das musst du mir erklären.

[Dennis07]

Zitat von Redeemer:

Als erster Post im Forum ein Post zum Thema Zeigerarithmetik ist seltsam.

????

Zitat von Redeemer:

Warum ^TRGBTriple und nicht PBytes wie jeder normale Mensch es nutzen würde?

Wattt? Wenn "jeder normale Mensch" für dich nur du selber bist, dann vielleicht. Aber alle "normalen Menschen" die ich kenne benutzen TRGBTriple.

Zitat von Redeemer:

Assign ohne var halte ich für kritisch.

Wiebitte? Was genau meinst du?

Ansonsten kann ich dir nur zustimmen. Aber wäre wirklich nett, wenn du mir mal etwas Erleuchtung bringst, ich blicke da manchmal nicht recht was du sagen willst.