c++.de :: wxwidgets Problem mit Programm

mdpse Mitglied 20:35:03 07.04.2012 Titel:

Hallo!

Hab mich hier gerade erst registriert, da ich ein Problem mit einem Programm habe.

Es handelt sich dabei um eine numerische Integration.
Ich habe das Programm schon als Konsolen-Programm geschrieben und da funktioniert es auch perfekt. Dabei werden die berechneten Werte in eine .txt Datei gespeichert.
Jetzt soll es aber auch eine GUI bekommen. Da ich ein Programmieranfänger bin habe ich von wxwidgets auch nicht so viel ahnung.

Also habe ich mal herumprobiert, einfach einen Knopf erstellt, wenn der Knopf gedrückt wird soll das Programm die Werte in eine .txt Datei speichern. Das ganze also noch ohne direkte Eingabe von Werten. Dabei habe ich die Klasse und die Methoden aus dem OOP-Programm in das GUI-Programm eingefügt. Das ganze funktioniert auch, allerdings nur bei bestimmten Werten.

Hier mal der Code:

C++:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65	#include "wxVersuchFrm.h" #include "rakete.h" #include <fstream> #include <iostream> using namespace std; //---------------------------------------------------------------------------- // wxVersuchFrm //---------------------------------------------------------------------------- BEGIN_EVENT_TABLE(wxVersuchFrm,wxFrame) EVT_CLOSE(wxVersuchFrm::OnClose) EVT_BUTTON(ID_WXBUTTON1,wxVersuchFrm::WxButton1Click) END_EVENT_TABLE() wxVersuchFrm::wxVersuchFrm(wxWindow parent, wxWindowID id, const wxString &title, const wxPoint &position, const wxSize& size, long style) : wxFrame(parent, id, title, position, size, style) { CreateGUIControls(); } wxVersuchFrm::~wxVersuchFrm() { } void wxVersuchFrm::CreateGUIControls() { WxPanel1 = new wxPanel(this, ID_WXPANEL1, wxPoint(1, -2), wxSize(301, 299)); WxButton1 = new wxButton(WxPanel1, ID_WXBUTTON1, wxT("WxButton1"), wxPoint(17, 26), wxSize(75, 25), 0, wxDefaultValidator, wxT("WxButton1")); SetTitle(wxT("wxVersuch")); SetIcon(wxNullIcon); SetSize(8,8,320,334); Center(); } void wxVersuchFrm::OnClose(wxCloseEvent& event) { Destroy(); } / * WxButton1Click */ void wxVersuchFrm::WxButton1Click(wxCommandEvent& event) { Rakete rakete1; rakete1.erzeuge(590000,90000,4000,125,3500,0.35,10.8,0); rakete1.Raketengleichung(0.1,2000); }

Die zwei Funktionen am Schluss (erzeuge(...) und Raketengleichung(...)) sind aus meiner Klasse "Rakete".

Die Funktionen sind wie folgt definiert:

C++:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192	#include "rakete.h" // class's header file #include <cstdlib> #include <iostream> #include <cmath> #include <fstream> #include <iomanip> using namespace std; // Funktion : Abhängigkeit der Erdanziehung mit der Höhe float Rakete :: gravitation (float h) { float go = 9.81; float Re = 6400000; float gH; gH = (go * pow (Re,2))/ pow (Re + h,2); return gH; } // Barometrische Höhenformel laut ISA, gilt eigentlich nur bis zur Tropopause!! float Rakete :: dichte (float h) { float rho0 = 1.225; // Dichte in MSL laut ISA kg/m³ float a = 0.0065; // Temperaturgradient laut ISA K/m (bis zur Tropopause) float M = 0.02896; // molare Masse kg/mol float go = 9.81; // Erdanziehung m/s² float To = 288.15; // Temperatur in MSL laut ISA K float ho = 0; // Meereshöhe float R = 8.314; // universelle Gaskonstante für Luft J/Kmol float rhoH; // Dichte in der Höhe h if ( h < 40000 ) // Die Dichte muss ab ca 40000m auf 0 gesetzt werden, da sie sonst zu einer komplexen Zahl wird -> NAN // -> zieht alle anderen Werte mit runter!! { rhoH = rho0 pow((1- (a * ( h - ho )/To)), (( M * go)/(R * a) - 1)); } else { rhoH = 0; } return rhoH; } // Abnahme der Masse m mit der Zeit float Rakete :: mass (float m0, float ml, float mp, float tb, float t) { float mT; mT = ( m0 - mp * t) * ( t < tb ) + ml * ( t >= tb); return mT; } // erzeuge Methode void Rakete :: erzeuge ( float x, float b, float c, float d, float e, float f, float g, float h ) // Die Parametervariablen dürfen nicht gleich sein wie die Variablen // die die Werte übernehmen { m0 = x; ml = b; mp = c; tb = d; vg = e; cw = f; Sref = g; ho = h; } // Raketengleichung void Rakete :: Raketengleichung ( float dt,int a) { float v = 0; // Anfangsgeschwindigkeit float t = 0; // Anfangszeitpunkt float vp; // Ableitung der Geschwindigkeit float dh; // Änderung der Höhe in der Zeit dt float dv; // Änderung der Geschwindigkeit in der Zeit dt float* vT; float* hT; float* T; //dient nur dazu die Zeit für die Excell Datei zu speichern vT = new float [a]; hT = new float [a]; T = new float [a]; vT [0] = 0; // Setzt Anfangszustand fest hT [0] = ho; // Setzt Anfangszustand fest T [0] = 0; for ( int i = 1 ; i < a; i++) { vp = ( (mp * vg)(t < tb) - mass (m0,ml,mp,tb,t) gravitation ( hT [i-1]) - (cw * dichte (hT [i-1]) * vT [i-1] * fabs (vT [i-1]) * Sref)/2) / mass (m0,ml,mp,tb,t); dv = vp * dt; vT [i] = vT [i - 1] + dv; dh = vT [i] * dt; hT [i] = hT [i - 1] + dh; t = t + dt; T [i+1] = t; } //-------------SPEICHERN IN TXT DATEI------------------ //----------------------------------------------------- // Die berechneten Daten werden in ein .txt Datei gespeichert, // abhängig von dem sich ergebenden "a" werden die Werte in einem bestimmten Intervall gespeichert ofstream program; program.open("Raketengleichung.txt"); int b; // Setzt die größe des Intervalls fest b = 1; int c = a/b; // Setzt fest wie viele Werte eingelesen werden float* velocity; // speichert die Werte der Geschwindigkeit in "velocity" velocity = new float [ c ]; velocity [0] = 0; for ( int i = 1; i < c ; i++) { velocity [i] = vT [ i * b ]; // Jeder b. Wert wird gespeichert } float* altitude; // speichert die Werte der Höhe in "altitude" altitude = new float [ c ]; altitude [0] = 0; for ( int i = 1; i < c ; i++ ) { altitude [i] = hT [ i * b ]; } float* time; // speichert die Werte der Zeit in "time" time = new float [ c ]; time [0] = 0; for ( int i = 0; i < c; i++ ) { time [i] = T [ i * b ]; } program << " " << " t " << " " << " v " << " " << " h " << endl; for ( int i = 0; i < c ; i ++ ) // Gibt die Werte in der .txt Datei { if ( altitude [i-1] < 0 ) // stoppt die Ausgabe wenn die ´Höhe negativ wird { break; } program << " " << time [i] << " " << velocity [i] << " " << altitude [i] << endl; } program.close(); delete [] hT; delete [] vT; delete [] T; }

Wenn ich in der Funktion "Raketengleichung" für "dt" = 1 und "a" = 200 eingebe funktioniert es normal, sobald ich aber ein zu großes "a" wähle oder "dt" kleiner mache (zb dt = 0.1), kommt beim Drücken des Knopfes eine Fehlermeldung.

Was könnte der Fehler sein?
Wie gesagt die Formeln usw. sind alle richtig und funktionieren normal auch.

MfG md

Unregistrierter

Benenne mal ID_WXBUTTON1 um, in irgendwas ohne wx im Namen. Die Callback Funktion genauso.
wx als Präfix ist für wxWidgets reserviert.

Unregistrierter

Kommt eine Fehlermeldung - hervorragende Beschreibung, damit kann man etwas anfangen.

In Zeile 117 schreibst du mit T[i+1] = t an eine ungültige Position!

mdpse Mitglied 12:26:23 08.04.2012 Titel:

Fehlermeldung lautet:
wxVersuch.exe funktioniert nicht mehr
Das Programm wurde aufgrund eines Problems nicht richtig ausgeführt. Das Programm wird geschlossen und Sie werden benachrichtigt, wenn eine Lösung verfügbar ist.

Das mit Zeile 117 versteh ich jetzt nicht ganz, aber es geht ja auch nicht um die Funktionen, die einwandfrei funktionieren.
Kann es sein, dass das Programm ein Problem mit dem Speicherplatz bekommt, weil dieser zum Beispiel beschränkt ist? Bei "dt" = 0.1 ergibt sich zB bei einer Flugdauer von 200s eine Arraygröße von 2000 also 2000 mal float, normalerweise ja kein Problem.

Unregistrierter

Wenn i = a-1 schreibst du an die Stelle T[a]. Da das Array die Länge a hat, schreibst du hinter den dir zugewiesenen Bereich. Das kann die Ursache für dein Problem sein, es könnte aber auch noch weitere Fehler geben.
Nur weil ein Programm zufällig nicht abstürzt, funktioniert es noch nicht korrekt.

mdpse Mitglied 18:02:29 08.04.2012 Titel:

Hey super. Das wars, Zeile 117 war wirklich das Problem.
Danke für die Hilfe.