GUI login

[Pottery]

This is getting very frustrating :\..

When I click the login button it should popup the message "Good", but I keep getting "Bad" :/

This is just testing to see if it's going to login or not, it's meant to be a client so you login and go to your user control panel which I have in another file, but I can't unless I figure this out and I need some help; here's the code..

expandcollapse popup

; Gone2pwn user client (version 1.0)

#include <ButtonConstants.au3>
#include <EditConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#Region ### START Koda GUI section ### Form=
$Form1_1 = GUICreate("Form1", 667, 267, 192, 124)
$Pic1 = GUICtrlCreatePic("", 0, 0, 665, 185, BitOR($SS_NOTIFY,$WS_GROUP,$WS_CLIPSIBLINGS))
$Label1 = GUICtrlCreateLabel("Username:", 8, 192, 70, 22)
GUICtrlSetFont(-1, 10, 800, 0, "Palatino Linotype")
GUICtrlSetColor(-1, 0x000000)
$Label2 = GUICtrlCreateLabel("Password:", 8, 216, 67, 22)
GUICtrlSetFont(-1, 10, 800, 0, "Palatino Linotype")
GUICtrlSetColor(-1, 0x000000)
$user = GUICtrlCreateInput("", 96, 192, 121, 21)
$pass = GUICtrlCreateInput("", 96, 216, 121, 21)
$Button1 = GUICtrlCreateButton("LOGIN", 8, 240, 75, 17, $WS_GROUP)
$Group1 = GUICtrlCreateGroup("", 264, 192, 353, 65)
$Label3 = GUICtrlCreateLabel("Gone2pwn user client version 1.", 272, 200, 199, 17)
$Label4 = GUICtrlCreateLabel("Next version will include actual graphics instead of just a plain window.", 272, 224, 336, 17)
GUICtrlCreateGroup("", -99, -99, 1, 1)
GUISetState(@SW_SHOW)
#EndRegion ### END Koda GUI section ###

While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
        Case $Button1
            CheckIfValid()
    EndSwitch
WEnd

Func CheckIfValid()
    If $user = "1" And $pass = "1" Then
        MsgBox(0, "1", "Good")
    Else
        MsgBox(0, "0", "Bad")
        Exit
    EndIf
EndFunc

[somdcomputerguy]

Try this. Change If $user = "1" And $pass = "1" Then to If GUICtrlRead($user) = "1" And GUICtrlRead($pass) = "1" Then

[Pottery]

Works! Thanks alot

[somdcomputerguy]

You bet.