WinForms

Απ: Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

Q-apprentice — Thu, 06 May 2010 17:42:09 GMT

μια όθόνη που έχω για Login την καλώ με την LoginFrm.ShowDialog() μέσα από την CentralScreen.frm και το application.exit() το χρησιμοποιώ γιατί με βολεύει.....τι άλλο μπορώ να κάνω για να είναι πιο σωστή η εφαρμογή ?

Απ: Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

KelMan — Wed, 05 May 2010 23:35:31 GMT

Το πρόβλημα δεν είναι η σωστή επιλογή handler, είναι το ότι κάνεις Application.Exit μέσα στο FormClosing. Το Application.Exit έχει ως αποτέλεσμα να τρέχουν όλοι οι FormClosing handlers από τις διάφορες φόρμες. Το ερώτημα είναι γιατί κάνεις Application.Exit μέσα από το FormClosing; Είσαι σε κάποια child φόρμα;

Απ: Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

Q-apprentice — Wed, 05 May 2010 23:19:34 GMT

Τελικά ήταν λάθος επιλογή event handler...το σωστό κομμάτι κωδικα είναι το εξής:

    Private Sub Form1_FormClosed(ByVal sender As Object, ByVal e As FormClosedEventArgs) _
         Handles Me.FormClosed
        Try
            LogoutRec()
        Catch ex As Exception
            MsgBox(ex.ToString)
        End Try
        Application.Exit()
    End Sub

Το δοκίμασα και δουλεύει κανονικά....

Απ: Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

Q-apprentice — Wed, 05 May 2010 23:14:28 GMT

Αν κατάλαβα καλά θέλεις να βγάλω την 2η γραμμή που κάνει το handling .....το έκανα αλλά δεν περνάει καν απο εκεί..απλά κλείνει την εφαρμογή....στην προηγούμενη περίπτωση με το που το τρέχει περνάει από το application.exit() αλλά δεν το κλεινει ! ...θα κοιτάξω από εκεί μπας και έχω ξεχάσει κάτι...απλά μου κάνει εντύπωση.....αν έχεις να προτείνεις κάτι άλλο ευχαρίστως να το ακούσω...

Απ: Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

KelMan — Wed, 05 May 2010 21:15:35 GMT

Επίσης, αν ο κώδικάς σου είναι production θα πρέπει απαραιτήτως να αλλάξεις αυτά τα string concatenations με parameterized queries γιατί διαφορετικά μπορεί οποιοσδήποτε κακόβουλος να σου κάνει SQL injection attack. Κάνε μια σχετική αναζήτηση και στο dotNETZone, το έχουμε ξανακουβεντιάσει το θέμα αρκετές φορές.

Απ: Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

KelMan — Wed, 05 May 2010 21:12:57 GMT

Επειδή λογικά αυτό δεν θα έπρεπε να συμβαίνει, κάνε την εξής δοκιμή:

Αφαίρεσε το Handles Me.FormClosing από το Private Sub Form1_FormClosing(...) και δοκίμασε να τρέξεις ξανά την εφαρμογή. Όταν φτάσεις στο επίμαχο σημείο, βάλε ένα breakpoint στην κλήση της LogoutRec και όταν το χτυπήσει, δες το Call Stack για να καταλάβεις πώς έχει φτάσει εκεί η εκτέλεση. Μπορεί κάπου αλλού να καλείται η FormClosing, μπορεί να υπάρχει κανένα AddHandler που να την προσθέτει δεύτερη φορά ως event handler για το συγκεκριμένο event. Αν δεν χτυπήσει καμία φορά το breakpoint, βάλε ξανά το Handles Me.FormClosing και δοκίμασε πάλι την ίδια διαδικασία και έλεγξε πάλι το Call Stack κάθε μία από τις δύο φορές που θα χτυπήσει. Αν δεν χτυπήσει δύο φορές αλλά μόνο μια, πάει να πει ότι κάπου αλλού υπάρχει το bug.

Το query τρέχει 2 φορές κατά το κλείσιμο της φόρμας....

Q-apprentice — Wed, 05 May 2010 17:46:25 GMT

Καλημέρα σε όλους. Είχα ρωτήσει πιο παλιά για το πως μπορώ να ελέγχω events κατά την διάρκεια του form closing και επιλύθηκε μια χαρά χάρις την βοήθεια των ανθρώπων εδώ. Αυτό που αντιμετωπίζω όμως είναι το εξής :

Όταν κλείνει η φόρμα τρέχω ένα query το οποίο καταγράφει την έξοδο των χρηστών στην βάση (κάνω το ίδιο και για την είσοδο). Το query, κατά την έξοδο όμως, τρέχει 2 φορές (!) με αποτέλεσμα να έχω διπλή εγγραφή στην βάση για την έξοδο του ίδιου χρήστη.

το module που έχω φτιάξει για login-logout, είναι το εξής :

Imports System.Windows.Forms
Imports System.Data.SqlClient
Module General
    '==================================================================================================
    'Query Εισόδου Χρηστών...
    Public login As String = "INSERT INTO [20_UserLog] " _
                            & "(UserID, FName, LName, Logdate, In_Out)" _
                            & "VALUES " _
                            & "(" & Convert.ToInt32(CentralScreen.UserIDText.Text) & ", " _
                            & "(SELECT FName FROM [20_Users] WHERE UserID = " & Convert.ToInt32(CentralScreen.UserIDText.Text) & "), " _
                            & "(SELECT LName FROM [20_Users] WHERE UserID = " & Convert.ToInt32(CentralScreen.UserIDText.Text) & "), " _
                            & "GETDATE(), " _
                            & "'I')"
    'Query Εξόδου Χρηστών...
    Public logout As String = "INSERT INTO [20_UserLog] " _
                            & "(UserID, FName, LName, Logdate, In_Out)" _
                            & "VALUES " _
                            & "(" & Convert.ToInt32(CentralScreen.UserIDText.Text) & ", " _
                            & "(SELECT FName FROM [20_Users] WHERE UserID = " & Convert.ToInt32(CentralScreen.UserIDText.Text) & "), " _
                            & "(SELECT LName FROM [20_Users] WHERE UserID = " & Convert.ToInt32(CentralScreen.UserIDText.Text) & "), " _
                            & "GETDATE(), " _
                            & "'O')"
    '==================================================================================================
    'Ορισμός των SQL Commands για το Login/Logout των χρηστών...
    Public loginCMD As New SqlCommand(login, SQLConn.Connection)
    Public logoutCMD As New SqlCommand(logout, SQLConn.Connection)
    '==================================================================================================
    Public Sub LoginRec()
        loginCMD.ExecuteNonQuery()
        SQLConn.Connection.Close()
    End Sub
    Public Sub LogoutRec()
        logoutCMD.ExecuteNonQuery()
        SQLConn.Connection.Close()
    End Sub
End Module

...και καλείται από την παρακάτω ρουτίνα η οποία καλείται όταν ο χρήστης πατήσει το κουμπί "Χ" της φόρμας...

    Private Sub Form1_FormClosing(ByVal sender As Object, ByVal e As FormClosingEventArgs) _
         Handles Me.FormClosing
        LogoutRec()
        Application.Exit()
    End Sub

..αν μπορείτε να βοηθείσετε θα το εκτιμήσω πολύ....

ευχαριστώ εκ των προτέρων....