Sorry to have to ask but this is driving me up the wall.

What i want to eventually want to do is to buy 1 lot of EURUSD and at the same time SELL 1 lot of GBPUSD by just double clicking a script whilst a chart of EURUSD is open.

I know i have to use the market info command and the buy command works.

When i try to add the sell part it doesnt. I have stripped it right down to the point where i am just trying to sell at market price the GBPUSD on a EURUSD chart and i have this


//+------------------------------------------------------------------+
//| sell.mq4 |
//| Copyright © 2008, MetaQuotes Software Corp. |
//| Forex Trading Software: Forex Trading Platform MetaTrader 4 |
//+------------------------------------------------------------------+
#property copyright "Copyright © 2008, MetaQuotes Software Corp."
#property link "http://www.metaquotes.net"

#include <stdlib.mqh>
#include <WinUser32.mqh>

//+------------------------------------------------------------------+
//| script program start function |
//+------------------------------------------------------------------+
int start()
{
//----


int ticket=OrderSend("GBPUSD",OP_SELL,1,MarketInfo("GB PUSD",MODE_BID),0,"expert comment",255,0,CLR_NONE);

if(ticket<1)
{
int error=GetLastError();
Print("Error = ",ErrorDescription(error));
return;
}
//----
OrderPrint();
return(0);
}






Would someone be kind enough to tell me what i am doing wrong

Thanks Paul

from what I see 2 parameters are missing in your OrderSend function, stoploss, and takeprofit and at least in what you put there is a space in "GB(space)PUSD" but that maybe comes from Copy Paste

int OrderSend( string symbol, int cmd, double volume, double price, int slippage, double stoploss, double takeprofit, string comment=NULL, int magic=0, datetime expiration=0, color arrow_color=CLR_NONE)

An what error do you get in the log file when you execute the script?

Hi jipl.

Thanks for your reply its very much appreciated.

Firstly yes that is due to copy paste.

I do not get any error, it says script loaded ok and then script removed ok

If i use this it works




int start()
{
//----
int ticket=OrderSend("EURUSD",OP_BUY,1.0,MarketInfo("E URUSD",MODE_ASK),0,"expert comment",255,0,CLR_NONE);

if(ticket<1)

{
int error=GetLastError();
Print("Error = ",ErrorDescription(error));
return;
}
//----
OrderPrint();
return(0);
}



But if i change to this it doesnt

int start()
{
//----
int ticket=OrderSend("GBPUSD",OP_SELL1.0,MarketInfo("G BPUSD",MODE_BID),0,"expert comment",255,0,CLR_NONE);

if(ticket<1)

{
int error=GetLastError();
Print("Error = ",ErrorDescription(error));
return;
}
//----
OrderPrint();
return(0);
}



As you can see the parameters for sl and tp are missing in the one that works so thats why i didnt put them in the one that doesnt.

I will try it though thanks

Paul

jlpi, thanks i have tried adding the sl and tp parameters and it is fine now. Still dont know why the buy script works without them but hey at least its working.

Again thanks for your help

Paul

no problem, you are welcome

A comma is missing.

FerruFx

__________________
FerruFx / www.ervent.net - Professional Coding Services (EAs/Indicators/Alerts)

BBVPS.com - Reliable Windows VPS For MT4 Hosting