B4J Question [solved] Test Null?

MarkusR

Well-Known Member
Licensed User
Longtime User
its a bug?
output is
null
null
B4X:
'B4J 7.51
Sub Test2
    
    Dim Params As Map
    Params.Initialize
    'Params.Put("Key","ABC") 'without this row .Get Return something that can Not be Test As null
    
    Dim MyString As String
    MyString = Params.Get("Key")
    Log(MyString)
    'If IsNull(MyString) Then MyString = "123"
    If MyString = Null Or MyString = "" Then MyString = "123"
    Log(MyString)
    
End Sub

Sub IsNull(O As Object) As Boolean
    Return (O=Null)
End Sub
 

amykonio

Active Member
Licensed User
Longtime User
Hi.
What happens if you change:
B4X:
MyString = Null
to
B4X:
MyString = "null"


Does it work?

Anyway. The way I would write it is
B4X:
Sub Test2
 Dim Params As Map
 Dim o As Object
 Params.Initialize
' Params.Put("Key","ABC") 'without this row .Get Return something that can Not be Test As null
 
 Dim MyString As String
 o = Params.Get("Key")
 If o Is Object Then
  MyString = o
 Else
  MyString = "123"
 End If
 Log(MyString)
End Sub

 
Upvote 0

MarkusR

Well-Known Member
Licensed User
Longtime User

hi erel, ahhh
that Null is converted to "null" string is special.
because missing quotes maybe you can modify this window that is it more clear what kind of value is displayed.
MyString null String
MyString null Object
or
MyString = "null" to be equal to source code


 
Upvote 0

amykonio

Active Member
Licensed User
Longtime User
I'm not sure this would be helpful...
All values for strings in this window are displayed without quotes. Also, how can we know if the value "null" wasn't set as a string?
This is the main reason I believe Erel suggests to avoid working with nulls - strings...
 
Upvote 0

MarkusR

Well-Known Member
Licensed User
Longtime User
intuitiv i would expect empty string "" if there is no key found. and a method Params.KeyExists("Key")
other languages would raise an error but i not like error handling for every method.
B4X:
Dim MyString As String
MyString = Params.Get("Key")
 
Upvote 0
Cookies are required to use this site. You must accept them to continue using the site. Learn more…