This `for` loop can be replaced with an iterative for loop, just like is done with `map.keySet()`

I would recommend:

```
   for(Character character : characters){
      if(character.getFirstName() == val){
         chars.add(character);
      }
   }