GDI+ und ColorDialog, Problem bei der Farbänderung einer Shape (Line)

  • C#
  • .NET (FX) 4.5–4.8

Es gibt 4 Antworten in diesem Thema. Der letzte Beitrag () ist von Duster.

    GDI+ und ColorDialog, Problem bei der Farbänderung einer Shape (Line)

    Hallo Leute,
    ich hoffe alle sind gesund?! :)

    Ich habe folgendes Problem:
    Ich möchte die Farbe von Shapes die ich mit GDI+ erstellt habe mit durch einen ColorDialog ändern.
    Die Farbe ändert sich auch aber total falsch, es sieht so aus als würden die R, G, B -Werte total wild durcheinander gemixt!?
    Ich hab ein kleines Beispielprg. erstellt bei dem der Code genauso stukturiert ist und das gleiche Problem auftritt. Ich hab versucht den Code so kurz zu halten wie es geht.
    Das Projekt hab ich natürlich auch angehängt!

    Code in der Form:

    C#-Quellcode

    1. using System;
    2. using System.Drawing;
    3. using System.Drawing.Drawing2D;
    4. using System.Windows.Forms;
    6. namespace ColorDialog_Problem
    7. {
    8. public partial class Form1 : Form
    9. {
    10. private bool _drawState = true;
    11. private string _shapeName = "Line";
    13. private Line _line = new();
    14. private Point _lineCoords;
    15. private int _alpha = 255;
    16. private int _red = 255;
    17. private int _green = 201;
    18. private int _blue = 14;
    21. public Form1()
    22. {
    23. InitializeComponent();
    24. }
    26. protected override void OnPaint(PaintEventArgs e)
    27. {
    28. base.OnPaint(e);
    29. _drawState = true;
    31. _line.Draw(e.Graphics);
    32. }
    33. private void CmdStart_Click(object sender, EventArgs e)
    34. {
    35. if (DlgColor.ShowDialog() == DialogResult.OK)
    36. {
    37. _red = DlgColor.Color.R;
    38. _blue = DlgColor.Color.G;
    39. _green = DlgColor.Color.B;
    41. switch (_shapeName)
    42. {
    43. case "Line":
    44. _line = new Line(_alpha, _red, _green, _blue, 10, DashStyle.Dash)
    45. { LineLocation = _lineCoords, Size = new Size(200, 0) };
    46. break;
    47. default:
    48. break;
    49. }
    50. }
    51. Invalidate();
    52. }
    53. private void Form1_Load(object sender, EventArgs e)
    54. {
    55. InitializeShapes();
    56. }
    57. public void InitializeShapes()
    58. {
    59. _lineCoords = new Point(300, 200);
    61. _line = new Line(_alpha, _red, _green, _blue, 10, DashStyle.Dash)
    62. { LineLocation = _lineCoords, Size = new Size(200, 0) };
    63. }
    64. }
    65. }


    Code in der Klasse Shape:

    C#-Quellcode

    1. using System;
    2. using System.Drawing;
    3. using System.Drawing.Drawing2D;
    5. namespace ColorDialog_Problem
    6. {
    7. public abstract class Shape
    8. {
    9. public Point LineLocation;
    11. public abstract string Name { get; set; }
    12. public int Alpha { get; set; }
    14. public int Width { get; set; }
    15. public DashStyle DStyle { get; set; }
    16. public Size Size { get; set; }
    18. public Pen Pen;
    19. public abstract void Draw(Graphics e);
    20. }
    21. }


    Code in der Klasse Line:

    C#-Quellcode

    1. using System.Drawing;
    2. using System.Drawing.Drawing2D;
    4. namespace ColorDialog_Problem
    5. {
    6. class Line : Shape
    7. {
    8. public override string Name
    9. {
    10. get => "Line";
    11. set { }
    12. }
    14. public Line() { }
    16. public Line(int alpha, int red, int green, int blue, int width, DashStyle dStyle)
    17. {
    18. Pen = new Pen(Color.FromArgb(alpha, red, green, blue), width)
    19. {
    20. DashStyle = dStyle
    21. };
    22. }
    23. public override void Draw(Graphics e)
    24. {
    25. e.DrawLine(Pen, LineLocation, LineLocation + Size);
    26. }
    27. }
    28. }


    Wäre super wenn Ihr mir helfen könntet!
    Vielen Dank vorab für Eure Mühe!!!!
    ;)
    Duster
    Dateien

    Zeile#38, #39: so ist es momentan

    C#-Quellcode

    1. _blue = DlgColor.Color.G;
    2. _green = DlgColor.Color.B;
    Wie es sein sollte, findest Du leicht raus. Schau nochmal genau hin ;)
    Dieser Beitrag wurde bereits 5 mal editiert, zuletzt von „VaporiZed“, mal wieder aus Grammatikgründen.

    Aufgrund spontaner Selbsteintrübung sind all meine Glaskugeln beim Hersteller. Lasst mich daher bitte nicht den Spekulatiusbackmodus wechseln.

    @Duster Da hilft tatsächlich nur Debuggen.
    Nicht glauben, was der Code tut, sondern es wissen:
    Debuggen, Fehler finden und beseitigen
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!