Korrekte DB-Abfrage mit try catch finally

  • C#

Es gibt 3 Antworten in diesem Thema. Der letzte Beitrag () ist von RodFromGermany.

    Korrekte DB-Abfrage mit try catch finally

    Hallo zusammen

    Wie muss ich das korrekt programmieren?

    Quellcode

    1. internal class DB_Auslesen
    2. {
    3. private string s;
    4. public DB_Auslesen()
    5. { }
    7. public string ZeitreiheAuslesen(string zp, string linie)
    8. {
    9. object val;
    10. try
    11. {
    12. string ConnectionString = @"Server...";
    13. SqlConnection connection = new SqlConnection(ConnectionString);
    14. using (connection)
    15. {
    16. SqlCommand command = new SqlCommand($"SELECT zeitreihen_id FROM zero_dm.d_zpt_header WHERE zaehlpunkt_id = '{zp}' AND linie = '{linie}'", connection);
    17. connection.Open();
    18. SqlDataReader reader = command.ExecuteReader();
    19. while (reader.Read()) // maximal ein Treffer mööglich
    20. {
    21. val = reader["zeitreihen_id"];
    22. s = val.ToString();
    23. reader.Close();
    24. //connection.Close();
    25. }
    26. if (string.IsNullOrEmpty(s))
    27. {
    28. s = "Kein Treffer";
    29. }
    30. }
    31. }
    32. catch (Exception ex)
    33. {
    34. s = "Fehler";
    35. MessageBox.Show(ex.ToString());
    36. }
    37. finally
    38. {
    39. //reader.Close();
    40. connection.Close();
    41. return s;
    42. }
    43. }
    44. }


    Problem 1: im finally-Block kann ich nicht auf connection zugreifen ("connection" ist im aktuellen Kontext nicht vorhanden)
    Problem 2: Fehler bei return S im finally-Block (Das Steuerelement kann den Text einer finally-Klausel nicht verlassen
    Problem 3: Eigentlich möchte ich das als Statische Klasse und Funktion

    Kann mir jemand weiterhelfen?
    Besten Dank

    @Murmeli81 Sieh Dir mal dies an:
    Spoiler anzeigen

    C#-Quellcode

    1. internal static class DB_Auslesen
    2. {
    3. public static string ZeitreiheAuslesen(string zp, string linie)
    4. {
    5. SqlConnection connection = null;
    6. try
    7. {
    8. string ConnectionString = @"Server...";
    9. connection = new SqlConnection(ConnectionString);
    10. using (connection)
    11. {
    12. SqlCommand command = new SqlCommand($"SELECT zeitreihen_id FROM zero_dm.d_zpt_header WHERE zaehlpunkt_id = '{zp}' AND linie = '{linie}'", connection);
    13. connection.Open();
    14. using (SqlDataReader reader = command.ExecuteReader())
    15. {
    16. if (reader.Read()) // maximal ein Treffer mööglich
    17. {
    18. object val = reader["zeitreihen_id"];
    19. string s = val.ToString();
    20. return s;
    21. }
    22. reader.Close();
    23. }
    25. return "Kein Treffer";
    26. }
    27. }
    28. catch (Exception ex)
    29. {
    30. MessageBox.Show(ex.ToString());
    31. return "Fehler";
    32. }
    33. finally
    34. {
    35. if (connection != null)
    36. {
    37. connection.Close();
    38. }
    39. }
    40. }
    41. }
    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

    Besten Dank. Genau das, was ich wollte.

    Noch eine allgemeine Frage.
    Wenn innerhalb eines Programms mehrmals auf eien DB zugegriffen wird, will ich mich nicht jedes mal die Zugangsdaten eingeben müssen. Zu Testzwecken habe ich hier erstmal User und PW in den ConnectionString reingeschrieben.
    Speichert man User und PW nach der Anmeldung einfach in eine Variable? Ich bin jetzt davon ausgegengen, dass die Verbindung jeweils nur für den Zugriff geöffnet, und danach gfleich wieder geschlossen werden sollte, also nicht während der ganzen Zeit offen bleibt, bis das Programm geschlossen wird.

    @Murmeli81 Mach Dir eine Klasse, in der alle DB-Zugriffe ablaufen und speichere diese Daten in Membern dieser Klasse.
    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!

Facebook
Telefonspende