Wish Show in quick info window that Map.GetKeyAt and GetValueAt are deprecated

[raphaelcno]

When a map is used, I suggest to show in the quick info window that GetKeyAt and GetValueAt methods are deprecated, and that For Each loop or B4XOrderedMap should be used instead:

Edit after post #3: For Each loop

 

[thetahsk]

..

take a look here..

[B4X] "Code Smells" - common mistakes and other tips

"Code smells" are common patterns that can indicate that there is a problem in the code. A problem doesn't mean that the code doesn't work, it might be that it will be difficult to maintain it or that there are more elegant ways to implement the same thing. Remember that not everything is clear...

www.b4x.com

 

[raphaelcno]

Yes, you're right, the For Each loop should be used instead:

For Each key As String In Map1.Keys
    Dim value As String = Map1.Get(key)
Next

I have edited post #1.