Slideshow

pictureBox1->Image->Dispose()

Helfender Helfer

Ach du solltest natürlich auch noch beachten das das Image am anfang null ist! also:

private: System::Void TimerEventProcessor(System::Object^ sender, System::Timers::ElapsedEventArgs^ e) {                   
                if (this->lFileNames->Count > 0) {       
                   this->tTimer->Stop();
                   if(this->pictureBox1->Image != null)
                          this->pictureBox1->Image->Dispose();
                   this->pictureBox1->Image = System::Drawing::Image::FromFile(this->lFileNames[0]);
                   this->lFileNames->Remove(lFileNames[0]);
                   this->tTimer->Start();
                } else {
                   das Gleiche, nur wird halt zusätzlich die Liste neu gefüllt
                }

}

magifrei

Ah sorry, hatte es jetzt nur mal auf die Schnelle hineingeschrieben, um die Fehlermeldung posten zu können.

Bei meinen ersten Versuchen hab' ich das "Image" mit drin gehabt

c:\users\marian\documents\visual studio 2005\projects\fotoslider\fotoslider\Form1.h(288) : error C2039: 'Dispose': Ist kein Element von 'System::Drawing::Image'
        c:\windows\microsoft.net\framework\v2.0.50727\system.drawing.dll: Siehe Deklaration von 'System::Drawing::Image'
        Sie sollten stattdessen den Destruktor, "~Image", aufrufen.

achso C++ mach ein

delete(this->pictureBox1->Image);

anstatt das

this->pictureBox1->Image->Dispose();

magifrei

AHHHHHHHHHHHHHHH, ich halts nicht aus. Ich Volldepp

War so verzweifelt, dass ich Sachen wie ~this->pictureBox1->Image etc. versucht habe.
delete() ... es darf nicht wahr sein. Dabei hab' ich das sogar mal in einem Projekt zum Beenden desselbigen verwendet, bis ich dann irgendwann entdeckt habe, dass man das Formular über Close() beenden kann.

Danke, jetzt verbraucht mein Programm ziemlich konstant 30MB bei 10MP-Dateien; das wird wohl die ungefähre Größe des entsprechenden Bitmaps sein.

Du hast meine Party echt gerettet, tausend Dank

magifrei

Und da bin ich leider auch schon wieder

Nachdem nun alles inkl. Vollbildmodus und genügsamen Speichermanagement funktioniert und sich mein Projekt auf einem anderen Rechner auch zu einer lauffähigen Exe kompilieren ließ, habe ich nun ein dickes Problem hier.

Bewegt man in dem Moment des Bildwechsels ein anderes Fenster über die Picturebox, so stürzt mein Programm mit folgender Meldung ab:

************** Ausnahmetext **************
System.ArgumentException: Ungültiger Parameter.
bei System.Drawing.Image.get_Width()
bei System.Drawing.Image.get_Size()
bei System.Windows.Forms.PictureBox.ImageRectangleFromSizeMode(PictureBoxSizeMo
de mode)
bei System.Windows.Forms.PictureBox.get_ImageRectangle()
bei System.Windows.Forms.PictureBox.OnPaint(PaintEventArgs pe)
bei System.Windows.Forms.Control.PaintWithErrorHandling(PaintEventArgs e, Int16 layer, Boolean disposeEventArgs)
bei System.Windows.Forms.Control.WmPaint(Message& m)
bei System.Windows.Forms.Control.WndProc(Message& m)
bei System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
bei System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
bei System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

Mein Bild wird in der Mainform in einer Picturebox angezeigt. Ausgelöst wird der Bildwechsel durch ein Timerevent:

private: System::Void TimerEventProcessor(System::Object^ sender, System::Timers::ElapsedEventArgs^ e) {					

			 //Speicher vom alten Bild befreien
			 delete(this->pictureBox1->Image);

			 //Wenn die Dateinamenliste noch Einträge enthält...
			 if (this->lFileNames->Count > 0) {		

				this->tTimer->Stop();

				//bRandom enthält den Bool für die zufällige Bildwiedergabe
				if (bRandom == true) {
					int x = this->lFileNames->Count;
					int y = this->random->Next(0, x);

					//Überprüfung, für den Fall, dass inzwischen ein Bild gelöscht wurde
					while ((this->lFileNames->Count > 0) && (System::IO::File::Exists(this->lFileNames[y]) == false)) {
						this->lFileNames->Remove(lFileNames[y]);
						x = this->lFileNames->Count;
						y = this->random->Next(0, x);
					}
					//Wenn die Dateinamenliste danach immer noch Einträge enthält...
					if (this->lFileNames->Count > 0) {
						this->pictureBox1->Image = System::Drawing::Image::FromFile(this->lFileNames[y]);
						this->lFileNames->Remove(lFileNames[y]);
					}

				} else {
					//Überprüfung, für den Fall, dass inzwischen ein Bild gelöscht wurde
					while ((this->lFileNames->Count > 0) && (System::IO::File::Exists(this->lFileNames[0]) == false)) {
						this->lFileNames->Remove(lFileNames[0]);
					}
					//Wenn die Dateinamenliste danach immer noch Einträge enthält...
					if (this->lFileNames->Count > 0) {	
						this->pictureBox1->Image = System::Drawing::Image::FromFile(this->lFileNames[0]);
						this->lFileNames->Remove(lFileNames[0]);
					}
				}

				this->tTimer->Start();

			} else {

				this->tTimer->Stop();

				loadFileNames();

				if (bRandom == true) {
					int x = this->lFileNames->Count;
					int y = this->random->Next(0, x);
					this->pictureBox1->Image = System::Drawing::Image::FromFile(this->lFileNames[y]);
					this->lFileNames->Remove(lFileNames[y]);
				} else {
					this->pictureBox1->Image = System::Drawing::Image::FromFile(this->lFileNames[0]);
					this->lFileNames->Remove(lFileNames[0]);
				}

				this->tTimer->Start();

			}
		 }

Der Absturz erfolgt auch, wenn man die Fenstergröße während des Bildwechsel verändert.
Ich würde es mit einer try-Anweisung versuchen, hab aber keine Ahnung, wie das entsprechende catch lauten muss

magifrei

Ich habe es in meiner showPicture-Methode einfach mal so probiert:

private: System::Void showPicture(int x) {
			 try {
				this->pictureBox1->Image = System::Drawing::Image::FromFile(this->lFileNames[x]);
				this->lFileNames->Remove(lFileNames[x]);
			 }
			 catch(...) {}
		 }

Der Absturz erfolgt aber nach wie vor. Wenn ich mir die Fehlermeldung so anschaue, könnte es auch an der Picturebox ansich, bzw dem SizeMode liegen.

this->pictureBox1->SizeMode = System::Windows::Forms::PictureBoxSizeMode::Zoom;

Hat jemand eine Idee?

mach doch mal den fehler im debug modus und sag wos hängt ^^

mfg Helfender Helfer

magifrei

Eine nicht behandelte Ausnahme des Typs "System.ArgumentException" ist in System.Drawing.dll aufgetreten.

Zusätzliche Informationen: Ungültiger Parameter.

und er schickt mich zum return meiner main.

In der "Aufrufliste" steht ein kleiner gelber Pfeil bei System.Drawing.dll!System.Drawing.Image.Width.get() + 0x54 Bytes.

Auf meinem PC (also mit installiertem VS) tritt der Fehler aber nur auf, wenn man die Fenstergröße (damit auch die Picturebox) während des Bildwechsels ändert.
Ein Rumhantieren mit anderen Dialogfenstern stört ihn (im Gegensatz zu dritten PCs ohne VS) nicht.

Ich überlege gerade, ob es eventuell hilft, den Fenstermodus beim Laden eines Bilder auf "nicht veränderbar" zu setzen.
Jedoch wäre das nur die Lösung für das Problem an meinem PC. Auf dritten Rechner reicht es wie gesagt schon, wenn man zum Zeitpunkt des Bildwechsel ein fremdes Dialogfenster über meiner Hauptform mit der Picturebox bewegt. Da hilft meine Idee natürlich nicht weiter.

Nachtrag: Meine Idee funktioniert leider nicht.

magifrei

Eines haben schlaflose Nächte für sich. Wenn man nicht einschläft denkt man über allen möglichen Kram nach.

Mein Programm verabschiedet sich, da ich vor dem Laden des neuen Bildes das alte per delete() lösche. Die Resize-Funktion des Fensters scheint das wohl nicht zu mögen, da ein Bild, dessen Größe es verändern will, auf einmal nicht mehr da ist.

Daher kam es auch auf meinem Win2000 System nicht zum Absturz. Dort wird die Änderung der Fenstergröße nämlich erst übernommen, wenn man die Maus loslässt.
WinXP und Vista machen das ja quasi just in time.

Das Ändern der Fenstereigenschaft auf Fixed ist hilft aber leider nur auf den ersten Blick. Maximiert man nämlich genau während des Bildwechsel, kommt es wieder zum gleichen Problem.

Ich müsste in meiner Methode eine Überprüfung haben, ob die Fenstergröße aktuell verändert wird. Falls ja, soll er solange mit dem Bildwechsel warten.

michme

Stop den Timer für den Bildwechsel im ResizeBegin Event und starte ihn wieder im ResizeEnd Event.

Z. B. so:

private: System::Void Form1_ResizeBegin(System::Object^  sender, System::EventArgs^  e) {
                 this->timer1->Stop();
             }
private: System::Void Form1_ResizeEnd(System::Object^  sender, System::EventArgs^  e) {
                 this->timer1->Start();
             }

magifrei

Danke, das funktioniert gut.
Kann mir nochjemand sagen, welches Events durch Klicken der Maximieren/Minimieren-Schaltflächen ausgelöst werden?

Ich finde leider nur MaximizedBoundsChanged bzw MinimumSizeChanged und das ist ja leider was anderes.
Komisch, dass es nicht unter this->MaximizeBox->Click oder so zu finden ist.

In der Hilfe habe ich dann entdeckt, dass durch das Klicken dieser Schaltflächen ein Resize-Ereignis ausgelöst wird, blöderweise scheint das durch meine ResizeBegin und ResizeEnd jedoch nicht abgedeckt zu sein.
Schafft man es nämlich, genau im Moment des Bildwechsels auf Maximieren zu klicken, schmiert es mir nach wie vor mit genannter Meldung ab.

Man findet zwar auch noch this->Resize, wo man einen Eventhandler einfügen könnte ... nur wie starte ich den Timer dann wieder? Bzw. ist das überhaupt das Richtige?

Hilfe, hätte ich am Anfang gewusst, dass dieses kleine Ding so in Arbeit ausartet, hätte ich es gelassen
Aber wenn, dann soll es natürlich auch bugfrei sein

michme

Dann ruf im Resize Event noch folgendes auf:

System::Void Form1::Form1_Resize(System::Object^ sender, System::EventArgs^ e)
{
    this->OnResizeBegin(e->Empty);
    this->OnResizeEnd(e->Empty);
}

Dies löst nacheinander Form1_ResizeBegin und Form1_ResizeEnd aus, welche ja den Timer stoppen und starten.

HTH Micha

magifrei

Das funktioniert leider nicht

Der Code lässt sich zwar kompilieren, das Problem ist aber trotzdem noch da.
Hab's auch zusätzlich mit sowas probiert, weil ich dachte, den Timer so auf jeden Fall erst einmal stoppen zu können:

private: System::Void Form1_Resize(System::Object^ sender, System::EventArgs^ e) {
			this->tTimer->Stop();
			this->OnResizeEnd(e->Empty);
		 }

Liefert aber das gleiche Ergebnis

michme

Wie hoch bzw. niedrig ist den dein Timerinterval? Ich lass zwar nur nen Counter im Label hochzählen, sehe aber deutlich das der Timer angehalten und wieder gestartet wurde.

magifrei

Da es nicht so einfach ist, genau im passenden Moment auf Maximieren zu klicken, hab ich den Timer in diesem Fall auf 1s gestellt.

michme

Hmm, dann versuch doch die ArgumentException zu fangen im Resize Event.

System::Void Form1::Form1_Resize(System::Object^ sender, System::EventArgs^ e)
{
    try
    {
        this->OnResizeBegin(e->Empty);
    }
    catch (System::ArgumentException^ e)
    {
        //MessageBox::Show(e->Message); // oder auch e->StackTrace
    }
    finally
    {
        this->OnResizeEnd(e->Empty);
    }
}

Im try wird der Timerstop ausgelöst, im Falle eines Falles die ArgumentException gefangen(im catch) und im finally der Timer wieder gestartet.
Mangels Absturz auf meiner Seite, kann ich jetzt nicht sagen ob es dir tatsächlich hilft.

magifrei

Eine nicht behandelte Ausnahme des Typs "System.ArgumentException" ist in System.Drawing.dll aufgetreten.

Zusätzliche Informationen: Ungültiger Parameter.

Hatte aber Haltepunkte gesetzt und kann sagen, dass er sowohl try, catch, als auch finally durchlaufen hat. Gleich danach kam es dann zum Absturz.

Vielleicht sollte ich die Maxi- und Minimieren-Schaltflächen einfach entfernen

Übrig bleibt wahrscheinlich das Problem, dass es zum Absturz kommt, wenn man ein weiteres Dialogfenster zum Zeitpunkt des Bildwechsels über meiner PictureBox bewegt. Dieses Problem kann ich aber zur Zeit auch nicht nachstellen, da es auf meinen PC nicht vorkommt; wohl aber auf dem Notebook meiner Freudnin und auf PCs von Bekannten, die es freundlicherweise für mich getestet haben.

magifrei

Oh man, Problem gelöst

Nach

delete(this->pictureBox1->Image);

musste einfach ein

this->pictureBox1->Image = nullptr;

eingefügt werden. Nun scheint's zu laufen

My sorry war meine schuld hab vergessen das mit zuschreiben...