also teilweise ist man stolz, dass man die kompliziertesten Dinge hinbekommt und dann scheitert man an absoluten Kleinigkeiten. Ich bitte daher nochmal freundlichst um Eure Hilfe:

In meiner Main.cpp mit

__fastcall TForm1::TForm1(TComponent* Owner)

habe ich am Anfang eingebaut:

Anweisung->Caption = "Updateprüfung, bitte warten";
   Form1->Show();
   Application->ProcessMessages();
   versionspruefung = updatepruefung();

Wenn ein gewisser Wert zurückgeliefert wird, ist ein Update verfügbar.
Um dem User das mitzuteilen, soll ein Infofenster (keine Messagebox) eingeblendet werden. Also:

if (versionspruefung == 1)   
   {
      Form_Update->Show();
      execl("Q:\\Updater.exe", "Q:\\Updater.exe", NULL, NULL);
   }

Aber was ich zu sehen bekomme, ist leider nicht das Fenster "Form_Update", sondern eine nette Fehlermeldung:

Im Projekt AccessControl.exe ist eine Exception der Klasse EAccessViolation aufgetreten. Meldung: 'Zugriffsverletzung bei Adresse 400EE72E in Modul 'vlc60.bpl'. Lesen von Adresse 000002EC'. Prozeß wurde angehalten.

Zumindest erhalte ich diese "Luxusmeldung", wenn ich das Projekt aus dem Builder heraus starte. Starte ich die EXE direkt von der Platte, bekomme ich nur die Meldung "EAccessViolation" ohne Erklärung.

Die Form_Update.h habe ich selbstverständlich in die Main.cpp eingebunden.
Der Fehler liegt ganz klar an der Zeile

Form_Update->Show();

denn wenn ich die als Kommentar markiere, läuft alles ohne Probleme.
Aber dann beginnt halt direkt ohne Vorwarnung das Update und der User kriegt gar nicht mit, was da gerade passiert.

Kann mir bitte einer mein Brett vorm Kopf lösen und sagen, wie ich dieses zweite Fenster angezeigt bekomme?

DANKE.

Syntaktisch ist das schon in Ordnung. Nur ist Form_Update in der betreffenden Zeile ein ungültiger Pointer. Du must diesem erstmal eine gültige Form-Instanz zuweisen. Grundkenntnisse der Speicherverwaltung in C++ sind hier notwendig.

bis bald
akari

/edit: etwas getrödelt.

Ihr meint doch sicherlich:

TForm *Form_Update;
Form_Update = new TForm(this);
Form_Update->Visible = true;

oder?

Aber damit generiere ich doch während der Laufzeit ein ganz neues Fenster.
Das will ich doch gar nicht.
Ich hab doch mein Fenster (Form2 umbenannt in Form_Update) im Builder gebastelt.
Dazu zugehörend der (wenn auch leere) Quellcode Updatehinweis.cpp.
Die Updatehinweis.h ist in meine Main.cpp eingebunden (<include>).

Ich will doch einfach nur dieses Fenster, dass doch zur Laufzeit bereits bekannt ist (oder etwa nicht?) wenn eine Bedingung zutrifft einblenden.

Sorry, aber für ein kleines Stück Quellcode wäre ich echt dankbar, ich raff das gerde mal überhaupt nicht.

Danke.

WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR, int)
{
	try
	{
		Application->Initialize();
		Application->CreateForm(__classid(TForm1), &Form1);
		Application->CreateForm(__classid(TForm_Update), &Form_Update);
		Application->Run();
	}
	catch (Exception &exception)
	{
		Application->ShowException(&exception);
	}
	return 0;
}

d.h. dein 2. Formular wird erst dann erzeugt, nachdem das 1. Formular komplett erzeugt wurde.

Du solltest nur im Konstruktor einer Form auf andere Objekte zugreifen, wenn du genau weißt, was du tust - hier anscheinend weißt du es nicht -)

Am besten du lagerst die Update-Prüfung in das Ereignis 'OnFormActivate' bzw. 'OnFormShow' aus. Evtl. mußt du dann ein Flag setzen, dass es nicht mehrmals aufgerufen wird.

Versuch's doch mal so :

^Aufruf eines modalen Forms :^

Form2 = new TForm2(this);
Form2->ShowModal();
delete Form2;
Form2 = NULL;

^Aufruf eines nicht modalen Forms :^

Form2 = new TForm2(this);
Form2->Show();

^Deleten mußt das dann an geeigneter Stelle...^

Du mußt jetzt nur noch Deine Form-Namen einsetzen...

Alles ,außer dem Hauptformular, wird aus der Liste "Automatisch erzeugen" in die Liste "Verfügbare Formulare" gepackt. Jetzt kannst Du jedes verfügbare Form auch schon im Konstruktor des Hauptformulars aufrufen (wenn man das will).

Nash

Du solltest nur im Konstruktor einer Form auf andere Objekte zugreifen, wenn du genau weißt, was du tust - hier anscheinend weißt du es nicht -)

Am besten du lagerst die Update-Prüfung in das Ereignis 'OnFormActivate' bzw. 'OnFormShow' aus. Evtl. mußt du dann ein Flag setzen, dass es nicht mehrmals aufgerufen wird.

Danke für den Tip, daran bastel ich jetzt schon zwei Tage rum. Ich bekomme zwar damit mein zweites Fenster angezeigt, aber das Update nicht mehr ohne Fehler gestartet.
Also mein Konstruktor macht jetzt nix mehr außer die Buttons im Hauptformular unsichtbar. Diese sollen erst sichtbar werden, wenn wirklich eine sinnvolle Funktion dafür geladen worden ist. Klappt auch.

Meine OnActivate sieht wie folgt aus:

void __fastcall TForm_Main::FormActivate(TObject *Sender)
{
   short int update = 0;

   if (activated == 0)
   {
      update = updatepruefung();      // Prüfung auf vorhandene Updates

      read_registry();       // Access-Pfade aus Registry einlesen
      newsanzeige();         // News anzeigen falls vorhanden
      path_ini_einlesen();   // DB-Pfade einlesen und Buttons aktivieren

      activated = 1;         // damit dieser Prozess nur einmal anläuft
   }
}

Und das ist der entsprechende Teil aus der Funktion updatepruefung:

Form_Update->Show();
      Application->ProcessMessages();
      ::Sleep(3000);
      Form_Update->Close();
      execl("Q:\\Anwendungen_DLZ\\Access Control\\AccessControlUpdater.exe", "Q:\\Anwendungen_DLZ\\Access Control\\AccessControlUpdater.exe", NULL, NULL);

Er ruft zwar das Updateprogramm auf, bringt mir dabei aber den Fehler:
"Eigenschaft Visible kann in OnShow oder OnHide nicht verändert werden".
Ich habe zwar für keines der beiden Ereignisse etwas programmiert, aber da "execl ()" die Anwendung, aus der es aufgerufen wird, schließt, nachdem die zu öffnende Anwendung erfolgreich geöffnet werden konnte, produziert dies diesen Fehler.

Also wenn ich das Ganze aus dem Konstruktor oder aber einer Funktion, die aus dem Konstruktor heraus aufgerufen worden ist, ausführe, dann bekomme ich eine "Zugriffsverletzung bei Adresse ....".
Wenn ich da Ganze aus dem Ereignis OnActivate oder OnShow bzw. einer Funktion, die aus OnActivate oder OnShow heraus aufgerufen wurde, ausführe, dann bekomme ich die Fehlermeldung "Eigenschaft Visible kann nicht geändert werden".

Wenn ich wenigstens an was super kompliziertem Verzweifeln würde, dann ginge das ja, aber ich bin einfach nur zu blöd, aus einem Hauptfenster heraus ein zweites Fenster zu öffnen, wieder zu schließen und eine andere Anwendung aufzurufen, ohne mir zwischendurch einen Fehler einzufangen.

Bitte, bitte helft mir nochmal.

DANKE.

^Aufruf eines nicht modalen Forms :^

Form2 = new TForm2(this);
Form2->Show();

^Deleten mußt das dann an geeigneter Stelle...^
Nash

Danke, klappt. FREU * HÜPF * SPRING

Aber gestattet mir bitte trotzdem die Frage:
Ist das jetzt ok, wenn ich im Konstruktor mein zweites Fenster aufrufe?
Oder gibt es da Dinge die ich beachten sollte?
Ich frage, weil ich diesen Satz ja durchaus ernst nehme:

Du solltest nur im Konstruktor einer Form auf andere Objekte zugreifen,
wenn du genau weißt, was du tust - hier anscheinend weißt du es nicht -)

Und ich gebe es ja zu, dass ich es nicht weiß.

Wenn du jetzt also das 2. Formular nicht-modal aufrufst, dann solltest du nur dafür sorgen, daß es irgendwann wieder gelöscht wird. Da du es anscheinend nur einmalig in deinem Programm aufrufst, wird der Speicherverbrauch also nicht so groß sein, so daß es beim Löschen des Hauptformulars (beim Beenden des Programms) automatisch mitgelöscht wird.

Da du das 2. Fenster ja anscheinend nur als Info-Fenster temporär anzeigst, kannst du einfach das Erzeugen und Löschen selber durchführen:

TForm *frm = new TForm(this);
frm->Show();
Application->ProcessMessages();
::Sleep(3000);
frm->Close();
delete frm;
int nResult = execl("Q:\\Anwendungen_DLZ\\Access Control\\AccessControlUpdater.exe", NULL);
// check result
if(nResult == -1)
{
   // couldn't start update program
   // check global var 'errno'

   // ...
}

Bzgl. der Funktion 'execl': du brauchst den Dateinamen(incl. Pfad) nicht doppelt angeben (obiges reicht aus, außer du willst deiner Anwendung noch den eigenen Namen+Pfad als Parameter übergeben, aber das ist wohl unnötig).
Außerdem solltest du den Rückgabewert prüfen (nResult, s.o.), denn es kann immer mal Probleme geben...

Auch solltest du evtl. den Pfad für das Update-Programm nicht fest im Quellcode ablegen (evtl. besser relativ zum eigentlichen Programm angeben, d.h. z.B. "./AccessControlUpdater.exe").

Joe_M. schrieb:

Zum einen sollte man niemals die vorgebenen Namen für manuell erstellte Forms verwenden (wenn überhaupt, dann mit der CreateForm-Methode).

Kannst Du das mal ein wenig ausführen, warum das unsauber ist ?
Ich hab damit noch nie Probleme gehabt und ich programmiere nicht erst seit gestern...

Nash

Wenn du den von der IDE vorgegebenen Pointer zum Form benutzt, dann machst du dich immer davon abhängig. Das geht nur solange wie du nur eine Instanz dieses Formes brauchst. Wenn du jemals dynamisch mehrere Instanzen der Formklasse brauchst must du immer über this gehen.

bis bald
akari

Ja gut, OK. Den Fall hatte ich bisher eher selten, um nicht zu sagen eigentlich nie (soweit mir gerade einfällt)... doch einmal, in einer Bilderverwaltungs-DLL. Die konnte mehrfach aufgerufen werden. Da hatte ich das auch anders gelöst. Aber das ist für mich, wie gesagt, eher selten der Fall.

Aber so schlimm kann's auch nicht sein, denn sogar in der BCB-Hilfe (z.B.: Formulare dynamisch erstellen) wird das so gemacht...

Nash

Du kannst schon im Konstruktor ein 2. Formular aufrufen (oder sonstige beliebige Dinge machen), nur mir schien es so, daß du nicht so viel Ahnung von der Speicherverwaltung hast.

Ich sag es mal so: Ich tue mich schon ein wenig schwer, was Programmierung für Windows angeht. Ich habe C-Progammierung mit einem Borland Compiler unter DOS für DOS gelernt. Selbst als ich meinen ersten Borland Compiler unter Windows hatte, habe ich weiterhin für DOS programmiert. Und ich gebe ehrlich zu, dass zu dieser Zeit Anwendungen unter DOS entstanden sind, die ich mir unter Windows nicht zutrauen würde zu programmieren. So sicher wie ich in DOS bin, was ich mache, so unsicher bin ich unter Windows.
Doch grundlegende Kenntnisse der Speicherverwaltung kann ich natürlich auch unter Windows weiter nutzen. Dazu zählt für mich ganz klar (auch in Zeichen von 4 GB RAM) arbeitsspeicherschonendes programmieren (z.B.: warum int, wenn auch short int möglich ist, warum global, wenn ich die Variable nur lokal benötige?) und natürlich auch alles, was ich irgendwann mal aufmache (z.B.: filestream, Formular) sofort wieder zu schließen, wenn ich es nicht mehr benötige.

Th schrieb:

Bzgl. der Funktion 'execl': du brauchst den Dateinamen(incl. Pfad) nicht doppelt angeben (obiges reicht aus, außer du willst deiner Anwendung noch den eigenen Namen+Pfad als Parameter übergeben, aber das ist wohl unnötig).

Ich hatte diesbezüglich eigentlich in der Hilfe nachgesehen, wie die Funktion aussieht:

int execl (char *path, char *arg0 ..., NULL);

Der Parameter path gibt den Dateinamen des untergeordneten Prozeßes an.
Sämtliche exec-Funktionen müssen zumindest ein Argument an den untergeordneten Prozeß übergeben, nämlich arg0. Dieses Argument enthält per Konvention eine Kopie von path.

Und als Beispiel führt

execl("C:\\Programme\\Office\\msaccess.exe", "C:\\Datenbanken\\Database.mdb");

nur dazu, dass Access geöffnet wird.
Willst Du wirklich die Datenbank aufmachen, muss es

execl("C:\\Programme\\Office\\msaccess.exe", "C:\\Programme\\Office\\msaccess.exe", "C:\\Datenbanken\\Database.mdb");

heißen

Th schrieb:

Außerdem solltest du den Rückgabewert prüfen (nResult, s.o.), denn es kann immer mal Probleme geben...

Yepp, 100% Zustimmung. Mach ich auch noch.

Th schrieb:

Auch solltest du evtl. den Pfad für das Update-Programm nicht fest im Quellcode ablegen (evtl. besser relativ zum eigentlichen Programm angeben, d.h. z.B. "./AccessControlUpdater.exe").

Werde ich wohl nicht hinbekommen, da die Anwendung lokal auf dem Rechner liegt und die "Updater.exe" auf verschiedenen Servern. Jedoch hat jeder PC immer nur einen von diesen Servern eingebunden und das immer unter Q:\[...].
Ich möchte die "Updater.exe" auch nicht lokal auf die Rechner legen, weil diese definitiv noch weiterentwickelt werden wird. Wenn Sie aber lokal auf mehreren Rechnern liegt, dann komme ich da nicht mehr ohne weiteres dran.
Und falls ich Änderungen an dieser (durch ein Update) vornehmen möchte, kann ich das dann nicht machen, da die Datei sich ja nicht selber austauschen kann wenn sie läuft.
Überlegung daher:
- Anwendung lokal, "Updater.exe" auf dem Server.
- wenn Update vorhanden, Start der "Updater.exe" über execl
- nach erfolgreichem Start der "Updater.exe" schließt sich die lokale Hauptanwendung selber, was ja in der Natur von execl liegt.
Damit sind alle Dateien der lokalen Hauptanwendung geschlossen und austauschbar. Und eine neue Version der "Updater.exe" kann ich auf dem Server austauschen.

Ich weiß ja nicht, wann ich das nächste Mal auf ein Problem stoße und Eure Hilfe benötige, daher an dieser Stelle schon mal ein gang ganz großes Dankeschön für Eure Unterstützung.
Ich werde auch hin und wieder mal reinsehen, vielleicht kann ich selber ja auch mal jemandem helfen.