There must be a neat way to express this

I often find myself writing (coroutine) code that does something like this:

Code (CSharp):

1. if (value < target) {
2.     value += speed * Time.deltaTime;
3.     someParameter.value = value;
4.     if (value >= target)
5.         yield break;
6.     else
7.         yield return null;
8. }
9. else if (value > target) {
10.     value -= speed * Time.deltaTime;
11.     someParameter.value = value;
12.     if (value <= target)
13.         yield break;
14.     else
15.         yield return null;
16. }

I _hate_ that "if" statement, but it seems necessary because I need to know whether I'm doing =< or >=. I'm sure there must be a more compact way of expressing this?

 

You want to lerp something towards a target with a speed? Why dont you just do Mathf.MoveTowards in your coroutine, until your value is approx. equal to target?

 

Try something like this.

Code (CSharp):

1. float dist = target - value;
2. float delta = speed * Mathf.Sign(dist) * Time.deltaTime;
3. value += delta;
4.  
5. if(Mathf.Abs(delta) > Mathf.Abs(dist))
6.   yield return null;
7. else
8.   yield break;

 

Hey didn't know about that function! That's *exactly* what I was looking for