Sign in to follow this  
Followers 0
cparadis

isnumber() fails

4 posts in this topic

I don't know what i doing wrong. I want to check if 127.0.0.1 is numbers.

And i removed port and the dot's, but it fails :)

$ip = "127.0.0.1:1234"

;Remove the port
$var = stringsplit($ip, chr(58))

;Replace . with nothing.
$var = stringreplace($var[1],chr(46),"")

;If variable is number, msgbox..
if IsNumber($var) == 1 then
    
    msgbox(0, "", $var) 
else
    msgbox(0, "", "failed")
EndIf

Share this post


Link to post
Share on other sites



it fails because ''$var'' it's a string, no matter if they are number it is treated as string, you have to convert it to number first

$ip = "127.0.0.1:1234"

;Remove the port
$var = stringsplit($ip, chr(58))

;Replace . with nothing.
$var = stringreplace($var[1],chr(46),"")

;If variable is number, msgbox..
if IsNumber(Number($var)) == 1 then

    msgbox(0, "", $var)
else
    msgbox(0, "", "failed")
EndIf

Share this post


Link to post
Share on other sites

cparadis,

$var is a string - you need to convert it to a number first.

$var = Number(stringreplace($var[1],chr(46),""))

or

IsNumber(Number($var))

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

#4 ·  Posted (edited)

Haha, that was the problem. Thanks for the help :)

Edit:

Fixed it, thankls for the help :party:

Edited by cparadis

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