VB + MySQL: DataGridView update/refresh live bzw. in realtime

  • VB.NET

Es gibt 7 Antworten in diesem Thema. Der letzte Beitrag () ist von dennis1702.

    VB + MySQL: DataGridView update/refresh live bzw. in realtime

    Hallo Zusammen,

    zuerst mal vorweg, es ist das ERSTE MAL, dass ich mich innerhalb eines Forums bezüglich Probleme/Fragen zu Programmieren/Datenbanken wende.
    Habt ein wenig nachsicht, wenn ich mich evtl. unklar ausdrücke oder bei beantworteten Sachen nicht direkt antworte (Familie hat bei mir immernoch vorrang ;) )

    So wo das geklärt ist, kommen wir zu meinem Anliegen:

    Ich habe ein bisher kleines "Tool" welches sich aber WEIT vergrößern soll, sobald ich das Problem lösen kann, Grundlegend könnte man sagen es ist ein ToDo/Aufgaben Programm, welches aber im späteren Verlauf auch eine Art Ticketsystem (Ähnlich zu Jira z.B.) bekommen soll. Aber soweit ist es noch nicht. Zu Anfang meldet sich jeder erstmal an und dafür habe ich bisher in VB einen Anmeldebildschirm gebastelt, sieht so aus -> Siehe Bild Anmeldefenster.jpg
    Dieses Anmeldefenster meldet den Benutzer (Je nachdem wer es ist) an einer MySQL Datenbank im Remoteverfahren an.

    VB.NET-Quellcode

    1. Imports MySql.Data.MySqlClient
    3. Public Class Form1
    5. Private Sub LinkLabel1_LinkClicked(sender As Object, e As LinkLabelLinkClickedEventArgs) Handles LinkLabel1.LinkClicked
    6. System.Diagnostics.Process.Start("mailto:dennis@fahrdienst-rhein-main.de")
    7. End Sub
    9. Private Sub OK_Click(sender As Object, e As EventArgs) Handles OK.Click
    10. con = NewCon
    11. con.ConnectionString = "server=123456789url.de;userid=user;password=passwort;database=ramar;SslMode=none"
    13. Try
    14. con.Open()
    15. Dim Query As String
    16. Query = "select * from ramar.user where username='" & usertextbox.Text & "' and password='" & passwordtextbox.Text & "' "
    17. cmd = New MySqlCommand(Query, con)
    19. con.Close()
    21. Catch ex As MySqlException
    22. MessageBox.Show(ex.Message)
    24. Finally
    25. con.Dispose()
    27. End Try
    29. If usertextbox.Text = "User1" And passwordtextbox.Text = "Passwort" Then
    30. Call Ralf()
    31. ElseIf usertextbox.Text = "User2" And passwordtextbox.Text = "Passwort" Then
    32. Call Kristin()
    33. ElseIf usertextbox.Text = "User3" And passwordtextbox.Text = "passwort" Then
    34. Call Vera()
    35. ElseIf usertextbox.Text = "User4" And passwordtextbox.Text = "passwort" Then
    36. Call Hasan()
    37. ElseIf usertextbox.Text = "UserA" And passwordtextbox.Text = "passwort" Then
    38. Call Admin()
    39. ElseIf usertextbox.Text = "" Or passwordtextbox.Text = "" Then
    40. MsgBox("Benutzer- und Passwortfeld dürfen nicht leer sein!", MsgBoxStyle.Critical, "ACHTUNG!!!")
    41. Else
    42. MsgBox("Falscher Benutzername oder Passwort!", MsgBoxStyle.Critical, "ACHTUNG!!!")
    43. End If
    45. End Sub
    47. Private Sub UserA()
    49. UserAForm.Show()
    50. Me.Hide()
    52. End Sub
    54. Private Sub User1()
    56. User1Form.Show()
    57. Me.Hide()
    59. End Sub
    61. Private Sub User2()
    63. User2Form.Show()
    64. Me.Hide()
    66. End Sub
    68. Private Sub User3()
    70. User3Form.Show()
    71. Me.Hide()
    73. End Sub
    75. Private Sub User4()
    77. User4Form.Show()
    78. Me.Hide()
    80. End Sub
    82. Private Sub Cancel_Click(sender As Object, e As EventArgs) Handles Cancel.Click
    83. Me.Close()
    84. End Sub
    85. End Class


    Datenbank liegt mittels Portweiterleitung im Internet. Die Version von MySQL ist 5.5, falls das wichtig ist.
    Nachdem sich User oder Admin angemeldet hat, hat man eine Art Menü für (später irgendwann) mehrere Tabellen/Funktionen.
    In diesem Fall nehmen wir als Beispiel mal die Personal Tabelle, wo 2-3 User neues Personal in einem Unternehmen anlegen können. Form -> Siehe Bild PersonalFenster.jpg
    Die Linken TextBoxen ignorieren wir vorerst (Habe zuerst mit der ListView Option gebaut).
    Hier mal der Code den ich bisher in der Personalform habe:

    VB.NET-Quellcode

    1. Imports MySql.Data.MySqlClient
    3. Public Class PersonalForm
    5. ''Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click
    6. ''Try
    7. 'FALSCH -> sql = "INSERT INTO 'ramar'.'Personal'('Vorname','Nachname','Abteilung','E-Mail','Telefon') VALUES('" & TextBox1.Text & "','" & TextBox2.Text & "','" & TextBox3.Text & "','" & TextBox4.Text & "','" & TextBox5.Text & "')"
    8. 'RICHTIG v'
    9. '' sql = "INSERT INTO `ramar`.`Personal` (`Vorname`, `Nachname`, `Abteilung`, `E-Mail`, `Telefon`) VALUES ('" & TextBox1.Text & "','" & TextBox2.Text & "','" & TextBox3.Text & "','" & TextBox4.Text & "','" & TextBox5.Text & "')"
    10. 'ConnectionRoot()
    11. '' cmd = New MySqlCommand(sql, NewCon)
    12. '' cmd.ExecuteNonQuery()
    13. '' MsgBox("Person angelegt")
    14. ''Catch ex As Exception
    15. '' MsgBox(ex.Message)
    16. ''Finally
    17. '' cmd.Dispose()
    18. '' NewCon.Close()
    19. '' End Try
    21. 'Me.PersonalTableAdapter.Fill(Me.RamarData.Personal)
    23. '' End Sub
    25. Private Sub Button3_Click(sender As Object, e As EventArgs) Handles Button3.Click
    27. Me.Close()
    29. End Sub
    31. Private Sub Button2_Click(sender As Object, e As EventArgs) Handles Button2.Click
    32. TextBox1.Text = ""
    33. TextBox2.Text = ""
    34. TextBox3.Text = ""
    35. TextBox4.Text = ""
    36. TextBox5.Text = ""
    37. End Sub
    39. 'Private Sub PersonalBindingNavigatorSaveItem_Click(sender As Object, e As EventArgs)
    40. 'Me.Validate()
    41. 'Me.PersonalBindingSource.EndEdit()
    42. 'Me.PersonalTableAdapter.Update(Me.RamarData.Personal)
    44. 'End Sub
    46. Private Sub PersonalForm_Load(sender As Object, e As EventArgs) Handles MyBase.Load
    47. 'TODO: Diese Codezeile lädt Daten in die Tabelle "RamarData.Personal". Sie können sie bei Bedarf verschieben oder entfernen.
    48. Me.PersonalTableAdapter.Fill(Me.RamarData.Personal)
    49. 'Me.PersonalDataGridView.DataSource = Me.RamarData.Personal
    50. 'Me.PersonalDataGridView.Refresh()
    51. If RamarData.HasChanges(DataRowState.Added) Then
    52. Me.Validate()
    53. PersonalBindingSource.EndEdit()
    54. Me.PersonalTableAdapter.Update(Me.RamarData.Personal)
    55. End If
    56. If RamarData.HasChanges(DataRowState.Deleted) Then
    57. Me.Validate()
    58. PersonalBindingSource.EndEdit()
    59. Me.PersonalTableAdapter.Update(Me.RamarData.Personal)
    60. End If
    61. If RamarData.HasChanges(DataRowState.Modified) Then
    62. Me.Validate()
    63. PersonalBindingSource.EndEdit()
    64. Me.PersonalTableAdapter.Update(Me.RamarData.Personal)
    65. End If
    67. Timer1.Start()
    69. 'If Me.InvokeRequired Then
    70. 'Me.Invoke(Sub() 'lampda
    72. 'End Sub)
    73. 'Else
    75. 'End If
    77. End Sub
    79. Private Sub Button4_Click(sender As Object, e As EventArgs) Handles Button4.Click
    81. Me.Validate()
    82. PersonalBindingSource.EndEdit()
    83. Me.PersonalTableAdapter.Update(Me.RamarData.Personal)
    85. End Sub
    87. Private Sub Timer1_Tick(sender As Object, e As EventArgs) Handles Timer1.Tick
    88. Me.PersonalDataGridView.DataSource = Me.RamarData.Personal
    89. Me.PersonalDataGridView.Refresh()
    90. End Sub
    92. Private Sub PersonalDataGridView_CellContentClick(sender As Object, e As DataGridViewCellEventArgs) Handles PersonalDataGridView.CellContentClick
    94. End Sub
    96. Private Sub ButtonNew_Click(sender As Object, e As EventArgs) Handles ButtonNew.Click
    97. PersonalBindingSource.AddNew()
    98. End Sub
    100. 'Private Sub Tableupdate()
    102. 'If RamarData.HasChanges(DataRowState.Added) Then
    104. ' New rows have been added to the dataset, add appropriate code.
    105. 'Else
    106. ' No new rows have been added to the dataset, add appropriate code.
    107. 'End If
    108. 'End Sub
    110. End Class


    Wie man sieht, ich habe ein paar Sachen bisher ausprobiert (Siehe auskommentierte Zeilen).
    Mein Problem ist nur, entweder es muss ein Aktualisieren Button gedrückt werden, um das DataGridView aktuell zu halten oder ich brauche einen Timer, der aber beim Aktualisieren nicht jedesmal los meckert, dass bei der Eingabe von neuem Personal kein "NULL" zulässig ist.
    Die Daten, die ich im BindingSource eingebe, werden, man glaubt es kaum, direkt ins DataSet geschrieben. Dies konnte ich anhand des ersten Test's mit dem Timer beim Verändern der bereits eingetragenen Daten erkennen, dass der Timer direkt aktualisiert hat. Das geht allerdings nicht, wenn man neues Personal anlegt, da das Dataset ja dann direkt erstmal eine "NULL" Zeile erhält und sobald der Timer durchläuft erkennt er Fehleingaben.

    Folgendes ist für mich wichtig, bzw. würde ich gerne erreichen: Eine Automatische aktualisierung des DataGridViews, sobald ein User eine Zeile einträgt, sodass auch alle anderen Benutzer des Tools erkennen "Aha, da ist was eingetragen worden, bzw. ah, der Kollege X hat bereits den neuen Kollegen Y angelegt". Im Idealfall (Für Technisch unbegabte Mitarbeiter) ohne einen Akualisierungsbutton. Halt Automatisch.

    Meine Frage an euch ist jetzt, gibt es eine Methode, wo sowas möglich ist? Ich habe bereits dutzende Stunden im Netz mit der Suche verbracht und muss jetzt einfach hier fragen, so unangenehm wie es mir auch ist, weil ich die befürchtung habe, dass es sowas ganz banales ist, wo ich aber nicht drauf gekommen bin. ?(

    Ich danke euch schon mal im Voraus und entschuldige mich, falls einiges an Text hier total unnötig ist. ;)

    Gruß Dennis

    EDIT: Ich vergaß eine Frage: Wo liegt eigentlich der Unterschied zwischen DataGridView.Refresh() und Me.PersonalDataTable.Update(...)??
    Bilder

    • Anmeldefenster.jpg

      558,29 kB, 1.920×1.080, 195 mal angesehen

    • PersonalFenster.jpg

      536,49 kB, 1.920×1.080, 209 mal angesehen
    Programmieren sollte für Personen, die was ordentliches machen wollen, eine Leidenschaft sein. Ansonsten entsteht nur Müll und davon haben wir bereits genug in der Gesellschaft. :D

    Wie heißt es so schön, willkommen im Forum.

    Wenn ich deinen bisherigen Code so anschauen, wird das bald nicht mehr wartbar.
    Du hast pro User eine namentliche Sub und ein Form. Warum? Wenn ein User hinzukommt oder wenn einer wegfällt musst du alles umschreiben.
    Du solltest deine Controls vernünftig benennen, dann hast du mehr Überblick.

    Deine ganze Anmeldung ist offen für SQL Injection und strotzt vor, bereits erwähnten, hard coded Kram.

    Den Unterschied zwischen dem Update und dem Refresh hätte man schon nach kurzer Suche in der MSDN finden können.

    Zu dem Update der anderen Clients wenn ein User schon einen Eintrag erfasst hat, woher sollen die anderen Clients etwas davon wissen? Dafür benötigt man einen Server Dienst der aktiv die Clients über Änderungen benachrichtigt. Zum Beispiel SignalR.

    Grundlegend fehlt es an so viel, dass das Erlernen der Basics eine gute Idee wäre.
    Die deutsche Sprache ist Freeware, du kannst sie benutzen, ohne dafür zu bezahlen. Sie ist aber nicht Open Source, also darfst du sie nicht verändern, wie es dir gerade passt.

    Hier einige Tipps meinerseits:
    1. Benenne deine Controls immer sauber
    2. Lagere den DB Zugriff in eine separate Klasse aus
    3. Nutze die SQL Command Parameters damit man keine SQL injection durchführen kann
    4. Arbeite Objektorientiert
    5. Lerne wie man Forms korrekt instanziiert
    6. Obligatorisch: Arbeite Option Strict On (falls noch nicht passiert)

    Dein Code ist so wie er jetzt ist früher oder später nicht mehr wartbar (Spaghetticode). Ich würde die automatische aktualisierunge erstmal komplett weg lassen. Das kann man alter immer noch einbauen wenn das Grundgerüst steht. Lieber nen Time laufen lassen der informativ anzeigt das es nen neues Ticket gibt. Dann kann der User immer noch entscheiden ob er manuell die Daten neu läd...
    "Gib einem Mann einen Fisch und du ernährst ihn für einen Tag. Lehre einen Mann zu fischen und du ernährst ihn für sein Leben."

    Wie debugge ich richtig? => Debuggen, Fehler finden und beseitigen
    Wie man VisualStudio nutzt? => VisualStudio richtig nutzen

    Danke MrTrebron und danke mrMo für eure Antworten.

    Ich stimme euch beiden zu, der Code ist wahrlich sehr wüst. Lag daran, dass ich mehrfach das Projekt gestartet habe (immer wieder anders) und meist an diesem Problem gescheitert bin und den Aufbau immer neu gestaltet habe.
    Daher könnte es nun sein, dass ich einiges erstmal total besch*** geschrieben habe (unübersichtlich) weil ich dieses Problem erstmal überwinden wollte.
    Aber ich werde mich die Tage hinsetzen und mal alles übersichtlich gestalten.

    Den Unterschied zwischen dem Update und dem Refresh hätte man schon nach kurzer Suche in der MSDN finden können.


    Ja natürlich, da stimme ich dir zu, allerdings habe ich VIELE Seiten gefunden, wo sowohl das eine, als auch das andere für dasselbe Benutzt wurden. Daher wollte ich hier mal fragen.

    Du hast pro User eine namentliche Sub und ein Form. Warum? Wenn ein User hinzukommt oder wenn einer wegfällt musst du alles umschreiben.


    Das liegt daran das meine Idee war, dass jeder User seine eigene Oberfläche mit wichtigen Funktionen für diesen User hat. Da ja jeder was anderes braucht.
    Sozusagen, braucht die Abteilung Fuhrpark keine Aufgabenverteilung an Abteilung IT, da diese nicht direkt zusammen arbeiten (zumindest bei mir), Geschäftsleitung darf Grundsätzlich alles und kann jede Funktion nutzen. Abteilung Buchhaltung soll eine Zeiterfassung für die Mitarbeiter machen können, sowas soll aber Abteilung Fuhrpark, IT und andere nicht. Das war dann für mich das schnellste Gerüst.

    Also werde ich am Montag erstmal ein wenig umbauen und würde mich dann hier noch mal melden.
    Vielen Dank nochmal.
    Programmieren sollte für Personen, die was ordentliches machen wollen, eine Leidenschaft sein. Ansonsten entsteht nur Müll und davon haben wir bereits genug in der Gesellschaft. :D

    Also zunächst solltest du dir ein Datenmodell überlegen. Zudem Strukturen, Einstellungen, Funktionen die dein Programm nachher haben soll. Du brauchst also ein KONZEPT. Ohne das, codest du wild irgendwas zusammen... Also: Papier und Stift und Programm konzeptionieren.

    Was brauche ich?
    - Benutzer
    - Benutzerrollen
    - Einstellungen
    - DB Zugriff
    - Tickets
    - Ticket Status (neu, bearbeitung, erledigt, usw.)
    - und und und

    Ich würde den Benutzern, wie @MrTrebron schrieb, mit Benutzerrollen arbeiten. Über die Rolle steuerst du dann was wer machen darf (löschen, bearbeiten, usw.) und was wer sehen kann (Auswertungen, Einstellungen, Fuhrpark, usw.).

    Zudem rate ich davon ab, alles hart rein zu programmieren. Versuche dein Programm einstellbar zu machen. Ist zwar mehr zum programmieren, dafür bist du hinten raus flexibler...
    "Gib einem Mann einen Fisch und du ernährst ihn für einen Tag. Lehre einen Mann zu fischen und du ernährst ihn für sein Leben."

    Wie debugge ich richtig? => Debuggen, Fehler finden und beseitigen
    Wie man VisualStudio nutzt? => VisualStudio richtig nutzen

    Papier und Stift, klingt wie Back to the Roots. Aber hast schon recht.
    Prinzipiell habe ich den Anfang ja, bis auf Benutzerrollen und Einstellungen. Da bin ich ja aber durch MrTrebron drauf gebracht worden, dass so zu machen.
    Werde mir da mal eine Aufstellung bzw. das Konzept zusammenstellen und erst dann mal alles überflüssige (wie das Auskommentierte z.B.) raus werfen. So wird das ganze dann hoffentlich aufgeräumter.
    Den DB-Zugriff habe ich btw. in eine eigene Klasse ausgelagert, aber habe im Anmeldeformular irgendwie noch die Anmelderoutine ausgeschrieben (k.A. warum).
    Wie gesagt, ich schreib erstmal auf, wie es nach der Anmeldung weiter gehen soll und sobald das Konzept fertig steht kehre ich zurück.
    Programmieren sollte für Personen, die was ordentliches machen wollen, eine Leidenschaft sein. Ansonsten entsteht nur Müll und davon haben wir bereits genug in der Gesellschaft. :D