SendMail with Price?

[dsraider]

Hi Tim,

I'm trying to get SendMail() to include PnL so I get something like: ES Stop Filled: +3.5. I'm now trying to define var1 in variables but keep getting error messages. It doesn't like private int or private double. Is there one I should be using?

Thanks,
Dave

[NinjaTrader_Tim]

Hi dsraider,

If you are including the "+", you can try a string type. You may want to separate the variable to include your just the number value, as a double then add the + in the SendMail()

[dsraider]

Hi Tim,

I admit not knowing what you mean exactly but can you tell me why lastTrade works perfectly and lastTrade.ProfitPoints stops my strat from working? From everything I've read, I don't understand why that would be an issue and I think that's what's stopping my progress.

Thanks,
Dave

[NinjaTrader_JoshP]

Dave,

There may be a point in time when lastTrade is empty or null and trying to access its properties when it is in such a state will cause problems. If you are just accessing the object itself it won't have an issue telling you it is empty though. This all depends on your code and when exactly you are printing.

[Baruch]

I'm waiting for this thread to hit 100 posts. This is my small contribution.
UNBELIEVABLE

[dsraider]

Hey Josh,

I tried the following but it caused it not to email at all:

if (stopLossTokens.Contains(order.Token))
            {
                // Check order for terminal state
                if (order.OrderState == OrderState.Filled)
                {
                    if(lastTrade != null)
                    {    
                    // Print out information about the order
                    Print(order.ToString());
                    SendMail("", "[email protected]", Instrument.MasterInstrument.Name + " Cross2050 Stop Filled " + order.AvgFillPrice + ": " + (lastTrade.ProfitPoints/4 >= 0 ? "+" + Convert.ToString(lastTrade.ProfitPoints/4) : Convert.ToString(lastTrade.ProfitPoints/4)), "");
                    Print("SendMail: " + Instrument.MasterInstrument.Name + " Cross2050 Stop Filled " + order.AvgFillPrice + ": " + lastTrade.ProfitPoints);
                    }
                if (order.OrderState == OrderState.Cancelled || order.OrderState == OrderState.Filled || order.OrderState == OrderState.Rejected)    
                    // Remove from collection
                    {
                    stopLossTokens.Remove(order.Token);
                        entryOrder1 = null;
                        entryOrder2 = null;
                    }
                    
                }
                // Print out the current stop loss price
                else
                    Print("The order name " + order.Name + " stop price is currently " + order.StopPrice);
            } 

Am I on the right track?

Thanks.

P.S. That makes 81, Baruch.

[NinjaTrader_Tim]

Hi dsraider,

I suggest removing all the email stuff for now and just focus on debugging this. Add unique Print() statements inside each level of "if" conditions to ensure each condition is being entered.

[dsraider]

Turns out all I had to do was combine some code. If anyone is interested, this is the working result:

if (stopLossTokens.Contains(order.Token))
            {
                // Check order for terminal state
                if (order.OrderState == OrderState.Filled && stopSent == false)
                {
                    // Print out information about the order
                    if (Performance.RealtimeTrades.Count > 0) 
                        {     
                            // Get the last completed real-time trade (at index 0) 
                            Trade lastTrade = Performance.AllTrades[Performance.AllTrades.Count - 1];
 
                            if (lastTrade != null) 
                            {
                                if(lastTrade.ProfitPoints >= 0)
                                {    
                                Print("The last trade profit is " + (lastTrade.ProfitPoints >= 0 ? "+" + Convert.ToString(lastTrade.ProfitPoints) : Convert.ToString(lastTrade.ProfitPoints/4)));
                                SendMail("", "[email protected]", Instrument.MasterInstrument.Name + " Cross2050 Stop Filled " + order.AvgFillPrice + ": +" + lastTrade.ProfitPoints, "");
                                stopSent = true;    
                                }
                                else 
                                {
                                    Print("The last trade profit is " + (lastTrade.ProfitPoints >= 0 ? "+" + Convert.ToString(lastTrade.ProfitPoints) : Convert.ToString(lastTrade.ProfitPoints)));
                                    SendMail("", "[email protected]", Instrument.MasterInstrument.Name + " Cross2050 Stop Filled " + order.AvgFillPrice + ": " + lastTrade.ProfitPoints, "");
                                stopSent = true;
                                }
                            } 
                        }    
                if (order.OrderState == OrderState.Cancelled || order.OrderState == OrderState.Filled || order.OrderState == OrderState.Rejected)    
                    // Remove from collection
                    {
                    stopLossTokens.Remove(order.Token);
                        entryOrder1 = null;
                        entryOrder2 = null;
                    }
 
                }
                // Print out the current stop loss price
                else
                    Print("The order name " + order.Name + " stop price is currently " + order.StopPrice);
            }
 
 
            // Process profit target orders
            if (profitTargetTokens.Contains(order.Token))
            {
                // Check order for terminal state
                if (order.OrderState == OrderState.Filled && targetSent == false)
                {
                    // Print out information about the order
                    if (Performance.RealtimeTrades.Count > 0) 
                        {     
                            // Get the last completed real-time trade (at index 0) 
                            Trade lastTrade = Performance.AllTrades[Performance.AllTrades.Count - 1];
 
                            if (lastTrade != null)
                            Print("The last trade profit is " + (lastTrade.ProfitPoints >= 0 ? "+" + Convert.ToString(lastTrade.ProfitPoints) : Convert.ToString(lastTrade.ProfitPoints/4)));
                            SendMail("", "[email protected]", Instrument.MasterInstrument.Name + " Cross2050 Target Filled " + order.AvgFillPrice + ": +" + lastTrade.ProfitPoints, "");
                            targetSent = true;
                        }
                    if (order.OrderState == OrderState.Cancelled || order.OrderState == OrderState.Filled || order.OrderState == OrderState.Rejected)
                    {    
                    // Remove from collection
                    profitTargetTokens.Remove(order.Token);
                        entryOrder1 = null;
                        entryOrder2 = null;
                    }
                }
 
                // Print out the current stop loss price
                else
                    Print("The order name " + order.Name + " limit price is currently " + order.LimitPrice);
            }  
        } 

Dave

[Rienb]

Interesting topic!

Is it also possible to send a screen-shot of the chart by mail?

[NinjaTrader_Tim]

Hi Rienb,

Unfortunately not. For you reference here is the SendMail() parameters.

http://www.ninjatrader-support.com/HelpGuideV6/helpguide.html?SendMail1