Sequence with coroutines.

Quick question for the forums, I have combos in which if you hit certain objects within the time then you get bonus points the only thing I can't figure out is how to do it in sequence.

Code (csharp):

1.  
2.  IEnumerator ComboOne()
3.     {
4.  
5.          if (Target1.Hit == true)
6.                 yield return new WaitForSeconds(3);
7.             if (Target2.Hit == true)
8.                yield return new WaitForSeconds(3);
9.                 if (Target3.Hit == true)
10.                 StartCoroutine(Anna());
11. }
12.  
13.  

The problem is when I hit them in any order they still do the same thing, so just needing to know how to hit in a sequence order?


Thanks

 

This is actually what you have :
So they are all checked at the same time. It will check for Target1, then 2 and then 3.

Code (csharp):

1.  
2.  IEnumerator ComboOne(){
3.  
4.          if (Target1.Hit == true)
5.                 yield return new WaitForSeconds(3);
6.          if (Target2.Hit == true)
7.                yield return new WaitForSeconds(3);
8.          if (Target3.Hit == true)
9.                 StartCoroutine(Anna());
10. }
11.  

If you want to force 1 first, then the other :

Code (csharp):

1.  
2.  IEnumerator ComboOne(){
3.  
4.     if (Target1.Hit == true){
5.            yield return new WaitForSeconds(3);
6.  
7.            if (Target2.Hit == true){
8.                 yield return new WaitForSeconds(3);
9.  
10.                 if (Target3.Hit == true){
11.                      StartCoroutine(Anna());
12.                 }
13.            }
14.       }
15. }
16.  

You basically need to put the "{*}" sign in your if statement so they are sequential.

 

Thanks for the reply, this what I did to solve this,

Code (csharp):

1.  
2.      IEnumerator ComboOne(){
3.      HitCount = 0
4.         if (Target1.Hit == true){
5.              HitCount = 1;
6.                yield return new WaitForSeconds();
7.                 HitCount = 0;
8.                if (Target2.Hit == true  HitCount == 1){
9.                       HitCount = 2;
10.                        yield return new WaitForSeconds(3);
11.                      HitCount = 0;
12.            if (Target3.Hit == true  HitCount == 2){
13.                          StartCoroutine(());
14.                     }
15.                }
16.           }
17.     }
18.  

Basically, it works but hopefully on one gets confused, when the Hit Count is set a specific Target, then as long as you are within the the time frame then HitCount increments and if you are 0 you must hit the first target to get past the hit count= 0 and then move forward, if that makes sense.

 

Glad I could help!

Did you try your code? Does it work?
I have the feeling that after your first timer which has no time value btw, the next statement won't work:

Code (csharp):

1.  
2. yield return new WaitForSeconds(); //no time here
3. HitCount = 0; // you reset the HitCount which prevent the next statement to be True
4.      if (Target2.Hit == true  HitCount == 1){ ...
5.  

You do it twice.

Make sense?

 

Actually it does work, I set my hit count to =0 and as long as I am within the time frame it executes perfectly.
That is my bad I do add 3 seconds to execute the next code. Just forgot to show it here.

 

The nesting looks terribly unwieldy. Any way to sequentially execute Coroutines within a certain context?

 

It depends on the context/conditions. You can certainly make more effective designs by altering how you think about conditions. The above *idea* is okay (doesn't work as is), but the formatting is extremely bad and there are piles of errors in the code.

LINQ could possibly help clean it up in some cases. Sometimes you can do returns/breaks to exit the routine instead of nesting if's deeper and deeper. Sometimes its best to modularize the routines differently just for the sake of maintainability and readability.

 

I tend to do this for sequences

Code (CSharp):

1. IEnumerator CoThingie (){
2.     // Do something
3.     while (!someCondition) yield return null;
4.     // Do the next thing
5. }

You can also throw a timer inside the while and yield return break if there is no action within the time limit.