It's more of a consistency/guidelines thing really, we should avoid hard code strings in this manner. It only looks like we use this activity in this class, do we use the other activity types elsewhere in the code?