Linie in WPF zeichnen (Code behind)

[Joe (Gast)]

Hallo Leute,

versuche mit Visual C# 2010 eine Linie zu zeichnen (wpf-Projekt).
Mein Code (MainWindow.xaml.cs) sieht folgendermaßen aus;

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

namespace DrawSomeLines
{
    /// <summary>
    /// Interaktionslogik für MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();


            Line myLine = new Line();
            Grid myGrid = null;
            myLine.Stroke = System.Windows.Media.Brushes.LightSteelBlue;
            myLine.X1 = 1;
            myLine.X2 = 50;
            myLine.Y1 = 1;
            myLine.Y2 = 50;
            myLine.HorizontalAlignment = HorizontalAlignment.Left;
            myLine.VerticalAlignment = VerticalAlignment.Center;
            myLine.StrokeThickness = 2;
            myGrid.Children.Add(myLine);



        }
    }
}

Mit F5 starte ich das Debugging und erhalte folgende Fehlermeldung:

"TargetInvocationException wurde nicht behandelt"


Was mache ich falsch?

GRÜSSE, JOE

[Borislav B. (boris_b)]

Um's kurz zu machen: myGrid ist null

[jmp (Gast)]

Grid myGrid = null;
und
myGrid.Children.Add(myLine);

==> myGrid ist nicht instantiiert

--jmp

[Joe (Gast)]

OK, folgender Code kompiliert jetzt. Sehe aber keine Linie.

public MainWindow()
        {
            InitializeComponent();


            Line myLine = new Line();
            Grid myGrid = new Grid();
            myLine.Stroke = System.Windows.Media.Brushes.Black;
            myLine.X1 = 150;
            myLine.X2 = 50;
            myLine.Y1 = 150;
            myLine.Y2 = 50;
            myLine.HorizontalAlignment = HorizontalAlignment.Left;
            myLine.VerticalAlignment = VerticalAlignment.Center;
            myLine.StrokeThickness = 4;
            myGrid.Children.Add(myLine);
        }

[jmp (Gast)]

1. Normalerweise werden Zeichenfunktionen in einem OnDraw-Event 
ausgeführt.
2. Deine Linie scheint mir ein Punkt zu sein (gleiche x- und 
y-Koordinaten)

--jmp

[Borislav B. (boris_b)]

Joe schrieb:
> Sehe aber keine Linie.

Mit deinem Grid machst du ja auch nichts. Das wird gleich wieder vom 
Garbage Collector gefressen wenn die Funktion verlassen wird. Klar dass 
da nichts gezeichnet wird...

[Joe (Gast)]

Ich dachte, dass in folgender Zeile "myGrid" verwendet wird:

myGrid.Children.Add(myLine);

[Karl H. (kbuchegg)]

Joe schrieb:
> Ich dachte, dass in folgender Zeile "myGrid" verwendet wird:
>
> myGrid.Children.Add(myLine);

Da steht:
füge die Linie zum Grid dazu.

Gut.

Aber was ist mit dem Grid selber?
Woher soll das Window wissen, dass du diesen einen Grid angezeigt haben 
willst? Du kannst tausende Gridsd haben, jeder mit anderen Linien. Aber 
welche davon sollen denn jetzt gemalt werden und welche nicht?

[Jonas B. (jibi)]

Dir fehlen ganz klar die Grundlagen in C# und WPF.
Die solltest du dir erstmal aneignen.

Gruß Jonas

[Joe (Gast)]

Ok, habe verstanden.

public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();            
        }

        private void Window_Loaded(object sender, RoutedEventArgs e)
        {
            Line myLine = new Line();
            Grid myGrid = new Grid();
            myGrid.Width = 400;
            myGrid.Height = 400;


            myLine.Stroke = System.Windows.Media.Brushes.Black;
            myLine.Fill = System.Windows.Media.Brushes.Black;
            myLine.X1 = 100;
            myLine.X2 = 150;
            myLine.Y1 = 100;
            myLine.Y2 = 150;
            //myLine.HorizontalAlignment = HorizontalAlignment.Left;
            //myLine.VerticalAlignment = VerticalAlignment.Center;
            myLine.StrokeThickness = 4;
            myGrid.Children.Add(myLine);

            myGrid.ShowGridLines = true;

            this.Content = myGrid;
            this.Show();
        }

        
    }

Ist die verwendete Lösung sinnvoll oder wie (bzw. in welcher 
Ereignis-Funktion) integriere ich meinen Code?


GRÜSSE, JOe

[Borislav B. (boris_b)]

Joe schrieb:
> Ist die verwendete Lösung sinnvoll

Nein. Das Konzept von WPF ist es Code und GUI zu TRENNEN. D.h. deine GUI 
gehört in die XAMLs, der Code in die CSs. Das manuelle Zeichnen per Code 
ist nicht sinnvoll.

[Jonas B. (jibi)]

Also wenn die Linie da immer gezeichnet werden soll, also quasi Teil der 
Gui wird - dann gehört die in ein XAML Document, wird also nur 
beschrieben. Wie
die dann gezeichnet wird, darum kümmert sich die Grafikengine, die dir 
WPF freundlicherweise spendiert.

Gruß Jonas

[Joe (Gast)]

Ok, sehe ich ein. Ich versuche die Linie zu bewegen, daher das Zeichnen 
im C#-Code

public partial class MainWindow : Window
    {       

        private Line myLine = new Line();
        private Grid myGrid = new Grid();              


        public MainWindow()
        {
            InitializeComponent();            
        }

        private void Window_Loaded(object sender, RoutedEventArgs e)
        {
            Timer myTimer = new Timer(3000);

            myTimer.Elapsed += new ElapsedEventHandler(TimerElapsed);
            myTimer.Enabled = true;

            myLine.Stroke = System.Windows.Media.Brushes.Black;
            myLine.Fill = System.Windows.Media.Brushes.Black;
            myLine.StrokeThickness = 4;
            myGrid.Children.Add(myLine);
            myGrid.ShowGridLines = true;

            this.Content = myGrid;
            this.Show();           
            
        }


        static void TimerElapsed(object sender, ElapsedEventArgs e)
        {
            DrawLine();
        }


        private void DrawLine()
        {
            if (this.myLine.X1 < 1900)
                this.myLine.X1 = this.myLine.X1 + 100;
            else
                this.myLine.X1 = 0;
        }
        
    }

Ich erhalte die Fehlermeldung:

Fehler  1  Für das nicht statische Feld, die Methode oder die 
Eigenschaft "DrawSomeLines.MainWindow.DrawLine()" ist ein Objektverweis 
erforderlich.  C:\Users\tt.GD\Documents\Visual Studio 
2010\Projects\DrawSomeLines\DrawSomeLines\MainWindow.xaml.cs  54  13 
DrawSomeLines


Anscheinend kann ich aus einer statischen Methode keine nicht-statische 
Methode aufrufen?

GRÜSSE; Joe

[Jonas B. (jibi)]

>Anscheinend kann ich aus einer statischen Methode keine nicht-statische
>Methode aufrufen?

>>Dir fehlen ganz klar die Grundlagen in C# und WPF.
>>Die solltest du dir erstmal aneignen.

Gruß Jonas

[Jonas B. (jibi)]

>Timer myTimer = new Timer(3000);

Und wieder hast du den Timer in einem Event definiert, sobald das Event 
abgearbeitet wurde frisst der GC deinen Timer...

Gruß Jonas

[Joe (Gast)]

Mit diesem Beispiel versuche ich meine Grundlagen zu schaffen.


So falsch kann der Zähler nicht sein. Wenn ich statt des 
DrawLine-Aufrufes
mir eine MessageBox anzeigen lasse, so klappt das auch im 3sec-Abstand.

Kann mir jemand erklären, was bei dem Aufruf von DrawLine() falsch ist?

GRÜSSE, Joe

[Joe (Gast)]

Jonas, wie würdest Du es richtig machen?

• Wo würdest Du den Timer definieren?
• Wo würdest Du nach Ablauf des Timers die neue X-Koordinate zeichnen?

[Borislav B. (boris_b)]

Wenn du Animationen machen willst, geht das auch ohne so einen seltsamen 
Workaround. Das bietet WPF quasi schon von Haus aus:

http://msdn.microsoft.com/de-de/library/ms752312%28v=vs.110%29.aspx

[Joe (Gast)]

Danke für den Link. Bevor ich mir den genauer anschaue, würde ich gerne 
mein Beispiel hinbekommen, mit den gezeigten Teilen.


GRÜSSE, JOE

[Karl H. (kbuchegg)]

Joe schrieb:
> Mit diesem Beispiel versuche ich meine Grundlagen zu schaffen.
>

Dann empfehle ich dir dringend den Ankauf von Literatur.
Du wirst hier im Forum niemanden finden, der dir einen 300 Seiten C# 
Schmöker anlassbezogen vorbetet.


> So falsch kann der Zähler nicht sein. Wenn ich statt des
> DrawLine-Aufrufes
> mir eine MessageBox anzeigen lasse, so klappt das auch im 3sec-Abstand.

Das hängt ganz davon ab, wann der Garbage Collector dann irgendwann mal 
das Timer-Objekt entsorgt.

> Kann mir jemand erklären, was bei dem Aufruf von DrawLine() falsch ist?

die Funktion ist static, gehört also zu keinem spezifischen Objekt, 
sondern hängt nur allgemein so in der Klasse rum, damit sie im 
Namensraum der Klasse existiert. Aber wenn sie aufgerufen wird, dann 
wird sie nicht für irgend ein spezifisches Objekt aufgerufen. Ergo 
kannst du auch nicht auf nicht-statische Dinge in deiner Klasse 
zugreifen, denn die sind ja an ein einziges, bestimmtes Objekt gebunden 
(jedes jemals derartige Objekt hat seine eigenen Satz an 
nicht-statischen Variablen).


> • Wo würdest Du den Timer definieren?

Den Timer muss es geben, so lange es das Window gibt.
Genauso wie den Grid oder (bei dir momentan) die Linie.
Also: Wo und wie hast du den Grid und die Linie definiert bzw. erzeugt 
und warum hast du das mit dem Timer nicht genauso gemacht?

[Joe (Gast)]

OK, danke für Eure Vorschläge und Erläuterungen.


Folgender modifizierter Code lässt sich starten, hat dann aber einen 
Laufzeitfehler:

namespace DrawSomeLines
{
    /// <summary>
    /// Interaktionslogik für MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {       

        private Line myLine = new Line();
        private Grid myGrid = new Grid();              


        public MainWindow()
        {
            InitializeComponent();            
        }

        private void Window_Loaded(object sender, RoutedEventArgs e)
        {
            Timer myTimer = new Timer(3000);

            myTimer.Elapsed += new ElapsedEventHandler(TimerElapsed);
            myTimer.Enabled = true;

            myLine.Stroke = System.Windows.Media.Brushes.Black;
            myLine.Fill = System.Windows.Media.Brushes.Black;
            myLine.StrokeThickness = 4;
            myLine.X1 = 100;
            myLine.Y1 = 200;
            myGrid.Children.Add(myLine);
            myGrid.ShowGridLines = true;

            this.Content = myGrid;
            this.Show();           
            
        }


        private void TimerElapsed(object sender, ElapsedEventArgs e)
        {
            DrawLine();            
            
        }


        private void DrawLine()
        {
            if (this.myLine.X1 < 1900)   //(*)
                this.myLine.X1 = this.myLine.X1 + 100;
            else
                this.myLine.X1 = 0;
        }
        
    }
}

Fehlermeldung: (*)
Der aufrufende Thread kann nicht auf dieses Objekt zugreifen, da sich 
das Objekt im Besitz eines anderen Threads befindet.



GRÜSSE, JOE

[Borislav B. (boris_b)]

Joe schrieb:
> Der aufrufende Thread kann nicht auf dieses Objekt zugreifen, da sich
> das Objekt im Besitz eines anderen Threads befindet.

Dann weißt du ja wo das Problem liegt und kannst es beheben.

Willst jetzt hier wirklich mit JEDEM Problem aufschlagen anstatt einfach 
mal ein Tutoarial/Buch durchzuarbeiten?

[Karl H. (kbuchegg)]

Tja.
da wird dann eben wahrscheinlich ein Invoke fehlen.

Vielleicht verstehst du jetz schön langsam, warum die Methode 'Ich 
probier einfach so lange rum bis xyz (zutreffendes einsetzen)' nicht 
funktioniert. Es gibt hunderte (wenn nicht sogar tausende) Dinge, die du 
wissen und beherrschen musst. Und nur wenn du alle beherrscht, dann 
landest du auch bei einem sauberen Programm. Nur: woher sollst du denn 
das alles wissen, was es zu beachten gilt? Eben. Du kannst es nicht 
wissen. Aber dein Buch weiß es.

[Joe (Gast)]

Hmmm, ok. Ich habe ein Buch: "Einstieg in Visual C# 2010"

Dort finde ich leider nicht viel zu WPF und auch nicht wie
ich Multi-Threading mache. Die Frage ist doch, wie kann ich obiges 
Beispiel
sauber und simpel lösen. Wenn ich Schritte zur Lösung erfahre, dann 
lerne ich
doch am besten oder nicht? Weil ich dann nämlich weiß, wie man es 
richtig macht.

>Es gibt hunderte (wenn nicht sogar tausende) Dinge, die du
>wissen und beherrschen musst.

JA, das gebe ich Dir Recht. Man muss aber am lebenden Beispiel erfahren 
und lernen, was man alles falsch machen kann. Die Beispiele im Buch sind 
immer
sehr simpel und funktionieren natürlich immer.

Grüsse, JOE

[Karl H. (kbuchegg)]

Joe schrieb:
> Hmmm, ok. Ich habe ein Buch: "Einstieg in Visual C# 2010"
>
> Dort finde ich leider nicht viel zu WPF und auch nicht wie
> ich Multi-Threading mache.

Ich kann mir nicht vorstellen, dass Invoke da drinnen nicht beschrieben 
ist. Denn das, und das Zusammenspiel mit Delegates, braucht man 
eigentlich alle Nase lang, wenn man C# mit .Net einsetzt.

Insofern hat das nichts mit WPF zu tun. WPF ist ja nur ein Framework, 
welches dir eine bestimmte Organisation vorgibt. Die Basismechanismen, 
wie die Dinge in C# funktionieren sind ja nach wie vor dieselben.

> JA, das gebe ich Dir Recht. Man muss aber am lebenden Beispiel erfahren
> und lernen, was man alles falsch machen kann. Die Beispiele im Buch sind
> immer
> sehr simpel und funktionieren natürlich immer.

Die Beispiele schweben aber nicht im freien Raum, sondern sind in 
Kapitel eingebettet in denen Erläuterungen stehen. Das wichtige sind ja 
nicht die Beispiele, sondern dass man versteht, worum es im jeweiligen 
Kapitel geht.
D.h. man muss so ein Buch auch durcharbeiten! Kapitel für Kapitel lesen, 
darüber nachdenken was man gelesen hat, die Beispiele als Illustration 
des gelesenen nehmen und im Idealfall dann auch ein wenig abwandeln bzw. 
zur Selbstkontrolle selbst ein paar Beispiele dazu erfinden, wenn am 
Kapitelende keine Übungsbeispiele angegeben sind.

Dann wird das auch was.

[Joe (Gast)]

>Ich kann mir nicht vorstellen, dass Invoke da drinnen nicht beschrieben
>ist. Denn das, und das Zusammenspiel mit Delegates, braucht man
>eigentlich alle Nase lang, wenn man C# mit .Net einsetzt.

Zumindest im Stichwortverzeichnis finde ich "Invoke" und "Delegates" 
nicht.

[Eugler (Gast)]

Joe schrieb:
>>Ich kann mir nicht vorstellen, dass Invoke da drinnen nicht
> beschrieben
>>ist. Denn das, und das Zusammenspiel mit Delegates, braucht man
>>eigentlich alle Nase lang, wenn man C# mit .Net einsetzt.
>
> Zumindest im Stichwortverzeichnis finde ich "Invoke" und "Delegates"
> nicht.

http://technet.microsoft.com/de-de/

Manchmal verstehe ich es nicht ... .

[Joe (Gast)]

Manchmal verstehe ich die Arroganz einiger Forums-"Teilnehmer" auch 
nicht ;-)