C#: Fehlermeldung NullReferenceException

[csharp_user (Gast)]

Guten Morgen,

wie kann in einer anderen Methode auf das Objekt imageStream zugeriffen 
werden? Nach Ausführung der Buttonmethode btn_Load_Click soll der Button 
btn_SendImage_Click ausgeführt werden. Bei der folgenden Zeile,

while ((bytesSize = imageStream.Read(downBuffer, 0, downBuffer.Length)) > 0)

 zeigt Visual Studio einen Fehler an. Es handelt sich hierbei um einen 
NullReferenceException Fehler. Was könnte ich tun, damit ich auch in der 
Methode btn_SendImage_Click auf das Objekt imageStream zugreifen kann?

public partial class FormSendImage : Form
    {
        FileStream imageStream;

        public FormSendImage()
        {
            InitializeComponent();
        }

        public void btn_SendImage_Click(object sender, EventArgs e)
        {
            int bytesSize = 0;
            byte[] downBuffer = new byte[32 - 2];
            byte[] sendBuffer = new byte[32];


            int chunkNumber = 0;
            while ((bytesSize = imageStream.Read(downBuffer, 0, downBuffer.Length)) > 0)
            {

            }

            imageStream.Close();
        }

        public void btn_Load_Click(object sender, EventArgs e)
        {
            if (ImageOpenFileDialog.ShowDialog() == DialogResult.OK)
            {
                using (FileStream imageStream = new FileStream(ImageOpenFileDialog.FileName, FileMode.Open, FileAccess.Read))
                {

                }
            }
        }
    }

[JensM (Gast)]

Der Zugriff auf imageStream ist in der Funktion möglich.
Alles korrekt.

Das Objekt imageStream ist noch null (NullReferenceException).

Es muss irgendwo vor dem ersten Zugriff ein Objekt erzeugt werden.

z.B.
imageStream = new FileStream();

Gruß JensM

[Borislav B. (boris_b)]

Dir ist schon klar, dass du zwei imageStreams hast?
Und nur einer davon wird initialisiert (aber auch gleich darauf wieder 
abgeräumt).

csharp_user schrieb:
> FileStream imageStream;

csharp_user schrieb:
> using (FileStream imageStream = new FileStream...

[csharp_user (Gast)]

Ich möchte zuerst das Bild in ein FileStream laden. Im Anschluss möchte 
ich mit der anderen Funktion btn_SendImage_Click auf den gleichen 
FileStream zugreifen können, ohne das die Fehlermeldung erscheint.

[csharp_user (Gast)]

>Dir ist schon klar, dass du zwei imageStreams hast?
>Und nur einer davon wird initialisiert (aber auch gleich darauf wieder
>abgeräumt).

Habe ich bereits berichtigt.

Trotzdem erscheint nun eine andere Fehlermeldung: sieh Anhang

[Borislav B. (boris_b)]

Wie sieht dein Code denn jetzt aus? Verwendest du immernoch das using?
Das macht deinen Stream natürlich dicht. Wenn du den also weiter 
verwenden willst, solltest du das using entfernen.

[Lenny D. (le-do)]

Hm Boris hat Recht, du hast 2 mal die Deklaration "FileStream 
imageStream" mit gleichem Namen, komisch dass das überhaupt kompiliert.
Eine using-Direktive begrenzt den Scope einer Variablen, d.h. am Ende 
des using Blocks wird der Stream wieder deinitialisiert (weil du ihm ja 
sagst ich brauche ihn nur innerhalb des Bereichs).

Wenn das using keinen guten Grund hat machs wieder weg (dann musst du 
aber selbst später den Stream wieder schließen, ich denke mit .Close() 
oder .Dispose() oder so, schau mal im Internet), außerdem reicht da dann 
imageStream ohne "FileStream" davor, das steht ja schon global einmal.

Dann müsstest du nach dem Laden von überall Zugriff auf ImageStream 
haben.
Allerdings sollte in dem Send Button trotzdem nochmal auf !=null 
checken, da er unter Umständen ja vor dem Laden gedrükt wird (selbst 
wenn du ihn vorher ausgraust oder so ist es kein guter Stil so einen 
Fallstrick offen zu lassen). Hier also eine kleine Verbesserung:

public partial class FormSendImage : Form
    {
        FileStream imageStream;

        public FormSendImage()
        {
            InitializeComponent();
        }

        public void btn_SendImage_Click(object sender, EventArgs e)
        {
            if(imageStream == null)
                return; //sollte sowieso nich passieren, aber hier checken so kommt wenigstens keine Exception
            int bytesSize = 0;
            byte[] downBuffer = new byte[32 - 2];
            byte[] sendBuffer = new byte[32];


            int chunkNumber = 0;
            while ((bytesSize = imageStream.Read(downBuffer, 0, downBuffer.Length)) > 0)
            {
                //hier passiert was, schreib das immer dazu wenn du code löscht zum posten
            }

            imageStream.Close();
        }

        public void btn_Load_Click(object sender, EventArgs e)
        {
            if (ImageOpenFileDialog.ShowDialog() == DialogResult.OK)
            {
                imageStream = new FileStream(ImageOpenFileDialog.FileName, FileMode.Open, FileAccess.Read)
            }
        }

        //TODO: OnFormClose oder so spätestens den Stream schließen mit .Dispose o.Ä.
    }

[csharp_user (Gast)]

using habe ich nun entfernt:

[/c]
if (ImageOpenFileDialog.ShowDialog() == DialogResult.OK)
            {
                imageStream = new 
FileStream(ImageOpenFileDialog.FileName, FileMode.Open, 
FileAccess.Read);
}
[/c]

Allerdings ist byteSize in der while Schleife immer null:

while ((bytesSize = imageStream.Read(downBuffer, 0, downBuffer.Length)) > 0)

[csharp_user (Gast)]

Wenn ich das Bild in der Methode btn_SendImage öffne, dann wird die 
while Schleife korrekt ausgeführt.

        public void btn_SendImage_Click(object sender, EventArgs e)
        {
            string filePath = @"C:\Test\";
            string fileName = "Image.jpg";

            imageStream = new FileStream(filePath + fileName, FileMode.Open, FileAccess.Read);

            if(imageStream == null)
                return; //sollte sowieso nich passieren, aber hier checken so kommt wenigstens keine Exception
            int bytesSize = 0;
            byte[] downBuffer = new byte[32 - 2];
            byte[] sendBuffer = new byte[32];


            int chunkNumber = 0;
            while ((bytesSize = imageStream.Read(downBuffer, 0, downBuffer.Length)) > 0)
            {
                //hier passiert was, schreib das immer dazu wenn du code löscht zum posten
            }

            imageStream.Close();
        }

[Borislav B. (boris_b)]

csharp_user schrieb:
> Wenn ich das Bild in der Methode btn_SendImage öffne, dann wird die
> while Schleife korrekt ausgeführt.

Das sollte keinen Unterschied machen.

Frage: machst du vorher schon etwas mit dem Stream? Also NACHDEM du ihn 
geöffnet hast und BEVOR du btn_SendImage_Click aufrufst?
Falls du z.B. schon daraus gelesen hast, kannst du das nicht ohne 
Weiteres noch ein zweites mal tun.

[csharp_user (Gast)]

Ja. Auf der Oberfläche wird die Byte Größe des Bildes und das Bild 
ausgegeben. Was kann ich da tun, damit ich trotzdem in der Methode 
btn_Load_Click die Bytegröße sowie das Bild ausgeben kann?

[Borislav B. (boris_b)]

csharp_user schrieb:
> Ja. Auf der Oberfläche wird die Byte Größe des Bildes und das Bild
> ausgegeben. Was kann ich da tun, damit ich trotzdem in der Methode
> btn_Load_Click die Bytegröße sowie das Bild ausgeben kann?

Warum willst du denn den FileStream wiederverwenden? Ich glaube das ist 
nicht sinnvoll...

Öffne doch einfach immer einen neuen Filestream an den Stellen, wo du 
ihn benötigst. Dann kannst du dort auch immer das using verwenden, was 
dir ein sauberes Aufräumen garantiert.

[csharp_user (Gast)]

            if (ImageOpenFileDialog.ShowDialog() == DialogResult.OK)
            {
                imageStream = new FileStream(ImageOpenFileDialog.FileName, FileMode.Open, FileAccess.Read);

                lbl_Size.Text = "Picture size [bytes]: " + imageStream .Length.ToString();
                    
                pictureBox.Image = System.Drawing.Image.FromStrea(imageStream);
            }

[Lenny D. (le-do)]

Ein Stream geht Zeichen für Zeichen durch die Quelle und schreibt diese 
in seinen (größenbegrenzten) Buffer den du dann stückweise oder komplett 
liest. Bei jedem gelesenen Zeichen geht er daher natürlich eins weiter. 
Wenn du das Bild einmal ausliest bist du am Ende des Streams angekommen, 
da folgen keine weiteren Daten.
Mit Stream.Seek kannst du an den Anfang (oder andere Positionen) des 
Streams zurückspringen.
http://msdn.microsoft.com/de-de/library/system.io.filestream.seek%28v=vs.110%29.aspx

Ansonsten sind Boris' Alternativen auch gut.

[csharp_user (Gast)]

Verwende nun die Methode Seek. Nun funktioniert es so wie ich es möchte.

Vielen vielen Dank!

[Borislav B. (boris_b)]

Vergiss aber nicht, den Stream zu disposen!