Android Question [B4X] shorter if

[Alexander Stolte]

hey, can this be simplified somehow?

If tmp_m.Get("type") = 1 Or tmp_m.Get("type") = 2 Or tmp_m.Get("type") = 3 Or tmp_m.Get("type") = 4 Or tmp_m.Get("type") = 5 Or tmp_m.Get("type") = 6 Or tmp_m.Get("type") = 7 Or tmp_m.Get("type") = 8 Then
                
Log("test")           
                
End If

 

[angel_]

If 9 > tmp_m.Get("type") Then
               
Log("test")
               
End If

 

[MikeSW17]

Why Not:

If tmp_m.Get("type") > 0 and tmp_m.Get("type") < 9 Then
    Log("test")
End If

 

[Erel]

I wouldn't call Map.Get multiple times like this. Not for performance reasons, but to make it easier to change the key in the future.

Dim i As Int = tmp_m.Get("type")
If i >= 1 And i <= 9 Then

 

[Alexander Stolte]

Is this possible?:

If in array(1,2,3,6,7,8,20) = tmp_m.Get("type") then

 

[angel_]

Maybe this:

Dim tipo As String = tmp_m.Get("type")
Dim Lista As List = Array As String(1, 2, 3, 6, 7, 8, 20)
Log(Lista.IndexOf("" & tipo) > -1)

 

[Sandman]

In theory it might be possible to do something like this. Doesn't work though, IndexOf isn't supported by arrays. Perhaps there's an alternative way of writing this that works?

If (Array As String(1, 2, 3, 6, 7, 8, 20)).IndexOf(tmp_m.Get("type")) > -1 Then
   ' Code goes here
End If

 

[Erel]

Sub InArray(Item As Object, Arr As List) As Boolean
 Return Arr.IndexOf(Item) <> -1
End Sub

If InArray(7, Array(1, 2, 3, 4, 5, 6)) Then

 

[Mahares]

Here is my 2 cents worth in my suitCASE or briefCASE:

Select 5
        Case  1, 2, 3, 4, 5, 6
            Log("I am here")
        Case Else
            Log("I don't exist")
    End Select

 

[Sandman]

Here is my 2 cents worth in my suitCASE or briefCASE

I would hope the compiler adjusted your code to:

Log("I am here")