isn't getTagId() a kind of expensive O(n) operation?  It seems like you could use the integer constants here rather than looking up these strings at runtime.