Sign in to follow this  
Followers 0
MSLx Fanboy

Error verifying GUI data

4 posts in this topic

Having a bit o' trouble verifying that the stuff entered into a GUI is what is actually valid. File attached.

Any help would be greatly appreciated, I don't use GUIs very often.

TSAuto_Kick.au3


Writing AutoIt scripts since
_DateAdd("d", -2, _NowCalcDate())

Share this post


Link to post
Share on other sites



check your syntax on the if statements I believe NOT in string comparison is incorrect

Try for example:

If StringInStr(StringUpper(GUICtrlRead($CtrlIPAddr)),"IP ADDRESS") Then
                MsgBox(0, "Teamspeak Auto-Kick Tool", "No IP Address Entered")
            Else
                $IPAddr = GUICtrlRead($CtrlIPAddr)
                If StringLen(TCPNametoIP($IPAddr)) > 0 Then
                    $IPAddr = TCPNametoIP($IPAddr)
                    $good += 1
                Else
                    MsgBox(0, "Teamspeak Auto-Kick Tool", "Bad IP Address")
                EndIf
            EndIf

SciTE for AutoItDirections for Submitting Standard UDFs

 

Don't argue with an idiot; people watching may not be able to tell the difference.

 

Share this post


Link to post
Share on other sites

#3 ·  Posted (edited)

Thanks! Now I just need to do some better testing for the TCPNametoIP() line

Edit: :( was using two different variables for the IP Address. Wow, I'm pretty slow there. Current version of the GUI below

5/14 1:21PM Updated script with checks for checkboxes, have a few more things to add...then to add in the real scripting part.

TSAuto_Kick.au3

Edited by MSLx Fanboy

Writing AutoIt scripts since
_DateAdd("d", -2, _NowCalcDate())

Share this post


Link to post
Share on other sites

I've finished the GUI part of it, made it into a func i plan to include in a separate file, for simplicity's sake.

Taking a break now...I'll come back and write the half of the script that actually does the work!

GUI Code below

TSAuto_Kick.au3


Writing AutoIt scripts since
_DateAdd("d", -2, _NowCalcDate())

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0