MQL4のエラーの解決法

Question

前提・実現したいこと

コンパイルしたらエラーがでてしまいどうしても解決できない。

発生している問題・エラーメッセージ

'if' - expressions are not allowed on a global scope	手法.mq4	150	9
'oncebar' - unexpected token, probably type is missing?	手法.mq4	184	9
'box' - undeclared identifier	手法.mq4	184	19
'oncebar' - variable already defined	手法.mq4	184	9
   see previous declaration of variable 'oncebar'	手法.mq4	17	5
'}' - expressions are not allowed on a global scope	手法.mq4	186	4
declaration of 'box' hides global variable	手法.mq4	106	7
   see previous declaration of 'box'	手法.mq4	184	19
'Ticket2' - undeclared identifier	手法.mq4	123	26
'ture' - undeclared identifier	手法.mq4	123	55
'Ticketw' - undeclared identifier	手法.mq4	125	36
implicit conversion from 'number' to 'string'	手法.mq4	145	49
'}' - semicolon expected	手法.mq4	147	11
'Ticket1' - undeclared identifier	手法.mq4	154	26
'ture' - undeclared identifier	手法.mq4	154	55
implicit conversion from 'number' to 'string'	手法.mq4	177	50
11 errors, 3 warnings		12	4

#property copyright "Copyright 2020, MetaQuotes Software Corp."
#property link      "https://www.mql5.com"
#property version   "1.00"
#property strict 





extern int maxspread = 20;//スプレッド
extern int closeperiod = 20;//有効期限
extern double rot =0.1;//ロット

//1本の足で1回の取引関係
extern string  onceonly= "--- entry only once per 1 bar --------------------------------";
extern bool entrybar = true;//trueにすれば、この機能が有効になる。
int oncebar = 0;//前回の値動きの時の足の本数を格納しておくための変数






int OnInit()
  {
  
  //1本の足で1回の取引関係
  oncebar = Bars;//現在のチャートの足の本数を返す。

  
   return(INIT_SUCCEEDED);
  }






void OnDeinit(const int reason)
  {

  }




void OnTick()
  {
  
  //口座残高
  //double AccountBalance();
  
  //前日足の四本値
  double H1 = iHigh(NULL,0,1);
  double L1 = iLow(NULL,0,1);
  double O1 = iOpen(NULL,0,1);
  double C1 = iClose(NULL,0,1);

  //週足の四本値
  double WH1 = iHigh(NULL,PERIOD_W1,1);
  double WL1 = iLow(NULL,PERIOD_W1,1);
  double WO1 = iOpen(NULL,PERIOD_W1,1);
  double WC1 = iClose(NULL,PERIOD_W1,1);
  
  //前週足の四本値
  double WH2 = iHigh(NULL,PERIOD_W1,2);
  double WL2 = iLow(NULL,PERIOD_W1,2);
  double WO2 = iOpen(NULL,PERIOD_W1,2);
  double WC2 = iClose(NULL,PERIOD_W1,2);
  
  //月足の四本値
  double MNH1 = iHigh(NULL,PERIOD_MN1,1);
  double MNL1 = iLow(NULL,PERIOD_MN1,1);
  double MNO1 = iOpen(NULL,PERIOD_MN1,1);
  double MNC1 = iClose(NULL,PERIOD_MN1,1);
  
  //前月足の四本値
  double MNH2 = iHigh(NULL,PERIOD_MN1,2);
  double MNL2 = iLow(NULL,PERIOD_MN1,2);
  double MNO2 = iOpen(NULL,PERIOD_MN1,2);
  double MNC2 = iClose(NULL,PERIOD_MN1,2);
  
  //最高値
  int highestprice = iHighest(NULL,PERIOD_D1,MODE_HIGH,20,1);
  
  //最高値の4本値
  double highestH = iHigh(NULL,0,highestprice);
  double highestL = iLow(NULL,0,highestprice);
  double highestO = iOpen(NULL,0,highestprice);
  double highestC = iClose(NULL,0,highestprice);
  
  //最安値
  int cheapestprice = iLowest(NULL,PERIOD_D1,MODE_LOW,20,1);
  
  //最安値の4本値
  double cheapestH = iHigh(NULL,0,cheapestprice);
  double cheapestL = iLow(NULL,0,cheapestprice);
  double cheapestO = iOpen(NULL,0,cheapestprice);
  double cheapestC = iClose(NULL,0,cheapestprice);
  
  //損切値制限
  double buyson = cheapestH-cheapestL;
  double sellson = highestH-highestL;
  
  //1本の足で1回の取引関係
  int box = Bars;
  
    // 仕掛けフィルター（１本の足で１回だけ仕掛ける）
    if(oncebar)
      {
        // 新しい足を生成した時ではない場合は、仕掛けない
        if(box == oncebar)
          {
            oncebar = box;
            return;
          }
      }
      
        if(cheapestH < C1  && MarketInfo(NULL,MODE_SPREAD) < maxspread)//買いの条件・現在のスプレッド・スプレッド上限
          {
          
          //買いポジションの決済
          if(OrderSelect(Ticket2,SELECT_BY_TICKET) == ture)
          {
           bool close = OrderClose(Ticketw,rot,C1,10,Magenta);
          }
          
          
           int Ticket1 = OrderSend(//買いの処理
           Symbol(),//通貨ペア
           OP_BUYLIMIT,//買いの指値
           rot,//ロット数
           cheapestH,//注文価格
           10,//スリップページ
           cheapestL,//損切
           0,//利確
           NULL,//コメント
           1212,//マジックナンバー
           closeperiod,//有効期限
           Magenta//色
           );
           
           //通知
           Alert("Buy",Symbol(),"at",Ask);
           SendNotification("Buy"+Symbol()+"at"+Ask)
            
          }
         }
        
        if(highestL > C1 && MarketInfo(NULL,MODE_SPREAD) < maxspread)//売りの条件・現在のスプレッド・スプレッド上限
          {
          
          //売りポジションの決済
          if(OrderSelect(Ticket1,SELECT_BY_TICKET) == ture)
            {
             bool close = OrderClose(Ticket1,rot,C1,10,Magenta);
            }
          
          
           int Ticket2 = OrderSend(//売りの処理
           Symbol(),//通貨ペア
           OP_SELLLIMIT,//売りの指値
           rot,//ロット数
           highestL,//注文価格
           10,//スリップページ
           highestH,//損切
           0,//利確
           NULL,//コメント
           1111,//マジックナンバー
           closeperiod,//有効期限
           Magenta//色
           );
         
        if(Ticket1 > 0)
           {
           Alert("Sell",Symbol(),"at",Bid);
           SendNotification("Sell"+Symbol()+"at"+Bid);
           }
         
          }
        
        
        //1本の足で1回の取引関係
        oncebar = box;

   }

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[1T2R3M4]

なにも試していないようですが、
括弧の数くらい確認していますか。

[nisei2]

括弧が一つ多かったみたいです、ありがとうございます。

[1T2R3M4]

解決したようでよかったです。
質問をClose(解決済に)してくださいね。

[nisei2]

すいません、まだこのサイトの事がよくわからず質問を解決済みにする方法がわかりません。
どうしたら解決済みになるのでしょうか？

[1T2R3M4]

質問したことないので正確にはわかりませんが、
回答からBEST ANSWERを選ぶとClose処理ができるのではないでしょうか。
最初の質問はCloseしてますよね。
回答がない場合は自身で回答つけて自己解決処理ができるはずです。

[nisei2]

そうだったのですね、確かに最初ベストアンサーした記憶があります！
しかし、前どうやってベストアンサーのボタンを出したのか思い出せません。
回答者さんのどこかにボタンがあるのでしょうか？

[1T2R3M4]

回答してみたのでCloseしてみてください。

Answer 1

           //通知
           Alert("Buy",Symbol(),"at",Ask);
           SendNotification("Buy"+Symbol()+"at"+Ask)

          }
         }

括弧が多いのではないでしょうか。

Comments

[nisei2]

ありがとうございます！

[1T2R3M4]

おっ、マイナスが付いた。なぜ今頃。

[nisei2]

覚えがないのでたぶんなにかの拍子に押してしまっただけだと思います、すみません。

[1T2R3M4]

nisei2さんが押したのだったら、もう一度押すとキャンセルできますよ。
気を使っていただき、ありがとうございます。