Need help returning value from function

int locate_price()                                                                  // Function to locate price between Gann lines
   {                                                                                // Check all lines 0-8
      for(k=0; k<9; k++)
      {
         if(level[10] == 1)
         {
            if(Bid>level[k] && Bid<level[k+1])
            {
               return(k);
            }
            else 
            {
               return(0);
            }
         }
         if(level[10] == 2)
         {
            if(Bid<level[k] && Bid>level[k+1]){
            return(k);
            }
            else {
            return(0);
            }
         }
      }
   } 

Quoting stryker251

Disliked

I wrote the below function in an EA to check to see the location of current price between 0-8 prices stored in an array that I have calculated. I thought you could return a value from a function but I keep getting zero instead of what the count was when price was located. Can anyone help me by telling me where I'm wrong? I int k at the beginning of the EA...

Thanks

[php]
int locate_price() // Function to locate price between Gann lines
{...

Ignored

// Function to locate price between Gann lines
// Check all lines 0-8
 

int locate_price( int& level[ ] ) {
      for( k = 0; k < 9; k ++ ) {
         if( level[ 10 ] == 1 ) {
            if( Bid > level[ k ] && Bid < level[ k + 1 ] ) {
               return( k );
            } else {
               return( 0 )
            }
         }

         if( level[ 10 ] == 2 ) {
            if( Bid < level[ k ] && Bid > level[ k + 1 ] ) {
            return( k );
            }  else {
            return( 0 );
            }
         }
      }

Quoting stryker251

Disliked

The array is set to [11] but I just printed the values in the function and the level[10] is stuck on 2...which still doesn't explain returning zero but something to check on...

Ignored

Quoting CockeyedCowboy

Disliked

OK, my eyes are bad, I dont look so good, I did not see you have 4 return statments there, the two that return K are returning the index number of the for loop. what is it you what it to return?


If it whats contained in the array then you need to know?

return( level[ k ] );

if you want to return a changed element of an array then you need to pass it as a parameter as in the example. and call the function as in;

locate_price( level );

This way you are passing the whole array and any changes you make to it will stay out side the function....

Ignored

int locate_price()                                       // Function to locate price between which lines
   {                                                     // Check all lines 0-8
      for(k=0; k<9; k++)
      {
         if(level[10] == 1)                              // 1 = Blue = Up     
         {
            if(Bid>level[k] && Bid<level[k+1])
            {
               return(level[k]);
            }
            else 
            {
               return(0);
            }
         }
         if(level[10] == 2)                              // 2 = Red = Down 
         {
            if(Bid<level[k] && Bid>level[k+1]){
            return(level[k]);
            }
            else {
            return(0);
            }
         }
      }
   }  
//+------------------------------------------------------------------+
//| expert start function                                            |
//+------------------------------------------------------------------+
int start()
  {
  gann_lvl=iCustom(NULL,0,"DKGann_SQ9",0,0);
 
//PREFIX "_lev" lines are previous levels and not considered
#define PREFIX "_lev"
   for(int iObj=ObjectsTotal()-1; iObj >= 0; iObj--)
   {
       hline = ObjectName(iObj);                                                    // Find objects that are trendlines only
       if(ObjectType(hline) == OBJ_TREND && ObjectGet(hline, OBJPROP_PRICE1) == ObjectGet(hline, OBJPROP_PRICE2))
       {
            if (StringFind(hline, PREFIX) || 0)                                     //Only current trendlines start with "#", all others with "_lev"
            {
               for(int i; i=0; i<9; i++)
               {
                  if(StrToInteger(StringSubstr(hline, StringLen(hline) - 1)) == i)  // Find lines 0-8 by ending number
                  {
                     if(ObjectGet(hline, OBJPROP_COLOR) == 16748574)                // Determine if line are calculated up ("1") or down ("2") from move
                     {
                        level[10]=1;                                                   // by looking at object color, Long Color Blue=16748574
                     }
                     if(ObjectGet(hline, OBJPROP_COLOR) == 4678655)
                     {
                        level[10]=2;                                                   // Short Color Red=4678655 -- level[10] is the state only
                     }
                     level[i]=ObjectGet(hline, OBJPROP_PRICE2);                     // Store price of lines 0-8
                  }
               }
            }
       }
   }
//+------------------------------------------------------------------+
   located=locate_price();