NullReferenceException bei Aufruf einer Methode

[Schlumpf (Gast)]

Hallo, beim Ausführen des unten gezeigten Programmteiles gibt es eine 
Fehlermeldung (als Kommentar angefügt).
Der Fehler ist zwar behoben, wenn ich, wie es VisualStudio vorschlägt, 
eine neue Instanz anlege, aber ich will mich ja auf genau das Objekt 
beziehen, was ich in "twincat_ads_vorbereitung()" erstellt habe.
Was muss ich tun, damit der Aufruf auch in einer anderen Methode 
funktioniert?

public partial class Form1
    {
        public TcAdsClient adsClient;
        private int handle_start;
        bool aktivieren = false;
        
        public static void form_geladen()
        {
            Form1 instanz = new Form1();    
            instanz.twincat_ads_vorbereitung();   
        }

        public void twincat_ads_vorbereitung()
        {
            adsClient = new TcAdsClient();

            try
            {
                adsClient.Connect(851);
                handle_start = adsClient.CreateVariableHandle("MAIN.aktivieren");
            }
            catch (Exception ausnahme)
            {
                MessageBox.Show("Fehlerquelle: " + ausnahme.Source+
                                "\nBeschreibung: "+ausnahme.Message,
                                "Fehler ADS-Vorbereitung", MessageBoxButtons.OK,
                                MessageBoxIcon.Error);
            }
        }

        public void twincat_ads_dispose()
        {
            MessageBox.Show("TwinCAT ADS", "Beenden", MessageBoxButtons.OK, MessageBoxIcon.Hand);
            adsClient.Dispose(); //System.NullReferenceException wurde nicht behandelt.Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.

        }
    }
}

[Sam P. (Gast)]

Hast du geprüft, ob ..._vorbereitung auch wirklich vor ..._dispose 
aufgerufen wurde und Erfolg hatte? Wird ..._dispose vielleicht zweimal 
aufgerufen?

Einfache printf-Statements (ich kenne C# nicht, ersetze durch was 
gleichwertiges) am Anfang der Funktionen könnten dir weiterhelfen.

[Schlumpf (Gast)]

ja, die Reihenfolge ist geprüft und war auch richtig. Ich habe das 
Programm noch etwas umgestellt und mal einen Schreibzugriff ausprobiert. 
Das funktioniert auch wunderbar - solange ich in einer Methode bleibe -. 
Ich würde aber gerne aus mehreren Methoden auf das Handle zugreifen.

    class Funktionen
    {
        private TcAdsClient adsClient = new TcAdsClient();
        
        private int handle_start;
        bool aktivieren = false;

        public static void form_geladen()
        {         
            Funktionen instanz = new Funktionen();   
            instanz.twincat_ads_vorbereitung();
        }

        public void twincat_ads_vorbereitung()
        {
            try
            {
                adsClient.Connect(851);
                handle_start = adsClient.CreateVariableHandle("MAIN.aktivieren");
                adsClient.WriteAny(handle_start, true); //Hier funktioniert es, SPS läuft los
                Console.WriteLine("Geöffnet\n");
            }
            catch (Exception ausnahme)
            {
                MessageBox.Show("Fehlerquelle: " + ausnahme.Source+
                                "\nBeschreibung: "+ausnahme.Message,
                                "Fehler ADS-Vorbereitung", MessageBoxButtons.OK,
                                MessageBoxIcon.Error);
            }
        }

        
        public static void ads_schreiben()
        {
            Funktionen inst = new Funktionen();
            inst.adsClient.WriteAny(inst.handle_start, inst.aktivieren);
            //HIER FEHLER
        }
    }
}

[Warhawk (Gast)]

Da nicht der ganze Quelltext vorliegt, kann ich nur ratten.
Du legst eine neue Instanz von Funktionen an, rufst aber deine 
Vorbereitenmethode nie für dieses Objekt auf.

Versuch mal folgendes:

Funktionen inst = new Funktionen();
inst.twincat_ads_vorbereitung();
inst.adsClient.WriteAny(inst.handle_start, inst.aktivieren);

Grüße, Warhawk

[Schlumpf (Gast)]

Danke, damit scheint es zu funktionieren!

Wird dabei dann jedes mal ein neues Handle erzeugt? Ich will ja 
eigentlich nur einen adsClient haben, mit dem ich mich auch nur einmal 
verbinde. Die anderen MEthoden sollen nur darauf zugreifen.

[Warhawk (Gast)]

Ok dann mach mal folgendes:

    class Funktionen
    {
        private static Funktionen m_Inst = null;

        private TcAdsClient adsClient = new TcAdsClient();
        
        bool aktivieren = false;

        public static void form_geladen()
        {         
            m_Inst = new Funktionen();   
            m_Inst.twincat_ads_vorbereitung();
        }

        public void twincat_ads_vorbereitung()
        {
            try
            {
                adsClient.Connect(851);
                handle_start = adsClient.CreateVariableHandle("MAIN.aktivieren");
                adsClient.WriteAny(handle_start, true); //Hier funktioniert es, SPS läuft los
                Console.WriteLine("Geöffnet\n");
            }
            catch (Exception ausnahme)
            {
                MessageBox.Show("Fehlerquelle: " + ausnahme.Source+
                                "\nBeschreibung: "+ausnahme.Message,
                                "Fehler ADS-Vorbereitung", MessageBoxButtons.OK,
                                MessageBoxIcon.Error);
            }
        }

        
        public static void ads_schreiben()
        {
            m_Inst.adsClient.WriteAny(inst.handle_start, inst.aktivieren);
            //HIER FEHLER
        }
    }

Das ist aber aus Objekt orientierter Sicht nicht ganz sauber.
Ich würde ohne die static-Methoden arbeiten und die Referenz auf die 
Instanz von Funktionen an alle Klassen übergeben die damit arbeiten 
sollen.

Du legst in deiner Klasse für die Anwendungslogik eine Instanz von 
Funktionen an, speicherst sie in einer Membervariable und reicht sie von 
dort aus an alle Klassen weiter die damit arbeiten sollen.