Coroutines with yield return new waitforseconds(); go for execution of other functions?

Question

0.00/5 (No votes)

See more:

I am using several coroutines as following

Script1:

Reads an input values that are x,y,z etc.

passes x and there values to script2 for execution one by one

Script2:

a function receives that values and checks for each if it is x with positive values it calls XPOS coroutine else if y .....

then that coroutine moves objects accordingly but when it reaches yield return new WaitForSeconds(0.2f); it go back to Script1 and start execution next step then come back and start execution after yield return new WaitForSeconds(0.2f); . If Script1 is finish then the coroutines switches with one another during yield return new WaitForSeconds(0.2f);

following is codes I am using;

Script2 Code: Receiving values from Scipt1 and calling coroutine against each

private  void ObjSupport_PositionArrived(string Coordinate, double Value)
   {
       Debug.Log("Cordinate" + Coordinate);
       Debug.Log("Value" + Value);

       float ivalue = (float)Value;
       switch (Coordinate)
       {
           case "X+":
               {
                   if (ivalue == 0)
                   {

                   }
                   else
                   {

                       StartCoroutine("FuncXPlus", ivalue);

                   }

               }
               break;
           case "X-":
               {
                   if (ivalue == 0)
                   {

                   }
                   else
                   {

                       StartCoroutine("FuncXMinus", ivalue);
                   }
               }
               break;

           case "Z+":
               {
                   if (ivalue == 0)
                   {

                   }
                   else
                   {

                       StartCoroutine("FuncZPlus", ivalue);



                   }
               }
               break;
           case "Z-":
               {
                   if (ivalue == 0)
                   {


                   }
                   else
                   {

                       StartCoroutine("FuncZMinus", ivalue);

                   }
               }
               break;

       }

After that following are coroutine functions

IEnumerator FuncZPlus(float ivalue)
   {
       Vector3 Beemposition = Beem.transform.localPosition;
       float BeemCurrentXpos = Beemposition.y;
       Beemposition.y = Mathf.Clamp(Beemposition.y + ivalue, 0.04f, 0.07f);
       while (BeemCurrentXpos <= Beemposition.y)
       {
           BeemCurrentXpos += 0.0001f;
           Vector3 posX = new Vector3(Beemposition.x, BeemCurrentXpos, Beemposition.z);
           Beem.transform.localPosition = posX;

           yield return new WaitForSeconds(0.2f);

       }

   }

   IEnumerator FuncZMinus(float ivalue)
   {
       Debug.Log("FunZMinus");
       Vector3 Beemposition = Beem.transform.localPosition;
       float BeemCurrentYpos = Beemposition.y;
       Beemposition.y = Mathf.Clamp(Beemposition.y + ivalue, 0.04f, 0.07f);
       while (BeemCurrentYpos >= Beemposition.y)
       {
           BeemCurrentYpos -= 0.0001f;
           Vector3 posX = new Vector3(Beemposition.x, BeemCurrentYpos,Beemposition.z);
           Beem.transform.localPosition = posX;

           yield return new WaitForSeconds(0.2f);

       }

   }
   IEnumerator FuncXPlus(float ivalue)
   {
       Debug.Log("FunXplus");
       Vector3 positionX = HeadRight.transform.localPosition;
       float CurrentXpos = positionX.x;
       positionX.x = Mathf.Clamp(positionX.x + ivalue, -0.02f, -0.002f);
       HeadRight.transform.localPosition = positionX;



       while (CurrentXpos <= positionX.x)
       {
           CurrentXpos += 0.0001f;
           Vector3 posX = new Vector3(CurrentXpos, positionX.y, positionX.z);
           HeadRight.transform.localPosition = posX;

           yield return new WaitForSeconds(0.2f);

       }

   }


   IEnumerator FuncXMinus(float ivalue)
   {
       Debug.Log("FunXminus");
       Vector3 xposition = HeadRight.transform.localPosition;
       float CurrentXpos = xposition.x;
       xposition.x = Mathf.Clamp(xposition.x - ivalue, -0.02f, 0f);
     while (CurrentXpos <= xposition.x)
       {
           Debug.Log("X-While");
           CurrentXpos -= 0.0001f;
           Vector3 posX = new Vector3(CurrentXpos, xposition.y, xposition.z);
           HeadRight.transform.localPosition = posX;

           yield return new WaitForSeconds(0.2f);

       }

   }

What I have tried:

The codes i have given are what i have tried

Posted 2-Oct-20 2:14am

Member 11391151

Updated 30-Oct-20 21:10pm

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

George Swan · Answer 1 · 2020-10-02T12:59:00

I am not sure what you are trying to do exactly but it seems to me that the switch statement can be simplified. You are checking that ivalue is not zero in every case. Why not test for ivalue==0 just once on the first line of the method and return if it is?

Your coroutine functions are returning an IEnumerator, my preference would be to return an IEnumerable<float> then you can iterate over the collection using a simple foreach statement.

The method call StartCoroutine(“FuncXPlus”, ivalue) looks like itis going to look up a function and pass ivalue to it. A switch statement using the format introduced in C#8 can simplify matters considerably and give you the same functionality in a few lines of code - something like this.

C#

private static IEnumerable<float> SwitchCoordinateFunction(string coordinate, float value)
 {
     if (value == 0) return null;
     return coordinate switch
     {
         "X+" => FuncXPlus(value),
         "X-" => FuncXMinus(value),
         "Z+" => FuncZPlus(value),
         "Z-" => FuncXMinus(value),
         _ => null,
     };
 }
 //example useage
    string coordinate = "X+";
     float value = 2;
     var functionResults = SwitchCoordinateFunction(coordinate, value);
     if (functionResults == null) return;

     foreach (float result in functionResults)
     {
         Console.Write($"{result}, ");
     }

Member 11391151 · Answer 2 · 2020-10-30T21:11:00

Solution 2

Everything is going fine in my code but during the

yield return new WaitForSeconds(0.2f);

other codes are executed .And i dont want that.

Posted 30-Oct-20 21:11pm

Member 11391151

Comments

Richard Deeming 2-Nov-20 11:15am

If you want to reply to a solution, click the "Have a Question or Comment?" button under that solution and post a comment.

Do not post your comment as a new "solution".