SendMail with Price?

[dsraider]

Hi,

I currently have my strat set up to email me upon entry, breakeven and exit. However, these are just basic notifications and I was wondering if there was a way to add more detail to the message, namely the actual price of entry and exit.

Thanks,
Dave

[NinjaTrader_JoshP]

You can send such information if you grab it from the TradeCollection object.

[dsraider]

Well, I went here: http://www.ninjatrader-support.com/H...helpguide.html but it's gibberish to me. Any other links or a hint?

Thanks,
Dave

[NinjaTrader_JoshP]

Dave,

What you can do is save your entry price when you are in the trade with Position.AvgPrice. Store it in a variable and then send that variable along with your email. For your exit price you can pretty much grab the price at the time your order was closed to get the exit price. If you are using limit orders you will know the limit price and can just use that instead. Store those into a different variable and also send it in your email.

[dsraider]

Sir,

I'm really flying blindly on this one. Would the following work?:

Variables
private IOrder Position.Avg.Price = null;

...

SendMail("", "[email protected]", "Position.Avg.Price", "");

Thanks,
Dave

[NinjaTrader_JoshP]

Dave,

No, Position.AvgPrice is a variable you would use in OnBarUpdate() directly. Whenever you hold a position. The average entry price for that position is reflected by the Position.AvgPrice variable. You do not need to make an IOrder for this.

SendMail("", "[email protected]", "Some email subject", "Some email text: My average entry price was: " + Position.AvgPrice);

[dsraider]

Josh,

The good news is that it worked. The bad news is it sent me a million emails. I'd like only one when it enters, moves my stop to even and exits. I guess I can't use built-ins like Position.AvgPrice?

Thanks,
Dave

[NinjaTrader_Bertrand]

Dave, you can still use Position.AvgPrice to access this info, however you need to be mindful where to issue the SendMail() statement then to avoid duplicate emails, for example you include it as part of your entry rules to send only when you enter and not on each following OnBarUpdate() bar per se.

[dsraider]

Hey Bertrand,

This is what I did:

if (CrossAbove(EMA(20), EMA(50), 1))
{
entryOrder = EnterLong(DefaultQuantity, "2050 Cross");
DrawArrowUp("", true, 0, Low[0] - 12 * TickSize, Color.Blue);
DrawTriangleUp("My triangle up" + CurrentBar, false, 0, Low[0] - 12 * TickSize, Color.Blue);
SendMail("", "[email protected]", "ES Long " + Position.AvgPrice, "");

I got about 15 emails before stopping the strat.

[NinjaTrader_JoshP]

dsraider,

When you put it in such a condition you will receive an email every single time that condition is true. If you are using CalculateOnBarClose = false, that condition could be true hundreds of times within a bar. That would mean hundreds of emails.

You should limit the number of times you send the email. Use a bool variable within your if-statement.

In Variables region of your code

private bool someBool = true;

In OnBarUpdate()

if (CrossAbove(EMA(20), EMA(50), 1))
{
     if (someBool == true)
     {
          someBool = false;
          SendMail(....);
     }
}
else
      someBool = true;

[dsraider]

Josh,

I like the idea of using a bool but wouldn't I run into the same issue since my strat's set to CalculateOnBarClose = false? Not sure what I can use as a one-time signal. Would it be easier to go back to Trade or TradeCollection? I didn't really understand that code, though, so I'd need another pointer

Dave

[NinjaTrader_JoshP]

No, you would not run into the same issue. The code in my previous post is a general frame work. You would need to still put in the logic you want for when to reset your bool variable to true. Only when it is true will the mail be sent. Once set to false it will remain false till set back to true in the code again.

[dsraider]

Okay, the good news is I got one and ONLY one email. Thanks for that. The bad news is my email read "ES Long 0". Here's my code:

if (CrossAbove(EMA(20), EMA(50), 1))
            {
                entryOrder = EnterLong(DefaultQuantity, "2050 Cross");
				DrawArrowUp("", true, 0, Low[0] - 12 * TickSize, Color.Blue);
				DrawTriangleUp("My triangle up" + CurrentBar, false, 0, Low[0] - 12 * TickSize, Color.Blue);
				
				if(longEntry == true)
				{longEntry = false;
				SendMail("", "[email protected]", "ES Long " + Position.AvgPrice, "");
				}
            }

What I do wrong?

Thanks.

[NinjaTrader_Bertrand]

At this point of sending the MarketPosition has not registered the long price value yet, so try adding a check for it - if (Position.MarketPosition != MarketPosition.Flat)

[dsraider]

Bertrand and Josh,

I've decided that the two of you enjoy torturing me! Today is my birthday and I'm thinking one example of clearly written code would be a great present (hint hint, nudge nudge). It might even get the first to post my drink ticket at next year's expo

Dave