How to compare a single value to multiple options?

Quoting mtbb

Disliked

How can this work?... if(Minute()==(11 || 15 || 18) test=true; Is there a way to have a single value compare multiple options like above? or does it have to be if(Minute()==11 || Minute()==15 || Minute()==18);

Ignored

void OnStart()
{
   int minutes[] = {1,5,15,23,35,45,53};
   Print(is_in(35, minutes)); //true
}
template<typename T>
bool is_in(T check_value, T &array[])
{
   for(int i=ArraySize(array)-1; i>=0; i--)
      if(check_value == array[i])
         return true;
   return false;
}

Quoting mtbb

Disliked

Thanks for the quick reply Nicholishen.. I was really trying to stay away from having to make functions and or dealing with arrays for such a simple thing.. So i guess the easy (dumb) answer is .. if(Minute()==11 || Minute()==15 || Minute()==18)test=true;

Ignored

Quoting mtbb

Disliked

Ok... Will only call the minute function once.. I am surprised you cant compare if(x==(y || z ||w))do something;

Ignored

if x in [y, z, w]:

Quoting Nicholishen

Disliked

{quote} There isn't an easy way to check membership like in python. You will need to create a func... void OnStart() { int minutes[] = {1,5,15,23,35,45,53}; Print(is_in(35, minutes)); //true } template<typename T> bool is_in(T check_value, T &array[]) { for(int i=ArraySize(array)-1; i>=0; i--) if(check_value == array[i]) return true; return false; }

Ignored

void OnStart()
{
    int minutes[] = {1,5,15,23,35,45,53};
    Print(is_in(35, minutes)); //true
}
template<typename T>
bool is_in(T check_value, T &array[])
{
   for(int i=ArraySize(array)-1; i>=0; i--)
   if(check_value == array[i]) return true;
  return false;
}

template<typename Tval,typename Tarr>
int TArraySearch(Tval val,Tarr &arr[])
  {
   for(int i=0; i<ArraySize(arr);++i)
     {
      if(val==arr[i])return i;//return index if val found
     }
   return -1;// val not found
  }
 
void OnTick()
  {
    int minutes[]={1,5,15,23,35,45,53};
    if(TArraySearch(Minute(),minutes)!=-1)test=true; //just like checking if(OrderOpen(Symbol(),etc...)!=-1)...
  }

int index=TArraySearch(Minute(),minutes);
//minutes[index] gives u now full access inside and u can modify it if the array is defined outside OnTick() function
minutes[index] =16;

Quoting masterx584

Disliked

{quote}i read some of your comments where u were "teaching" about complex classes when u have problems with such simple things.... your code void.

Ignored

Quote

Disliked

...you didn't even created a loop

Quote

Disliked

u use a template just to look better but u don't know what is a template...

string num = "one";
int nums[] = {1,2,3};
if(is_in(num, nums))
   ...

Quote

Disliked

what's the point to search an array if not returning the index??

if(is_in(value, list))

if(is_in(value, list) >= 0)

Quote

Disliked

Returning the index gives you access inside the array

Quoting Nicholishen

Disliked

{quote} Yes, master, I could give lessons on the correct way to use MQL classes -- just like I'm about to give you a lesson on your poorly thought out and aggressive rebuttal... So let's begin. {quote} Not only is this incorrect, but the loop in my code is more efficient than the loop in yours. When you compare our functions you are counting up and calling the ArraySize function on each iteration (O(n)) while my loop counts backwards and only calls ArraySize once (O(1)). I clearly win this one. {quote} I know very well how to create a template and...

Ignored

for(int i=ArraySize(array)-1; i>=0; i--)
if(check_value == array[i]) return true;
return false;

for(int i=0;i<bullshit;i++)
 {
   this is a loop
  }

template<typename T>
bool is_in(T check_value, T &array[])

template<typename Tval,typename Tarr>
int TArraySearch(Tval val,Tarr &arr[])

Quoting masterx584

Disliked

{quote} Are u serious??? a loop look like if(int i=0;i<bullshit;i++) { this is a loop } U missed the brackets bro.You return false always.U don't check inside the arrayGo compile your bullshit and then come back here to teach ...lol... and the template ... can't use twice the same T as u tried to.I bet u write your code into your sleep.. bool is_in(T check_value, T &array[]) this is not possible. welcome to my ignore list.

Ignored

template<typename T>
bool is_in(T check_value, T &array[])
{
   for(int i=ArraySize(array)-1; i>=0; i--)
      if(check_value == array[i])
         return true;
   return false;
}

template<typename T>
bool is_in(T check_value, T &array[])
{
   for(int i=ArraySize(array)-1; i>=0; i--)
   {
      if(check_value == array[i])
      {
         return true;
      }
   }
   return false;
}

Quote

Disliked

bool is_in(T check_value, T &array[]) this is not possible.

Quoting Nicholishen

Disliked

{quote} I did compile my original code and I tested it before I posted it.... so let's continue our lesson. In MQL, just like in C and C++, you don't need braces. If you omit the braces following a loop or expression the control flow is passed to the next loop, expression, or statement in succession. You only need braces to identify a code-block with more than one successive expression. Therefore, these two code blocks are identical. template<typename T> bool is_in(T check_value, T &array[]) { for(int i=ArraySize(array)-1; i>=0; i--) if(check_value...

Ignored

Quoting braintheboss

Disliked

{quote} You dont forget he is master ( 584 times ) in C++... If he say you need use braces, accept it

Ignored

Quoting Nicholishen

Disliked

{quote} Hahaha, touche. I graciously accept my defeat. Logic prevails. /s

Ignored

Quoting sakisf

Disliked

static int blabla = 0; static datetime m1time = 0; if(m1time < iTime(NULL, 1, 0)) { m1time = iTime(NULL, 1, 0); blabla = Minute(); switch (blabla){ case 11: test = true; break; case 15: test = true; break; case 18: test = true; break; } }

Ignored

   bool test = false;
   switch (Minute()){
      case 11:
      case 15:
      case 18:
         test = true;
   }