There are two issues with the use of stopChan here:
* Channels are typically used to communicate between separate goroutines. It's a code smell to be putting and pulling onto the same channel in a single function.
* In the <-stopChan case below, all you're doing is setting ok to false. But this if condition only fires if ok is *already* false.
In other words, you can get rid of stopChan altogether and just change the if clause to:
```go
    if !ok {
        break
    }
```