# The spy among us help

**URL:** <https://discourse.codecombat.com/t/the-spy-among-us-help/9216>\
**Category:** Ambassador\
**Created:** [October 6, 2016, 10:01pm UTC](https://discourse.codecombat.com/t/the-spy-among-us-help/9216 "2016-10-06T22:01:16Z")\
**Posts on this page:** 3\
**Page:** 1

<div class="post-metadata">

**Author:** ![mattgamer](https://avatars.discourse-cdn.com/v4/letter/m/b4bc9f/32.png) [@mattgamer](https://discourse.codecombat.com/u/mattgamer)\
**Post date:** [October 6, 2016, 10:01pm UTC](https://discourse.codecombat.com/t/the-spy-among-us-help/9216/1 "2016-10-06T22:01:16Z")

</div>

I don’t know if something is wrong with the function but my hero says the spy name as a friend no matter what.  
here is my code:

```python
function letterInWord(word, letter) {
    for (var i = 0; i < word.length; i++) {
        var character = word[i];
        if (character == letter) {
            return true;
        }
        return false;
    }
}
var spyLetter = "z";
var friends = hero.findFriends();
for (var j = 0; j < friends.length; j++) {
    var friendName = friends[j].id;
    if (letterInWord(friendName, spyLetter)) {
        hero.say(friendName + " is a spy!!!");
    } else {
        hero.say(friendName + " is a friend.");
    }
}

```

---

<div class="post-metadata">

**Author:** ![Bryukh](https://sea2.discourse-cdn.com/flex016/user_avatar/discourse.codecombat.com/bryukh/32/19597_2.png) [@Bryukh](https://discourse.codecombat.com/u/Bryukh)\
**Post date:** [October 7, 2016, 4:32am UTC](https://discourse.codecombat.com/t/the-spy-among-us-help/9216/2 "2016-10-07T04:32:58Z")

</div>

> [@mattgamer](#):
>
> for (var i = 0; i \< word.length; i++) {  
> var character = word[i];  
> if (character == letter) {  
> return true;  
> }  
> return false;  
> }

Your `return false` is too early. It’s called after the first iteration (letter).

---

<div class="post-metadata">

**Author:** ![mattgamer](https://avatars.discourse-cdn.com/v4/letter/m/b4bc9f/32.png) [@mattgamer](https://discourse.codecombat.com/u/mattgamer)\
**Post date:** [October 7, 2016, 6:51pm UTC](https://discourse.codecombat.com/t/the-spy-among-us-help/9216/3 "2016-10-07T18:51:38Z")

</div>

ok thanks, I moved it and it worked