forループで1回だけダイアログボックスを表示 - 終了

Question

このコードは、空のテキストボックスが複数ある場合に複数のダイアログボックスのプロンプトを除いて正常に動作しますが、一度のみプロンプトを表示します。例えば、1,1、（null）、（null）、d、gを入力すると、空のテキストボックスが2つあるのでダイアログボックスは2回表示されますが、一度のみプロンプトを表示する必要があります。

どうすればこの問題を解決できますか？

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
    { 
     for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
     { 
      if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
      { 
     MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      } 
     } 
    } 

Answer 1

私はそれを解決する別の方法を見つけました。

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
    { 
     int tt = 0; 
     for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
     { 
      if (tt == 0) 
      { 
       if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
       { 
        MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
        tt = 1; 
       } 
      } 
     } 
    } 

Answer 2

あなたは、単にフラグを導入することができます

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    bool hasEmpty = false; 

    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      hasEmpty = true; 
     } 
    } 

    if (hasEmpty) { 
     MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
    } 
} 

Answer 3

それはチェックを停止するようにループから抜け出すないのはなぜ？

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; // <-- 
     } 
    } 
} 

returnもこの状況で動作します。

---

テストBeforeSave場合は、二回実行されている：ヨーヨーが見ることができるように

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    MessageBox.Show("Test"); // <-- 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; // <-- 
     } 
    } 
} 

は私が（ループ外）メソッドの上部に新しい「テスト」のメッセージを追加し、あなたが重複表示される場合」コードを使用しているときは、BeforeSaveが2回実行されていることを意味します。

その場合、なぜそれが2回実行されているかを確認し、それを修正する必要があります。それが修正可能でない場合は、いくつかのsyncronizationソリューションがあるかもしれない...など：

private int canSave; 

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    if (Interlocked.CompareExchange(ref canSave, 0, 1) != 1) 
    { 
     // Any cancelation logic that's appropiate here 
     return; 
    } 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; // <-- 
     } 
    } 
} 

次に、あなたのコードがそれを許可しないように0を実行できるようにcanSave 1を参照してください。 Interlockedオペレーションでは、BeforeSaveのコードは、canSaveをコードのどこか1に設定するまで再実行されません（実行時に自動的に0に設定されるため、複数のスレッドが混乱する可能性はありません）。

私はBeforeSaveの二重実行を制御するソリューションを提供していますが、予想よりも2倍速で実行されている場合は、別の問題があることを示しています。 ）。

Answer 4

フラグを使用するか、Linqを使用してください。

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) { 
    bool flag = false; 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      flag = true; 
      break; 
     } 
    } 
    if (flag) 
     MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
} 

あなたのBeforeSaveメソッドが複数回呼び出さ取得されている場合でも、あなたのために働く必要があります私は道に続いて、LINQソリューション

Answer 5

を提供するために使用されるオブジェクトを十分に知りません。

private bool _isMessageBoxShown; 
public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{  
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
    if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
    { 
     if(!_isMessageBoxShown) 
     { 
      _isMessageBoxShown = true; 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; 
     } 
    } 
    }    
} 

ちょうどあなたがあなたのメッセージボックスは、次の時間を示すことにしたいとき、あなたは_isMessageBoxShown = false;を設定する必要があることを確認してください。