i dont know the full context of your code, but if you're just talking about getting a list of the IDs based on a store, then yes, that's probably about as simple as you'll get. however, you should probably technically do:

getSelectionModel().getSelections()

I have come to like Ext.each() instead of the for loop, but that's not really consequential one way or the other.