Compiled EA.ex4 and mqh Files

#include "C:\Users\Owner\Documents\abc\def.mqh"

Quoting losja

Disliked

1) No, you don't. Just the .ex4 file. 2) No! mqh files are just added internally to your EA (mq4 file) by MetaEditor when you click compile. So, it is like writting in the mq4 file the same code that you have in your mqh file. mqh files are good for organizing code or make use of some common code among different EAs, indicators and scripts. 3) Yes. This way: #include "C:\Users\Owner\Documents\abc\def.mqh"

Ignored

int enBUY(string symbol)
  {
   int Output=0;
   if(BUY Conditions) Output=1; // for BUY Signal
   return(Output);
  }
//+------------------------------------------------------------------+
int enSEL(string symbol)
  {
   int Output=0;
   if(SELL Conditions) Output=1; // for SELL Signal
   return(Output);
  }

#include <C:\Users\Owner\Documents\abc\defBUY.mqh>
#include <C:\Users\Owner\Documents\abc\defSEL.mqh>

Quote

Disliked

I'm currently using #include <C:\Users\Owner\Documents\abc\def.mqh> and you suggested #include "C:\Users\Owner\Documents\abc\def.mqh", what are the differences ?

Quoting losja

Disliked

{quote} No difference. Just I prefer to use quotation marks because angle brackets are also used for files in the Include folder. I can't tell you why your EA only opens sell orders without seeing more code of your EA. Each mqh file will tolerate everything an mq4 file tolerates. You don't need to create a different mqh file for buy and sell conditions.

Ignored

Quoting losja

Disliked

{quote} You don't need to create a different mqh file for buy and sell conditions.

Ignored

// In mqh file (filename : def.mqh)
 
int enSig(string symbol)
  {
   int Output=0;
 
   if(BUY Conditions) Output=1; // BUY entry signal conditions
   if(SELL Conditions) Output=2; // SELL entry signal conditions
 
   return(Output);
  }
 
//+------------------------------------------------------------------+
// Codes of EA
 
#include <C:\Users\Owner\Documents\abc\def.mqh>
 
if(... && enSig(symbol)==1 && ...) // BUY signal
 
if(... && enSig(symbol)==2 && ...) // SELL signal

int enSig(string symbol)
  {
   if(BUY Conditions) return 1; // BUY entry signal conditions
   if(SELL Conditions) return 2; // SELL entry signal conditions
 
   return 0;
  }

Quoting losja

Disliked

That's really good! But if you still want to use two functions, you can go ahead. It will also work. Take a look at this alternative to your code: int enSig(string symbol) { if(BUY Conditions) return 1; // BUY entry signal conditions if(SELL Conditions) return 2; // SELL entry signal conditions return 0; }

Ignored

int enSig(string symbol)
  {
   if(BUY Conditions) return 1;
 
// If the BUY entry signal conditions are met, it shall return to "1",
// no question at all. But due to descending order nature, the program
// goes to lower level below and test the SELL signal condition AGAIN :-
 
   if(SELL Conditions) return 2;
 
// program after tested the SELL entry signal condition, confirmed
// there is no SELL signal and therefore the program returns from the
// above "1" to become "0"
 
   return 0;
 
// eventually on this enSig block, the return is "0", no BUY entry signal
// of "1" because it has been superceded by "0" of the SELL signal
// conditions.
 
  }

int enSig(string symbol)
  {
   if(BUY Conditions) return 1; // BUY entry signal conditions
 
   else if(SELL Conditions) return 2; // SELL entry signal conditions
 
   else return 0;
  }

Quoting reteid2222

Disliked

1 mqh code is compiled in your ex file so no problem 2 no just have one instaaltion ofmt4 where you put all mqh files in the create the ex4 files and put it on vps server…ex4 are stand allone until you refer to other files öike dlls 3. no need see point 3 what you want to do use ifdefs to use both files…

Ignored

Quoting losja

Disliked

That's really good! But if you still want to use two functions, you can go ahead. It will also work. Take a look at this alternative to your code: int enSig(string symbol) { if(BUY Conditions) return 1; // BUY entry signal conditions if(SELL Conditions) return 2; // SELL entry signal conditions return 0; }

Ignored

Quoting chimoong

Disliked

{quote} Hi, Losja, thanks, I'll try your way. I love the merged style (2 blocks in 1 block) to reduce the codes and yet retains the efficiency of the EA. I have another question on logic of why my EA only placed SELL order but without BUY order, this may due to the programming is in descending order (lower level of codes), let me explain the following logic for your opinion :- int enSig(string symbol) { if(BUY Conditions) return 1; // If the BUY entry signal conditions are met, it shall return to "1", // no question at all. But due to descending...

Ignored

int enSig(string symbol)
  {
   if(BUY Conditions) return 1;
   // If Buy Conditions are met, the function ends here and returns 1, otherwise the function executes the next statement

   if(SELL Conditions) return 2;
   // If Sell Conditions are met, the function ends here and returns 2, otherwise the function executes the next statement.

   return 0;
  // If neither Buy nor Sell conditions are met, then the function reaches this point and returns 0
  }

Quoting losja

Disliked

{quote} We will get the same result for both code snippets. They are equivalent. int enSig(string symbol) { if(BUY Conditions) return 1; // If Buy Conditions are met, the function ends here and returns 1, otherwise the function executes the next statement if(SELL Conditions) return 2; // If Sell Conditions are met, the function ends here and returns 2, otherwise the function executes the next statement. return 0; // If neither Buy nor Sell conditions are met, then the function reaches this point and returns 0 }

Ignored