Любой вопрос новичка, чтоб не захламлять форум. Профи, не проходите мимо. Без вас никуда - 6. - страница 495

 

Терминал обновился с 509 на 600 и советник при компиляции выдает такое предупреждение:

declaration of 'ErrNum' hides global declaration at line 81

и часть кода:

string ErrorDescription(int ErrNum)
{
  switch(ErrNum)
  {

Подскажите пожалуйста в чем проблема?

 
rustein:

Терминал обновился с 509 на 600 и советник при компиляции выдает такое предупреждение:

declaration of 'ErrNum' hides global declaration at line 81

и часть кода:

Подскажите пожалуйста в чем проблема?

У вас есть ErrNum, объявленная на глобальном уровне. Переменная с тем же именем внутри функции скрывает значение глобальной переменной. В функции задайте другое имя
 

Спасибо, то есть поменял ErrNum на ErrNumber и все? Предупреждение исчезло, все будет работать правильно?

string ErrorDescription(int ErrNumber)
{
  switch(ErrNum)
  {
    case 0: return("NO ERROR");
    case 1: return("NO RESULT");                                 
    case 2: return("COMMON ERROR");                              
    case 3: return("INVALID TRADE PARAMETERS");                  
    case 4: return("SERVER BUSY");                               
    case 5: return("OLD VERSION");                               
    case 6: return("NO CONNECTION");                             
    case 7: return("NOT ENOUGH RIGHTS");   
 
Супер, спасибо огромное еще раз.

И последнее предупреждение в коде:

not all control paths return a value

double FractalUp()
{
  if(iCustom(Symbol(),0,"SR",FractalRange,false,1,2,TradeBar) != EMPTY_VALUE)  
  return(iCustom(Symbol(),0,"SR",FractalRange,false,1,2,TradeBar));
}

Подскажите и здесь что не так, пожалуйста.

 
rustein:
Супер, спасибо огромное еще раз.

И последнее предупреждение в коде:

not all control paths return a value

Подскажите и здесь что не так, пожалуйста.

Если условие выполняется, то есть возврат значения из функции.

Иначе - возврата из функции нет. Верните из функции какое-нибудь умолчательное значение

 
хехе :),

сделал так и предупреждение исчезло:

double FractalUp()
{
  if(iCustom(Symbol(),0,"SR",FractalRange,false,1,2,TradeBar) != EMPTY_VALUE)
  {
    return(iCustom(Symbol(),0,"SR",FractalRange,false,1,2,TradeBar));
  }
  else return (0);
}

надеюсь что хорошо, жизнь в мт4 продолжается :), спасибо еще раз!.

 
rustein:
хехе :),

сделал так и предупреждение исчезло:

надеюсь что хорошо, жизнь в мт4 продолжается :), спасибо еще раз!.

Не за что.
 

Ребят, есть 4 простые функции собственного производства, в них по всей видимости есть ошибка. Помогите пожалуйста начинающему!

(название каждой функции говорит само за себя, что должна делать эта функция)

extern double LOT =0.1;


double GetLastOrderProfit()
{
  int time = 0; double profit = 0; 
  for(int i = OrdersHistoryTotal()-1; i>=0; i--)
  {
    if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY))
    {
      if(OrderSymbol() == Symbol()&& OrderMagicNumber() == Magic)
      {
        if(time<OrderCloseTime())
        {
          time=OrderCloseTime();
          profit=OrderProfit();
        }
      }
    }
  }
  return(profit);
}

//+------------------------------------------------------------------+
double GetLastLot()
{
  int time = 0; double Lot = 0; 
  for(int i = OrdersHistoryTotal()-1;i>=0;i--)
  {
    if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY))
    {
      if(OrderSymbol() == Symbol()&& OrderMagicNumber() == Magic)
      {
        if(time<OrderCloseTime())
        time=OrderCloseTime();
        Lot = OrderLots(); 
      }
    }
  }

  if(Lot <= 0) Lot = LOT;

  return(Lot);
}

//+------------------------------------------------------------------+
double GetLastTenOrdersProfit()
{
  double profit = 0; int count = 0; 
  for(int i = OrdersHistoryTotal()-1;i>=0;i--)
  {
    if(OrderSelect(i,SELECT_BY_POS,MODE_HISTORY))
    {
      if(OrderSymbol() == Symbol()&& OrderMagicNumber() == Magic)
      {
        if (count<10)
        {
          profit=profit+OrderProfit();
          count++;
        } 
      }
    }
  }

  return(profit);
}
//+------------------------------------------------------------------+
double GetLot()
{
  double Lot = 0; double n = GetLastLot();
  
  if (GetTotalProfit() < 0 || GetLastLotProfit() < 0)
  Lot = NormalizeDouble (n * MartinStep,2);
  
  if (GetTotalProfit() >= 0 || GetLastLotProfit() >= 0)
  Lot = LOT;
  
  return (Lot);
}

 

помогите разобраться

double b=NormalizeDouble(ObjectGet("b",OBJPROP_TIME1),Digits);

int shiftb=iBarShift(NULL,0,b);

пишет одно предупреждение possible loss of data due to type conversion ссылаясь на выделенную b, да, она не datetime, но если сделать так:

datetime b=ObjectGet("b",OBJPROP_TIME1);

int shiftb=iBarShift(NULL,0,b);

то предупреждение будет почему-то на datetime b, вроде все правильно и не должно быть предупреждений.

 
Alexandr24:

помогите разобраться

double b=NormalizeDouble(ObjectGet("b",OBJPROP_TIME1),Digits);

int shiftb=iBarShift(NULL,0,b);

пишет одно предупреждение possible loss of data due to type conversion ссылаясь на выделенную b, да, она не datetime, но если сделать так:

datetime b=ObjectGet("b",OBJPROP_TIME1);

int shiftb=iBarShift(NULL,0,b);

то предупреждение будет почему-то на datetime b, вроде все правильно и не должно быть предупреждений.

А задайте int b
Причина обращения: