Why won't this work?

[SirDarknight1200]

#include <ButtonConstants.au3>
#include <EditConstants.au3>
#include <GUIConstantsEx.au3>
#include <WindowsConstants.au3>

$Form1 = GUICreate("Form1", 615, 438, 192, 124)
$Input1 = GUICtrlCreateInput("Input1", 24, 32, 265, 21)
$Input2 = GUICtrlCreateInput("Input2", 24, 80, 265, 21)
$Button1 = GUICtrlCreateButton("Button1", 296, 240, 225, 73)
GUISetState(@SW_SHOW)


While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
        Case $Button1
            If $Input1 > $Input2 Then
                MsgBox(0,"Test", "Greater")
            Else
                MsgBox(0, "Test", "Smaller")
            EndIf

    EndSwitch
WEnd

What I want is if I put 2 in input1 and 1 in input2 and press the button. the msgbox will show greater. But it shows smaller everytime

[TheSaint]

You are just getting the ID of the Inputs.

You need to read the value you set in the Inputs.

Case $Button1
            If GUICtrlRead($Input1) > GUICtrlRead($Input2) Then
                MsgBox(0,"Test", "Greater")
            Else
                MsgBox(0, "Test", "Smaller")
            EndIf

You also aren't checking for when they are equal.

[TheSaint]

All up, you would be better off assigning what you read, to variables.

Case $Button1
            $val_1 = GUICtrlRead($Input1)
            $val_2 = GUICtrlRead($Input2)
            If $val_1 > $val_2 Then
                MsgBox(0,"Test", "Greater")
            ElseIf $val_1 < $val_2 Then
                MsgBox(0,"Test", "Smaller")
            Else
                MsgBox(0, "Test", "Equal")
            EndIf

[SirDarknight1200]

Thanks a lot