Richtiges Vorgehen zum Beenden einer vorherigen Instanz

  • VB.NET

Es gibt 11 Antworten in diesem Thema. Der letzte Beitrag () ist von Achilleus.

    Richtiges Vorgehen zum Beenden einer vorherigen Instanz

    Hallo Leute,

    ich habe eine kleine Mini-Konsolenanwendung und muss bei einigen Aufrufen verhindern, dass es eine zweite Instanz gibt.
    Hierzu habe ich folgende Programmroutine erstellt, die auch funktioniert.

    VB.NET-Quellcode

    1. ​
    2. ' Anwendung bereits gestartet?
    3. Public Function CleanAppPrevInstance(Optional ByVal bShowMsg As Boolean = True, Optional ByVal bAppActivate As Boolean = True) As Boolean
    4. Dim p() As Process
    5. Dim appName As String = Process.GetCurrentProcess.ProcessName
    7. p = Process.GetProcessesByName(appName)
    8. If p.Count > 1 Then
    9. ' Process is running
    10. If bShowMsg Then
    11. Select Case MessageBox.Show("Das Programm Crypto kann nicht mehrfach gestartet werden." & vbNewLine _
    12. & "Die vorherige Instanz muss geschlossen werden." & vbNewLine _
    13. & "" & vbNewLine _
    14. & "Möchten Sie fortfahren?", "Mehrfachstart von Crypto", MessageBoxButtons.YesNo, MessageBoxIcon.Question)
    16. Case DialogResult.Yes
    17. Application.DoEvents()
    18. Process.GetProcessesByName(appName)(1).Kill()
    19. Return True
    20. Case DialogResult.No
    21. Return False
    22. End Select
    24. End If
    25. End If
    27. Return True
    28. End Function


    Meine Frage ist nun, ist das auch wirklich ein sauberes vorgehen oder gibt es eine bessere Lösung? ?(

    Vielen Dank im Voraus.

    Gruß Achilleus

    @Achilleus Besser ist es, Du arbeitest mit einem Mutex, da bekommst Du beim Start mitgeteilt, ob schon eine Instanz läuft.
    Ich hab ein Beispiel in C# für WinForm, das musst Du dann an Deine Bedürfnisse anpassen:
    Spoiler anzeigen

    C#-Quellcode

    1. using System;
    2. using System.Diagnostics;
    3. using System.Runtime.InteropServices;
    4. using System.Threading;
    5. using System.Windows.Forms;
    7. namespace DeviceControl
    8. {
    9. static class Program
    10. {
    11. /// <summary>
    12. /// Prozeduren zur Anzeige der ersten instanz
    13. /// </summary>
    14. private static class NativeMethods
    15. {
    16. [DllImport("User32.dll")]
    17. [return: MarshalAs(UnmanagedType.Bool)]
    18. internal static extern bool SetForegroundWindow(IntPtr hWnd);
    20. [DllImport("User32.dll")]
    21. [return: MarshalAs(UnmanagedType.Bool)]
    22. internal static extern bool ShowWindowAsync(IntPtr hWnd, int nCmdShow);
    23. }
    25. //----------------------------------------------------------
    26. /// <summary>Der Haupteinstiegspunkt für die Anwendung.</summary>
    27. [STAThread]
    28. static void Main()
    29. {
    30. Application.EnableVisualStyles();
    31. Application.SetCompatibleTextRenderingDefault(false);
    33. bool isFirstInstance;
    34. // Please use a unique name for the mutex to prevent conflicts with other programs
    35. using (Mutex mtx = new Mutex(true, "Application Test", out isFirstInstance))
    36. {
    37. if (isFirstInstance)
    38. {
    39. Application.Run(new MainForm());
    40. }
    41. else
    42. {
    43. // The application is already running
    44. // TODO: Display message box or change focus to existing application instance
    45. MessageBox.Show("Application already running!", "Info", MessageBoxButtons.OK, MessageBoxIcon.Information);
    47. Process current = Process.GetCurrentProcess();
    48. foreach (Process process in Process.GetProcessesByName(current.ProcessName))
    49. {
    50. if (process.Id != current.Id)
    51. {
    52. // Fenster des Prozesses in den Vordergrund holen
    53. NativeMethods.SetForegroundWindow(process.MainWindowHandle);
    54. NativeMethods.ShowWindowAsync(process.MainWindowHandle, 9);
    55. break;
    56. }
    57. }
    58. }
    59. } // releases the Mutex
    60. }
    61. }
    62. }
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Hallo @RodFromGermany,

    vielen Dank für die Info.
    Ich werde mir das einmal in Ruhe anschauen und versuchen es umzusetzen.

    Ich wunder mich immer wieder, was es alles gibt. :D

    Gruß Achilleus


    Hallo @RodFromGermany,

    also ich habe es nicht hinbekommen, aber ich habe meine Programmroutine etwas verändert.

    VB.NET-Quellcode

    1. Public Function CleanAppPrevInstance(Optional ByVal bShowMsg As Boolean = True, Optional ByVal bAppActivate As Boolean = True) As Boolean
    2. Dim current As Process = Process.GetCurrentProcess()
    4. For Each process As Process In Process.GetProcessesByName(current.ProcessName)
    6. If process.Id <> current.Id Then
    7. If bShowMsg Then
    8. Select Case MessageBox.Show("Das Programm Crypto kann nicht mehrfach gestartet werden." & vbNewLine _
    9. & "Die vorherige Instanz muss geschlossen werden und es werden alle eingefügten Dateien gelöscht." & vbNewLine _
    10. & "" & vbNewLine _
    11. & "Möchten Sie fortfahren?", "Mehrfachstart von Crypto", MessageBoxButtons.YesNo, MessageBoxIcon.Question)
    13. Case DialogResult.Yes
    14. Application.DoEvents()
    15. process.Kill()
    16. Return True
    17. Case DialogResult.No
    18. Return False
    19. End Select
    20. End If
    22. Exit For
    23. End If
    24. Next
    26. Return True
    27. End Function


    Nun taucht aber ein anderes Problem auf und zwar befindet sich mein Programm im Netz und wir haben eine Terminalsystem.
    Anscheinend meldet mein Programm jetzt auch eine zweite Instanz an, wenn ein anderer Nutzer mein Programm gestartet hat.

    Ich bräuchte das ja aber nur, wenn ich selber eine zweite Instanz öffnen möchte. Es muss doch so etwas wie Owner geben oder?

    Hast du oder auch andere eine Idee hierzu?
    Das wäre mir schon sehr wichtig.

    Vielen Dank im Voraus.

    Gruß Achilleus

    Beiträge zusammengefügt. ~Thunderbolt

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von „Thunderbolt“ ()

    Achilleus schrieb:

    und zwar befindet sich mein Programm im Netz und wir haben eine Terminalsystem.
    Solch unwichtige Information gehört in den eröffnungspost :!:
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Ich mach das so, dass mein Programm eine leere Datei X erstellt und diese geöffnet lässt. Wenn eine 2. Instanz im Netz gestartet wird, versucht die 2. Instanz, die Datei X zu löschen. Wenn es geht, läuft die 2. Instanz alleine im Netz, wenn eine Zugriffsverletzung kommt, läuft logischerweise noch die 1. Instanz im Netz.
    Dieser Beitrag wurde bereits 5 mal editiert, zuletzt von „VaporiZed“, mal wieder aus Grammatikgründen.

    Aufgrund spontaner Selbsteintrübung sind all meine Glaskugeln beim Hersteller. Lasst mich daher bitte nicht den Spekulatiusbackmodus wechseln.

    Ihr solltet euch zweimal überlegen, ob ihr einen Mutex verwendet.
    Wenn Programm A schon auf dem Server aktiv ist und ihr entwickelt ein Programm B, das auch auf dem Server laufen soll, dann kann es passieren, dass Programm B Schwierigkeiten bekommt. Ich hatte das auf der Arbeit. Der Trick ist dann, Objekte zu nutzen, die mehrere Threads vertragen, wie z.B. ein Concurrent Dictionary    . (Ja, ich weiß, ist nicht das Thema).

    Ich prüfe wie folgt, ob bereits eine Programminstanz läuft:
    ​bool anotherInstanceOfTheProgramIsRunning = System.Diagnostics.Process.GetProcessesByName(System.IO.Path.GetFileNameWithoutExtension(System.Reflection.Assembly.GetEntryAssembly().Location)).Count() > 1;

    GetProcessesByName dürfte aber nur auf dem lokalen PC funktionieren, oder?

    ##########

    Ah, es gibt ja ne Überladung mit Angabe des ZielPCs, auch wenn Du diese anscheinend nicht nutzt.
    Dieser Beitrag wurde bereits 5 mal editiert, zuletzt von „VaporiZed“, mal wieder aus Grammatikgründen.

    Aufgrund spontaner Selbsteintrübung sind all meine Glaskugeln beim Hersteller. Lasst mich daher bitte nicht den Spekulatiusbackmodus wechseln.

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von „VaporiZed“ ()

    Also ich habe den Lösungsvorschlag von @VaporiZed mit einer Lockdatei genommen, die ich einfach mit einem Streamwriter öffne und erst beim Beenden schließe und lösche. Das Verzeichnis überwache ich mit einem Filesystemwatcher, sodass ich die Änderungen im Verzeichnis auch von außen mitbekomme. Das funktioniert in den ersten Tests einwandfrei und ich kann damit den Aufruf einer 2. Instanz sinnvoll verhindern.

    Für diese kleine Anwendung ist meine Lösung völlig ausreichend. Ist am einfachsten und am sichersten so.

    Vielen Dank noch einmal an alle für Eure Unterstützung.

    Gruß Achilleus