Yield not working in C#

I'm trying to build FPS counter that will change only once per second in C#.

I have this in class definition:

float FPS;

This in `void OnGUI()`:

GUI.Label(new Rect(0, 30, 600, 30), string.Format("FPS: {0}", this.FPS));

And this function in class:

IEnumerator ShowFPS()
    FPS = 1 / Time.smoothDeltaTime;
    yield return new WaitForSeconds(1);

The FPS counter still changes every frame although the `yield return new WaitForSeconds(1);`

If I put the StartCoroutine(ShowFPS()); in Update the same thing happens and if I put it in Start it just says FPS: Infinity.

How do I make the yield return new WaitForSeconds(1); wait 1 second?

Currently it is waiting a second - the issue is that nothing happens after that one second, and every frame you're calling another coroutine which does the same thing - changes the fps, then waits a second before doing nothing

What you should do instead is something like:

void Start()
    InvokeRepeating("UpdateFPS", 1, 1);

void UpdateFPS()
    FPS = 1 / Time.smoothDeltaTime;

Which doesn't use coroutines, but easier to understand code

Found the problem myself, adding `while(true)` inside the coroutine and moving the `StartCoroutine` to `Start` did the trick.