I like that, but instead of looping through a Set I would just call them out explicitly:

```java
excludeContentStateFromQuery(ContentState.NEED_REVIEW, query)
excludeContentStateFromQuery(ContentState.NEW, query)
excludeContentStateFromQuery(ContentState.REJECTED, query)
```
Makes it easier to read as it's only 3, plus we don't have a set lying around there for nothing.