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:
"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...