Shouldn't this be rather an enum?